<div dir="ltr">These issues should be resolved by r295013 and r295014.</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Feb 9, 2017 at 1:55 PM, Hans Wennborg <span dir="ltr"><<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I'm happy as long as it builds :-)<br>
<div class="HOEnZb"><div class="h5"><br>
On Wed, Feb 8, 2017 at 2:26 PM, Zachary Turner <<a href="mailto:zturner@google.com">zturner@google.com</a>> wrote:<br>
> Should we do that now, as a way to fix this issue, or should we try to get<br>
> another fix in first so we have more time to think about using std<br>
> call_once?<br>
><br>
> On Wed, Feb 8, 2017 at 2:19 PM Reid Kleckner <<a href="mailto:rnk@google.com">rnk@google.com</a>> wrote:<br>
>><br>
>> I think we can. MSVC's std::once_flag default constructor is constexpr<br>
>> now. It still generates dynamic initialization code if you use it as a<br>
>> static local, but MSVC defaults to using thread safe static initialization,<br>
>> so that isn't a problem unless you disable it, which we don't. We disable it<br>
>> in compiler-rt (/Zc:threadSafeInit-), but that doesn't use this code.<br>
>><br>
>> On Wed, Feb 8, 2017 at 1:18 PM, Zachary Turner <<a href="mailto:zturner@google.com">zturner@google.com</a>> wrote:<br>
>>><br>
>>> Is this the right review?  <a href="https://reviews.llvm.org/D5922" rel="noreferrer" target="_blank">https://reviews.llvm.org/D5922</a><br>
>>><br>
>>> Being that this was over 2 years ago, I suspect it was when we were<br>
>>> supporting MSVC2012 and 2013.  Now that we're requiring MSVC2015, is it time<br>
>>> to reconsider? I don't see any links to an MS Connect issue, so I don't know<br>
>>> what the original bug was to know if it has been fixed.<br>
>>><br>
>>><br>
>>> On Wed, Feb 8, 2017 at 1:03 PM Reid Kleckner <<a href="mailto:rnk@google.com">rnk@google.com</a>> wrote:<br>
>>>><br>
>>>> It's a sad story. Read the comments and the review threads. It's<br>
>>>> hilarious.<br>
>>>><br>
>>>> On Wed, Feb 8, 2017 at 1:00 PM, Zachary Turner via lldb-dev<br>
>>>> <<a href="mailto:lldb-dev@lists.llvm.org">lldb-dev@lists.llvm.org</a>> wrote:<br>
>>>>><br>
>>>>> Why doesn't llvm::call_once() just use std::call_once on Windows?<br>
>>>>><br>
>>>>> On Wed, Feb 8, 2017 at 12:40 PM Hans Wennborg <<a href="mailto:hans@chromium.org">hans@chromium.org</a>><br>
>>>>> wrote:<br>
>>>>>><br>
>>>>>> The Win64 lldb build seems broken (at 294367).<br>
>>>>>><br>
>>>>>> I ran into this when trying to build the weekly snapshot<br>
>>>>>> (<a href="http://www.llvm.org/builds/" rel="noreferrer" target="_blank">http://www.llvm.org/builds/</a>) which includes LLDB these days.<br>
>>>>>><br>
>>>>>> I suspect this might be related to Kamil's changes a few days ago. I<br>
>>>>>> see Pavel committed something to fix Darwin afterwards.<br>
>>>>>><br>
>>>>>> Zach, do you know what's going on here? Do we have any buildbot<br>
>>>>>> coverage for this?<br>
>>>>>><br>
>>>>>>    Creating library lib\liblldb.lib and object lib\liblldb.exp<br>
>>>>>> lldbHost.lib(HostInfoWindows.<wbr>cpp.obj) : error LNK2019: unresolved<br>
>>>>>> external symbo<br>
>>>>>> l "void __cdecl llvm::sys::_mm_mfence(void)"<br>
>>>>>> (?_mm_mfence@sys@llvm@@YAXXZ) refer<br>
>>>>>> enced in function "void __cdecl llvm::call_once<class<br>
>>>>>> <lambda_e212a11f7f891e804e<br>
>>>>>> 713e15728a6adc> >(struct llvm::once_flag &,class<br>
>>>>>> <lambda_<wbr>e212a11f7f891e804e713e1<br>
>>>>>> 5728a6adc> &&)"<br>
>>>>>> (??$call_once@V<lambda_<wbr>e212a11f7f891e804e713e15728a6a<wbr>dc>@@$$V@ll<br>
>>>>>><br>
>>>>>> vm@@YAXAEAUonce_flag@0@$$QEAV<<wbr>lambda_<wbr>e212a11f7f891e804e713e15728a6a<wbr>dc>@@@Z)<br>
>>>>>> bin\liblldb.dll : fatal error LNK1120: 1 unresolved externals<br>
>>>>>> LINK failed. with 1120<br>
>>>>><br>
>>>>><br>
>>>>> ______________________________<wbr>_________________<br>
>>>>> lldb-dev mailing list<br>
>>>>> <a href="mailto:lldb-dev@lists.llvm.org">lldb-dev@lists.llvm.org</a><br>
>>>>> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/lldb-dev</a><br>
>>>>><br>
>><br>
><br>
</div></div></blockquote></div><br></div>