In float64_to_uint64_round_to_zero() a typo meant that we were
taking the uint64_t return value from float64_to_uint64() and
putting it into an int64_t variable before returning it as
uint64_t again. Use uint64_t instead of pointlessly casting it
back and forth to int64_t.
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
---
Spotted while reading the code...
fpu/softfloat.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fpu/softfloat.c b/fpu/softfloat.c
index c295f31..218b375 100644
--- a/fpu/softfloat.c
+++ b/fpu/softfloat.c
@@ -7386,7 +7386,7 @@ uint64_t float64_to_uint64_round_to_zero(float64 a, float_status *status)
{
signed char current_rounding_mode = status->float_rounding_mode;
set_float_rounding_mode(float_round_to_zero, status);
- int64_t v = float64_to_uint64(a, status);
+ uint64_t v = float64_to_uint64(a, status);
set_float_rounding_mode(current_rounding_mode, status);
return v;
}
--
2.7.4
Le 03/02/2017 à 19:59, Peter Maydell a écrit : > In float64_to_uint64_round_to_zero() a typo meant that we were > taking the uint64_t return value from float64_to_uint64() and > putting it into an int64_t variable before returning it as > uint64_t again. Use uint64_t instead of pointlessly casting it > back and forth to int64_t. > > Signed-off-by: Peter Maydell <peter.maydell@linaro.org> Reviewed-by: Laurent Vivier <laurent@vivier.eu> > --- > Spotted while reading the code... > > fpu/softfloat.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/fpu/softfloat.c b/fpu/softfloat.c > index c295f31..218b375 100644 > --- a/fpu/softfloat.c > +++ b/fpu/softfloat.c > @@ -7386,7 +7386,7 @@ uint64_t float64_to_uint64_round_to_zero(float64 a, float_status *status) > { > signed char current_rounding_mode = status->float_rounding_mode; > set_float_rounding_mode(float_round_to_zero, status); > - int64_t v = float64_to_uint64(a, status); > + uint64_t v = float64_to_uint64(a, status); > set_float_rounding_mode(current_rounding_mode, status); > return v; > } >
On 02/03/2017 03:59 PM, Peter Maydell wrote: > In float64_to_uint64_round_to_zero() a typo meant that we were > taking the uint64_t return value from float64_to_uint64() and > putting it into an int64_t variable before returning it as > uint64_t again. Use uint64_t instead of pointlessly casting it > back and forth to int64_t. > > Signed-off-by: Peter Maydell <peter.maydell@linaro.org> Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org> > --- > Spotted while reading the code... > > fpu/softfloat.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/fpu/softfloat.c b/fpu/softfloat.c > index c295f31..218b375 100644 > --- a/fpu/softfloat.c > +++ b/fpu/softfloat.c > @@ -7386,7 +7386,7 @@ uint64_t float64_to_uint64_round_to_zero(float64 a, float_status *status) > { > signed char current_rounding_mode = status->float_rounding_mode; > set_float_rounding_mode(float_round_to_zero, status); > - int64_t v = float64_to_uint64(a, status); > + uint64_t v = float64_to_uint64(a, status); > set_float_rounding_mode(current_rounding_mode, status); > return v; > } >
03.02.2017 21:59, Peter Maydell wrote: > In float64_to_uint64_round_to_zero() a typo meant that we were > taking the uint64_t return value from float64_to_uint64() and > putting it into an int64_t variable before returning it as > uint64_t again. Use uint64_t instead of pointlessly casting it > back and forth to int64_t. Applied to -trivial, thanks! /mjt
© 2016 - 2024 Red Hat, Inc.