re PR c++/63283 (constexpr function called by templated function is not treated as...

	PR c++/63283
	* constexpr.c (potential_constant_expression_1): Handle reference
	args in templates.

From-SVN: r219686

diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog
index 543f4d9a4d96eaf8974b2a39223426396964cfa9..7ad7737d582792ec933939db05ca901547391a7b 100644 (file)
--- a/gcc/cp/ChangeLog
+++ b/gcc/cp/ChangeLog
@@ -1,3 +1,9 @@
+2015-01-15  Jason Merrill  <jason@redhat.com>
+
+       PR c++/63283
+       * constexpr.c (potential_constant_expression_1): Handle reference
+       args in templates.
+
 2015-01-15  Thomas Schwinge  <thomas@codesourcery.com>
            James Norris  <jnorris@codesourcery.com>
            Cesar Philippidis  <cesar@codesourcery.com>

diff --git a/gcc/cp/constexpr.c b/gcc/cp/constexpr.c
index 14325069531e280024d8256f6315d6dc9cef1aed..e27a892a51d320ba36558bd06ab591ed06b52683 100644 (file)
--- a/gcc/cp/constexpr.c
+++ b/gcc/cp/constexpr.c
@@ -3881,7 +3881,11 @@ potential_constant_expression_1 (tree t, bool want_rval, bool strict,
         for (; i < nargs; ++i)
           {
             tree x = get_nth_callarg (t, i);
-           if (!RECUR (x, rval))
+           /* In a template, reference arguments haven't been converted to
+              REFERENCE_TYPE and we might not even know if the parameter
+              is a reference, so accept lvalue constants too.  */
+           bool rv = processing_template_decl ? any : rval;
+           if (!RECUR (x, rv))
              return false;
           }
         return true;

diff --git a/gcc/testsuite/g++.dg/cpp0x/constexpr-template8.C b/gcc/testsuite/g++.dg/cpp0x/constexpr-template8.C
new file mode 100644 (file)
index 0000000..7b2b9c7
--- /dev/null
+++ b/gcc/testsuite/g++.dg/cpp0x/constexpr-template8.C
@@ -0,0 +1,10 @@
+// PR c++/63283
+// { dg-do compile { target c++11 } }
+
+constexpr int array_length(int (&array)[3]) { return 3; }
+int a[] = { 1, 2, 3 };
+template <typename T> int f() {
+  struct { int e[array_length(a)]; } t;
+  return sizeof(t);
+}
+int main() { f<void>(); }

diff --git a/gcc/testsuite/g++.dg/cpp0x/static_assert10.C b/gcc/testsuite/g++.dg/cpp0x/static_assert10.C
index 216f2595b6bd9a29c72f75dfb2c4a44326346509..e7f728e3f4fa21a399ea4b4a1aaed44f47b838db 100644 (file)
--- a/gcc/testsuite/g++.dg/cpp0x/static_assert10.C
+++ b/gcc/testsuite/g++.dg/cpp0x/static_assert10.C
@@ -4,5 +4,5 @@
 template<typename T> bool foo(T)
 {
   int i;
-  static_assert(foo(i), "Error"); // { dg-error "non-constant condition|not usable" }
+  static_assert(foo(i), "Error"); // { dg-error "non-constant condition|not usable|non-constexpr" }
 }