about summary refs log tree commit diff
path: root/src/math/log1pf.c
diff options
context:
space:
mode:
authorSzabolcs Nagy <nsz@port70.net>2013-08-15 10:14:46 +0000
committerSzabolcs Nagy <nsz@port70.net>2013-08-15 10:14:46 +0000
commitc599f4f4d705ba215a2672b92baa1c29d9dd0735 (patch)
treef9931ee00f59bae9aba69b9dbcff18a5c1613018 /src/math/log1pf.c
parentcd18dc8525a275323819ac4c7826cc3404202f7b (diff)
downloadmusl-c599f4f4d705ba215a2672b92baa1c29d9dd0735.tar.gz
musl-c599f4f4d705ba215a2672b92baa1c29d9dd0735.tar.xz
musl-c599f4f4d705ba215a2672b92baa1c29d9dd0735.zip
math: fix asin, atan, log1p, tanh to raise underflow on subnormal
for these functions f(x)=x for small inputs, because f(0)=0 and
f'(0)=1, but for subnormal values they should raise the underflow
flag (required by annex F), if they are approximated by a polynomial
around 0 then spurious underflow should be avoided (not required by
annex F)

all these functions should raise inexact flag for small x if x!=0,
but it's not required by the standard and it does not seem a worthy
goal, so support for it is removed in some cases.

raising underflow:
- x*x may not raise underflow for subnormal x if FLT_EVAL_METHOD!=0
- x*x may raise spurious underflow for normal x if FLT_EVAL_METHOD==0
- in case of double subnormal x, store x as float
- in case of float subnormal x, store x*x as float
Diffstat (limited to 'src/math/log1pf.c')
-rw-r--r--src/math/log1pf.c7
1 files changed, 5 insertions, 2 deletions
diff --git a/src/math/log1pf.c b/src/math/log1pf.c
index 39832d28..c38e0bcb 100644
--- a/src/math/log1pf.c
+++ b/src/math/log1pf.c
@@ -43,9 +43,12 @@ float log1pf(float x)
 			return (x-x)/(x-x);         /* log1p(x<-1)=NaN */
 		}
 		if (ax < 0x38000000) {   /* |x| < 2**-15 */
-			/* raise inexact */
-			if (two25 + x > 0.0f && ax < 0x33800000)  /* |x| < 2**-24 */
+			/* if 0x1p-126 <= |x| < 0x1p-24, avoid raising underflow */
+			if (ax < 0x33800000 && ax >= 0x00800000)
 				return x;
+#if FLT_EVAL_METHOD != 0
+			FORCE_EVAL(x*x);
+#endif
 			return x - x*x*0.5f;
 		}
 		if (hx > 0 || hx <= (int32_t)0xbe95f619) { /* sqrt(2)/2- <= 1+x < sqrt(2)+ */