aarch64: Fix f{max,min}{f} build for GCC 4.9 and 5
commit58a813bf6e732211af53e690c92c14a50bb06e0e
authorAdhemerval Zanella <adhemerval.zanella@linaro.org>
Thu, 16 Nov 2017 21:52:59 +0000 (16 19:52 -0200)
committerAdhemerval Zanella <adhemerval.zanella@linaro.org>
Fri, 17 Nov 2017 11:23:07 +0000 (17 09:23 -0200)
tree3e1e9d399d6ee88abf654de335d310255c2ad882
parent6a6aa1f38959a5d17ff776b252604bece45864dc
aarch64: Fix f{max,min}{f} build for GCC 4.9 and 5

GCC 4.9 and 5 do not generate a correct f{max,min}nm instruction for
__builtin_{fmax,fmin}{f} without -ffinite-math-only.  It is clear a
compiler issue since the instruction can handle NaN and Inf correctly
and GCC6+ does not show this issue.

We can backport a fix to GCC 5, raise the minimum required GCC version
for aarch64 (since GCC 4.9 branch is now closed [1]) and/or add
configure check to check for this issue.  However I think
-ffinite-math-only should be safe for these specific implementations
and it is a simpler solution.

Checked on aarch64-linux-gnu with GCC 5.3.1.

* sysdeps/aarch64/fpu/Makefile (CFLAGS-s_fmax.c, CFLAGS-s_fmaxf.c,
CFLAGS-s_fmin.c, CFLAGS-s_fminf.c): New rule: add -ffinite-math-only.

[1] https://gcc.gnu.org/ml/gcc/2016-08/msg00010.html

Signed-off-by: Adhemerval Zanella <adhemerval.zanella@linaro.org>
Reviewed-by: Szabolcs Nagy <szabolcs.nagy@arm.com>
ChangeLog
sysdeps/aarch64/fpu/Makefile