<div dir="ltr">Yea I think this is fine.</div><br><div class="gmail_quote"><div dir="ltr">On Wed, Jan 27, 2016 at 9:29 AM Hans Wennborg <<a href="mailto:hans@chromium.org">hans@chromium.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Zach is the lldb cmake owner. Zach, does this seem OK for 3.8?<br>
<br>
On Wed, Jan 27, 2016 at 7:13 AM, Pavel Labath <<a href="mailto:labath@google.com" target="_blank">labath@google.com</a>> wrote:<br>
> Hi,<br>
><br>
> could we get this patch applied to the 3.8 branch?<br>
><br>
> thanks,<br>
> pl<br>
><br>
><br>
> On 27 January 2016 at 11:02, Pavel Labath via lldb-commits<br>
> <<a href="mailto:lldb-commits@lists.llvm.org" target="_blank">lldb-commits@lists.llvm.org</a>> wrote:<br>
>> Author: labath<br>
>> Date: Wed Jan 27 05:02:02 2016<br>
>> New Revision: 258921<br>
>><br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=258921&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=258921&view=rev</a><br>
>> Log:<br>
>> Fix linking with LLVM_LINK_LLVM_DYLIB=ON<br>
>><br>
>> Linking with LLVM shared libraries currently produces linker errors.  This works around the issue<br>
>> (pr24953) by disabling linking with llvm so for lldb libraries.<br>
>><br>
>> Patch by Evangelos Foutras.<br>
>><br>
>> Reviewers: zturner<br>
>><br>
>> Subscribers: lldb-commits<br>
>><br>
>> Differential Revision: <a href="http://reviews.llvm.org/D16293" rel="noreferrer" target="_blank">http://reviews.llvm.org/D16293</a><br>
>><br>
>> Modified:<br>
>>     lldb/trunk/cmake/modules/AddLLDB.cmake<br>
>><br>
>> Modified: lldb/trunk/cmake/modules/AddLLDB.cmake<br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project/lldb/trunk/cmake/modules/AddLLDB.cmake?rev=258921&r1=258920&r2=258921&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/lldb/trunk/cmake/modules/AddLLDB.cmake?rev=258921&r1=258920&r2=258921&view=diff</a><br>
>> ==============================================================================<br>
>> --- lldb/trunk/cmake/modules/AddLLDB.cmake (original)<br>
>> +++ lldb/trunk/cmake/modules/AddLLDB.cmake Wed Jan 27 05:02:02 2016<br>
>> @@ -56,7 +56,7 @@ macro(add_lldb_library name)<br>
>>    if (PARAM_OBJECT)<br>
>>      add_library(${name} ${libkind} ${srcs})<br>
>>    else()<br>
>> -    llvm_add_library(${name} ${libkind} ${srcs})<br>
>> +    llvm_add_library(${name} ${libkind} DISABLE_LLVM_LINK_LLVM_DYLIB ${srcs})<br>
>><br>
>>      lldb_link_common_libs(${name} "${libkind}")<br>
>><br>
>> @@ -93,7 +93,7 @@ macro(add_lldb_library name)<br>
>>  endmacro(add_lldb_library)<br>
>><br>
>>  macro(add_lldb_executable name)<br>
>> -  add_llvm_executable(${name} ${ARGN})<br>
>> +  add_llvm_executable(${name} DISABLE_LLVM_LINK_LLVM_DYLIB ${ARGN})<br>
>>    set_target_properties(${name} PROPERTIES FOLDER "lldb executables")<br>
>>  endmacro(add_lldb_executable)<br>
</blockquote></div>