[cfe-commits] r54567 - in /cfe/trunk: include/clang/Parse/Action.h lib/Parse/ParseDeclCXX.cpp lib/Sema/Sema.h lib/Sema/SemaDeclCXX.cpp

Argiris Kirtzidis akyrtzi at gmail.com
Fri Aug 8 17:39:29 PDT 2008


Author: akirtzidis
Date: Fri Aug  8 19:39:29 2008
New Revision: 54567

URL: http://llvm.org/viewvc/llvm-project?rev=54567&view=rev
Log:
Passing right brace location to ActOnFinishCXXClassDef is redundant, since it gets passed to ActOnFinishCXXMemberSpecification too.

Modified:
    cfe/trunk/include/clang/Parse/Action.h
    cfe/trunk/lib/Parse/ParseDeclCXX.cpp
    cfe/trunk/lib/Sema/Sema.h
    cfe/trunk/lib/Sema/SemaDeclCXX.cpp

Modified: cfe/trunk/include/clang/Parse/Action.h
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Parse/Action.h?rev=54567&r1=54566&r2=54567&view=diff

==============================================================================
--- cfe/trunk/include/clang/Parse/Action.h (original)
+++ cfe/trunk/include/clang/Parse/Action.h Fri Aug  8 19:39:29 2008
@@ -613,7 +613,7 @@
 
   /// ActOnFinishCXXClassDef - This is called when a class/struct/union has
   /// completed parsing, when on C++.
-  virtual void ActOnFinishCXXClassDef(DeclTy *TagDecl,SourceLocation RBrace) {
+  virtual void ActOnFinishCXXClassDef(DeclTy *TagDecl) {
   }
 
   //===----------------------- Obj-C Declarations -------------------------===//

Modified: cfe/trunk/lib/Parse/ParseDeclCXX.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Parse/ParseDeclCXX.cpp?rev=54567&r1=54566&r2=54567&view=diff

==============================================================================
--- cfe/trunk/lib/Parse/ParseDeclCXX.cpp (original)
+++ cfe/trunk/lib/Parse/ParseDeclCXX.cpp Fri Aug  8 19:39:29 2008
@@ -626,5 +626,5 @@
   // Leave the class scope.
   ExitScope();
 
-  Actions.ActOnFinishCXXClassDef(TagDecl, RBraceLoc);
+  Actions.ActOnFinishCXXClassDef(TagDecl);
 }

Modified: cfe/trunk/lib/Sema/Sema.h
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/Sema.h?rev=54567&r1=54566&r2=54567&view=diff

==============================================================================
--- cfe/trunk/lib/Sema/Sema.h (original)
+++ cfe/trunk/lib/Sema/Sema.h Fri Aug  8 19:39:29 2008
@@ -603,7 +603,7 @@
                                                  SourceLocation LBrac,
                                                  SourceLocation RBrac);
 
-  virtual void ActOnFinishCXXClassDef(DeclTy *TagDecl,SourceLocation RBrace);
+  virtual void ActOnFinishCXXClassDef(DeclTy *TagDecl);
   
 
   // Objective-C declarations.

Modified: cfe/trunk/lib/Sema/SemaDeclCXX.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/SemaDeclCXX.cpp?rev=54567&r1=54566&r2=54567&view=diff

==============================================================================
--- cfe/trunk/lib/Sema/SemaDeclCXX.cpp (original)
+++ cfe/trunk/lib/Sema/SemaDeclCXX.cpp Fri Aug  8 19:39:29 2008
@@ -468,7 +468,7 @@
               FieldCollector->getCurNumFields(), LBrac, RBrac);
 }
 
-void Sema::ActOnFinishCXXClassDef(DeclTy *D,SourceLocation RBrace) {
+void Sema::ActOnFinishCXXClassDef(DeclTy *D) {
   Decl *Dcl = static_cast<Decl *>(D);
   assert(isa<CXXRecordDecl>(Dcl) &&
          "Invalid parameter, expected CXXRecordDecl");





More information about the cfe-commits mailing list