r160365, thanks!<br><br><div class="gmail_quote">On Tue, Jul 17, 2012 at 7:14 PM, Benjamin Kramer <span dir="ltr"><<a href="mailto:benny.kra@gmail.com" target="_blank">benny.kra@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5"><br>
On 17.07.2012, at 16:53, Alexey Samsonov <<a href="mailto:samsonov@google.com">samsonov@google.com</a>> wrote:<br>
<br>
> Hi!<br>
><br>
> This patch improves the behavior of DebugInfoEntryMinimal::getSubprogramName() introduced in r159512.<br>
> To fetch a subprogram name we should not only inspect the DIE for this subprogram, but optionally inspect<br>
> its specification, or its abstract origin (even if there is no inlining), or even specification of an abstract origin.<br>
> New test case checks that llvm-dwarfdump is able to correctly detect function name in the latter case.<br>
><br>
> Code review: <a href="http://codereview.appspot.com/6409051/" target="_blank">http://codereview.appspot.com/6409051/</a><br>
<br>
</div></div>Posted some style nits in the code review tool, otherwise the patch looks good to me.<br>
<br>
- Ben<br>
<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div>Alexey Samsonov, MSK</div><br>