dawn added inline comments.

================
Comment at: tools/lldb-server/CMakeLists.txt:42
@@ +41,3 @@
+  if (LLDB_VERS_GENERATED_FILE)
+    add_dependencies(lldb-server swig_wrapper)
+  endif()
----------------
dawn wrote:
> brucem wrote:
> > Why is this needed?
> You get undefined symbol otherwise (if you build from scratch).
I should clarify - lldb-server needs to link with LLDB_vers.c or you get 
unresolved symbol (that's the fix at line 38).  This sets up the dependency - 
see example in source/API/CMakeLists.txt.


Repository:
  rL LLVM

http://reviews.llvm.org/D13535



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to