Re: [Qemu-devel] [V2 PATCH] target-ppc: Bug: VSX Convert to Integer Should Truncate

2014-03-26 Thread Richard Henderson
On 03/26/2014 08:05 AM, Tom Musta wrote: > On 3/25/2014 1:06 PM, Peter Maydell wrote: >> Looking at this a little more closely, why aren't we >> just using the _round_to_zero versions of the float >> to int conversion softfloat functions? (This is how >> we implement fctiwz vs fctiw, for instance.)

Re: [Qemu-devel] [V2 PATCH] target-ppc: Bug: VSX Convert to Integer Should Truncate

2014-03-26 Thread Tom Musta
On 3/25/2014 1:06 PM, Peter Maydell wrote: > Looking at this a little more closely, why aren't we > just using the _round_to_zero versions of the float > to int conversion softfloat functions? (This is how > we implement fctiwz vs fctiw, for instance.) Unfortunately, the softfloat list of these fu

Re: [Qemu-devel] [V2 PATCH] target-ppc: Bug: VSX Convert to Integer Should Truncate

2014-03-25 Thread Tom Musta
On 3/25/2014 1:06 PM, Peter Maydell wrote: > > Looking at this a little more closely, why aren't we > just using the _round_to_zero versions of the float > to int conversion softfloat functions? (This is how > we implement fctiwz vs fctiw, for instance.) > > thanks > -- PMM > Fair question, Pe

Re: [Qemu-devel] [V2 PATCH] target-ppc: Bug: VSX Convert to Integer Should Truncate

2014-03-25 Thread Peter Maydell
On 23 March 2014 21:52, Tom Musta wrote: > The various VSX Convert to Integer instructions should truncate the > mantissa. This fix forces the softfloat rounding mode to "round to > zero" prior to performing the conversion. After the conversion is > completed, the internal rounding mode is resto

[Qemu-devel] [V2 PATCH] target-ppc: Bug: VSX Convert to Integer Should Truncate

2014-03-23 Thread Tom Musta
The various VSX Convert to Integer instructions should truncate the mantissa. This fix forces the softfloat rounding mode to "round to zero" prior to performing the conversion. After the conversion is completed, the internal rounding mode is restored from the PowerPC FPSCR bits. Signed-off-by: T