[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 13:09:44 PST 2016


beanz added a comment.

Moving the option into config-ix is fine, and basing the default off whether or not the SDK is present seems like a good way to handle this.

I'll defer to you whether `WIN32` or `MSVC` is correct. `WIN32` is for if you are targeting Windows, and `MSVC` is if your compiler is MSVC-compatible. `WIN32` seems right to me, but I don't know.


https://reviews.llvm.org/D26255





More information about the llvm-commits mailing list