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)
commit4192ffd74c69e3fb6d761becc8e5117c3de42052
tree48954a81d48f8b5cf2ac3dc60e36dfa1360af84e
parentc68cae86bbe78100f263cc73ba9cbe452778f9e4
c++: Fix tsubst ICE with invalid code [PR97993, PR97187]

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]