<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Jan 29, 2020 at 2:28 AM Pavel Labath <<a href="mailto:pavel@labath.sk" target="_blank">pavel@labath.sk</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 28/01/2020 20:37, David Blaikie via llvm-dev wrote:<br>
> and one idea I had was to have the DWARFDataExtractor return a new<br>
> DWARFDataExtractor that was specifically bounded to the parsed length<br>
> for this reason.<br>
<br>
I think this would be great. In fact, I was getting ready to propose<br>
something like that myself. :)<br>
<br>
FWIW, lldb DataExtractors already support this kind of slicing.<br></blockquote><div><br>Oh, great - I'm sort of inundated with code reviews and things these days - so could I prevail upon you to implement that & pick up some of these line table reviews to either coax the use of such an entity there, or port a few places (at least all the DWARF32/64 length parsing duplicates - there's 3-4 copies of the code that does the 32/64 length dance and validates the length of a header or similar in there)?<br><br>Tagged you in a few reviews (some committed already/approved - but places that could benefit from cleanup with such a device) <a href="https://reviews.llvm.org/D72155">https://reviews.llvm.org/D72155</a> <a href="https://reviews.llvm.org/D73618">https://reviews.llvm.org/D73618</a> <a href="https://reviews.llvm.org/D72900">https://reviews.llvm.org/D72900</a><br><br> </div></div></div>