about summary refs log tree commit diff
path: root/src/math/sinh.c
diff options
context:
space:
mode:
authorSzabolcs Nagy <nsz@port70.net>2020-01-20 20:38:45 +0000
committerRich Felker <dalias@aerifal.cx>2020-02-21 23:42:12 -0500
commitd20558148d8a2c52229b02668627697e83ca3840 (patch)
treee4f3373bb8649db621bfeb871ed335e13886b343 /src/math/sinh.c
parentb3797d3b2e10e6fff2a6b04af917e61e95838b08 (diff)
downloadmusl-d20558148d8a2c52229b02668627697e83ca3840.tar.gz
musl-d20558148d8a2c52229b02668627697e83ca3840.tar.xz
musl-d20558148d8a2c52229b02668627697e83ca3840.zip
math: fix sinh overflows in non-nearest rounding
The final rounding operation should be done with the correct sign
otherwise huge results may incorrectly get rounded to or away from
infinity in upward or downward rounding modes.

This affected sinh and sinhf which set the sign on the result after
a potentially overflowing mul. There may be other non-nearest rounding
issues, but this was a known long standing issue with large ulp error
(depending on how ulp is defined near infinity).

The fix should have no effect on sinh and sinhf performance but may
have a tiny effect on cosh and coshf.
Diffstat (limited to 'src/math/sinh.c')
-rw-r--r--src/math/sinh.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/math/sinh.c b/src/math/sinh.c
index 00022c4e..a01951ae 100644
--- a/src/math/sinh.c
+++ b/src/math/sinh.c
@@ -34,6 +34,6 @@ double sinh(double x)
 
 	/* |x| > log(DBL_MAX) or nan */
 	/* note: the result is stored to handle overflow */
-	t = 2*h*__expo2(absx);
+	t = __expo2(absx, 2*h);
 	return t;
 }