qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH] mips: Fix BC1ANY[24]F instructions


From: Andreas Färber
Subject: Re: [Qemu-devel] [PATCH] mips: Fix BC1ANY[24]F instructions
Date: Wed, 18 Apr 2012 23:00:00 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:11.0) Gecko/20120312 Thunderbird/11.0

Am 26.11.2011 14:37, schrieb Richard Sandiford:
> There's some dodgy application of De Morgan's law in the emulation
> of the MIPS BC1ANY[24]F instructions: they end up branching only
> if all CCs are false, rather than if one CC is.
> 
> Tested on mips64-linux-gnu, where it fixes the GCC MIPS3D tests.
> 
> Signed-off-by: Richard Sandiford <address@hidden>
> ---
>  target-mips/translate.c |    8 ++++----
>  1 files changed, 4 insertions(+), 4 deletions(-)

Ping! Digging out another unreviewed mips patch for 1.1.

Andreas

> diff --git a/target-mips/translate.c b/target-mips/translate.c
> index ba45eb0..2b977b3 100644
> --- a/target-mips/translate.c
> +++ b/target-mips/translate.c
> @@ -6017,7 +6017,7 @@ static void gen_compute_branch1 (CPUState *env, 
> DisasContext *ctx, uint32_t op,
>              TCGv_i32 t1 = tcg_temp_new_i32();
>              tcg_gen_shri_i32(t0, fpu_fcr31, get_fp_bit(cc));
>              tcg_gen_shri_i32(t1, fpu_fcr31, get_fp_bit(cc+1));
> -            tcg_gen_nor_i32(t0, t0, t1);
> +            tcg_gen_nand_i32(t0, t0, t1);
>              tcg_temp_free_i32(t1);
>              tcg_gen_andi_i32(t0, t0, 1);
>              tcg_gen_extu_i32_tl(bcond, t0);
> @@ -6041,11 +6041,11 @@ static void gen_compute_branch1 (CPUState *env, 
> DisasContext *ctx, uint32_t op,
>              TCGv_i32 t1 = tcg_temp_new_i32();
>              tcg_gen_shri_i32(t0, fpu_fcr31, get_fp_bit(cc));
>              tcg_gen_shri_i32(t1, fpu_fcr31, get_fp_bit(cc+1));
> -            tcg_gen_or_i32(t0, t0, t1);
> +            tcg_gen_and_i32(t0, t0, t1);
>              tcg_gen_shri_i32(t1, fpu_fcr31, get_fp_bit(cc+2));
> -            tcg_gen_or_i32(t0, t0, t1);
> +            tcg_gen_and_i32(t0, t0, t1);
>              tcg_gen_shri_i32(t1, fpu_fcr31, get_fp_bit(cc+3));
> -            tcg_gen_nor_i32(t0, t0, t1);
> +            tcg_gen_nand_i32(t0, t0, t1);
>              tcg_temp_free_i32(t1);
>              tcg_gen_andi_i32(t0, t0, 1);
>              tcg_gen_extu_i32_tl(bcond, t0);

-- 
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg



reply via email to

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