re PR c++/64080 ([C++14]ICE in cxx_eval_store_expression)
authorJason Merrill <jason@redhat.com>
Thu, 4 Dec 2014 20:37:24 +0000 (15:37 -0500)
committerJason Merrill <jason@gcc.gnu.org>
Thu, 4 Dec 2014 20:37:24 +0000 (15:37 -0500)
PR c++/64080
* constexpr.c (cxx_eval_store_expression): Handle non-decl store
targets.

From-SVN: r218401

gcc/cp/ChangeLog
gcc/cp/constexpr.c
gcc/testsuite/g++.dg/cpp1y/constexpr-ref1.C [new file with mode: 0644]

index 0e2548fbdb8822d4b1b9077d975289337b763b69..2db8bd7c581825d1c373ffedb489a13911717e12 100644 (file)
@@ -1,3 +1,9 @@
+2014-12-03  Jason Merrill  <jason@redhat.com>
+
+       PR c++/64080
+       * constexpr.c (cxx_eval_store_expression): Handle non-decl store
+       targets.
+
 2014-12-03  Paolo Carlini  <paolo.carlini@oracle.com>
 
        PR c++/63558
index d18025f457c5a7490ada94c8508385919f7fb090..9426d8558e2398974705d8fce60d6b8ce29bf626 100644 (file)
@@ -2583,19 +2583,22 @@ cxx_eval_store_expression (const constexpr_ctx *ctx, tree t,
 
        default:
          object = probe;
-         gcc_assert (DECL_P (object));
        }
     }
 
   /* And then find/build up our initializer for the path to the subobject
      we're initializing.  */
-  tree *valp = ctx->values->get (object);
+  tree *valp;
+  if (DECL_P (object))
+    valp = ctx->values->get (object);
+  else
+    valp = NULL;
   if (!valp)
     {
       /* A constant-expression cannot modify objects from outside the
         constant-expression.  */
       if (!ctx->quiet)
-       error ("modification of %qD is not a constant-expression", object);
+       error ("modification of %qE is not a constant-expression", object);
       *non_constant_p = true;
       return t;
     }
diff --git a/gcc/testsuite/g++.dg/cpp1y/constexpr-ref1.C b/gcc/testsuite/g++.dg/cpp1y/constexpr-ref1.C
new file mode 100644 (file)
index 0000000..6f88f82
--- /dev/null
@@ -0,0 +1,9 @@
+// PR c++/64080
+// { dg-do compile { target c++14 } }
+
+constexpr void foo (int&& i) { i = 0; }
+
+void bar(int&& i)
+{
+  bool b = noexcept(foo(static_cast<int&&>(i)));
+}