[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 02/21] target-s390x: split FPU ops
From: |
Blue Swirl |
Subject: |
Re: [Qemu-devel] [PATCH 02/21] target-s390x: split FPU ops |
Date: |
Tue, 4 Sep 2012 19:40:34 +0000 |
On Tue, Sep 4, 2012 at 6:42 PM, Richard Henderson <address@hidden> wrote:
> On 09/02/2012 10:33 AM, Blue Swirl wrote:
>> +/* fpu_helper.c */
>> +uint32_t set_cc_f32(float32 v1, float32 v2);
>> +uint32_t set_cc_f64(float64 v1, float64 v2);
>> +uint32_t set_cc_nz_f32(float32 v);
>> +uint32_t set_cc_nz_f64(float64 v);
>> +
>
> I think that the CC handling should stay together, regardless of FPU-ness.
> These functions are quite small and can be usefully inlined by the compiler.
>
> OTOH, this is much easier to do with my translate.c rewrite, so maybe I'll
> just take responsibility for moving them back...
The problem is that these are used by some FPU ops as well as
condition code ops. Maybe it's better to move them to cpu.h as inline
functions?
It could be also possible to upgrade condition code functions to full
helpers, that could help implementing lazy condition code evaluation.
I noticed that the helpers don't use TCGv registers for register
access but instead the helpers access env->regs[] and env->fregs[]
directly, this probably would need to be changed too.
>
>
> r~
- [Qemu-devel] [PATCH 01/21] target-s390x: fix style, (continued)
- [Qemu-devel] [PATCH 03/21] target-s390x: split condition code helpers, Blue Swirl, 2012/09/02
- [Qemu-devel] [PATCH 02/21] target-s390x: split FPU ops, Blue Swirl, 2012/09/02
- Re: [Qemu-devel] [PATCH 02/21] target-s390x: split FPU ops, Richard Henderson, 2012/09/04
- Re: [Qemu-devel] [PATCH 02/21] target-s390x: split FPU ops,
Blue Swirl <=
- Re: [Qemu-devel] [PATCH 02/21] target-s390x: split FPU ops, Richard Henderson, 2012/09/04
- Re: [Qemu-devel] [PATCH 02/21] target-s390x: split FPU ops, Alexander Graf, 2012/09/04
- Re: [Qemu-devel] [PATCH 02/21] target-s390x: split FPU ops, Richard Henderson, 2012/09/05
- Re: [Qemu-devel] [PATCH 02/21] target-s390x: split FPU ops, Alexander Graf, 2012/09/05
- Re: [Qemu-devel] [PATCH 02/21] target-s390x: split FPU ops, Alexander Graf, 2012/09/06
- Re: [Qemu-devel] [PATCH 02/21] target-s390x: split FPU ops, Richard Henderson, 2012/09/06
- Re: [Qemu-devel] [PATCH 02/21] target-s390x: split FPU ops, Alexander Graf, 2012/09/07
- Re: [Qemu-devel] [PATCH 02/21] target-s390x: split FPU ops, Andreas Färber, 2012/09/07
- Re: [Qemu-devel] [PATCH 02/21] target-s390x: split FPU ops, Aurelien Jarno, 2012/09/07
[Qemu-devel] [PATCH 04/21] target-s390x: split integer helpers, Blue Swirl, 2012/09/02