[PATCH] D58109: [llvm] [cmake] Provide split include paths in LLVMConfig
Michał Górny via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Feb 12 12:22:23 PST 2019
mgorny added a comment.
Yeah, I was thinking of doing that too (in particular making `TableGen.cmake` use it instead of `LLVM_MAIN_INCLUDE_DIR`) but I'm not sure if someone doesn't have some obscure use case I'd break this way. My main concern is that other packages had `LLVM_MAIN_INCLUDE_DIR` as cache variable for stand-alone builds, so people could've overrode it independently of `LLVM_INCLUDE_DIRS`.
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D58109/new/
https://reviews.llvm.org/D58109
More information about the llvm-commits
mailing list