[PATCH] D70081: DWARFDebugLoclists: add location list "interpretation" logic

[Pavel Labath via Phabricator via llvm-commits]

labath added a comment.

In D70081#1743535 <https://reviews.llvm.org/D70081#1743535>, @SouraVX wrote:

> Now, I'm assuming "DW_LLE*" dumping/other stuff works fine with llvm-objdump and objdump GDB. since they share the same encoding's {pre/post DWARF5}?
>  But thier's one more slight glitch, for case of debug_loc{No split} section. We are still emitting raw ranges in a sense --
>
>   llvm-dwarfdump -debug-loc -v a.out   --using latest master
>   .debug_loc contents:
>   0x00000000:
>               [0x0000000000000000,  0x0000000000000004): DW_OP_reg5 RDI
>               [0x0000000000000004,  0x0000000000000018): DW_OP_reg3 RBX
>
>
> Now I understand it's not priority, but atleast for sake of uniformity and saving some confusion. We should use it in both ways or completely drop it.
>  Minor addition --  objdump is still broken to handle [DWARF5] stuff. for debug_loc and loc.dwo it dumps raw ranges.


You're absolutely right, and I am planning to do that. I already have a patch which does just that -- I just need to clean it up before submission (I'll do that today or tomorrow).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70081