[pkgsrc/trunk]: pkgsrc/math/fftw fftw: Disable "generic" GCC simd intrinsics

[nia <nia%pkgsrc.org@localhost>]

details:   https://anonhg.NetBSD.org/pkgsrc/rev/294f30b3385e
branches:  trunk
changeset: 454478:294f30b3385e
user:      nia <nia%pkgsrc.org@localhost>
date:      Sat Jun 12 07:30:23 2021 +0000

description:
fftw: Disable "generic" GCC simd intrinsics

These produce poor code and it's always better to use the CPU-specific
intrinsics instead, which we already do on platforms where the test
suite doesn't fail.

diffstat:

 math/fftw/options.mk |  8 ++------
 1 files changed, 2 insertions(+), 6 deletions(-)

diffs (28 lines):

diff -r 11f6fad6ac71 -r 294f30b3385e math/fftw/options.mk
--- a/math/fftw/options.mk      Sat Jun 12 07:04:57 2021 +0000
+++ b/math/fftw/options.mk      Sat Jun 12 07:30:23 2021 +0000
@@ -1,12 +1,11 @@
-# $NetBSD: options.mk,v 1.11 2021/05/16 10:16:26 nia Exp $
+# $NetBSD: options.mk,v 1.12 2021/06/12 07:30:23 nia Exp $
 
 PKG_OPTIONS_VAR=       PKG_OPTIONS.fftw
-PKG_SUPPORTED_OPTIONS= fftw-fortran openmp mpi
+PKG_SUPPORTED_OPTIONS= fftw-fortran openmp mpi simd
 
 .include "../../mk/bsd.prefs.mk"
 
 .if !empty(PKGSRC_COMPILER:M*gcc*) || !empty(PKGSRC_COMPILER:M*clang*)
-PKG_SUPPORTED_OPTIONS+=        simd
 .  if ${MACHINE_ARCH} == "x86_64"
 PKG_SUPPORTED_OPTIONS+=        avx
 .  endif
@@ -24,9 +23,6 @@
 .endif
 
 .if !empty(PKG_OPTIONS:Msimd)
-# Generic compiler vector abstractions (GCC extension).
-FFTW_FLOAT_OPTS+=      --enable-generic-simd128 --enable-generic-simd256
-FFTW_DOUBLE_OPTS+=     --enable-generic-simd128 --enable-generic-simd256
 .  if !empty(MACHINE_ARCH:Mpowerpc*)
 FFTW_FLOAT_OPTS+=      --enable-altivec
 .  endif