[PATCH] D69462: [DebugInfo]: Support for debug_loclists.dwo section in llvm and llvm-dwarfdump.

[David Blaikie via Phabricator via llvm-commits]

dblaikie added a comment.

In D69462#1755029 <https://reviews.llvm.org/D69462#1755029>, @probinson wrote:

> > Hmm, @aprantl @probinson - I can't find the wording in the DWARF5 spec about the use of loclists_base and rnglists_base in split units. It's unclear if these need to be specified in the split unit, or if they're assumed to be zero (or, technically, zero + sizeof(list header))? Currently LLVM doesn't generate a rnglists_base for split DWARFv5 units (I haven't checked the history on that change - but I might've had a hand in it) & this change is going to propagate that choice & do similarly for loclists_base.
>
> If you look at Table F.1, p.395 in the DWARF 5 spec, it shows DW_AT_loclists_base/rnglists_base as used in conventional units but not skeleton/split units.  Given that there's an index, I think the attribute itself would be superfluous.


Ah, perfect - thanks a bunch!


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

https://reviews.llvm.org/D69462