[cfe-commits] r48302 - /cfe/trunk/Driver/RewriteTest.cpp

[Steve Naroff]

Author: snaroff
Date: Wed Mar 12 16:09:20 2008
New Revision: 48302

URL: http://llvm.org/viewvc/llvm-project?rev=48302&view=rev
Log:
Avoid using the "unnamed struct field" extension (enabled with -fms-extensions). This feature/extension silently does the wrong thing in g++. As far as I can tell, g++ simply throws the field away entirely (note that it works fine with gcc). Since I am now always casting the object (for other reasons), accessing protected/public fields simply requires the cast refer to the defining class. This solution is simpler all around (thanks to Chris for suggesting it).

Modified:
    cfe/trunk/Driver/RewriteTest.cpp

Modified: cfe/trunk/Driver/RewriteTest.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/Driver/RewriteTest.cpp?rev=48302&r1=48301&r2=48302&view=diff

==============================================================================
--- cfe/trunk/Driver/RewriteTest.cpp (original)
+++ cfe/trunk/Driver/RewriteTest.cpp Wed Mar 12 16:09:20 2008
@@ -774,7 +774,11 @@
     if (const PointerType *pType = IV->getBase()->getType()->getAsPointerType()) {
       ObjCInterfaceType *intT = dyn_cast<ObjCInterfaceType>(pType->getPointeeType());
       if (CurMethodDecl->getClassInterface() == intT->getDecl()) {
-        std::string RecName = intT->getDecl()->getIdentifier()->getName();
+        // lookup which class implements the instance variable.
+        ObjCInterfaceDecl *clsDeclared = 0;
+        intT->getDecl()->lookupInstanceVariable(D->getIdentifier(), clsDeclared);
+        assert(clsDeclared && "RewriteObjCIvarRefExpr(): Can't find class");
+        std::string RecName = clsDeclared->getIdentifier()->getName();
         RecName += "_IMPL";
         IdentifierInfo *II = &Context->Idents.get(RecName.c_str());
         RecordDecl *RD = new RecordDecl(Decl::Struct, SourceLocation(), II, 0);
@@ -2189,14 +2193,9 @@
     if (RCDecl && ObjCSynthesizedStructs.count(RCDecl)) {
       Result = "\n    struct ";
       Result += RCDecl->getName();
-      if (LangOpts.Microsoft)
-        Result += "_IMPL";
-
-      // Note: We don't name the field decl. This simplifies the "codegen" for
-      // accessing a superclasses instance variables (and is similar to what gcc
-      // does internally). The unnamed struct field feature is enabled with
-      // -fms-extensions. If the struct definition were "inlined", we wouldn't
-      // need to use this switch. That said, I don't want to inline the def.
+      Result += "_IMPL ";
+      Result += RCDecl->getName();
+      Result += "_IVARS";
       Result += ";\n";
       
       // insert the super class structure definition.
@@ -2240,14 +2239,9 @@
     endBuf += Lexer::MeasureTokenLength(LocEnd, *SM);
     Result += " {\n    struct ";
     Result += RCDecl->getName();
-    if (LangOpts.Microsoft)
-      Result += "_IMPL";
-
-    // Note: We don't name the field decl. This simplifies the "codegen" for
-    // accessing a superclasses instance variables (and is similar to what gcc
-    // does internally). The unnamed struct field feature is enabled with
-    // -fms-extensions. If the struct definition were "inlined", we wouldn't
-    // need to use this switch. That said, I don't want to inline the def.
+    Result += "_IMPL ";
+    Result += RCDecl->getName();
+    Result += "_IVARS";
     Result += ";\n};\n";
     ReplaceText(LocStart, endBuf-startBuf, Result.c_str(), Result.size());
   }