[Lldb-commits] [PATCH] D68456: [lldb] [cmake] Support linking against clang-cpp dylib

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri Oct 4 04:33:28 PDT 2019

labath accepted this revision.
labath added inline comments.
This revision is now accepted and ready to land.

Comment at: lldb/cmake/modules/AddLLDB.cmake:94-99
+      target_link_libraries(${name} PRIVATE clang-cpp)
+    else()
+      target_link_libraries(${name} PRIVATE ${PARAM_CLANG_LIBS})
+    endif()
mgorny wrote:
> labath wrote:
> > I don't know whether this makes any difference, but I am wondering if we should pass these (both clang-cpp and PARAM_CLANG_LIBS) as the LINK_LIBS argument to the llvm_add_library call above. Seems like it would be more consistent with the approach of letting the llvm function manage linking. WDYT?
> I presumed it doesn't, so went for the simpler approach ;-). I can change it if you prefer.
Pfft.. I don't know.. We can leave that as is..



More information about the lldb-commits mailing list