Omit the field name when including the super class structure template.
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
diff --git a/Driver/RewriteTest.cpp b/Driver/RewriteTest.cpp
index 2a189ad..3ec5a10 100644
--- a/Driver/RewriteTest.cpp
+++ b/Driver/RewriteTest.cpp
@@ -1648,8 +1648,11 @@
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 @@
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());