replica
/
hanchenye-llvm-project
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Fix RecursiveASTVisitor to traverse the ctor initializer list before 

traversing the ctor body when traversing a CXXConstructorDecl.
Reviewed by chandlerc and csilvers.

llvm-svn: 107304
This commit is contained in:
Zhanyong Wan 2010-06-30 18:34:52 +00:00
parent 0f19afb9d4
commit 88d5a91f0d
1 changed files with 11 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
clang/include/clang/AST/RecursiveASTVisitor.h
View File

@ -992,6 +992,16 @@ bool RecursiveASTVisitor<Derived>::TraverseFunctionHelper(FunctionDecl *D) {
TRY_TO(TraverseType(D->getResultType()));
TRY_TO(TraverseDeclContextHelper(D)); // Parameters.
if (CXXConstructorDecl *Ctor = dyn_cast<CXXConstructorDecl>(D)) {
// Constructor initializers.
for (CXXConstructorDecl::init_iterator I = Ctor->init_begin(),
E = Ctor->init_end();
I != E; ++I) {
TRY_TO(TraverseConstructorInitializer(*I));
}
}
if (D->isThisDeclarationADefinition()) {
TRY_TO(TraverseStmt(D->getBody())); // Function body.
}
@ -1011,18 +1021,9 @@ DEF_TRAVERSE_DECL(CXXMethodDecl, {
})
DEF_TRAVERSE_DECL(CXXConstructorDecl, {
TRY_TO(TraverseFunctionHelper(D));
// FIXME: traverse the initializers before traversing the
// constructor body.
for (CXXConstructorDecl::init_iterator I = D->init_begin(),
E = D->init_end();
I != E; ++I) {
TRY_TO(TraverseConstructorInitializer(*I));
}
// We skip decls_begin/decls_end, which are already covered by
// TraverseFunctionHelper().
return true;
return TraverseFunctionHelper(D);
})
// CXXConversionDecl is the declaration of a type conversion operator.