[PATCH] D36300: WIP: DebugInfo: Use base address and offset pairs in debug_loc.dwo to reduce required relocations

[David Blaikie via llvm-commits]

Oh, and I meant to say: if you'd like a hand, happy to go poke around in
LLDB, or review patches, etc.

On Sat, Aug 5, 2017 at 8:48 AM David Blaikie <dblaikie at gmail.com> wrote:

> On Sat, Aug 5, 2017 at 8:34 AM Tamas Berghammer via Phabricator <
> reviews at reviews.llvm.org> wrote:
>
>> tberghammer added a comment.
>>
>> 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.
>>
>
> 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.
>
> 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)
>
>
>>
>>
>> https://reviews.llvm.org/D36300
>>
>>
>>
>>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170805/67fc502a/attachment.html>