diff options
| author | Richard Henderson | 2022-04-01 15:22:39 +0200 |
|---|---|---|
| committer | Richard Henderson | 2022-04-27 05:01:55 +0200 |
| commit | 9343c884445201cfd84955f199b13783fa829372 (patch) | |
| tree | f81e3608749428b883dccda046c10e55bfe0af3a /fpu | |
| parent | softfloat: Fix declaration of partsN_compare (diff) | |
| download | qemu-9343c884445201cfd84955f199b13783fa829372.tar.gz qemu-9343c884445201cfd84955f199b13783fa829372.tar.xz qemu-9343c884445201cfd84955f199b13783fa829372.zip | |
softfloat: Use FloatRelation within partsN_compare
As the return type is FloatRelation, it's clearer to
use the type for 'cmp' within the function.
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Message-Id: <20220401132240.79730-3-richard.henderson@linaro.org>
Diffstat (limited to 'fpu')
| -rw-r--r-- | fpu/softfloat-parts.c.inc | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/fpu/softfloat-parts.c.inc b/fpu/softfloat-parts.c.inc index db3e1f393d..bbeadaa189 100644 --- a/fpu/softfloat-parts.c.inc +++ b/fpu/softfloat-parts.c.inc @@ -1327,16 +1327,19 @@ static FloatRelation partsN(compare)(FloatPartsN *a, FloatPartsN *b, float_status *s, bool is_quiet) { int ab_mask = float_cmask(a->cls) | float_cmask(b->cls); - int cmp; if (likely(ab_mask == float_cmask_normal)) { + FloatRelation cmp; + if (a->sign != b->sign) { goto a_sign; } - if (a->exp != b->exp) { - cmp = a->exp < b->exp ? -1 : 1; - } else { + if (a->exp == b->exp) { cmp = frac_cmp(a, b); + } else if (a->exp < b->exp) { + cmp = float_relation_less; + } else { + cmp = float_relation_greater; } if (a->sign) { cmp = -cmp; |