Re: [Qemu-devel] [PATCH 02/22] softfloat: Add float to 16bit integer conversions.

2013-12-31 Thread Richard Henderson
On 12/31/2013 06:22 AM, Peter Maydell wrote: > Or do you mean we should call float32_to_int32() instead? > I think that ought to work... Yes, that's what I meant. r~

Re: [Qemu-devel] [PATCH 02/22] softfloat: Add float to 16bit integer conversions.

2013-12-31 Thread Peter Maydell
On 31 December 2013 14:18, Richard Henderson wrote: > On 12/31/2013 05:35 AM, Peter Maydell wrote: >> +int64_t v; >> +int_fast16_t res; >> +int old_exc_flags = get_float_exception_flags(status); >> + >> +v = float32_to_int64(a STATUS_VAR); > > Any good reason not to use int32 as th

Re: [Qemu-devel] [PATCH 02/22] softfloat: Add float to 16bit integer conversions.

2013-12-31 Thread Richard Henderson
On 12/31/2013 05:35 AM, Peter Maydell wrote: > +int64_t v; > +int_fast16_t res; > +int old_exc_flags = get_float_exception_flags(status); > + > +v = float32_to_int64(a STATUS_VAR); Any good reason not to use int32 as the intermediate, for the benefit of 32-bit hosts? Otherwise, R

[Qemu-devel] [PATCH 02/22] softfloat: Add float to 16bit integer conversions.

2013-12-31 Thread Peter Maydell
From: Will Newton ARMv8 requires support for converting 32 and 64bit floating point values to signed and unsigned 16bit integers. Signed-off-by: Will Newton [PMM: updated not to incorrectly set Inexact for Invalid inputs] Signed-off-by: Peter Maydell --- fpu/softfloat.c | 80 +