maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

Handle typedef function declarations correctly, such as 

typedef int (*Myfunc)(int);
       Myfunc func;

Reviewed by chandlerc

llvm-svn: 107450
This commit is contained in:
Craig Silverstein 2010-07-01 23:46:26 +00:00
parent 4ca8ddaceb
commit 19ebcd0f28
1 changed files with 12 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -977,17 +977,23 @@ bool RecursiveASTVisitor<Derived>::TraverseFunctionHelper(FunctionDecl *D) {
TRY_TO(TraverseNestedNameSpecifier(D->getQualifier())); TRY_TO(TraverseNestedNameSpecifier(D->getQualifier()));
// Visit the function type itself, which can be either // Visit the function type itself, which can be either
// FunctionNoProtoType or FunctionProtoType. // FunctionNoProtoType or FunctionProtoType, or a typedef. If it's
// not a Function*ProtoType, then it can't have a body or arguments,
// so we have to do less work.
Type *FuncType = D->getType().getTypePtr(); Type *FuncType = D->getType().getTypePtr();
if (FunctionNoProtoType *FuncNoProto = if (FunctionProtoType *FuncProto = dyn_cast<FunctionProtoType>(FuncType)) {
// Don't call Traverse*, or the result type and parameter types
// will be double counted.
TRY_TO(WalkUpFromFunctionProtoType(FuncProto));
} else if (FunctionNoProtoType *FuncNoProto =
dyn_cast<FunctionNoProtoType>(FuncType)) { dyn_cast<FunctionNoProtoType>(FuncType)) {
// Don't call Traverse*, or the result type will be double // Don't call Traverse*, or the result type will be double
// counted. // counted.
TRY_TO(WalkUpFromFunctionNoProtoType(FuncNoProto)); TRY_TO(WalkUpFromFunctionNoProtoType(FuncNoProto));
} else { } else { // a typedef type, or who knows what
// Don't call Traverse*, or the result type and parameter types TRY_TO(TraverseType(D->getType()));
// will be double counted. assert(!D->isThisDeclarationADefinition() && "Unexpected function type");
TRY_TO(WalkUpFromFunctionProtoType(dyn_cast<FunctionProtoType>(FuncType))); return true;
} }
TRY_TO(TraverseType(D->getResultType())); TRY_TO(TraverseType(D->getResultType()));