Implement a more sensible strategy for ObjC built-in types (addressing a long standing FIXME in Sema::GetObjcIdType()).

This removes several gross hacks to work around the previous "lazy" behavior.

Two notes:
- MinimalActions still needs to be taught about the built-in types (This breaks one of the -noop test cases). I started this, then added a FIXME.
- I didn't convert Sema::GetObjcProtoType() yet.



git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@43567 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/Lex/Preprocessor.cpp b/Lex/Preprocessor.cpp
index c1f4970..0d3ac52 100644
--- a/Lex/Preprocessor.cpp
+++ b/Lex/Preprocessor.cpp
@@ -369,33 +369,7 @@
     DefineBuiltinMacro(Buf, "__OBJC__=1");
   if (PP.getLangOptions().ObjC2)
     DefineBuiltinMacro(Buf, "__OBJC2__=1");
-  if (PP.getLangOptions().ObjC1) {
-    const char *ObjcType;
-    // Predefine all the ObjC goodies (traditionally declared in <objc/objc.h>).
-    // We define the following header guard for source compatibility. It has
-    // the effect of ignoring any explicit inclusion of <objc/objc.h>:-)
-    DefineBuiltinMacro(Buf, "_OBJC_OBJC_H_=1");
-    DefineBuiltinMacro(Buf, "OBJC_EXPORT=extern");
-    DefineBuiltinMacro(Buf, "OBJC_IMPORT=extern");
-    ObjcType = "typedef struct objc_class *Class;\n";
-    Buf.insert(Buf.end(), ObjcType, ObjcType+strlen(ObjcType));
-    ObjcType = "typedef struct objc_object { Class isa; } *id;\n";
-    Buf.insert(Buf.end(), ObjcType, ObjcType+strlen(ObjcType));
-    ObjcType = "typedef struct objc_selector *SEL;\n";
-    Buf.insert(Buf.end(), ObjcType, ObjcType+strlen(ObjcType));
-    ObjcType = "typedef id (*IMP)(id, SEL, ...);\n";
-    Buf.insert(Buf.end(), ObjcType, ObjcType+strlen(ObjcType));
-    ObjcType = "typedef signed char BOOL;\n";
-    Buf.insert(Buf.end(), ObjcType, ObjcType+strlen(ObjcType));
-    DefineBuiltinMacro(Buf, "YES=(BOOL)1");
-    DefineBuiltinMacro(Buf, "NO=(BOOL)0");
-    DefineBuiltinMacro(Buf, "Nil=0");
-    DefineBuiltinMacro(Buf, "nil=0");
-    ObjcType = "extern const char *sel_getName(SEL sel);\n";
-    Buf.insert(Buf.end(), ObjcType, ObjcType+strlen(ObjcType));
-    ObjcType = "extern SEL sel_getUid(const char *str);\n";
-    Buf.insert(Buf.end(), ObjcType, ObjcType+strlen(ObjcType));
-  }
+
   // Add __builtin_va_list typedef.
   {
     const char *VAList = PP.getTargetInfo().getVAListDeclaration();