<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Fri, Mar 23, 2018 at 11:18 AM Wolfgang Pieb via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">wolfgangp added inline comments.<br>
<br>
<br>
================<br>
Comment at: lib/DebugInfo/DWARF/DWARFDebugRnglists.cpp:134<br>
+                           *OffsetPtr - 1);<br>
+      uint64_t Base = Data.getAddress(OffsetPtr);<br>
+      CurrentRanges.push_back(RangeListEntry{EntryOffset, Encoding, Base, 0});<br>
----------------<br>
dblaikie wrote:<br>
> Some of our range dumping can now print which section an address refers to - should we do that for this range dumping? If so, we'd need to extract the section info here, I think?<br>
Currently the section is displayed when we're dumping the specific ranges a DIE refers to (DW_AT_ranges attribute). When we're dumping the contents of the entire .debug_ranges section we don't do it. Do you think we should do it there too?<br></blockquote><div><br>Ah, fair - thanks for pointing out the distinction.<br><br>I think at some point, it might make sense to dump the sections here too. But if we don't do it for the existing debug_ranges right now, that's not really an inconsistency/bug, just missing feature.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The full support for .debug_rnglists will turn the rangelists into DWARFAddressRangesVectors, so it will behave the same as the current implementation in this respect.<br></blockquote><div><br>*nod* that'll be good<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
<a href="https://reviews.llvm.org/D44811" rel="noreferrer" target="_blank">https://reviews.llvm.org/D44811</a><br>
<br>
<br>
<br>
</blockquote></div></div>