[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [PATCH v1 2/2] target-ppc: cmprb - truncate RA to 8bits
From: |
David Gibson |
Subject: |
Re: [Qemu-ppc] [PATCH v1 2/2] target-ppc: cmprb - truncate RA to 8bits |
Date: |
Wed, 3 Aug 2016 11:17:51 +1000 |
User-agent: |
Mutt/1.6.2 (2016-07-01) |
On Tue, Aug 02, 2016 at 01:23:00PM +0530, Nikunj A Dadhania wrote:
> Missed the following bit in the instruction coding.
>
> src1 ← EXTZ(RA(56:63))
>
> Reported-by: Anton Blanchard <address@hidden>
> Signed-off-by: Nikunj A Dadhania <address@hidden>
> ---
>
> Can be squashed with the original commit
Done.
>
> target-ppc/translate.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/target-ppc/translate.c b/target-ppc/translate.c
> index dc89e6a..14f4b68 100644
> --- a/target-ppc/translate.c
> +++ b/target-ppc/translate.c
> @@ -867,6 +867,7 @@ static void gen_cmprb(DisasContext *ctx)
> tcg_gen_trunc_tl_i32(src1, cpu_gpr[rA(ctx->opcode)]);
> tcg_gen_trunc_tl_i32(src2, cpu_gpr[rB(ctx->opcode)]);
>
> + tcg_gen_andi_i32(src1, src1, 0xFF);
> tcg_gen_ext8u_i32(src2lo, src2);
> tcg_gen_shri_i32(src2, src2, 8);
> tcg_gen_ext8u_i32(src2hi, src2);
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature