middle-end/114299 - missing error recovery from gimplify failure

When internal_get_tmp_var fails to gimplify the value the temporary
SSA name is supposed to be initialized with we can leak SSA names
with a NULL SSA_NAME_DEF_STMT into the IL.  That's bad, so recover
from this by instead returning a decl in that case.

	PR middle-end/114299
	* gimplify.cc (internal_get_tmp_var): When gimplification
	of VAL failed, return a decl.

	* gcc.target/i386/pr114299.c: New testcase.
This commit is contained in:
Richard Biener 2024-03-11 09:35:07 +01:00
parent dbe5ccda4d
commit 119f5ae045
2 changed files with 19 additions and 0 deletions

View File

@ -652,6 +652,11 @@ internal_get_tmp_var (tree val, gimple_seq *pre_p, gimple_seq *post_p,
gimplify_and_add (mod, pre_p);
ggc_free (mod);
/* If we failed to gimplify VAL then we can end up with the temporary
SSA name not having a definition. In this case return a decl. */
if (TREE_CODE (t) == SSA_NAME && ! SSA_NAME_DEF_STMT (t))
return lookup_tmp_var (val, is_formal, not_gimple_reg);
return t;
}

View File

@ -0,0 +1,14 @@
/* { dg-do compile { target lp64 } } */
/* { dg-options "-mgeneral-regs-only" } */
typedef __attribute__((__vector_size__(8))) __bf16 V;
typedef __attribute__((__vector_size__(16))) __bf16 W;
V v;
_Atomic V a;
W
foo(void) /* { dg-error "SSE" } */
{
return __builtin_shufflevector(v, a, 1, 2, 5, 0, 1, 6, 6, 4); /* { dg-error "invalid" } */
}