[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