[cfe-commits] r118451 - in /cfe/trunk: include/clang/Basic/Builtins.def lib/AST/ASTContext.cpp test/CodeGenObjC/implicit-objc_msgSend.m test/SemaObjC/builtin_objc_msgSend.m
Fariborz Jahanian
fjahanian at apple.com
Mon Nov 8 14:53:18 PST 2010
Author: fjahanian
Date: Mon Nov 8 16:53:18 2010
New Revision: 118451
URL: http://llvm.org/viewvc/llvm-project?rev=118451&view=rev
Log:
Provide a precise builtin declaration for objc_msgSend
to avoid a bogus warning. Fixes //rdar: //8632525
Added:
cfe/trunk/test/SemaObjC/builtin_objc_msgSend.m
Modified:
cfe/trunk/include/clang/Basic/Builtins.def
cfe/trunk/lib/AST/ASTContext.cpp
cfe/trunk/test/CodeGenObjC/implicit-objc_msgSend.m
Modified: cfe/trunk/include/clang/Basic/Builtins.def
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/Builtins.def?rev=118451&r1=118450&r2=118451&view=diff
==============================================================================
--- cfe/trunk/include/clang/Basic/Builtins.def (original)
+++ cfe/trunk/include/clang/Basic/Builtins.def Mon Nov 8 16:53:18 2010
@@ -29,6 +29,8 @@
// d -> double
// z -> size_t
// F -> constant CFString
+// G -> id
+// H -> SEL
// a -> __builtin_va_list
// A -> "reference" to __builtin_va_list
// V -> Vector, following num elements and a base type.
@@ -559,11 +561,8 @@
// POSIX setjmp.h
LIBBUILTIN(_longjmp, "vJi", "fr", "setjmp.h")
LIBBUILTIN(siglongjmp, "vSJi", "fr", "setjmp.h")
-
-// FIXME: This type isn't very correct, it should be
// id objc_msgSend(id, SEL)
-// but we need new type letters for that.
-LIBBUILTIN(objc_msgSend, "v*.", "f", "objc/message.h")
+LIBBUILTIN(objc_msgSend, "GGH.", "f", "objc/message.h")
BUILTIN(__builtin_objc_memmove_collectable, "v*v*vC*z", "nF")
// Builtin math library functions
Modified: cfe/trunk/lib/AST/ASTContext.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/AST/ASTContext.cpp?rev=118451&r1=118450&r2=118451&view=diff
==============================================================================
--- cfe/trunk/lib/AST/ASTContext.cpp (original)
+++ cfe/trunk/lib/AST/ASTContext.cpp Mon Nov 8 16:53:18 2010
@@ -5384,6 +5384,12 @@
case 'F':
Type = Context.getCFConstantStringType();
break;
+ case 'G':
+ Type = Context.getObjCIdType();
+ break;
+ case 'H':
+ Type = Context.getObjCSelType();
+ break;
case 'a':
Type = Context.getBuiltinVaListType();
assert(!Type.isNull() && "builtin va list type not initialized!");
Modified: cfe/trunk/test/CodeGenObjC/implicit-objc_msgSend.m
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/CodeGenObjC/implicit-objc_msgSend.m?rev=118451&r1=118450&r2=118451&view=diff
==============================================================================
--- cfe/trunk/test/CodeGenObjC/implicit-objc_msgSend.m (original)
+++ cfe/trunk/test/CodeGenObjC/implicit-objc_msgSend.m Mon Nov 8 16:53:18 2010
@@ -1,5 +1,5 @@
// RUN: %clang_cc1 -triple x86_64-apple-darwin9 -emit-llvm -o %t %s
-// RUN: grep -F 'declare i8* @objc_msgSend(...)' %t
+// RUN: grep -F 'declare i8* @objc_msgSend(i8*, i8*, ...)' %t
typedef struct objc_selector *SEL;
id f0(id x, SEL s) {
Added: cfe/trunk/test/SemaObjC/builtin_objc_msgSend.m
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/SemaObjC/builtin_objc_msgSend.m?rev=118451&view=auto
==============================================================================
--- cfe/trunk/test/SemaObjC/builtin_objc_msgSend.m (added)
+++ cfe/trunk/test/SemaObjC/builtin_objc_msgSend.m Mon Nov 8 16:53:18 2010
@@ -0,0 +1,3 @@
+// RUN: %clang_cc1 %s -fsyntax-only -verify
+// rdar: // 8632525
+extern id objc_msgSend(id self, SEL op, ...);
More information about the cfe-commits
mailing list