From d20558148d8a2c52229b02668627697e83ca3840 Mon Sep 17 00:00:00 2001 From: Szabolcs Nagy Date: Mon, 20 Jan 2020 20:38:45 +0000 Subject: 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. --- src/math/coshf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/math/coshf.c') diff --git a/src/math/coshf.c b/src/math/coshf.c index b09f2ee5..e739cff9 100644 --- a/src/math/coshf.c +++ b/src/math/coshf.c @@ -28,6 +28,6 @@ float coshf(float x) } /* |x| > log(FLT_MAX) or nan */ - t = __expo2f(x); + t = __expo2f(x, 1.0f); return t; } -- cgit v1.2.1