From d429265d8b28fda1550e22a58a5d19a080c6d788 Mon Sep 17 00:00:00 2001 From: Evan Bollig Date: Wed, 17 Feb 2021 10:09:34 -0600 Subject: Bugfix for x86_64 skylake and other archs where SIMD features are disabled by Fall back option. (#21727) --- var/spack/repos/builtin/packages/gromacs/package.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/var/spack/repos/builtin/packages/gromacs/package.py b/var/spack/repos/builtin/packages/gromacs/package.py index e273efedb8..1efb196e58 100644 --- a/var/spack/repos/builtin/packages/gromacs/package.py +++ b/var/spack/repos/builtin/packages/gromacs/package.py @@ -262,13 +262,16 @@ class Gromacs(CMakePackage): ('avx512', 'AVX_512') in simd_features): simd_features.remove(('avx512', 'AVX_512')) + feature_set = False for feature, flag in reversed(simd_features): if feature in target: options.append('-DGMX_SIMD:STRING={0}'.format(flag)) + feature_set = True break # Fall back - options.append('-DGMX_SIMD:STRING=None') + if not feature_set: + options.append('-DGMX_SIMD:STRING=None') # Use the 'rtdscp' assembly instruction only on # appropriate architectures -- cgit v1.2.3-70-g09d2