[PATCH] D11374: Make DW_AT_[MIPS]linkage_name optional, and off by default for SCE.

[Paul Robinson]

probinson added inline comments.

================
Comment at: test/DebugInfo/2010-04-06-NestedFnDbgInfo.ll:16
@@ -15,3 +15,3 @@
 ; CHECK-NOT: DW_TAG
-; CHECK: DW_AT_linkage_name {{.*}} "_ZZN1B2fnEvEN1A3fooEv"
+; CHECK: DW_AT_name {{.*}} "foo"
 ; And just double check that there's no out of line definition that references
----------------
echristo wrote:
> Why this change? Probably want to change the RUN line if you want to make this clean to run on sce.
In this case it looked like the linkage-name was not inherently interesting, but just a way to validate that the CHECK stream was in the correct DIE.  By using the unmangled name instead, it preserves the purpose of the check while being debugger-feature-neutral.
I did something similar in cases like recursive_inlining.ll, where the mangled name was just a lazy way to verify that the cross-DIE references were pointing to the right places; verifying the actual references rather than the names seemed like an improvement on its own, as well as avoiding a dependence on having mangled names.
In other tests, the mangled name was either inherently part of the test, or not so easy to find a substitute for, and in those cases I added the command-line option.



http://reviews.llvm.org/D11374