<div dir="ltr">Hello,<div><br></div><div>I just realized that it is blocking me too. Can we revert these changes?</div><div><br></div><div>Thanks,</div><div>Ilia</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Jan 10, 2015 at 10:17 AM, Vince Harron <span dir="ltr"><<a href="mailto:vharron@google.com" target="_blank">vharron@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><p dir="ltr">Sorry, just seeing this now.  I'll take a look tomorrow morning.</p><div class="HOEnZb"><div class="h5">
<div class="gmail_quote">On Jan 9, 2015 10:48 PM, "Bob Campbell" <<a href="mailto:bobcampbell_llvm@icloud.com" target="_blank">bobcampbell_llvm@icloud.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">> On Jan 9, 2015, at 10:47 AM, Dawn Perchik <<a href="mailto:dperchik@embarcadero.com" target="_blank">dperchik@embarcadero.com</a>> wrote:<br>
><br>
> Vince Harron <vharron <at> <a href="http://google.com" target="_blank">google.com</a>> writes:<br>
>><br>
>> Author: vharron<br>
>> Date: Wed Jan  7 15:35:07 2015<br>
>> New Revision: 225398<br>
>><br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=225398&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=225398&view=rev</a><br>
>> Log:<br>
>> lldb-platform - static link all llvm dependencies<br>
>><br>
>> Differential Revision: <a href="http://reviews.llvm.org/D6797" target="_blank">http://reviews.llvm.org/D6797</a><br>
>><br>
>> lldb-gdbserver statically links all llvm dependencies. This allows<br>
>> dead stripping code and reduces total binary size.<br>
>><br>
>> This change modifies lldb-plaform to static link llvm dependencies<br>
>> like lldb-gdbserver.<br>
><br>
> Our upstream build has been broken since this commit and is preventing us from<br>
> merging.   Vince, can you please have a look?  Build errors are:<br>
><br>
> [...]<br>
> Undefined symbols for architecture x86_64:<br>
>  "_liblldb_coreVersionString", referenced from:<br>
>      lldb_private::GetVersion() in lldb.cpp.o<br>
> ld: symbol(s) not found for architecture x86_64<br>
> clang: error: linker command failed with exit code 1 (use -v to see invocation)<br>
> [6/12] Linking CXX shared library lib/liblldb.3.6.0.dylib<br>
> ninja: build stopped: subcommand failed.<br>
><br>
> Build commands used are:<br>
> cmake -G Ninja .. "-DLLVM_TARGETS_TO_BUILD=ARM;X86;AArch64" \<br>
>   -DCMAKE_CXX_FLAGS="-std=c++11 -stdlib=libc++" -DCMAKE_BUILD_TYPE=Debug<br>
> ninja<br>
<br>
The change from:<br>
<br>
target_link_libraries(lldb-platform liblldb)<br>
<br>
to directly referencing the sources does not include some sources for Mac OS<br>
(and perhaps other hosts?) which are defined by the variables LLDB_WRAP_PYTHON<br>
and LLDB_VERS_GENERATED_FILE.<br>
<br>
Adding them back in, and also adding in the dependency to the target<br>
which generates these files seems to work.<br>
<br>
I have not extensively testing but this patch worked for me to build the target lldb-platform:<br>
<br>
diff --git a/tools/lldb-platform/CMakeLists.txt b/tools/lldb-platform/CMakeLists.txt<br>
index 1577765..d4a1e0a 100644<br>
--- a/tools/lldb-platform/CMakeLists.txt<br>
+++ b/tools/lldb-platform/CMakeLists.txt<br>
@@ -22,8 +22,14 @@ add_lldb_executable(lldb-platform<br>
   lldb-platform.cpp<br>
   ../../source/lldb-log.cpp<br>
   ../../source/lldb.cpp<br>
+  ${LLDB_WRAP_PYTHON}<br>
+  ${LLDB_VERS_GENERATED_FILE}<br>
   )<br>
<br>
+if (LLDB_WRAP_PYTHON OR LLDB_VERS_GENERATED_FILE)<br>
+  add_dependencies(lldb-platform swig_wrapper)<br>
+endif()<br>
+<br>
 # The Darwin linker doesn't understand --start-group/--end-group.<br>
 if (LLVM_COMPILER_IS_GCC_COMPATIBLE AND NOT "${CMAKE_SYSTEM_NAME}" MATCHES "Darwin")<br>
   target_link_libraries(lldb-platform<br>
<br>
<br>
><br>
<br>
> Thanks,<br>
> -Dawn<br>
><br>
> PS: I'm using my work e-mail since <a href="http://burble.org" target="_blank">burble.org</a> is down - when it recovers you<br>
> might get another e-mail about this - if so, please ignore :) .<br>
><br>
> _______________________________________________<br>
> lldb-commits mailing list<br>
> <a href="mailto:lldb-commits@cs.uiuc.edu" target="_blank">lldb-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits</a><br>
<br>
<br>
_______________________________________________<br>
lldb-commits mailing list<br>
<a href="mailto:lldb-commits@cs.uiuc.edu" target="_blank">lldb-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits</a><br>
</blockquote></div>
</div></div><br>_______________________________________________<br>
lldb-commits mailing list<br>
<a href="mailto:lldb-commits@cs.uiuc.edu">lldb-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits</a><br>
<br></blockquote></div><br></div>