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

Zachary Turner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 13 11:42:55 PST 2017


zturner added a comment.

I think it's because older versions of clang had their default hardcoded to VS 2013, //regardless of what environment was actually found//.  I think newer versions still have this issue, but it's not a problem in practice because the hardcoded version is the newest version.  But it will be in the future.  Long term we should default to whatever is detected in the headers and libraries that are found.


https://reviews.llvm.org/D41157





More information about the llvm-commits mailing list