<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jul 7, 2016 at 4:10 PM, Reid Kleckner <span dir="ltr"><<a href="mailto:rnk@google.com" target="_blank">rnk@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span class="">On Thu, Jul 7, 2016 at 3:45 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Yeah - is this necessary for CodeView? (does something break, or do you just get simple names where you'd prefer full mangled or qualified names)<br><br>It was chosen pretty deliberately to do it this way (use unqualified names) for gmlt to keep size down. Have you got numbers for the size delta for CodeView with this change? I'd really prefer to make the same choice for both - it'd seem a bit arbitrary to choose our size optimization based on differences in the kind of workloads we happen to have on these two platforms.</div></blockquote><div><br></div></span><div>It's problematic for CodeView because there is no equivalent field like DW_AT_linkage_name in any of the symbol records. There is only the display name, which includes scope qualifiers.</div><div><br></div><div>I'm suggesting we reverse our decision for DWARF because our space saving optimization breaks standard stack dumpers on Linux (gdb and addr2line),</div></div></div></div></blockquote><div><br></div><div>What breakage are you referring to here?<br><br>If we're talking about printing out the simple name - that comes up even in llvm-symbolizer if a function is inlined. So I think that's an intentional tradeoff that would apply to CV as well.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div> and that seems like a Bad Thing. Instead we've told users to user llvm-symbolizer, which is OK, but kind of crappy. </div></div></div></div></blockquote><div><br></div><div>I imagine that depends on how much it costs us - we've been pretty careful about binary size growth/minimization/etc for a while now, so it may be worth the tradeoff to say that llvm-symbolizer produces a better experience on this extra minimized data.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>For CodeView, because the linkage name isn't present anywhere, we can't actually do any better with llvm-symbolizer, and we need the fully scoped name.</div></div></div></div>
</blockquote></div><br></div></div>