diff options
author | Chris Lattner <sabre@nondot.org> | 2008-05-05 22:18:14 +0000 |
---|---|---|
committer | Chris Lattner <sabre@nondot.org> | 2008-05-05 22:18:14 +0000 |
commit | 95e2c71181c7ec1ffea0066bbae49e8742bd0687 (patch) | |
tree | d5ba3277134136539193040efbb46f018cc6fc7b /lib/Sema/SemaDecl.cpp | |
parent | d3dbcf4be7025fae5ce71125ec6d217d1ea3db8f (diff) |
Fix rdar://5905347 a crash on invalid builtin, due to the
params not getting installed for builtins when synthesized.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@50676 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Sema/SemaDecl.cpp')
-rw-r--r-- | lib/Sema/SemaDecl.cpp | 20 |
1 files changed, 16 insertions, 4 deletions
diff --git a/lib/Sema/SemaDecl.cpp b/lib/Sema/SemaDecl.cpp index f583ad1606..2624f5019d 100644 --- a/lib/Sema/SemaDecl.cpp +++ b/lib/Sema/SemaDecl.cpp @@ -143,8 +143,7 @@ Decl *Sema::LookupDecl(const IdentifierInfo *II, unsigned NSI, return 0; } -void Sema::InitBuiltinVaListType() -{ +void Sema::InitBuiltinVaListType() { if (!Context.getBuiltinVaListType().isNull()) return; @@ -161,8 +160,8 @@ ScopedDecl *Sema::LazilyCreateBuiltin(IdentifierInfo *II, unsigned bid, Builtin::ID BID = (Builtin::ID)bid; if (BID == Builtin::BI__builtin_va_start || - BID == Builtin::BI__builtin_va_copy || - BID == Builtin::BI__builtin_va_end) + BID == Builtin::BI__builtin_va_copy || + BID == Builtin::BI__builtin_va_end) InitBuiltinVaListType(); QualType R = Context.BuiltinInfo.GetBuiltinType(BID, Context); @@ -171,6 +170,19 @@ ScopedDecl *Sema::LazilyCreateBuiltin(IdentifierInfo *II, unsigned bid, SourceLocation(), II, R, FunctionDecl::Extern, false, 0); + // Create Decl objects for each parameter, adding them to the + // FunctionDecl. + if (FunctionTypeProto *FT = dyn_cast<FunctionTypeProto>(R)) { + llvm::SmallVector<ParmVarDecl*, 16> Params; + for (unsigned i = 0, e = FT->getNumArgs(); i != e; ++i) + Params.push_back(ParmVarDecl::Create(Context, New, SourceLocation(), 0, + FT->getArgType(i), VarDecl::None, 0, + 0)); + New->setParams(&Params[0], Params.size()); + } + + + // TUScope is the translation-unit scope to insert this function into. TUScope->AddDecl(New); |