From mboxrd@z Thu Jan  1 00:00:00 1970
From: gwenhael.goavec <gwenj@trabucayre.com>
Date: Sat, 14 Mar 2015 08:39:56 +0100
Subject: [Buildroot] [PATCH v2 1/1] fftw: add compile precision option
In-Reply-To: <20150313232521.29abacc5@free-electrons.com>
References: <1426242302-3676-1-git-send-email-gwenj@trabucayre.com>
 <20150313232521.29abacc5@free-electrons.com>
Message-ID: <20150314083956.740a5dc9@x230.trabucayre.com>
List-Id: <buildroot.busybox.net>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: buildroot@busybox.net

Dear Thomas Petazzoni

On Fri, 13 Mar 2015 23:25:21 +0100
Thomas Petazzoni <thomas.petazzoni@free-electrons.com> wrote:

> Dear Gwenhael Goavec-Merou,
> 
> On Fri, 13 Mar 2015 11:25:02 +0100, Gwenhael Goavec-Merou wrote:
> > From: Gwenhael Goavec-Merou <gwenhael.goavec-merou@trabucayre.com>
> > 
> > fftw has options to select compile precision between single, long-double and
> > quad. These options are exclusives. This patch adds choice to select precision
> > option.
> 
> Unfortunately, the quad precision support doesn't build, because not
> all toolchains have libquadmath in gcc:
> 
> checking for sinq in -lquadmath... no
> configure: error: quad precision requires libquadmath for quad-precision trigonometric routines
> 
> And the long double precision support also doesn't build:
> 
> checking for cosl... (cached) no
> configure: error: long-double precision requires long-double trigonometric routines
> 
> This was generated by the following defconfig:
> 
It's true. libquadmath is only enabled on x86 and x86_64. I must add a depend
on BR2_i386 || BR2_x86_64.

[SNIP]
> 
> Also, I see that FFTW has some NEON, Altivec, MMX and SSE/SSE2
> optimizations. Maybe it would make sense to enable them when the CPU
> supports them. However, beware that many of those optimizations only
> work in "single precision" mode.
> 
Yes. I plan to add these optimizations but that's for a future patch.
[SNIP]

Regards 
Gwen