[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] target-sparc: Revert setting cpu_dst to gen_des
From: |
Aurelien Jarno |
Subject: |
Re: [Qemu-devel] [PATCH] target-sparc: Revert setting cpu_dst to gen_dest_gpr |
Date: |
Mon, 29 Oct 2012 14:54:55 +0100 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Mon, Oct 29, 2012 at 03:50:20PM +1100, Richard Henderson wrote:
> There is some read-after-write error within the OP=2 insns which
> prevents setting cpu_dst to the real output register. Until this
> is found and fixed, always write to a temporary first.
>
> Cc: Blue Swirl <address@hidden>
> Cc: Aurelien Jarno <address@hidden>
> Signed-off-by: Richard Henderson <address@hidden>
> ---
> target-sparc/translate.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/target-sparc/translate.c b/target-sparc/translate.c
> index 4321393..9e46f14 100644
> --- a/target-sparc/translate.c
> +++ b/target-sparc/translate.c
> @@ -2633,7 +2633,7 @@ static void disas_sparc_insn(DisasContext * dc,
> unsigned int insn)
> case 2: /* FPU & Logical Operations */
> {
> unsigned int xop = GET_FIELD(insn, 7, 12);
> - TCGv cpu_dst = gen_dest_gpr(dc, rd);
> + TCGv cpu_dst = get_temp_tl(dc);
> TCGv cpu_tmp0;
>
> if (xop == 0x3a) { /* generate trap */
Tested-by: Aurelien Jarno <address@hidden>
--
Aurelien Jarno GPG: 1024D/F1BCDB73
address@hidden http://www.aurel32.net