[PR c++/87814] undefer deferred noexcept on tsubst if request

tsubst_expr and tsubst_copy_and_build are not expected to handle
DEFERRED_NOEXCEPT exprs, but if tsubst_exception_specification takes a
DEFERRED_NOEXCEPT expr with !defer_ok, it just passes the expr on for
tsubst_copy_and_build to barf.

This patch arranges for tsubst_exception_specification to combine the
incoming args with those already stored in a DEFERRED_NOEXCEPT, and
then substitute them into the pattern, when retaining a deferred
noexcept is unacceptable.

for  gcc/cp/ChangeLog

	PR c++/87814
	* pt.c (tsubst_exception_specification): Handle
	DEFERRED_NOEXCEPT with !defer_ok.

for  gcc/testsuite/ChangeLog

	PR c++/87814
	* g++.dg/cpp1z/pr87814.C: New.

From-SVN: r267155

diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog
index 0467ca277613e6f41f9e072a29e749e5394fce97..2dfb1fed6130c5826b02b481fa4e68bfa011e7d1 100644 (file)
--- a/gcc/cp/ChangeLog
+++ b/gcc/cp/ChangeLog
@@ -1,3 +1,9 @@
+2018-12-14  Alexandre Oliva <aoliva@redhat.com>
+
+       PR c++/87814
+       * pt.c (tsubst_exception_specification): Handle
+       DEFERRED_NOEXCEPT with !defer_ok.
+
 2018-12-14  Jason Merrill <jason@redhat.com>
 
        PR c++/86823

diff --git a/gcc/cp/pt.c b/gcc/cp/pt.c
index 3b378ee9ff48b328dec5b0590188270e9ca4b430..20f0d16efe107ebad7c57c22b7d38cd36e2c88ff 100644 (file)
--- a/gcc/cp/pt.c
+++ b/gcc/cp/pt.c
@@ -14151,9 +14151,17 @@ tsubst_exception_specification (tree fntype,
            }
        }
       else
-       new_specs = tsubst_copy_and_build
-         (expr, args, complain, in_decl, /*function_p=*/false,
-          /*integral_constant_expression_p=*/true);
+       {
+         if (DEFERRED_NOEXCEPT_SPEC_P (specs))
+           {
+             args = add_to_template_args (DEFERRED_NOEXCEPT_ARGS (expr),
+                                          args);
+             expr = DEFERRED_NOEXCEPT_PATTERN (expr);
+           }
+         new_specs = tsubst_copy_and_build
+           (expr, args, complain, in_decl, /*function_p=*/false,
+            /*integral_constant_expression_p=*/true);
+       }
       new_specs = build_noexcept_spec (new_specs, complain);
     }
   else if (specs)

diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index a0879161104dc29ff11b9efeeb494499f7d18978..9c24e473e5fe7da36477e3ffe4c4b34bf6f8c612 100644 (file)
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2018-12-14  Alexandre Oliva <aoliva@redhat.com>
+
+       PR c++/87814
+       * g++.dg/cpp1z/pr87814.C: New.
+
 2018-12-14  H.J. Lu  <hongjiu.lu@intel.com>
 
        * gcc.target/i386/cf_check-1.c: New test.

diff --git a/gcc/testsuite/g++.dg/cpp1z/pr87814.C b/gcc/testsuite/g++.dg/cpp1z/pr87814.C
new file mode 100644 (file)
index 0000000..37034bb
--- /dev/null
+++ b/gcc/testsuite/g++.dg/cpp1z/pr87814.C
@@ -0,0 +1,26 @@
+// { dg-do compile { target c++17 } }
+
+template<class Element>
+struct box {
+    template<class E>
+    constexpr box(E && e)
+        noexcept(noexcept(Element(e)))
+    {}
+};
+
+template<class... Ts>
+struct compressed_tuple_ : box<Ts> ... {
+    template<typename... Args>
+    constexpr compressed_tuple_(Args &&... args)
+        noexcept((noexcept(box<Ts>(args)) && ...))
+      : box<Ts>(args)...
+    {}
+};
+
+struct adaptor_cursor : compressed_tuple_<int*> {
+    using compressed_tuple_::compressed_tuple_;
+};
+
+int main() {
+    (void)noexcept(adaptor_cursor{(int*)0});
+}