From: Chris Lattner <sabre@nondot.org>
Date: Mon, 5 May 2008 22:18:14 +0000 (+0000)
Subject: Fix rdar://5905347 a crash on invalid builtin, due to the
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=95e2c71181c7ec1ffea0066bbae49e8742bd0687;p=clang

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
---

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);
   
diff --git a/test/Sema/builtins.c b/test/Sema/builtins.c
index ca3cbaf8dc..58cb92f513 100644
--- a/test/Sema/builtins.c
+++ b/test/Sema/builtins.c
@@ -30,3 +30,12 @@ void cfstring() {
   CFSTR("foo", "bar"); // expected-error {{ error: too many arguments to function }}
 }
 
+
+typedef __attribute__(( ext_vector_type(16) )) unsigned char uchar16;  // expected-warning {{extension}}
+
+// rdar://5905347
+unsigned char foo( short v ) {
+  uchar16 c;
+  return __builtin_ia32_vec_ext_v4si( c );  // expected-error {{too few arguments to function}}
+}
+