diff mbox

[3/7,v5] package/fftw: add ARM optimisations

Message ID 571d54234d6782550cbf3ecfe3df20d886ec9d01.1426465664.git.yann.morin.1998@free.fr
State Changes Requested
Headers show

Commit Message

Yann E. MORIN March 16, 2015, 12:31 a.m. UTC
Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
[based on a patch by Guillaume]
Cc: guillaume william brs <guillaume.bressaix@gmail.com>
Cc: Gwenhael Goavec-Merou <gwenhael.goavec-merou@trabucayre.com>
---
 package/fftw/fftw.mk | 3 +++
 1 file changed, 3 insertions(+)

Comments

Gwenhael Goavec-Merou March 16, 2015, 7:51 a.m. UTC | #1
Dear Yann E. MORIN
On Mon, 16 Mar 2015 01:31:43 +0100
"Yann E. MORIN" <yann.morin.1998@free.fr> wrote:

[SNIP
>  
> +# ARM optimisations
> +FFTW_CONF_OPTS += $(if $(BR2_ARM_CPU_HAS_NEON),--enable,--disable)-neon
> +
>  $(eval $(autotools-package))

--enable-neon must be enabled only if --enable-single is enabled. Without this,
configure fails.

Something like :

ifeq ($(BR2_ARM_CPU_HAS_NEON)$(BR2_PACKAGE_FFTW_PRECISION_SINGLE),yy)
FFTW_CONF_OPTS += --enable-neon
else
FFTW_CONF_OPTS += --disable-neon
endif
must be used 

Regards.
Gwenhael Goavec-Merou
diff mbox

Patch

diff --git a/package/fftw/fftw.mk b/package/fftw/fftw.mk
index fd21fd4..856adad 100644
--- a/package/fftw/fftw.mk
+++ b/package/fftw/fftw.mk
@@ -18,4 +18,7 @@  FFTW_CONF_OPTS += $(if $(BR2_PACKAGE_FFTW_PRECISION_QUAD),--enable,--disable)-qu
 FFTW_CONF_OPTS += $(if $(BR2_X86_CPU_HAS_SSE),--enable,--disable)-sse
 FFTW_CONF_OPTS += $(if $(BR2_X86_CPU_HAS_SSE2),--enable,--disable)-sse2
 
+# ARM optimisations
+FFTW_CONF_OPTS += $(if $(BR2_ARM_CPU_HAS_NEON),--enable,--disable)-neon
+
 $(eval $(autotools-package))