Changeset 60022 in vbox

Timestamp:

Mar 14, 2016 5:15:51 PM (9 years ago)

Author:

vboxsync

Message:

recompiler: shifting of negative values is undefined (thanks PVS)

File:

• trunk/src/recompiler/target-i386/translate.c (modified) (2 diffs)

trunk/src/recompiler/target-i386/translate.c

@@ -2706 +2706 @@
             gen_op_mov_reg_A0(1, R_ESP);
         else
-            gen_stack_update(s, (-2) << s->dflag);
+            gen_stack_update(s, -(2 << s->dflag));
     }
 }
@@ -2759 +2759 @@
     int i;
     gen_op_movl_A0_reg(R_ESP);
-    gen_op_addl_A0_im(-16 <<  s->dflag);
+    gen_op_addl_A0_im(-(16 <<  s->dflag));
     if (!s->ss32)
         gen_op_andl_A0_ffff();