fp-bench: fix update_random_ops

The second test in the branches is wrong; fix while converting
to a switch statement, which is easier to get right.

Signed-off-by: Emilio G. Cota <cota@braap.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
This commit is contained in:
Emilio G. Cota 2018-12-21 14:51:59 -05:00 committed by Alex Bennée
parent 952bc8b3c2
commit 446cfb0d34
1 changed files with 8 additions and 3 deletions

View File

@ -143,15 +143,20 @@ static void update_random_ops(int n_ops, enum precision prec)
for (i = 0; i < n_ops; i++) {
uint64_t r = random_ops[i];
if (prec == PREC_SINGLE || PREC_FLOAT32) {
switch (prec) {
case PREC_SINGLE:
case PREC_FLOAT32:
do {
r = xorshift64star(r);
} while (!float32_is_normal(r));
} else if (prec == PREC_DOUBLE || PREC_FLOAT64) {
break;
case PREC_DOUBLE:
case PREC_FLOAT64:
do {
r = xorshift64star(r);
} while (!float64_is_normal(r));
} else {
break;
default:
g_assert_not_reached();
}
random_ops[i] = r;