[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 06/21] target-sparc: Extract common code for flo
From: |
Richard Henderson |
Subject: |
Re: [Qemu-devel] [PATCH 06/21] target-sparc: Extract common code for floating-point operations. |
Date: |
Tue, 18 Oct 2011 15:21:16 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:7.0) Gecko/20110927 Thunderbird/7.0 |
On 10/18/2011 01:24 PM, Blue Swirl wrote:
>> #ifdef TARGET_SPARC64
>> -float64 helper_fabsd(CPUState *env, float64 src)
>> +float64 helper_fabsd(float64 src)
>
> This probably should go to previous patch.
Sure.
>> +/* Turn off the stupid always-inline hack in osdep.h. This gets in the
>> + way of the callback mechanisms we use in this file, generating warnings
>> + for always-inline functions called indirectly. */
>> +#define always_inline inline
>
> It would be better to just delete the offending (or all) inlines.
I certainly would like to delete the offending hack in osdep.h.
The inline markers themselves are generated by def-helper.h, and are required
so that we don't wind up with a corresponding number of defined-but-not-used
errors from the helper.h definitions.
I really didn't know any one way to handle this situation that would be
immediately acceptable to everyone. I assumed limiting the change to
the sparc front-end would minimize the pushback.
>> +static void gen_ne_fop_FF(DisasContext *dc, int rd, int rs,
>
> 'ne' is for no exception? How about noexcp or something?
no-exception when it's first introduced. Then after patch 11 it would
become no-env. Preferences for the intermediate stage?
r~
- [Qemu-devel] [PATCH 00/21] Sparc FPU/VIS improvements, Richard Henderson, 2011/10/18
- [Qemu-devel] [PATCH 05/21] target-sparc: Make VIS helpers const when possible., Richard Henderson, 2011/10/18
- [Qemu-devel] [PATCH 04/21] target-sparc: Pass float64 parameters instead of dt0/1 temporaries., Richard Henderson, 2011/10/18
- [Qemu-devel] [PATCH 11/21] target-sparc: Do exceptions management fully inside the helpers., Richard Henderson, 2011/10/18
- [Qemu-devel] [PATCH 13/21] target-sparc: Implement PDIST., Richard Henderson, 2011/10/18
- [Qemu-devel] [PATCH 12/21] sparc-linux-user: Handle SIGILL., Richard Henderson, 2011/10/18