[PATCH] D58109: [llvm] [cmake] Provide split include paths in LLVMConfig

Reid Kleckner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 12 10:26:09 PST 2019


rnk added a comment.

It seems like we could move clients of the single variable, LLVM_INCLUDE_DIR, to the LLVM_INCLUDE_DIRS list instead of doing this. Are there any reasons why that doesn't work for some clients?
This new arrangement bakes in the idea that LLVM has at most two include dirs, which is OK, I doubt we'll break that assumption.


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