<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Jun 6, 2014 at 5:33 AM, Timur Iskhodzhanov <span dir="ltr"><<a href="mailto:timurrrr@google.com" target="_blank">timurrrr@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The change looks good -- it simplifies the code and makes it faster.<br>
<br>
David,<br>
Do you have a good way to make sure we don't regress other than commit-and-see-if-it-explodes?<br></blockquote><div><br></div><div>Sadly not but our in-tree tests are pretty thorough for templates.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
================<br>
Comment at: lib/AST/MicrosoftMangle.cpp:678<br>
@@ +677,3 @@<br>
<div class="">+    if (Found == NameBackReferences.end()) {<br>
+      Out << BackReferenceKey;<br>
+      if (NameBackReferences.size() < 10) {<br>
</div>----------------<br>
This variable is no longer used as just a key.<br>
Please use something like `TemplateMangling` or `TDMangling` instead?<br>
<br>
<a href="http://reviews.llvm.org/D3983" target="_blank">http://reviews.llvm.org/D3983</a><br>
<br>
<br>
</blockquote></div><br></div></div>