
<html><head><meta http-equiv="Content-Type" content="text/html charset=iso-8859-1"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><br><div><div>On Feb 26, 2013, at 12:57 PM, Eric Christopher <<a href="mailto:echristo@gmail.com">echristo@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><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 style="">If this is for compatibility only then it should be under a flag. It sounds like this is just</div><div style="">

for correctness though and so the comment should reflect this.</div></div></div></div></blockquote>Hi Eric,</div><div><br></div><div>This is a correctness issue, it caused GNU gdb 6.3.50 to error out.<br><blockquote type="cite"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div style=""><br></div><div style="">Have you looked at other usages of AT_MIPS_linkage name and what's expected?</div></div></div></div></blockquote>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;">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>Thanks,</div><div>Manman<br><blockquote type="cite"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div style=""><br></div><div style="">-eric </div>

</div></div></div>

</blockquote></div><br></body></html>