]> granicus.if.org Git - clang/commitdiff
In ParseParenDeclarator match "D.setGroupingParens(true);" with another setGroupingPa...
authorArgyrios Kyrtzidis <akyrtzi@gmail.com>
Tue, 7 Oct 2008 10:21:57 +0000 (10:21 +0000)
committerArgyrios Kyrtzidis <akyrtzi@gmail.com>
Tue, 7 Oct 2008 10:21:57 +0000 (10:21 +0000)
Fixes this bug:
  int (x)(0); // error, expected function declarator where the '(0)' initializer is

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@57241 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Parse/ParseDecl.cpp
test/SemaCXX/direct-initializer.cpp

index 7d1a9bb4b653d5280960b2f658ce104832eec041..c541a1336f4d9e2a9112ccc6739c1bf74edcd9d4 100644 (file)
@@ -1279,6 +1279,7 @@ void Parser::ParseParenDeclarator(Declarator &D) {
   // direct-declarator: '(' declarator ')'
   // direct-declarator: '(' attributes declarator ')'
   if (isGrouping) {
+    bool hadGroupingParens = D.hasGroupingParens();
     D.setGroupingParens(true);
 
     if (Tok.is(tok::kw___attribute))
@@ -1287,6 +1288,8 @@ void Parser::ParseParenDeclarator(Declarator &D) {
     ParseDeclaratorInternal(D);
     // Match the ')'.
     MatchRHSPunctuation(tok::r_paren, StartLoc);
+
+    D.setGroupingParens(hadGroupingParens);
     return;
   }
   
index bb0aab6500a166884322f58e8316ec3d4257a78d..c756e473dd48ef8ee3e13854baeff4d5e7a79951 100644 (file)
@@ -1,8 +1,10 @@
-// RUN: clang -fsyntax-only %s 
+// RUN: clang -fsyntax-only -verify %s 
 
 int x(1);
+int (x2)(1);
 
 void f() {
   int x(1);
+  int (x2)(1); // expected-warning {{statement was disambiguated as declaration}}
   for (int x(1);;) {}
 }