fix fmaf wrong result
authorSzabolcs Nagy <nsz@port70.net>
Sun, 1 Apr 2018 20:02:01 +0000 (20:02 +0000)
committerRich Felker <dalias@aerifal.cx>
Mon, 2 Apr 2018 22:57:41 +0000 (18:57 -0400)
if double precision r=x*y+z is not a half way case between two single
precision floats or it is an exact result then fmaf returns (float)r.

however the exactness check was wrong when |x*y| < |z| and could cause
incorrectly rounded result in nearest rounding mode when r is a half
way case.

fmaf(-0x1.26524ep-54, -0x1.cb7868p+11, 0x1.d10f5ep-29)
was incorrectly rounded up to 0x1.d117ap-29 instead of 0x1.d1179ep-29.
(exact result is 0x1.d1179efffffffecp-29, r is 0x1.d1179fp-29)

src/math/fmaf.c

index aa57feb69567c67a79aae02fc7ea18e07d22fd12..80f5cd8a332fff28bb3022d6303546d216c78985 100644 (file)
@@ -50,7 +50,7 @@ float fmaf(float x, float y, float z)
        /* Common case: The double precision result is fine. */
        if ((u.i & 0x1fffffff) != 0x10000000 || /* not a halfway case */
                e == 0x7ff ||                   /* NaN */
-               result - xy == z ||                 /* exact */
+               (result - xy == z && result - z == xy) || /* exact */
                fegetround() != FE_TONEAREST)       /* not round-to-nearest */
        {
                /*