[PATCH] D26255: [cmake] Add LLVM_ENABLE_DIA_SDK option, and expose it in LLVMConfig

Zachary Turner via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 15 13:29:32 PST 2016


zturner added a comment.

Whether or not to use `WIN32` or `MSC_VER` is always a tricky question.  I would probably use `MSC_VER` in this case.  Because `WIN32` can be defined if you're using GCC / MinGW for example, and DIA won't won't work under those scenarios.


https://reviews.llvm.org/D26255





More information about the llvm-commits mailing list