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!

llvm-svn: 229584

llvm/cmake/modules/HandleLLVMOptions.cmake

index 3843aa8..24af2f8 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()