<div dir="ltr">They're essentially unrelated. I moved the other Lower* methods (LowerSTACKMAP, LowerPATCHPOINT) in to X86AsmPrinter, and did the same for LowerTlsAddr for consistency. I forgot to back this cosmetic change to LowerTlsAddr out before I committed. My apologies for that.<div>
<br></div><div>There is no functional change due to the signature change of LowerTlsAddr, so it can be safely reverted if you would prefer. The changes to the body of LowerTlsAddr should remain if possible though - they are valuable (though not strictly necessary) for the optimization.</div>
<div><br></div><div>- Lang.</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jul 24, 2014 at 2:36 PM, Philip Reames <span dir="ltr"><<a href="mailto:listmail@philipreames.com" target="_blank">listmail@philipreames.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Was there a review thread for this I missed?  I would have liked to see this before it went in.<div class=""><br>
<br>
On 07/24/2014 01:40 PM, Lang Hames wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
    case X86::TLS_base_addr64:<br>
-    return LowerTlsAddr(OutStreamer, MCInstLowering, *MI, getSubtargetInfo());<br>
+    return LowerTlsAddr(MCInstLowering, *MI);<br>
</blockquote></div>
This looks like it might be an accidental unrelated change?<span class="HOEnZb"><font color="#888888"><br>
<br>
Philip</font></span><div class="HOEnZb"><div class="h5"><br>
______________________________<u></u>_________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div>