<div dir="ltr">Seems this makes the flag (dwarf-linkage-names=*) a bit erroneous, no? Would it be better to trigger this directly off the debugger tuning flag now & drop the specific named flag?</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Apr 5, 2016 at 4:06 PM, Paul Robinson via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">probinson added a comment.<br>
<span class=""><br>
In <a href="http://reviews.llvm.org/D18706#391801" rel="noreferrer" target="_blank">http://reviews.llvm.org/D18706#391801</a>, @probinson wrote:<br>
<br>
> Built Clang with Clang 3 times: linkage names Enabled, linkage names Disabled but with the patch, linkage names Disabled without the patch (aka: all names, some names, no names).<br>
</span>>  'all names' and 'no names' are the options we have today; this patch replaces 'no names' with 'some names.'<br>
<br>
<br>
dblaikie wrote (in email):<br>
<span class=""><br>
> Which option(s) are you referring to here ^ (do we have a flag or somesuch that<br>
<br>
>  allows enabling/disabling linkage names?)<br>
<br>
<br>
</span>Yes, DwarfDebug.cpp has `-dwarf-linkage-names={Enable,Disable}` which is disabled by default when tuning for SCE, otherwise enabled by default.  This is what `DD->useLinkageNames()` is referring to, in addLinkageName(), giving us the all/none options.<br>
My patch obviously substitutes 'some' for 'none'.<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
<a href="http://reviews.llvm.org/D18706" rel="noreferrer" target="_blank">http://reviews.llvm.org/D18706</a><br>
<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div>