<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Sat, Aug 5, 2017 at 8:34 AM Tamas Berghammer via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">tberghammer added a comment.<br>
<br>
I checked LLDB and it doesn't have support for base address entries neither in .debug_loc nor in .debug_loc.dwo sections and it doesn't support reading .debug_loclists or .debug_loclists.dwo entries either. I will try to add support for them but it will require some refactoring so it will take some time.<br></blockquote><div><br>Cool - I mean, no particular rush, it seems, if GDB doesn't support it either so we'll need a fallback to the current behavior at least.<br><br>But if the benefit of fewer relocations in split-dwarf builds (especially optimized builds) is significant to you, this might be nice to have. (it's a smaller win compared to the debug_ranges improvement - which I need to come back around to (it's defaulted off behind a flag for now because Gold's gdb_index fails with it on 32 bit builds - so maybe I'll narrow the default-off down a little (32 bit builds targeting GDB)) & not sure if that's been fixed in LLDB yet)<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
<a href="https://reviews.llvm.org/D36300" rel="noreferrer" target="_blank">https://reviews.llvm.org/D36300</a><br>
<br>
<br>
<br>
</blockquote></div></div>