[PATCH 4/4] Do not wrap expf and exp2f
Arjan van de Ven
arjan@linux.intel.com
Thu Sep 14 00:31:00 GMT 2017
On 9/13/2017 1:19 PM, Joseph Myers wrote:
> On Wed, 13 Sep 2017, Szabolcs Nagy wrote:
>
>> * sysdeps/x86_64/fpu/w_expf.c: New file.
>
> I don't think having w_expf.c alongside w_expf.S in the same directory
> makes sense. (An IFUNC w_expf.c in the multiarch subdirectory, used to
> build both fma and non-fma versions and replacing the existing
> e_expf-fma.S, may well make sense if benchmarks show improvements on the
> fma and non-fma .S files.)
fwiw I am assuming we'll retire the x86-64 .S files and use the C version
(for both FMA and non-FMA with some IFUNC selector) once this lands.
Optimizing the .S file was fun for FMA, but the C version has a smaller
polynomial and is faster ... data speaks and this wins.
More information about the Libc-alpha
mailing list