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 + + if(CLANG_LINK_CLANG_DYLIB) + 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.. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68456/new/ https://reviews.llvm.org/D68456 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits