Remove unnecessary ICC flags affecting performance

Since we add -msse/../-mavx based on the acceleration we shouldn't
add -mtune=core2 anymore. Especially because it is added later
and takes precedence over the (higher) acceleration flag.

-ip and -funroll-all-loops could also be deleted because they don't
seem to give any significant performance improvement, and might
increase compilation time, but they don't hurt gromacs performance.

In theory it could help to use -xavx instead of -mavx but I can't
measure a difference.

Change-Id: Icd11c40c3cd3ef2ae6ef42f07d5d75c228593f51

diff --git a/cmake/gmxCFlags.cmake b/cmake/gmxCFlags.cmake
index bd979871f96db17a9292da7193045bf9b1e20274..8afe3fc25ec4548ac433f94d04ce868368bc8c3d 100644 (file)
--- a/cmake/gmxCFlags.cmake
+++ b/cmake/gmxCFlags.cmake
@@ -99,8 +99,6 @@ MACRO(gmx_c_flags)
             GMX_TEST_CFLAG(CFLAGS_WARN "-Wall" GMXC_CFLAGS)
             GMX_TEST_CFLAG(CFLAGS_STDGNU "-std=gnu99" GMXC_CFLAGS)
             GMX_TEST_CFLAG(CFLAGS_OPT "-ip -funroll-all-loops" GMXC_CFLAGS_RELEASE)
-            GMX_TEST_CFLAG(CFLAGS_X86 "-mtune=core2" GMXC_CFLAGS_RELEASE)
-            GMX_TEST_CFLAG(CFLAGS_IA64 "-mtune=itanium2" GMXC_CFLAGS_RELEASE)
         else()
             GMX_TEST_CFLAG(CFLAGS_WARN "/W2" GMXC_CFLAGS)
             GMX_TEST_CFLAG(CFLAGS_X86 "/Qip" GMXC_CFLAGS_RELEASE)
@@ -114,9 +112,6 @@ MACRO(gmx_c_flags)
             endif()
             GMX_TEST_CXXFLAG(CXXFLAGS_WARN "-Wall" GMXC_CXXFLAGS)
             GMX_TEST_CXXFLAG(CXXFLAGS_OPT "-ip -funroll-all-loops" GMXC_CXXFLAGS_RELEASE)
-            GMX_TEST_CXXFLAG(CXXFLAGS_X86 "-mtune=core2" GMXC_CXXFLAGS_RELEASE)
-            GMX_TEST_CXXFLAG(CXXFLAGS_IA64 "-mtune=itanium2" 
-                              GMXC_CXXFLAGS_RELEASE)
         else()
             GMX_TEST_CXXFLAG(CXXFLAGS_WARN "/W2" GMXC_CXXFLAGS)
             GMX_TEST_CXXFLAG(CXXFLAGS_X86 "/Qip" GMXC_CXXFLAGS_RELEASE)