<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Sun, Nov 12, 2017 at 12:53 PM Anton 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">xgsa added a comment.<br>
<br>
In <a href="https://reviews.llvm.org/D39622#919579" rel="noreferrer" target="_blank">https://reviews.llvm.org/D39622#919579</a>, @aprantl wrote:<br>
<br>
> For clarification: what is the "symbols table" you are referring to in the description?<br>
<br>
<br>
I meant the data dumped with "nm -an ./test".<br>
<br>
By the way, I haven't abandoned the patch, I have found one more case when my fix doesn't work and I am working on improvement. Nevertheless, it would be helpful to get answers to the questions in this review (about changing the "Indentation" field and about the test).<br></blockquote><div><br>I'm confused then - nm's output uses a demangler, the file doesn't actually have either "Impl<TagType::Tag1>" or "Impl<(TagType)0>" in it.<br><br>It's probably not OK/possible to ensure the type always matches lldb's demangled form. Other produces might have another idea of what its demangled form looks like.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D39622" rel="noreferrer" target="_blank">https://reviews.llvm.org/D39622</a><br>
<br>
<br>
<br>
</blockquote></div></div>