diff options
| -rw-r--r-- | Driver/RewriteTest.cpp | 14 |
1 files changed, 10 insertions, 4 deletions
diff --git a/Driver/RewriteTest.cpp b/Driver/RewriteTest.cpp index 2a189ad85b..3ec5a10b2c 100644 --- a/Driver/RewriteTest.cpp +++ b/Driver/RewriteTest.cpp @@ -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()); |