[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/4] openrisc: fix float32 and float64 helper de
From: |
Laurent Vivier |
Subject: |
Re: [Qemu-devel] [PATCH 2/4] openrisc: fix float32 and float64 helper definitions |
Date: |
Sun, 5 Nov 2017 15:39:48 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 |
Le 04/11/2017 à 22:20, Stafford Horne a écrit :
> Hello,
>
> On Fri, Nov 03, 2017 at 09:26:22PM +0100, Laurent Vivier wrote:
>> Signed-off-by: Laurent Vivier <address@hidden>
>
> There is no commit body here, do we need one? I can see in your cover letter
> its mentioned this was revealed after enabling "USE_SOFTFLOAT_STRUCT_TYPES".
>
> However, I'm just being curious why is this an issue? In softfloat I assume
> we
> pack our own floats so having the type that goes into a uint32 register
> wouldnt
> make much difference right?
It's not an issue. It's for consistency. We can store a float32 in an
uint32_t, but it's better to store float32 in float32 type.
And as said in include/fpu/softfloat.h:
"Use structures for soft-float types. This prevents accidentally mixing
them with native int/float types."
Thanks,
Laurent
- [Qemu-devel] [PATCH 0/4] softfloat: fix some helper definitions, Laurent Vivier, 2017/11/03
- [Qemu-devel] [PATCH 1/4] arm: fix float64 helper definitions, Laurent Vivier, 2017/11/03
- [Qemu-devel] [PATCH 4/4] softfloat: use floatXX_val() in XX_minmax(), Laurent Vivier, 2017/11/03
- [Qemu-devel] [PATCH 3/4] tricore: use make_float32() and float32_val(), Laurent Vivier, 2017/11/03
- Re: [Qemu-devel] [Qemu-arm] [PATCH 0/4] softfloat: fix some helper definitions, Philippe Mathieu-Daudé, 2017/11/03