[PATCH] AArch64 compile dbl-64/e_sqrt.c with -ffp-contract=off

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

[PATCH] AArch64 compile dbl-64/e_sqrt.c with -ffp-contract=off

Marcus Shawcroft-2
This patch fixes the numerical errors in aarch64 sqrt() due to the
compilers use of fnmsub (negated fused multiply-subtract).

Previous thread:
https://sourceware.org/ml/libc-alpha/2013-12/msg00473.html

I'll leave this on the list for 24 hours before committing to give
folks the opportunity to comment on the (in)appropriateness of my
solution...

Cheers
/Marcus

glibc-e_sqrt-flags.txt (1K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] AArch64 compile dbl-64/e_sqrt.c with -ffp-contract=off

Joseph Myers
On Tue, 17 Dec 2013, Marcus Shawcroft wrote:

> This patch fixes the numerical errors in aarch64 sqrt() due to the
> compilers use of fnmsub (negated fused multiply-subtract).
>
> Previous thread:
> https://sourceware.org/ml/libc-alpha/2013-12/msg00473.html
>
> I'll leave this on the list for 24 hours before committing to give
> folks the opportunity to comment on the (in)appropriateness of my
> solution...

The source file in question is architecture-independent, so the makefile
change should also be architecture-independent (i.e. in
sysdeps/ieee754/dbl-64/Makefile).

--
Joseph S. Myers
[hidden email]