]> git.ipfire.org Git - thirdparty/gcc.git/commitdiff
middle-end/119119 - re-gimplification of empty CTOR assignments
authorRichard Biener <rguenther@suse.de>
Thu, 6 Mar 2025 08:08:07 +0000 (09:08 +0100)
committerRichard Biener <rguenth@gcc.gnu.org>
Tue, 24 Jun 2025 11:09:05 +0000 (13:09 +0200)
The following testcase runs into a re-gimplification issue during
inlining when processing

  MEM[(struct e *)this_2(D)].a = {};

where re-gimplification does not handle assignments in the same
way than the gimplifier but instead relies on rhs_predicate_for
and gimplifying the RHS standalone.  This fails to handle
special-casing of CTORs.  The is_gimple_mem_rhs_or_call predicate
already handles clobbers but not empty CTORs so we end up in
the fallback code trying to force the CTOR into a separate stmt
using a temporary - but as we have a non-copyable type here that ICEs.

The following generalizes empty CTORs in is_gimple_mem_rhs_or_call
since those need no additional re-gimplification.

PR middle-end/119119
* gimplify.cc (is_gimple_mem_rhs_or_call): All empty CTORs
are OK when not a register type.

* g++.dg/torture/pr11911.C: New testcase.

(cherry picked from commit 3bd61c1dfaa2d7153eb4be82f423533ea937d0f9)

gcc/gimplify.cc
gcc/testsuite/g++.dg/torture/pr11911.C [new file with mode: 0644]

index a7b5a3883373e99d18b219672e81517f955cf166..ecbe58d2b0567b298501da6bf328a6a7909d38e9 100644 (file)
@@ -556,7 +556,7 @@ is_gimple_mem_rhs_or_call (tree t)
   else
     return (is_gimple_val (t)
            || is_gimple_lvalue (t)
-           || TREE_CLOBBER_P (t)
+           || (TREE_CODE (t) == CONSTRUCTOR && CONSTRUCTOR_NELTS (t) == 0)
            || TREE_CODE (t) == CALL_EXPR);
 }
 
diff --git a/gcc/testsuite/g++.dg/torture/pr11911.C b/gcc/testsuite/g++.dg/torture/pr11911.C
new file mode 100644 (file)
index 0000000..7dc836f
--- /dev/null
@@ -0,0 +1,21 @@
+// { dg-do compile }
+// { dg-require-effective-target c++17 }
+
+struct b {
+  int a;
+};
+struct c {
+  b d{};
+  c() = default;
+  c(c &) = delete;
+};
+struct e {
+  c a{};
+  e() {}
+};
+inline e f() { return {}; }
+struct g {
+  e cx;
+  g() : cx{f()} {}
+};
+void h() { g i; }