PR c++/93143 - incorrect tree sharing with constexpr.

We don't unshare CONSTRUCTORs as often during constexpr evaluation, so we
need to unshare them here.

	* constexpr.c (cxx_eval_outermost_constant_expr): Don't assume
	CONSTRUCTORs are already unshared.

From-SVN: r280127

diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog
index 96bbcc9166ae9262c436f73a29661cae89859062..1a36cd3c89ec44c5b2e6793640881b65e015c95c 100644 (file)
--- a/gcc/cp/ChangeLog
+++ b/gcc/cp/ChangeLog
@@ -1,5 +1,9 @@
 2020-01-10  Jason Merrill  <jason@redhat.com>
 
+       PR c++/93143 - incorrect tree sharing with constexpr.
+       * constexpr.c (cxx_eval_outermost_constant_expr): Don't assume
+       CONSTRUCTORs are already unshared.
+
        PR c++/93173 - incorrect tree sharing.
        PR c++/93033
        * cp-gimplify.c (cp_gimplify_init_expr, cp_gimplify_expr): Use

diff --git a/gcc/cp/constexpr.c b/gcc/cp/constexpr.c
index 9306a7dce4a1fd6642d283108e3594f55e643c5f..3ca3b9ecd653d684e37219f5e1860923a6a7898c 100644 (file)
--- a/gcc/cp/constexpr.c
+++ b/gcc/cp/constexpr.c
@@ -6347,10 +6347,10 @@ cxx_eval_outermost_constant_expr (tree t, bool allow_non_constant,
   if (!non_constant_p && overflow_p)
     non_constant_p = true;
 
-  /* Unshare the result unless it's a CONSTRUCTOR in which case it's already
-     unshared.  */
+  /* Unshare the result.  */
   bool should_unshare = true;
-  if (r == t || TREE_CODE (r) == CONSTRUCTOR)
+  if (r == t || (TREE_CODE (t) == TARGET_EXPR
+                && TARGET_EXPR_INITIAL (t) == r))
     should_unshare = false;
 
   if (non_constant_p && !allow_non_constant)

diff --git a/gcc/testsuite/g++.dg/cpp0x/constexpr-array22.C b/gcc/testsuite/g++.dg/cpp0x/constexpr-array22.C
new file mode 100644 (file)
index 0000000..80e24e2
--- /dev/null
+++ b/gcc/testsuite/g++.dg/cpp0x/constexpr-array22.C
@@ -0,0 +1,27 @@
+// PR c++/93143
+// { dg-do run { target c++11 } }
+
+struct A { char a[2]; };
+
+static constexpr A foo () { return A{1}; }
+
+void bar ()
+{
+  A a = foo ();
+  if (a.a[0] != 1)
+    __builtin_abort(); 
+}
+
+void foobar ()
+{
+  A x[] = { foo (), foo () };
+  A a = foo ();
+  if (a.a[0] != 1)
+    __builtin_abort(); 
+}
+
+int main()
+{
+  bar();
+  foobar();
+}