c++: Fix tsubst ICE with invalid code [PR97993, PR97187]
authorMarek Polacek <polacek@redhat.com>
Mon, 30 Nov 2020 19:53:24 +0000 (14:53 -0500)
committerMarek Polacek <polacek@redhat.com>
Wed, 2 Dec 2020 19:34:38 +0000 (14:34 -0500)
I had a strong sense of deja vu when looking into this, and no wonder,
since this is almost identical to c++/95728.

Since r11-423 tsubst_copy_and_build/TREE_LIST uses tsubst_tree_list
instead of open coding it.  While the latter could return an error
node wrapped in a TREE_LIST, the former can return a naked error node.

That broke in tsubst_copy_and_build/NEW_EXPR, because we were accessing
TREE_VALUE of an error node.

gcc/cp/ChangeLog:

PR c++/97187
PR c++/97993
* pt.c (tsubst_copy_and_build) <case NEW_EXPR>: Return error_mark_node
if init is erroneous.

gcc/testsuite/ChangeLog:

PR c++/97187
PR c++/97993
* g++.dg/eh/crash2.C: New test.
* g++.dg/template/crash132.C: New test.

gcc/cp/pt.c
gcc/testsuite/g++.dg/eh/crash2.C [new file with mode: 0644]
gcc/testsuite/g++.dg/template/crash132.C [new file with mode: 0644]

index 4fb0bc82c31e499f4be07a8179274d1e954a59ff..72d6cc3ad9854c920fc020b35423c19e198037da 100644 (file)
@@ -19829,6 +19829,8 @@ tsubst_copy_and_build (tree t,
           parameter packs are of length zero.  */
        if (init == NULL_TREE && TREE_OPERAND (t, 3) == NULL_TREE)
          init_vec = NULL;
+       else if (init == error_mark_node)
+         RETURN (error_mark_node);
        else
          {
            init_vec = make_tree_vector ();
diff --git a/gcc/testsuite/g++.dg/eh/crash2.C b/gcc/testsuite/g++.dg/eh/crash2.C
new file mode 100644 (file)
index 0000000..fff8e14
--- /dev/null
@@ -0,0 +1,20 @@
+// PR c++/97187
+// { dg-do compile { target c++14 } }
+// { dg-options "-fno-exceptions" }
+
+auto yp = [] { return 0; };
+
+template <class DI>
+DI
+zl ()
+{
+  auto au = [] () -> DI { return *new auto (true ? yp : throw); }; // { dg-error "exception handling disabled" }
+
+  return au ();
+}
+
+auto
+vd ()
+{
+  return zl <decltype (yp)> ();
+}
diff --git a/gcc/testsuite/g++.dg/template/crash132.C b/gcc/testsuite/g++.dg/template/crash132.C
new file mode 100644 (file)
index 0000000..f6f4863
--- /dev/null
@@ -0,0 +1,6 @@
+// PR c++/97993
+// { dg-do compile { target c++14 } }
+
+template <class T> T a;
+template <class T, class A, class=decltype(::new T (A()))> auto foo ();
+struct S decltype (foo <int, S>); // { dg-error "" }