labath added a comment. Seems reasonable. Just one quick question about the implementation.
================ 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() ---------------- 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? 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