aboutsummaryrefslogtreecommitdiff
path: root/ChangeLog
diff options
context:
space:
mode:
authorJoseph Myers <joseph@codesourcery.com>2015-02-06 15:44:07 +0000
committerJoseph Myers <joseph@codesourcery.com>2015-02-06 15:44:07 +0000
commit1c7a4a51a30dd001c81630156458ee55fc2e883c (patch)
tree3116ff19257a9cb385bc0f70d67becdb742d0395 /ChangeLog
parent61f006c12d8ff3a80a95a36fed81c908a2e39650 (diff)
downloadglibc-1c7a4a51a30dd001c81630156458ee55fc2e883c.tar
glibc-1c7a4a51a30dd001c81630156458ee55fc2e883c.tar.gz
glibc-1c7a4a51a30dd001c81630156458ee55fc2e883c.tar.bz2
glibc-1c7a4a51a30dd001c81630156458ee55fc2e883c.zip
soft-fp: Fix _FP_FMA when product is zero and third argument is finite (bug 17932).
soft-fp's _FP_FMA fails to set the result's exponent for cases where the result of the multiplication is 0, yielding incorrect (arbitrary, depending on uninitialized values) results for those cases. This affects libm for architectures using soft-fp to implement fma. This patch adds the exponent setting and tests for this case. Tested for ARM soft-float (which uses soft-fp fma), x86_64 and x86 (to verify not introducing new libm test failures there). (This bug showed up in testing my patch to move the Linux kernel to current soft-fp. math/Makefile has "override CFLAGS += -Wno-uninitialized" which would have stopped compiler warnings from showing up this problem, although I wouldn't be surprised if removing that shows spurious warnings from this code, if the compiler fails to follow that various cases where the exponent is uninitialized don't need it initialized because the class is set to a value meaning the uninitialized exponent isn't used.) [BZ #17932] * soft-fp/op-common.h (_FP_FMA): Set exponent of result in case where multiplication results in zero and third argument is finite and nonzero. * math/auto-libm-test-in: Add more tests of fma. * math/auto-libm-test-out: Regenerated.
Diffstat (limited to 'ChangeLog')
-rw-r--r--ChangeLog7
1 files changed, 7 insertions, 0 deletions
diff --git a/ChangeLog b/ChangeLog
index 32b766728e..8fa864c2a6 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,12 @@
2015-02-06 Joseph Myers <joseph@codesourcery.com>
+ [BZ #17932]
+ * soft-fp/op-common.h (_FP_FMA): Set exponent of result in case
+ where multiplication results in zero and third argument is finite
+ and nonzero.
+ * math/auto-libm-test-in: Add more tests of fma.
+ * math/auto-libm-test-out: Regenerated.
+
* soft-fp/op-common.h (_FP_CMP_CHECK_DENORM): New macro.
(_FP_CMP_CHECK_FLUSH_ZERO): Likewise.
(_FP_CMP): Use_FP_CMP_CHECK_DENORM and _FP_CMP_CHECK_FLUSH_ZERO.