[PATCH] D57406: build: remove use of llvm-config
Michał Górny via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sat Feb 9 04:30:44 PST 2019
mgorny added inline comments.
================
Comment at: CMakeLists.txt:13
+ set(LLVM_MAIN_SRC_DIR ${LLVM_BUILD_MAIN_SRC_DIR} CACHE PATH "Path to LLVM source tree")
+ set(LLVM_MAIN_INCLUDE_DIR ${LLVM_BUILD_MAIN_INCLUDE_DIR} CACHE PATH "Path to llvm/include")
----------------
So I've looked into the failures more closely, and it looks that `${LLVM_BUILD_MAIN_INCLUDE_DIR}` is wrong here. It should be `${LLVM_INCLUDE_DIR}`, i.e. the directory with installed headers. This is also what the old code was doing.
Repository:
rLLD LLVM Linker
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D57406/new/
https://reviews.llvm.org/D57406
More information about the llvm-commits
mailing list