CFieldCallback doesn't need to create an ExtensionRAIIObject: it's actually

automatically shadowed by the ExtensionRAIIObject created by
ParseStructDeclaration.

llvm-svn: 85941

1 files changed, 3 insertions, 14 deletions

diff --git a/clang/lib/Parse/ParseDecl.cpp b/clang/lib/Parse/ParseDecl.cpp
index be0df330671..a47065c2b54 100644
--- a/clang/lib/Parse/ParseDecl.cpp
+++ b/clang/lib/Parse/ParseDecl.cpp
@@ -1605,21 +1605,10 @@ void Parser::ParseStructUnionBody(SourceLocation RecordLoc,
           P(P), TagDecl(TagDecl), FieldDecls(FieldDecls) {}
 
         virtual DeclPtrTy invoke(FieldDeclarator &FD) {
-          const DeclSpec &DS = FD.D.getDeclSpec();
-          DeclPtrTy Field;
-
           // Install the declarator into the current TagDecl.
-          if (FD.D.getExtension()) {
-            // Silences extension warnings
-            ExtensionRAIIObject O(P.Diags);
-            Field = P.Actions.ActOnField(P.CurScope, TagDecl,
-                                         DS.getSourceRange().getBegin(),
-                                         FD.D, FD.BitfieldSize);
-          } else {
-            Field = P.Actions.ActOnField(P.CurScope, TagDecl,
-                                         DS.getSourceRange().getBegin(),
-                                         FD.D, FD.BitfieldSize);
-          }
+          DeclPtrTy Field = P.Actions.ActOnField(P.CurScope, TagDecl,
+                              FD.D.getDeclSpec().getSourceRange().getBegin(),
+                                                 FD.D, FD.BitfieldSize);
           FieldDecls.push_back(Field);
           return Field;
         }