[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2] temp-floating-point: Use make_float32/64 and
From: |
Chen Gang |
Subject: |
Re: [Qemu-devel] [PATCH v2] temp-floating-point: Use make_float32/64 and float32/64_val for the input register values |
Date: |
Wed, 7 Oct 2015 18:24:27 +0800 |
User-agent: |
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:31.0) Gecko/20100101 Thunderbird/31.7.0 |
On 10/7/15 17:29, Richard Henderson wrote:
> On 10/07/2015 01:56 AM, Chen Gang wrote:
>>> From 3655bbab04063bc0878510ab260832c7a6a2a925 Mon Sep 17 00:00:00 2001
>> From: Chen Gang<address@hidden>
>> Date: Tue, 6 Oct 2015 21:45:11 +0800
>> Subject: [PATCH v2] temp-floating-point: Use make_float32/64 and
>> float32/64_val for the input register values
>>
>> Original implementation use int*_to_float32 and float32_to_int*, which
>> will generate incorrect result.
>>
>> Also remove t_to_float64 and float64_to_t, use make_float64 and float64
>> instead of.
>>
>> Signed-off-by: Chen Gang<address@hidden>
>> ---
>> target-tilegx/fpu_helper.c | 63
>> +++++++++++-----------------------------------
>> 1 file changed, 15 insertions(+), 48 deletions(-)
>
> This is relative to... what? The v1 patch?
>
Yes, it is related with v1 patch (has the same name). The v1 patch is
based on patch "target-tilegx: Implement floating point temporarily".
Since they are temporary implementations, I guess, we can just left it
(do not integrate them into main line).
I sent these temporary implementations, because if someone need test
qemu tilegx, they can use these temporary patches (but of cause, I am
very glad if they can be integrated into main line, too).
Thanks.
--
Chen Gang (陈刚)
Open, share, and attitude like air, water, and life which God blessed