aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMarek Polacek <polacek@redhat.com>2020-06-09 18:08:45 -0400
committerMarek Polacek <polacek@redhat.com>2020-06-10 09:36:02 -0400
commit4fed5d5dd85e3f5d812d125f692351646a0417cb (patch)
treee63cab13fc0b63a916193c7171b407ccdd66de2b
parent2db262f29a4b1495885b1f23335b6dc363bf1143 (diff)
downloadgcc-4fed5d5dd85e3f5d812d125f692351646a0417cb.zip
gcc-4fed5d5dd85e3f5d812d125f692351646a0417cb.tar.gz
gcc-4fed5d5dd85e3f5d812d125f692351646a0417cb.tar.bz2
c++: Fix ICE with delayed parsing of noexcept-specifier [PR95562]
Here we ICE because a DEFERRED_PARSE expression leaked to tsubst_copy. We create these expressions for deferred noexcept-specifiers in cp_parser_save_noexcept; they are supposed to be re-parsed in cp_parser_late_noexcept_specifier. In this case we never got around to re-parsing it because the noexcept-specifier was attached to a pointer to a function, not to a function declaration. But we should not have delayed the parsing here in the first place; we already avoid delaying the parsing for alias-decls, typedefs, and friend function declarations. (Clang++ also doesn't delay the parsing for pointers to function.) gcc/cp/ChangeLog: PR c++/95562 * parser.c (cp_parser_direct_declarator): Clear CP_PARSER_FLAGS_DELAY_NOEXCEPT if the declarator kind is not cdk_id. gcc/testsuite/ChangeLog: PR c++/95562 * g++.dg/cpp0x/noexcept60.C: New test.
-rw-r--r--gcc/cp/parser.c5
-rw-r--r--gcc/testsuite/g++.dg/cpp0x/noexcept60.C13
2 files changed, 18 insertions, 0 deletions
diff --git a/gcc/cp/parser.c b/gcc/cp/parser.c
index b0b31d2..bc66e6e5 100644
--- a/gcc/cp/parser.c
+++ b/gcc/cp/parser.c
@@ -21283,6 +21283,11 @@ cp_parser_direct_declarator (cp_parser* parser,
/* DR 1207: 'this' is in scope after the cv-quals. */
inject_this_parameter (current_class_type, cv_quals);
+ /* If it turned out that this is e.g. a pointer to a
+ function, we don't want to delay noexcept parsing. */
+ if (declarator == NULL || declarator->kind != cdk_id)
+ flags &= ~CP_PARSER_FLAGS_DELAY_NOEXCEPT;
+
/* Parse the exception-specification. */
exception_specification
= cp_parser_exception_specification_opt (parser,
diff --git a/gcc/testsuite/g++.dg/cpp0x/noexcept60.C b/gcc/testsuite/g++.dg/cpp0x/noexcept60.C
new file mode 100644
index 0000000..d8efe1a
--- /dev/null
+++ b/gcc/testsuite/g++.dg/cpp0x/noexcept60.C
@@ -0,0 +1,13 @@
+// PR c++/95562
+// { dg-do compile { target c++11 } }
+
+template <bool Nothrow>
+struct Functions
+{
+ void (*func)(void*) noexcept(Nothrow);
+};
+
+void test()
+{
+ Functions<true> f{};
+}