[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 2/8] softfloat: use bits32 instead of uint32
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH v2 2/8] softfloat: use bits32 instead of uint32 |
Date: |
Tue, 4 Jan 2011 15:51:37 +0000 |
On 4 January 2011 15:15, Aurelien Jarno <address@hidden> wrote:
> Use bits32 instead of uint32 when manipulating floating point values
> directly for consistency reasons.
I'm not convinced this patch is particularly worthwhile, especially since
Andreas is working on a patchset which will convert all the bits32
uses back into uint32_t anyway, which is the direction to go if we
want to make the fpu/ code consistent about its type usage.
If you do want to do this, the commit message should be "uint32_t"
not "uint32" (which is a different type!)
> int float32_is_quiet_nan( float32 a1 )
> {
> float32u u;
> - uint64_t a;
> + bits32 a;
> u.f = a1;
> a = u.i;
> return ( 0xFF800000 < ( a<<1 ) );
This change is actually changing the type: shouldn't it be bits64 ?
It seems a bit inconsistent to change softfloat-native.c:float32_is_quiet_nan()
but not softfloat-native.c:float64_is_quiet_nan() (which uses uint64_t).
Personally I'd just drop this patch.
-- PMM
- [Qemu-devel] (no subject), Aurelien Jarno, 2011/01/04
- [Qemu-devel] [PATCH v2 1/8] softfloat: remove HPPA specific code, Aurelien Jarno, 2011/01/04
- [Qemu-devel] [PATCH v2 3/8] softfloat: rename *IsNaN variables to *IsQuietNaN, Aurelien Jarno, 2011/01/04
- [Qemu-devel] [PATCH v2 4/8] softfloat: fix float{32, 64}_maybe_silence_nan() for MIPS, Aurelien Jarno, 2011/01/04
- [Qemu-devel] [PATCH v2 5/8] softfloat: add float{x80, 128}_maybe_silence_nan(), Aurelien Jarno, 2011/01/04
- [Qemu-devel] [PATCH v2 8/8] target-ppc: Implement correct NaN propagation rules, Aurelien Jarno, 2011/01/04
- [Qemu-devel] [PATCH v2 2/8] softfloat: use bits32 instead of uint32, Aurelien Jarno, 2011/01/04
- Re: [Qemu-devel] [PATCH v2 2/8] softfloat: use bits32 instead of uint32,
Peter Maydell <=
- [Qemu-devel] [PATCH v2 6/8] softfloat: use float{32, 64, x80, 128}_maybe_silence_nan(), Aurelien Jarno, 2011/01/04
- [Qemu-devel] [PATCH v2 7/8] target-mips: Implement correct NaN propagation rules, Aurelien Jarno, 2011/01/04
- Re: [Qemu-devel] (no subject), Peter Maydell, 2011/01/04