[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