From: Richard Henderson <rth@twiddle.net>
Date: Wed, 23 Jan 2013 21:07:10 +0000 (-0800)
Subject: target-i386: no need to flush out cc_op before gen_eob
X-Git-Tag: qemu-xen-4.4.0-rc1~6^2~1055^2~42
X-Git-Url: http://xenbits.xensource.com/gitweb?a=commitdiff_plain;h=ccfcdd09bf91aabe039d2dae0b5ec3a05f083e59;p=qemu-upstream-4.5-testing.git

target-i386: no need to flush out cc_op before gen_eob

This makes code more similar to the other callers of gen_eob, especially
loopz/loopnz/jcxz.

Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Richard Henderson <rth@twiddle.net>
---

diff --git a/target-i386/translate.c b/target-i386/translate.c
index 6204764a5..71104fb92 100644
--- a/target-i386/translate.c
+++ b/target-i386/translate.c
@@ -2303,8 +2303,8 @@ static inline void gen_jcc(DisasContext *s, int b,
 {
     int l1, l2;
 
-    gen_update_cc_op(s);
     if (s->jmp_opt) {
+        gen_update_cc_op(s);
         l1 = gen_new_label();
         gen_jcc1(s, b, l1);
         set_cc_op(s, CC_OP_DYNAMIC);
@@ -2315,11 +2315,9 @@ static inline void gen_jcc(DisasContext *s, int b,
         gen_goto_tb(s, 1, val);
         s->is_jmp = DISAS_TB_JUMP;
     } else {
-
         l1 = gen_new_label();
         l2 = gen_new_label();
         gen_jcc1(s, b, l1);
-        set_cc_op(s, CC_OP_DYNAMIC);
 
         gen_jmp_im(next_eip);
         tcg_gen_br(l2);