[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v1 2/2] fpu/softfloat: raise float_invalid for N
From: |
Bastian Koppelmann |
Subject: |
Re: [Qemu-devel] [PATCH v1 2/2] fpu/softfloat: raise float_invalid for NaN in float_to_int |
Date: |
Thu, 12 Apr 2018 14:34:34 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 |
On 04/12/2018 02:26 PM, Peter Maydell wrote:
> On 12 April 2018 at 12:58, Alex Bennée <address@hidden> wrote:
>> Fixes https://bugs.launchpad.net/qemu/+bug/1759264
>>
>> Signed-off-by: Alex Bennée <address@hidden>
>> Cc: Bastian Koppelmann <address@hidden>
>> ---
>> fpu/softfloat.c | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/fpu/softfloat.c b/fpu/softfloat.c
>> index 9b99aa6ec8..ddc77c273c 100644
>> --- a/fpu/softfloat.c
>> +++ b/fpu/softfloat.c
>> @@ -1344,6 +1344,7 @@ static int64_t round_to_int_and_pack(FloatParts in,
>> int rmode,
>> case float_class_qnan:
>> case float_class_dnan:
>> case float_class_msnan:
>> + s->float_exception_flags = orig_flags | float_flag_invalid;
>> return max;
>> case float_class_inf:
>> return p.sign ? min : max;
>
> Don't we also need to raise the Invalid flag for float_class_inf ?
>
Yup, as mentioned in the bug report.
Cheers,
Bastian