]> granicus.if.org Git - clang/commitdiff
Omit the field name when including the super class structure template.
authorSteve Naroff <snaroff@apple.com>
Fri, 7 Dec 2007 22:15:58 +0000 (22:15 +0000)
committerSteve Naroff <snaroff@apple.com>
Fri, 7 Dec 2007 22:15:58 +0000 (22:15 +0000)
This allows us to access a superclasses ivars without deriving the absolute path.

The comments below say a bit more...

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@44688 91177308-0d34-0410-b5e6-96231b3b80d8

Driver/RewriteTest.cpp

index 2a189ad85b34c682d9c966167ab204ef673e1571..3ec5a10b2c45ed6f2dafeb55cd60a243ae2a6701 100644 (file)
@@ -1648,8 +1648,11 @@ void RewriteTest::SynthesizeObjcInternalStruct(ObjcInterfaceDecl *CDecl,
     if (RCDecl && ObjcSynthesizedStructs.count(RCDecl)) {
       Result = "\n    struct ";
       Result += RCDecl->getName();
-      Result += " _";
-      Result += RCDecl->getName();
+         // 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 += ";\n";
       
       // insert the super class structure definition.
@@ -1692,8 +1695,11 @@ void RewriteTest::SynthesizeObjcInternalStruct(ObjcInterfaceDecl *CDecl,
     endBuf += Lexer::MeasureTokenLength(LocEnd, *SM);
     Result += " {\n    struct ";
     Result += RCDecl->getName();
-    Result += " _";
-    Result += RCDecl->getName();
+       // 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 += ";\n};\n";
     Rewrite.ReplaceText(LocStart, endBuf-startBuf, 
                         Result.c_str(), Result.size());