<div dir="ltr">Actually, everything was busted trying to build LLDB. This was an LLDB use of the API you refactored in r226416 that didn't get updated. I've updated it in r226427 to try to get things building again. Sorry for the confusing email.</div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Jan 18, 2015 at 4:53 PM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com" target="_blank">chandlerc@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">MSVC still seems hosed. Here is a build that claims to have this commit as well:<div><br></div><div><a href="http://lab.llvm.org:8011/builders/lldb-x86-win7-msvc/builds/76/" target="_blank">http://lab.llvm.org:8011/builders/lldb-x86-win7-msvc/builds/76/</a><br></div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Jan 18, 2015 at 12:43 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: dblaikie<br>
Date: Sun Jan 18 14:43:57 2015<br>
New Revision: 226415<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=226415&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=226415&view=rev</a><br>
Log:<br>
Attempt to fix the MSVC build by working around a layering issue<br>
<br>
Since MCStreamer isn't part of Support, the dtor can't be called from<br>
here - so just pass by reference instead. This is rather imperfect, but<br>
will hopefully suffice.<br>
<br>
Modified:<br>
    llvm/trunk/include/llvm/Support/TargetRegistry.h<br>
<br>
Modified: llvm/trunk/include/llvm/Support/TargetRegistry.h<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Support/TargetRegistry.h?rev=226415&r1=226414&r2=226415&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Support/TargetRegistry.h?rev=226415&r1=226414&r2=226415&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/include/llvm/Support/TargetRegistry.h (original)<br>
+++ llvm/trunk/include/llvm/Support/TargetRegistry.h Sun Jan 18 14:43:57 2015<br>
@@ -381,7 +381,7 @@ namespace llvm {<br>
     /// createAsmPrinter - Create a target specific assembly printer pass.  This<br>
     /// takes ownership of the MCStreamer object.<br>
     AsmPrinter *createAsmPrinter(TargetMachine &TM,<br>
-                                 std::unique_ptr<MCStreamer> Streamer) const {<br>
+                                 std::unique_ptr<MCStreamer> &&Streamer) const {<br>
       if (!AsmPrinterCtorFn)<br>
         return nullptr;<br>
       return AsmPrinterCtorFn(TM, std::move(Streamer));<br>
<br>
<br>
_______________________________________________<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/mailman/listinfo/llvm-commits</a><br>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>