<div dir="ltr"><div>I applied this to a clean local copy, which has no other changes, and have the following test error, which may be pilot error on my part, but nevertheless, this test needs to be robust to changes in the line number.</div><div><br></div><div>llvm-svn/llvm/tools/clang/unittests/Tooling/QualTypeNamesTest.cpp:39: Failure</div><div>Value of: false</div><div>  Actual: false</div><div>Expected: true</div><div>Typename::getFullyQualifiedName failed for (anonymous struct)::un_in_st_1</div><div>   Actual: union (anonymous struct at input.cc:1:1)::(anonymous union at input.cc:2:27)</div><div> Exepcted: union (anonymous struct at input.cc:1:1)::(anonymous union at input.cc:2:31)</div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, May 4, 2018 at 9:22 AM, Mikhail Ramalho via Phabricator <span dir="ltr"><<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">mikhail.ramalho added a comment.<br>
<br>
Thanks.<br>
<br>
I just need someone to push it, as I don't have write access to the repo.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D36610" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D36610</a><br>
<br>
<br>
<br>
</blockquote></div><br></div>