[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 02/20] include/fpu/softfloat: remove USE_SOFT
From: |
Laurent Vivier |
Subject: |
Re: [Qemu-devel] [PATCH v2 02/20] include/fpu/softfloat: remove USE_SOFTFLOAT_STRUCT_TYPES |
Date: |
Tue, 9 Jan 2018 15:20:38 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.5.0 |
Le 09/01/2018 à 15:14, Peter Maydell a écrit :
> On 9 January 2018 at 14:12, Aurelien Jarno <address@hidden> wrote:
>> On 2018-01-09 13:27, Laurent Vivier wrote:
>>> Le 09/01/2018 à 13:22, Alex Bennée a écrit :
>>>> It's not actively built and when enabled things fail to compile. I'm
>>>> not sure the type-checking is really helping here. Seeing as we "own"
>>>> our softfloat now lets remove the cruft.
>>>
>>> I think it would be better to fix the build break than to remove the
>>> type-checking tool.
>>>
>>> but that's only my opinion...
>>
>> I agree with that. Those checks are useful for targets which call host
>> floating point functions for some instructions. This is ugly, but that's
>> what is still done for at least x86 for the trigonometrical functions.
>> The check prevents assigning a float or double value to a softfloat type
>> without calling the conversion function.
>
> Is gcc's codegen still bad enough that we have to default to not
> using the type-checking versions? If so, maybe we could at least
> enable the type-checking on an --enable-debug build, so it doesn't
> bitrot all the time.
What means "bad enough"? for some targets it works fine.
The problem with that is if it is not enabled all the time it becomes
broken really quick...
BTW, if it doesn't break Alex's work I'm volunteer to fix
USE_SOFTFLOAT_STRUCT_TYPES build.
Thanks,
Laurent
[Qemu-devel] [PATCH v2 05/20] include/fpu/softfloat: implement float16_set_sign helper, Alex Bennée, 2018/01/09
[Qemu-devel] [PATCH v2 03/20] include/fpu/softfloat: implement float16_abs helper, Alex Bennée, 2018/01/09
[Qemu-devel] [PATCH v2 06/20] include/fpu/softfloat: add some float16 constants, Alex Bennée, 2018/01/09