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