[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