[PATCH] D83624: [DWARFYAML] Implement the .debug_rnglists section.

Tue Jul 14 05:43:25 PDT 2020

jhenderson added a subscriber: probinson.
jhenderson added a comment.

In D83624#2150008 <https://reviews.llvm.org/D83624#2150008>, @Higuoxing wrote:

> In D83624#2149929 <https://reviews.llvm.org/D83624#2149929>, @jhenderson wrote:
>
> > Are we deliberately not supporting non-zero segment selector size values again? There's no test coverage for tables with non-zero values.
>
>
> The description of 'SegmentSelectorSize' isn't very clear to me in the spec. It doesn't mention how/where to insert a segment selector in the .debug_rnglists section.
>
> > "The segment size is given by the segment_selector_size field of the header, and the address size is given by the address_size field of the header. If the segment_selector_size field in the header is zero, the segment selector is omitted from the range list entries."
>
> Any idea?

Huh, that's not great, and might be a spec bug again. @aprantl/@probinson, could either of you provide any enlightenment on how the segment is supposed to be encoded in the .debug_rnglists section, if at all? The spec seems to be silent on the matter. (For reference, for the .debug_aranges section the segment selector appears immediately before the address).

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83624