qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH for-1.2] target-ppc: fix altivec instructions


From: Peter Maydell
Subject: Re: [Qemu-devel] [PATCH for-1.2] target-ppc: fix altivec instructions
Date: Sun, 26 Aug 2012 16:25:36 +0100

On 26 August 2012 15:14, Aurelien Jarno <address@hidden> wrote:
> Altivec instructions are not working anymore in PowerPC emulation,
> following commit d15f74fb, which inverted two registers in the call
> to helper. Fix that.
>
> Cc: Blue Swirl <address@hidden>
> Cc: Alexander Graf <address@hidden>
> Cc: Andreas Färber <address@hidden>
> Signed-off-by: Aurelien Jarno <address@hidden>
> ---
>  target-ppc/translate.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/target-ppc/translate.c b/target-ppc/translate.c
> index 91eb7a0..ac915cc 100644
> --- a/target-ppc/translate.c
> +++ b/target-ppc/translate.c
> @@ -6530,7 +6530,7 @@ static void glue(gen_, name)(DisasContext *ctx)         
>                 \
>      ra = gen_avr_ptr(rA(ctx->opcode));                                  \
>      rb = gen_avr_ptr(rB(ctx->opcode));                                  \
>      rd = gen_avr_ptr(rD(ctx->opcode));                                  \
> -    gen_helper_##name(rd, cpu_env, ra, rb);                             \
> +    gen_helper_##name(cpu_env, rd, ra, rb);                             \
>      tcg_temp_free_ptr(ra);                                              \
>      tcg_temp_free_ptr(rb);                                              \
>      tcg_temp_free_ptr(rd);                                              \

Reviewed-by: Peter Maydell <address@hidden>

(For these helpers, rd is an input to the helper, not an output,
which is why the cpu_env has to go first, unlike eg gen_helper_mulldo().)

-- PMM



reply via email to

[Prev in Thread] Current Thread [Next in Thread]