[cfe-commits] r152566 - in /cfe/trunk: lib/Sema/SemaDeclObjC.cpp test/Index/pch-with-errors.m

[Argyrios Kyrtzidis]

Author: akirtzidis
Date: Mon Mar 12 13:34:26 2012
New Revision: 152566

URL: http://llvm.org/viewvc/llvm-project?rev=152566&view=rev
Log:
[Sema] -Add an invalid objc category to the DeclContext so that it can be
serialized
-Don't add methods of invalid objc containers to the global method pool.
 This protects us from trying to serialize a method whose container was not
 serialized.

Part of rdar://11007039.

Added:
    cfe/trunk/test/Index/pch-with-errors.m
Modified:
    cfe/trunk/lib/Sema/SemaDeclObjC.cpp

Modified: cfe/trunk/lib/Sema/SemaDeclObjC.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/SemaDeclObjC.cpp?rev=152566&r1=152565&r2=152566&view=diff
==============================================================================
--- cfe/trunk/lib/Sema/SemaDeclObjC.cpp (original)
+++ cfe/trunk/lib/Sema/SemaDeclObjC.cpp Mon Mar 12 13:34:26 2012
@@ -766,6 +766,7 @@
     CDecl = ObjCCategoryDecl::Create(Context, CurContext, AtInterfaceLoc,
                                      ClassLoc, CategoryLoc, CategoryName,IDecl);
     CDecl->setInvalidDecl();
+    CurContext->addDecl(CDecl);
         
     if (!IDecl)
       Diag(ClassLoc, diag::err_undef_interface) << ClassName;
@@ -2018,6 +2019,10 @@
 
 void Sema::AddMethodToGlobalPool(ObjCMethodDecl *Method, bool impl,
                                  bool instance) {
+  // Ignore methods of invalid containers.
+  if (cast<Decl>(Method->getDeclContext())->isInvalidDecl())
+    return;
+
   if (ExternalSource)
     ReadMethodPool(Method->getSelector());
   

Added: cfe/trunk/test/Index/pch-with-errors.m
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Index/pch-with-errors.m?rev=152566&view=auto
==============================================================================
--- cfe/trunk/test/Index/pch-with-errors.m (added)
+++ cfe/trunk/test/Index/pch-with-errors.m Mon Mar 12 13:34:26 2012
@@ -0,0 +1,28 @@
+#ifndef HEADER
+#define HEADER
+
+ at interface I(cat)
+-(void)meth;
+ at end
+
+ at interface I2
+-(void)meth;
+ at end
+
+#else
+
+void foo(I2 *i) {
+  [i meth];
+}
+
+#endif
+
+// RUN: c-index-test -write-pch %t.h.pch %s -Xclang -detailed-preprocessing-record
+// RUN: c-index-test -test-load-source local %s -include %t.h -Xclang -detailed-preprocessing-record | FileCheck -check-prefix=CHECK-PARSE %s
+// RUN: c-index-test -index-file %s -include %t.h -Xclang -detailed-preprocessing-record | FileCheck -check-prefix=CHECK-INDEX %s
+
+// CHECK-PARSE: pch-with-errors.m:{{.*}} FunctionDecl=foo
+// CHECK-PARSE: pch-with-errors.m:{{.*}} ObjCMessageExpr=meth
+
+// CHECK-INDEX: [indexDeclaration]: kind: function | name: foo
+// CHECK-INDEX: [indexEntityReference]: kind: objc-instance-method | name: meth