Message ID | 1290538431-13170-12-git-send-email-peter.maydell@linaro.org |
---|---|
State | New |
Headers | show |
On Tue, Nov 23, 2010 at 06:53:50PM +0000, Peter Maydell wrote: > Use the softfloat conversion routines for conversion to 16 bit > integers, because just casting to a 16 bit type truncates the > value rather than saturating it at 16-bit MAXINT/MININT. > > Signed-off-by: Peter Maydell <peter.maydell@linaro.org> Reviewed-by: Nathan Froyd <froydnj@codesourcery.com> -Nathan
diff --git a/target-arm/helper.c b/target-arm/helper.c index 6a891da..96d6fb4 100644 --- a/target-arm/helper.c +++ b/target-arm/helper.c @@ -2567,7 +2567,7 @@ ftype VFP_HELPER(to##name, p)(ftype x, uint32_t shift, CPUState *env) \ return ftype##_zero; \ } \ tmp = ftype##_scalbn(x, shift, &env->vfp.fp_status); \ - return vfp_ito##p((itype)ftype##_to_##sign##int32_round_to_zero(tmp, \ + return vfp_ito##p(ftype##_to_##itype##_round_to_zero(tmp, \ &env->vfp.fp_status)); \ }
Use the softfloat conversion routines for conversion to 16 bit integers, because just casting to a 16 bit type truncates the value rather than saturating it at 16-bit MAXINT/MININT. Signed-off-by: Peter Maydell <peter.maydell@linaro.org> --- target-arm/helper.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)