objc_msgSend is not a builtin type in non-objc mode.

Fixes //rdar://8686888

llvm-svn: 119813

2 files changed, 21 insertions, 2 deletions

diff --git a/clang/lib/Sema/SemaExpr.cpp b/clang/lib/Sema/SemaExpr.cpp
index b70486a1b81..bc811d0e77b 100644
--- a/clang/lib/Sema/SemaExpr.cpp
+++ b/clang/lib/Sema/SemaExpr.cpp
@@ -4295,8 +4295,13 @@ Sema::BuildResolvedCallExpr(Expr *Fn, NamedDecl *NDecl,
     if (CheckFunctionCall(FDecl, TheCall))
       return ExprError();
 
-    if (unsigned BuiltinID = FDecl->getBuiltinID())
-      return CheckBuiltinFunctionCall(BuiltinID, TheCall);
+    if (unsigned BuiltinID = FDecl->getBuiltinID()) {
+      // When not in Objective-C mode, there is no builtin 'id' type.
+      // We won't have pre-defined library functions which use this type.
+      if (getLangOptions().ObjC1 ||
+          Context.BuiltinInfo.GetTypeString(BuiltinID)[0] != 'G')
+        return CheckBuiltinFunctionCall(BuiltinID, TheCall);
+    }
   } else if (NDecl) {
     if (CheckBlockCall(NDecl, TheCall))
       return ExprError();
diff --git a/clang/test/SemaCXX/builtin_objc_msgSend.cpp b/clang/test/SemaCXX/builtin_objc_msgSend.cpp
new file mode 100644
index 00000000000..0e90d54f804
--- /dev/null
+++ b/clang/test/SemaCXX/builtin_objc_msgSend.cpp
@@ -0,0 +1,14 @@
+// RUN: %clang_cc1 %s -fsyntax-only -verify
+// rdar://8686888
+
+typedef struct objc_selector *SEL;
+typedef struct objc_object *id;
+
+extern "C" __attribute__((visibility("default"))) id objc_msgSend(id self, SEL op, ...)
+    __attribute__((visibility("default")));
+
+inline void TCFReleaseGC(void * object)
+{
+ static SEL SEL_release;
+ objc_msgSend((id)object, SEL_release);
+}