[PATCH] D41157: [cmake] Explicitly set VS 2017 compatibility

[Shoaib Meenai via Phabricator via llvm-commits]

smeenai added a comment.

> The default is used only when the MSVC installation cannot be found. Successful compilation with clang-cl depends on finding the MSVC toolset. So either you have a toolset installed that's recent enough to compile or you don't, in which case overriding the compatibility level won't make much difference. Right? Is this because the detection method changed and that the old method won't find VC 2017?

This is for cross-compilation, in which case clang-cl explicitly won't be able to find an MSVC installation/toolset. We're supplying the MSVC toolset and Windows SDK headers and libraries manually.


https://reviews.llvm.org/D41157