diff options
| author | Richard Biener <rguenther@suse.de> | 2024-03-11 09:35:07 +0100 |
|---|---|---|
| committer | Richard Biener <rguenther@suse.de> | 2024-03-11 11:22:26 +0100 |
| commit | 119f5ae0455f02568159eafa9008a555605e7d71 (patch) | |
| tree | 938142b0f1e1ab8cb8b862b3a57e1836b047fb0b /gcc | |
| parent | dbe5ccda4dbbd064c703cd3ab2a58ea40f08dd1a (diff) | |
| download | gcc-119f5ae0455f02568159eafa9008a555605e7d71.zip gcc-119f5ae0455f02568159eafa9008a555605e7d71.tar.gz gcc-119f5ae0455f02568159eafa9008a555605e7d71.tar.bz2 | |
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.
Diffstat (limited to 'gcc')
| -rw-r--r-- | gcc/gimplify.cc | 5 | ||||
| -rw-r--r-- | gcc/testsuite/gcc.target/i386/pr114299.c | 14 |
2 files changed, 19 insertions, 0 deletions
diff --git a/gcc/gimplify.cc b/gcc/gimplify.cc index 6ebca96..d64bbf3 100644 --- a/gcc/gimplify.cc +++ b/gcc/gimplify.cc @@ -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; } diff --git a/gcc/testsuite/gcc.target/i386/pr114299.c b/gcc/testsuite/gcc.target/i386/pr114299.c new file mode 100644 index 0000000..b4f30b7 --- /dev/null +++ b/gcc/testsuite/gcc.target/i386/pr114299.c @@ -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" } */ +} |