<div dir="ltr">Oh, and I meant to say: if you'd like a hand, happy to go poke around in LLDB, or review patches, etc.</div><br><div class="gmail_quote"><div dir="ltr">On Sat, Aug 5, 2017 at 8:48 AM David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br></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_quote"><div dir="ltr">On Sat, Aug 5, 2017 at 8:34 AM Tamas Berghammer via Phabricator <<a href="mailto:reviews@reviews.llvm.org" target="_blank">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></div><div dir="ltr"><div class="gmail_quote"><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></blockquote></div>