We require MSVC 2013 Update 4 due to previous versions miscompiling ASTMatchers
authorFilipe Cabecinhas <me@filcab.net>
Tue, 17 Feb 2015 23:29:00 +0000 (23:29 +0000)
committerFilipe Cabecinhas <me@filcab.net>
Tue, 17 Feb 2015 23:29:00 +0000 (23:29 +0000)
Previous versions of MSVC 2013 would miscompile ASTMatchers (and/or their
tests). Bump up the requirement and make sure we know about the minor
revision.

Minimum required version found by Michael Edwards!

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@229584 91177308-0d34-0410-b5e6-96231b3b80d8

cmake/modules/HandleLLVMOptions.cmake

index 3843aa83450bfbb317e49de0d9790878d19a096d..24af2f81ceb3b1b09699f497f257bc50391f23ab 100644 (file)
@@ -41,8 +41,8 @@ int main() { return (float)x; }"
       set(CMAKE_REQUIRED_LIBRARIES ${OLD_CMAKE_REQUIRED_LIBRARIES})
     endif()
   elseif(CMAKE_CXX_COMPILER_ID MATCHES "MSVC")
-    if(CMAKE_CXX_COMPILER_VERSION VERSION_LESS 18.0)
-      message(FATAL_ERROR "Host Visual Studio must be at least 2013 (MSVC 18.0)")
+    if(CMAKE_CXX_COMPILER_VERSION VERSION_LESS 18.0.31101)
+      message(FATAL_ERROR "Host Visual Studio must be at least 2013 Update 4 (MSVC 18.0.31101)")
     endif()
   endif()
 endif()