[Lldb-commits] [PATCH] D86110: [WIP][DebugInfo] Lazily parse debug_loclist offsets
Pavel Labath via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Wed Aug 19 05:03:32 PDT 2020
labath added a comment.
In D86110#2224163 <https://reviews.llvm.org/D86110#2224163>, @dblaikie wrote:
> In D86110#2223905 <https://reviews.llvm.org/D86110#2223905>, @labath wrote:
>> This sounds perfectly reasonable to me. The offsets are present in memory already so there's no point in storing them in a vector as well.
> Thanks for taking a look! I'll take this as approval and commit soon.
The patch had a WIP label, so I was only looking/commenting on the high-level idea. But after looking at the implementation more closely, I think that looks good too.
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the lldb-commits