From: Roland Schulz Date: Wed, 22 Oct 2014 01:23:29 +0000 (-0400) Subject: Don't set CFLAGS for Phi in toolchain X-Git-Url: http://biod.pnpi.spb.ru/gitweb/?a=commitdiff_plain;h=1e1f18a50ac010fa3c8e1586f63ff9ac9009449d;p=alexxy%2Fgromacs.git Don't set CFLAGS for Phi in toolchain The recommended way to set those flags is with CMAKE_*_FLAGS_INIT but that is only passible in a platform file because those set in a toolchain file get overwritten. This fixes that extra CFLAGS passed on the command line don't get ignored. Also move setting of fftw out of platform file because those should be project independent. Change-Id: I5f4d70efd8369bdef5f5060959e3568026d6820c --- diff --git a/CMakeLists.txt b/CMakeLists.txt index eeeb3065be..581064bb66 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -181,10 +181,16 @@ gmx_option_multichoice( "${GMX_SUGGESTED_SIMD}" None SSE2 SSE4.1 AVX_128_FMA AVX_256 AVX2_256 MIC ARM_NEON ARM_NEON_ASIMD IBM_QPX IBM_VMX IBM_VSX Sparc64_HPC_ACE Reference) +if(GMX_TARGET_MIC) + set(GMX_FFT_LIBRARY_DEFAULT "mkl") +else() + set(GMX_FFT_LIBRARY_DEFAULT "fftw3") +endif() + gmx_option_multichoice( GMX_FFT_LIBRARY "FFT library" - "fftw3" + "${GMX_FFT_LIBRARY_DEFAULT}" fftw3 mkl "fftpack[built-in]") gmx_dependent_option( GMX_BUILD_OWN_FFTW diff --git a/cmake/Platform/Linux-Intel-C-XeonPhi.cmake b/cmake/Platform/Linux-Intel-C-XeonPhi.cmake new file mode 100644 index 0000000000..a08c51386c --- /dev/null +++ b/cmake/Platform/Linux-Intel-C-XeonPhi.cmake @@ -0,0 +1,35 @@ +# +# This file is part of the GROMACS molecular simulation package. +# +# Copyright (c) 2014,2015, by the GROMACS development team, led by +# Mark Abraham, David van der Spoel, Berk Hess, and Erik Lindahl, +# and including many others, as listed in the AUTHORS file in the +# top-level source directory and at http://www.gromacs.org. +# +# GROMACS is free software; you can redistribute it and/or +# modify it under the terms of the GNU Lesser General Public License +# as published by the Free Software Foundation; either version 2.1 +# of the License, or (at your option) any later version. +# +# GROMACS is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU +# Lesser General Public License for more details. +# +# You should have received a copy of the GNU Lesser General Public +# License along with GROMACS; if not, see +# http://www.gnu.org/licenses, or write to the Free Software Foundation, +# Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. +# +# If you want to redistribute modifications to GROMACS, please +# consider that scientific software is very special. Version +# control is crucial - bugs must be traceable. We will be happy to +# consider code for inclusion in the official distribution, but +# derived work must not be called official GROMACS. Details are found +# in the README & COPYING files - if they are missing, get the +# official version at http://www.gromacs.org. +# +# To help us fund GROMACS development, we humbly ask that you cite +# the research papers on the package. Check out http://www.gromacs.org. + +set(CMAKE_C_FLAGS_INIT "-mmic") diff --git a/cmake/Platform/Linux-Intel-CXX-XeonPhi.cmake b/cmake/Platform/Linux-Intel-CXX-XeonPhi.cmake new file mode 100644 index 0000000000..98d843fd34 --- /dev/null +++ b/cmake/Platform/Linux-Intel-CXX-XeonPhi.cmake @@ -0,0 +1,35 @@ +# +# This file is part of the GROMACS molecular simulation package. +# +# Copyright (c) 2014,2015, by the GROMACS development team, led by +# Mark Abraham, David van der Spoel, Berk Hess, and Erik Lindahl, +# and including many others, as listed in the AUTHORS file in the +# top-level source directory and at http://www.gromacs.org. +# +# GROMACS is free software; you can redistribute it and/or +# modify it under the terms of the GNU Lesser General Public License +# as published by the Free Software Foundation; either version 2.1 +# of the License, or (at your option) any later version. +# +# GROMACS is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU +# Lesser General Public License for more details. +# +# You should have received a copy of the GNU Lesser General Public +# License along with GROMACS; if not, see +# http://www.gnu.org/licenses, or write to the Free Software Foundation, +# Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. +# +# If you want to redistribute modifications to GROMACS, please +# consider that scientific software is very special. Version +# control is crucial - bugs must be traceable. We will be happy to +# consider code for inclusion in the official distribution, but +# derived work must not be called official GROMACS. Details are found +# in the README & COPYING files - if they are missing, get the +# official version at http://www.gromacs.org. +# +# To help us fund GROMACS development, we humbly ask that you cite +# the research papers on the package. Check out http://www.gromacs.org. + +set(CMAKE_CXX_FLAGS_INIT "-mmic") diff --git a/cmake/Platform/XeonPhi.cmake b/cmake/Platform/XeonPhi.cmake index 3e7956a83c..a23feaf9f8 100644 --- a/cmake/Platform/XeonPhi.cmake +++ b/cmake/Platform/XeonPhi.cmake @@ -1,7 +1,7 @@ # # This file is part of the GROMACS molecular simulation package. # -# Copyright (c) 2014, by the GROMACS development team, led by +# Copyright (c) 2014,2015, by the GROMACS development team, led by # Mark Abraham, David van der Spoel, Berk Hess, and Erik Lindahl, # and including many others, as listed in the AUTHORS file in the # top-level source directory and at http://www.gromacs.org. @@ -31,14 +31,13 @@ # # To help us fund GROMACS development, we humbly ask that you cite # the research papers on the package. Check out http://www.gromacs.org. + +set(CMAKE_SYSTEM_NAME Linux) +set(CMAKE_SYSTEM_PROCESSOR XeonPhi) if(NOT GMX_MPI) set(CMAKE_C_COMPILER "icc") set(CMAKE_CXX_COMPILER "icpc") else() - set(CMAKE_C_COMPILER "mpiicc") + set(CMAKE_C_COMPILER "mpiicc") #FindMPI doesn't work (#14991) set(CMAKE_CXX_COMPILER "mpiicpc") - set(GMX_PREFER_STATIC_LIBS ON CACHE BOOL "Shared libraries on Xeon Phi with MPI don't work.") endif() -set(CMAKE_CXX_FLAGS "-mmic" CACHE STRING "Flags used by the compiler during all build types.") -set(CMAKE_C_FLAGS "-mmic" CACHE STRING "Flags used by the compiler during all build types.") -set(GMX_FFT_LIBRARY "mkl" CACHE STRING "FFT library")