xiaobai added a comment. So I ran the lldb test suite from a standalone build tree, and this patch didn't change anything. I added some logging to the CMake and `LLVM_LIBRARY_DIR` is being set correctly. It appears to be getting it from the LLVMConfig we get from `find_package(LLVM)`. I think we could also get rid of `LLVM_BINARY_DIR` if that's the case, since it appears we only set it for lit.
From the looks of it, clang needs to set it manually because they rely on llvm-config instead of using `find_package`. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60180/new/ https://reviews.llvm.org/D60180 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
