Re: [Qemu-devel] [PATCH 00/22] tcg-i386 cleanup and improvement, v2

[Aurelien Jarno <aurelien@aurel32.net>]

On Wed, Apr 28, 2010 at 11:24:33AM -0700, Richard Henderson wrote:
> Changes v1->v2:
>   * Dropped controversial bswap changes; bswap16 continues to use rolw.
>   * Tidy data16 as the last of the hard-coded constants.
> 

I have applied 1-2 and 5-9, but I can't apply the other without patches
3 and 4. The others are fine except a minor issue on patch 13. Could you
please resend a series with the changes?

> 
> Richard Henderson (22):
>   tcg-i386: Allocate call-saved registers first.
>   tcg-i386: Tidy initialization of tcg_target_call_clobber_regs.
>   tcg-i386: Tidy ext8u and ext16u operations.
>   tcg-i386: Tidy ext8s and ext16s operations.
>   tcg-i386: Tidy bswap operations.
>   tcg-i386: Tidy shift operations.
>   tcg-i386: Tidy move operations.
>   tcg-i386: Eliminate extra move from qemu_ld64.
>   tcg-i386: Tidy jumps.
>   tcg-i386: Tidy immediate arithmetic operations.
>   tcg-i386: Tidy non-immediate arithmetic operations.
>   tcg-i386: Tidy movi.
>   tcg-i386: Tidy push/pop.
>   tcg-i386: Tidy calls.
>   tcg-i386: Tidy ret.
>   tcg-i386: Tidy setcc.
>   tcg-i386: Tidy unary arithmetic.
>   tcg-i386: Tidy multiply.
>   tcg-i386: Tidy xchg.
>   tcg-i386: Tidy lea.
>   tcg-i386: Use lea for three-operand add.
>   tcg-i386: Tidy data16 prefixes.
> 
>  tcg/i386/tcg-target.c |  718 +++++++++++++++++++++++++++++--------------------
>  1 files changed, 433 insertions(+), 285 deletions(-)
> 
> 
> 
> 

-- 
Aurelien Jarno                          GPG: 1024D/F1BCDB73
aurelien@aurel32.net                 http://www.aurel32.net