Fix Windows+Intel
authorRoland Schulz <roland.schulz@intel.com>
Mon, 16 Jul 2018 19:33:08 +0000 (12:33 -0700)
committerMark Abraham <mark.j.abraham@gmail.com>
Mon, 16 Jul 2018 21:09:36 +0000 (23:09 +0200)
Introduced by 95466e9d55b23

Change-Id: I275ab4e0ceb04ec03dc5d48067842540aa509621

src/gromacs/CMakeLists.txt

index 3bff3a9bc53101630b28b25135366b0338bd5f76..beeddfeaa058389f1a3ceab2c9644430258209cb 100644 (file)
@@ -300,7 +300,7 @@ string(REPLACE " " ";" IGNORED_CLANG_ALL_WARNINGS "${IGNORED_CLANG_ALL_WARNINGS}
 if (CMAKE_CXX_COMPILER_ID MATCHES "Clang" AND CMAKE_CXX_COMPILER_VERSION MATCHES "^6\.0")
    target_compile_options(libgromacs PRIVATE $<$<COMPILE_LANGUAGE:CXX>:-Weverything ${IGNORED_CLANG_ALL_WARNINGS}>)
 endif()
-if (MSVC)
+if (CMAKE_CXX_COMPILER_ID STREQUAL "MSVC")
    target_compile_options(libgromacs PRIVATE $<$<COMPILE_LANGUAGE:CXX>:/analyze /analyze:stacksize 70000
      #Control flow warnings are disabled because the commond line output is insufficient. There is no tool
      #to convert the xml report to e.g. HTML and even in Visual Studio the viewer doesn't work with cmake support.