[PATCH] D26255: [cmake] Add LLVM_ENABLE_DIA_SDK option, and expose it in LLVMConfig
Chris Bieneman via llvm-commits
llvm-commits at lists.llvm.org
Tue Nov 15 10:28:17 PST 2016
beanz added a comment.
If `LLVM_ENABLE_DIA_SDK` is `On` and you don't have a the SDK available that should be a fatal error. Treating it this way will also allow you to clean up some of the logic you're modifying in config-ix.cmake. Also gate the option under `WIN32` so it only shows up if you're targeting windows.
With those changes I think this looks reasonable.
https://reviews.llvm.org/D26255
More information about the llvm-commits
mailing list