labath added a comment.
Seems like I nice cleanup. I just have a question about the search paths.
(Also, I agree with @mgorny about not doing this so close to the branch point.)
================
Comment at: cmake/modules/LLDBStandalone.cmake:9
+ find_package(LLVM REQUIRED CONFIG
+ HINTS "${LLDB_PATH_TO_LLVM_BUILD}" NO_DEFAULT_PATH NO_CMAKE_FIND_ROOT_PATH)
+ find_package(Clang REQUIRED CONFIG
----------------
Why do you put `NO_DEFAULT_PATH` here? IIUC, the user will now have to specify
`LLDB_PATH_TO_LLVM_BUILD` in order to build this, whereas previously
llvm-config would be found on the path if it happened to be there (e.g. because
it was already installed).
Would it not make sense to keep this behavior?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D56531/new/
https://reviews.llvm.org/D56531
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
