<div dir="ltr">*nod* Thought so - would be good to know what scenario this is meant to support.<br><br><div class="gmail_quote"><div dir="ltr">On Fri, Feb 9, 2018 at 2:22 AM Jonas Devlieghere 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">JDevlieghere added a subscriber: clayborg.<br>
JDevlieghere added a comment.<br>
<br>
In <a href="https://reviews.llvm.org/D43092#1002677" rel="noreferrer" target="_blank">https://reviews.llvm.org/D43092#1002677</a>, @dblaikie wrote:<br>
<br>
> Could you check the revision history here? I'm pretty sure the first version of this I reviewed from Greg wasn't recursive - and then it became recursive at some point to handle something needed, but maybe those decisions need to be reexamined?<br>
<br>
<br>
It was @clayborg himself that updated the implementation. (<a href="https://reviews.llvm.org/D40156" rel="noreferrer" target="_blank">https://reviews.llvm.org/D40156</a> <a href="https://reviews.llvm.org/rL319104" rel="noreferrer" target="_blank">https://reviews.llvm.org/rL319104</a>)<br>
<br>
> The previous implementation would only look 1 DW_AT_specification or DW_AT_abstract_origin deep. This means DWARFDie::getName() would fail in certain cases. I ran into such a case while creating a tool that used the LLVM DWARF parser to generate a symbolication format so I have seen this in the wild.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D43092" rel="noreferrer" target="_blank">https://reviews.llvm.org/D43092</a><br>
<br>
<br>
<br>
</blockquote></div></div>