[PATCH] D67097: [DWARF] Check for format mismatch between CU and Range List Table.

[Paul Robinson via Phabricator via llvm-commits]

probinson added a comment.

In D67097#1656268 <https://reviews.llvm.org/D67097#1656268>, @dblaikie wrote:

> I'm not sure if using "OffsetIn" in the diagnostic is the right thing to do.
>
> Honestly the fact that DWARFv5 specifies that the "rnglist base address" (& loclist bast address, and str_offsets base) point to the beginning of the list rather than the start of the header is super quirky and I don't think it's helpful to users to highlight that. (though I doubt DWARFvNext would bother fixing this - it'sjust quirky, not super problematic for any practical reasons I can think of)


IIRC this is one of Cary's, and argued that it could be directly used to index into section, instead of you having to step past the header to use it.  Anyway we let him get away with it, for better or worse.  It would have to be essentially unusable or horribly inefficient, for the committee to be willing to change it now.


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

https://reviews.llvm.org/D67097