[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH 0/4] target-arm: Make Neon helpers use correct F
From: |
Aurelien Jarno |
Subject: |
[Qemu-devel] Re: [PATCH 0/4] target-arm: Make Neon helpers use correct FP status |
Date: |
Mon, 4 Apr 2011 20:29:04 +0200 |
User-agent: |
Mutt/1.5.20 (2009-06-14) |
On Mon, Apr 04, 2011 at 11:46:31AM +0100, Peter Maydell wrote:
> This is a patchset which takes the approach discussed in the
> comments on my earlier patch to fix this bug. We compile *_helper.c
> with HELPER_CFLAGS, so targets don't need to put all their helper
> routines in op_helper.c. (This only changes behaviour for target-arm
> because that's the only target that has files matching that pattern.)
> Patches two and three are large but fairly mechanical changes to
> remove explicit CPUState* arguments in favour of just using global
> 'env' in the two files which can now access it. Finally, patch 4
> is a two-liner to use the Neon FP status word...
>
> Peter Maydell (4):
> Makefile.target: Allow target helpers to be in any *_helper.c file
> target-arm: Use global env in neon_helper.c helpers
> target-arm: Use global env in iwmmxt_helper.c helpers
> target-arm: Make Neon helper routines use correct FP status
>
> Makefile.target | 2 +-
> target-arm/helpers.h | 256
> ++++++++++++++++++++++----------------------
> target-arm/iwmmxt_helper.c | 80 ++++++--------
> target-arm/neon_helper.c | 147 ++++++++++++-------------
> target-arm/translate.c | 248 ++++++++++++++++++++-----------------------
> 5 files changed, 349 insertions(+), 384 deletions(-)
>
Thanks, all applied.
--
Aurelien Jarno GPG: 1024D/F1BCDB73
address@hidden http://www.aurel32.net