CVS commit: pkgsrc/math/fftw

["Nia Alarie" <nia%netbsd.org@localhost>]

Module Name:    pkgsrc
Committed By:   nia
Date:           Sat Jun 12 07:30:23 UTC 2021

Modified Files:
        pkgsrc/math/fftw: options.mk

Log Message:
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.


To generate a diff of this commit:
cvs rdiff -u -r1.11 -r1.12 pkgsrc/math/fftw/options.mk

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: pkgsrc/math/fftw/options.mk
diff -u pkgsrc/math/fftw/options.mk:1.11 pkgsrc/math/fftw/options.mk:1.12
--- pkgsrc/math/fftw/options.mk:1.11    Sun May 16 10:16:26 2021
+++ pkgsrc/math/fftw/options.mk Sat Jun 12 07:30:23 2021
@@ -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 @@ FFTW_DOUBLE_OPTS+=    --enable-${opt}
 .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