<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Oct 11, 2013 at 12:57 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank" class="cremed">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 class="im"><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div dir="ltr"><div></div><div>Is it reasonable for this commit to be enabled only for limit-debug-info, when no-limit-debug-info is on, we don't see effects of this patch?<br>
</div></div></blockquote><div><br></div></div><div>GCC does this by default, at least on Linux - I haven't tested on MacOS. I would guess it does so there too (but as you point out, there are different tradeoffs there, so I might be wrong).<br>

<br> I'd like to understand the performance tradeoff to justify such a large regression in debug info size (it's worth about ~20% on Clang/LLVM based on my experiments).</div></blockquote></div><br>All evidence is that for many even larger C++ code bases (Chromium, etc) this is even more important to be on by default. I would really rather fix this by ensuring the debugger has the information necessary to quickly find the definition than to emit the definition in so many more places.</div>
</div>