[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 2/2] fpu/softfloat: raise float_invalid for N
From: |
Bastian Koppelmann |
Subject: |
Re: [Qemu-devel] [PATCH v2 2/2] fpu/softfloat: raise float_invalid for NaN/Inf in round_to_int_and_pack |
Date: |
Fri, 13 Apr 2018 17:24:21 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 |
On 04/13/2018 04:07 PM, Peter Maydell wrote:
> On 13 April 2018 at 15:06, Bastian Koppelmann
> <address@hidden> wrote:
>> On 04/13/2018 04:03 PM, Alex Bennée wrote:
>>> The re-factor broke the raising of INVALID when NaN/Inf is passed to
>>> the float_to_int conversion functions. round_to_uint_and_pack got this
>>> right for NaN but also missed out the Inf handling.
>>>
>>> Fixes https://bugs.launchpad.net/qemu/+bug/1759264
>>>
>>> Signed-off-by: Alex Bennée <address@hidden>
>>> Cc: Bastian Koppelmann <address@hidden>
>>> ---
>>> fpu/softfloat.c | 3 +++
>>> 1 file changed, 3 insertions(+)
>>>
>>
>> Reviewed-by: Bastian Koppelmann <address@hidden>
>
> Could you run your tricore test case with this patch? I think
> Alex was having difficulty getting that running...
This fixes the ftoi problem, so
Tested-by: Bastian Koppelmann <address@hidden>
However, my risu-like tests found another flag raising problem with
float32_div. I'll investigate it on Monday.
Cheers,
Bastian
Re: [Qemu-devel] [PATCH v2 2/2] fpu/softfloat: raise float_invalid for NaN/Inf in round_to_int_and_pack, Peter Maydell, 2018/04/13
Re: [Qemu-devel] [PATCH v2 0/2] Softfloat Fixes for 2.12, Peter Maydell, 2018/04/16