<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Feb 26, 2013 at 2:36 PM, Manman Ren <span dir="ltr"><<a href="mailto:mren@apple.com" target="_blank">mren@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><br><div><div class="im"><div>On Feb 26, 2013, at 12:57 PM, Eric Christopher <<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>> wrote:</div>
<br><blockquote type="cite"><div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Feb 26, 2013 at 12:48 PM, Manman Ren <span dir="ltr"><<a href="mailto:mren@apple.com" target="_blank">mren@apple.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: mren<br>
Date: Tue Feb 26 14:48:29 2013<br>
New Revision: 176120<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=176120&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=176120&view=rev</a><br>
Log:<br>
Debug Info: for static member variables, move AT_MIPS_linkage_name from<br>
TAG_member inside a class to the specification DIE.<br>
<br>
Having AT_MIPS_linkage_name on TAG_member caused old gdb (GNU 6.3.50) to<br>
error out. Also gcc 4.7 has AT_MIPS_linkage_name on the specification DIE.<br></blockquote><div><br></div><div>If this is for compatibility only then it should be under a flag. It sounds like this is just</div><div>
for correctness though and so the comment should reflect this.</div></div></div></div></blockquote></div>Hi Eric,</div><div><br></div><div>This is a correctness issue, it caused GNU gdb 6.3.50 to error out.<div class="im">
<br></div></div></div></blockquote><div><br></div><div style>That doesn't necessarily make it a correctness issue versus a compatibility issue. gcc4.7</div><div style>emitting it in the way that we are after your patch i'll call correctness since it's a gcc specific (sorta) attribute. </div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div><div class="im"><blockquote type="cite"><div dir="ltr"><div class="gmail_extra">
<div class="gmail_quote"><div><br></div><div>Have you looked at other usages of AT_MIPS_linkage name and what's expected?</div></div></div></div></blockquote></div>I didn't go though all usages of AT_MIPS_linkage, but apparently this is causing debuginfo_test to fail:</div>
<div><a href="http://smooshlab.apple.com:8013/builders/clang-x86_64-darwin10-nobootstrap-RA/builds/2397" style="font-size:12px" target="_blank">http://smooshlab.apple.com:8013/builders/clang-x86_64-darwin10-nobootstrap-RA/builds/2397</a></div>
<div><br></div><div>I am looking at the failure and reverted the patch in the meantime.</div><div><br></div></div></blockquote><div><br></div><div style>OK. Please do both.</div><div style><br></div><div style>Thanks!</div>
<div style><br></div><div style>-eric </div></div></div></div>