Hi Mark, > -----Original Message----- > From: Mark H Weaver [mailto:m...@netris.org] > Sent: Thursday, May 03, 2018 6:26 PM > To: Eric Bavier > Cc: guix-devel@gnu.org > Subject: Re: 01/01: gnu: fftw: Build SIMD codelets. > > Hi Eric, > > ericbav...@centurylink.net (Eric Bavier) writes: > > > bavier pushed a commit to branch core-updates in repository guix. > > > > commit 65bb22796f854cbc3eae053a80b1d64365dad376 > > Author: Eric Bavier <bav...@cray.com> > > Date: Fri Apr 6 10:53:06 2018 -0500 > > > > gnu: fftw: Build SIMD codelets. > > > > * gnu/packages/algebra.scm (fftw)[arguments]: Remove 'no-native > phase; use > > configure cache value instead. Add configure flags for SIMD codelets. > > (fftwf)[arguments]: Add neon configuration flag for 32-bit arm. > > (fftw-avx): Remove variable. > > [...] > > > @@ -560,7 +569,10 @@ cosine/ sine transforms or DCT/DST).") > > (arguments > > (substitute-keyword-arguments (package-arguments fftw) > > ((#:configure-flags cf) > > - `(cons "--enable-float" ,cf)))) > > + (if (string-prefix? "arm" (or (%current-target-system) > > + (%current-system))) > > + `(cons "--enable-neon" ,cf) > > + cf)))) > > Did you intend to remove the "--enable-float" configure flag in fftwf? > You didn't mention this change in the commit log.
Indeed, I think this was unintentional. > Can you take a look? Yes, thanks for checking. `~Eric
