diff options
| author | Evan Bollig <ebbollig@amazon.com> | 2021-02-17 10:09:34 -0600 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-02-17 08:09:34 -0800 |
| commit | d429265d8b28fda1550e22a58a5d19a080c6d788 (patch) | |
| tree | 903f7d5df7201302338c44aa8c7b6ea289de6e96 | |
| parent | 3e72a12bfd32e34ac4347ed4ef9574ec15ae0f14 (diff) | |
| download | spack-d429265d8b28fda1550e22a58a5d19a080c6d788.tar.gz spack-d429265d8b28fda1550e22a58a5d19a080c6d788.tar.bz2 spack-d429265d8b28fda1550e22a58a5d19a080c6d788.tar.xz spack-d429265d8b28fda1550e22a58a5d19a080c6d788.zip | |
Bugfix for x86_64 skylake and other archs where SIMD features are disabled by Fall back option. (#21727)
| -rw-r--r-- | var/spack/repos/builtin/packages/gromacs/package.py | 5 |
1 files changed, 4 insertions, 1 deletions
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 |