[Top][All Lists]
[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