[cfe-commits] r100237 - /cfe/trunk/lib/CodeGen/CGObjCMac.cpp

Daniel Dunbar daniel at zuster.org
Fri Apr 2 15:29:40 PDT 2010


Author: ddunbar
Date: Fri Apr  2 17:29:40 2010
New Revision: 100237

URL: http://llvm.org/viewvc/llvm-project?rev=100237&view=rev
Log:
IRgen/NeXT: Remove my refactoring-enabling asserts, and add a FIXME.

Modified:
    cfe/trunk/lib/CodeGen/CGObjCMac.cpp

Modified: cfe/trunk/lib/CodeGen/CGObjCMac.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/CGObjCMac.cpp?rev=100237&r1=100236&r2=100237&view=diff
==============================================================================
--- cfe/trunk/lib/CodeGen/CGObjCMac.cpp (original)
+++ cfe/trunk/lib/CodeGen/CGObjCMac.cpp Fri Apr  2 17:29:40 2010
@@ -48,18 +48,9 @@
                                      const ObjCIvarDecl *Ivar) {
   const ObjCInterfaceDecl *Container = Ivar->getContainingInterface();
 
-  // Check that the Obj-C decl contexts match what we expect.
-  const ObjCContainerDecl *DC = cast<ObjCContainerDecl>(Ivar->getDeclContext());
-  assert(isa<ObjCInterfaceDecl>(DC) || isa<ObjCImplDecl>(DC) ||
-         isa<ObjCCategoryDecl>(DC));
-  if (isa<ObjCImplDecl>(DC)) {
-    assert(DC == ID);
-    assert(Container == cast<ObjCImplDecl>(DC)->getClassInterface());
-  } else if (isa<ObjCCategoryDecl>(DC)) {
-    assert(Container == cast<ObjCCategoryDecl>(DC)->getClassInterface());
-  }  else {
-    assert(Container == DC);
-  }
+  // FIXME: We should eliminate the need to have ObjCImplementationDecl passed
+  // in here; it should never be necessary because that should be the lexical
+  // decl context for the ivar.
 
   // If we know have an implementation (and the ivar is in it) then
   // look up in the implementation layout.





More information about the cfe-commits mailing list