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

[Adrian McCarthy via Phabricator via llvm-commits]

amccarth added a comment.

I'm confused by this.

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?


https://reviews.llvm.org/D41157