[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/4] softfloat: add float32_set_sign(), float32_
From: |
Christophe Lyon |
Subject: |
Re: [Qemu-devel] [PATCH 2/4] softfloat: add float32_set_sign(), float32_infinity, float64_half, float64_256 and float64_512. |
Date: |
Mon, 21 Feb 2011 10:59:22 +0100 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686 (x86_64); en-US; rv:1.9.2.13) Gecko/20101207 Lightning/1.0b2 Thunderbird/3.1.7 |
On 20.02.2011 23:20, Aurelien Jarno wrote:
> On Sun, Feb 20, 2011 at 10:09:46PM +0000, Peter Maydell wrote:
>> On 20 February 2011 21:52, Aurelien Jarno <address@hidden> wrote:
>>> While it's probably a good idea to define the commonly used values in
>>> softfloat.h, I don't think we should have all the values used by the
>>> different targets here. Infinity, one, half, two probably have their
>>> place here, I don't think it's the case of 256 and 512. It should be
>>> better to defined them at the target level.
>>
>> Are you happy with targets just doing make_float*() on a
>> bit pattern? I guess that's the most straightforward thing,
>
> Yes, I think it is the way to go.
>
OK I will change that.
>>> Also for consistency, I
>>> think it's better to define these value for all float size, or at least
>>> for all the common ones (32, 64, maybe 16).
>>
>> I wouldn't bother with 16, only ARM uses that and only for
>> conversions to other formats.
>>
>
> That's true, so let's do it float 32 and 64.
>
I will add these too.
Christophe.
- [Qemu-devel] [PATCH v4 0/4] ARM: fix Neon VRECPE and VRSQRTE instructions., Christophe Lyon, 2011/02/18
- [Qemu-devel] [PATCH 2/4] softfloat: add float32_set_sign(), float32_infinity, float64_half, float64_256 and float64_512., Christophe Lyon, 2011/02/18
- Re: [Qemu-devel] [PATCH 2/4] softfloat: add float32_set_sign(), float32_infinity, float64_half, float64_256 and float64_512., Aurelien Jarno, 2011/02/20
- Re: [Qemu-devel] [PATCH 2/4] softfloat: add float32_set_sign(), float32_infinity, float64_half, float64_256 and float64_512., Peter Maydell, 2011/02/20
- Re: [Qemu-devel] [PATCH 2/4] softfloat: add float32_set_sign(), float32_infinity, float64_half, float64_256 and float64_512., Aurelien Jarno, 2011/02/20
- Re: [Qemu-devel] [PATCH 2/4] softfloat: add float32_set_sign(), float32_infinity, float64_half, float64_256 and float64_512.,
Christophe Lyon <=
[Qemu-devel] [PATCH 3/4] target-arm: fix support for VRECPE., Christophe Lyon, 2011/02/18
[Qemu-devel] [PATCH 4/4] target-arm: fix support for VRSQRTE., Christophe Lyon, 2011/02/18
[Qemu-devel] [PATCH 1/4] softfloat: move all default NaN definitions to softfloat.h., Christophe Lyon, 2011/02/18