<p dir="ltr"><br>
On Jan 18, 2015 7:08 PM, "Chandler Carruth" <<a href="mailto:chandlerc@google.com">chandlerc@google.com</a>> wrote:<br>
><br>
> 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.</p>
<p dir="ltr">No worries - thanks a bunch to the investigation/fix!</p>
<p dir="ltr">><br>
> On Sun, Jan 18, 2015 at 4:53 PM, Chandler Carruth <<a href="mailto:chandlerc@google.com">chandlerc@google.com</a>> wrote:<br>
>><br>
>> MSVC still seems hosed. Here is a build that claims to have this commit as well:<br>
>><br>
>> <a href="http://lab.llvm.org:8011/builders/lldb-x86-win7-msvc/builds/76/">http://lab.llvm.org:8011/builders/lldb-x86-win7-msvc/builds/76/</a><br>
>><br>
>> On Sun, Jan 18, 2015 at 12:43 PM, David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br>
>>><br>
>>> 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">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">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">llvm-commits@cs.uiuc.edu</a><br>
>>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
>><br>
>><br>
><br>
</p>