[Lldb-commits] [PATCH] D130796: [LLDB][NativePDB] Switch to use DWARFLocationList.

[Zequan Wu via Phabricator via lldb-commits]

zequanwu added a comment.

In D130796#3691227 <https://reviews.llvm.org/D130796#3691227>, @zequanwu wrote:

>> If you find yourself needing to do extra work to work its limitations, we should fix that algorithm instead.
>
> That makes sense. I'll work on fixing `RangeVectorData`.

After thinking it again, not just `RangeVectorData` need to be written but also `RangeVector`, although they are similar. It requires rewrite the whole file `RangeMap.h` and all other places uses its APIs.
Can we just use `RangeVectorData` as it is right now and fix it later? So, this patch is not blocked. Otherwise, this local variables functionality is not almost unusable for NativePDB plugin.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D130796/new/

https://reviews.llvm.org/D130796