[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