<div dir="ltr">I don't think it's vitally pressing or anything - just trying to delegate some of the deluge of code review/coaxing all these DWARF changes (at least the line table improvements and the DWARF64 improvements - and I think there's one other group/project happening too? (DWARF5 features like the defaulted template parameters, implicit_pointer, etc))</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Jan 31, 2020 at 6:17 AM Pavel Labath <<a href="mailto:pavel@labath.sk">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 31/01/2020 01:31, David Blaikie wrote:<br>
> <br>
> <br>
> On Wed, Jan 29, 2020 at 2:28 AM Pavel Labath <<a href="mailto:pavel@labath.sk" target="_blank">pavel@labath.sk</a><br>
> <mailto:<a href="mailto:pavel@labath.sk" target="_blank">pavel@labath.sk</a>>> wrote:<br>
> <br>
>     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>
> <br>
> <br>
> Oh, great - I'm sort of inundated with code reviews and things these<br>
> days - so could I prevail upon you to implement that & pick up some of<br>
> these line table reviews to either coax the use of such an entity there,<br>
> or port a few places (at least all the DWARF32/64 length parsing<br>
> duplicates - there's 3-4 copies of the code that does the 32/64 length<br>
> 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<br>
> places that could benefit from cleanup with such a<br>
> device) <a href="https://reviews.llvm.org/D72155" rel="noreferrer" target="_blank">https://reviews.llvm.org/D72155</a> <a href="https://reviews.llvm.org/D73618" rel="noreferrer" target="_blank">https://reviews.llvm.org/D73618</a> <a href="https://reviews.llvm.org/D72900" rel="noreferrer" target="_blank">https://reviews.llvm.org/D72900</a><br>
> <br>
>  <br>
<br>
Thanks.<br>
<br>
Well, I am afraid in this case, "getting ready" is a pretty long-term<br>
process :/, and I don't think I'll be able to get to this next week,<br>
because I'll be travelling.<br>
<br>
But I think I should be able to squeeze that in the week after that.<br>
<br>
pl<br>
</blockquote></div>