fp-bench: remove wrong exponent raise in fill_random
authorEmilio G. Cota <cota@braap.org>
Fri, 21 Dec 2018 19:52:00 +0000 (14:52 -0500)
committerAlex Bennée <alex.bennee@linaro.org>
Tue, 22 Jan 2019 20:47:54 +0000 (20:47 +0000)
At this point random_ops[] only contains normals, so there's
no need to do anything to them. In fact, raising the exponent
here can make the output !normal, which is precisely
what the comment says we want to avoid.

Signed-off-by: Emilio G. Cota <cota@braap.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
tests/fp/fp-bench.c

index 546bac9c9c53f8b8e0d335b2949ab7ab48a14455..4ba5e1d2d4d321a4c9b3ee70dca2721c9561ddb7 100644 (file)
@@ -176,8 +176,6 @@ static void fill_random(union fp *ops, int n_ops, enum precision prec,
             if (no_neg && float32_is_neg(ops[i].f32)) {
                 ops[i].f32 = float32_chs(ops[i].f32);
             }
-            /* raise the exponent to limit the frequency of denormal results */
-            ops[i].f32 |= 0x40000000;
             break;
         case PREC_DOUBLE:
         case PREC_FLOAT64:
@@ -185,8 +183,6 @@ static void fill_random(union fp *ops, int n_ops, enum precision prec,
             if (no_neg && float64_is_neg(ops[i].f64)) {
                 ops[i].f64 = float64_chs(ops[i].f64);
             }
-            /* raise the exponent to limit the frequency of denormal results */
-            ops[i].f64 |= LIT64(0x4000000000000000);
             break;
         default:
             g_assert_not_reached();