aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMarek Polacek <polacek@redhat.com>2022-04-27 18:17:54 -0400
committerMarek Polacek <polacek@redhat.com>2022-04-28 08:37:45 -0400
commit851031b2fcd5210b96769c440db10130478d273c (patch)
treecb6706d1dc389ede6ac33d80d8cf188b3edeadc9
parent4e117418fb71f508c479e0144500f4da9cc92520 (diff)
downloadgcc-851031b2fcd5210b96769c440db10130478d273c.zip
gcc-851031b2fcd5210b96769c440db10130478d273c.tar.gz
gcc-851031b2fcd5210b96769c440db10130478d273c.tar.bz2
c++: global-namespace-qualified var after class def [PR90107]
Here we wrongly reject the definition of "::N::a" struct A; namespace N { extern A a; } struct A {} ::N::a; because our code to diagnose a missing ; after a class definition doesn't realize that :: can follow a class definition. PR c++/90107 gcc/cp/ChangeLog: * parser.cc (cp_parser_class_specifier_1): Accept :: after a class definition. gcc/testsuite/ChangeLog: * g++.dg/parse/qualified6.C: New test.
-rw-r--r--gcc/cp/parser.cc1
-rw-r--r--gcc/testsuite/g++.dg/parse/qualified6.C10
2 files changed, 11 insertions, 0 deletions
diff --git a/gcc/cp/parser.cc b/gcc/cp/parser.cc
index 169e6a6..2235da1 100644
--- a/gcc/cp/parser.cc
+++ b/gcc/cp/parser.cc
@@ -25933,6 +25933,7 @@ cp_parser_class_specifier_1 (cp_parser* parser)
case CPP_OPEN_PAREN:
case CPP_CLOSE_PAREN:
case CPP_COMMA:
+ case CPP_SCOPE:
want_semicolon = false;
break;
diff --git a/gcc/testsuite/g++.dg/parse/qualified6.C b/gcc/testsuite/g++.dg/parse/qualified6.C
new file mode 100644
index 0000000..68b51f7
--- /dev/null
+++ b/gcc/testsuite/g++.dg/parse/qualified6.C
@@ -0,0 +1,10 @@
+// PR c++/90107
+// { dg-do compile }
+
+struct A;
+namespace N { extern A a; }
+struct A {} ::N::a;
+
+struct A1;
+struct B { static A1 a1; };
+struct A1 {} ::B::a1;