qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] [patch] Bug in arm saturating instructions
@ 2005-05-01 10:35 Paul Brook
  0 siblings, 0 replies; only message in thread
From: Paul Brook @ 2005-05-01 10:35 UTC (permalink / raw)
  To: qemu-devel; +Cc: TAKAGO Daisuke

The patch below fixes a but in the arm saturating add/subtract instructions. 
We were writing the result back to the wrong register.

Paul

Index: target-arm/translate.c
===================================================================
RCS file: /cvsroot/qemu/qemu/target-arm/translate.c,v
retrieving revision 1.24
diff -u -p -r1.24 translate.c
--- target-arm/translate.c	27 Apr 2005 20:25:20 -0000	1.24
+++ target-arm/translate.c	1 May 2005 10:28:46 -0000
@@ -1032,7 +1032,7 @@ static void disas_arm_insn(CPUState * en
                 gen_op_subl_T0_T1_saturate();
             else
                 gen_op_addl_T0_T1_saturate();
-            gen_movl_reg_T0(s, rn);
+            gen_movl_reg_T0(s, rd);
             break;
         case 0x8: /* signed multiply */
         case 0xa:

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2005-05-01 10:42 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2005-05-01 10:35 [Qemu-devel] [patch] Bug in arm saturating instructions Paul Brook

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).