[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 2/5] target-m68k: fix gen_flush_flags()
From: |
Richard Henderson |
Subject: |
Re: [Qemu-devel] [PATCH v2 2/5] target-m68k: fix gen_flush_flags() |
Date: |
Fri, 13 Jan 2017 09:56:41 -0800 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.6.0 |
On 01/13/2017 04:52 AM, Laurent Vivier wrote:
> gen_flush_flags() is setting unconditionally cc_op_synced to 1
> and s->cc_op to CC_OP_FLAGS, whereas env->cc_op can be set
> to something else by a previous tcg fragment.
>
> We fix that by not setting cc_op_synced to 1
> (except for gen_helper_flush_flags() that updates env->cc_op)
>
> FIX: https://github.com/vivier/qemu-m68k/issues/19
>
> Signed-off-by: Laurent Vivier <address@hidden>
> ---
> target/m68k/translate.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
Reviewed-by: Richard Henderson <address@hidden>
r~
- [Qemu-devel] [PATCH v2 0/5] Fixes for target/m68k, Laurent Vivier, 2017/01/13
- [Qemu-devel] [PATCH v2 1/5] target-m68k: fix bit operation with immediate value, Laurent Vivier, 2017/01/13
- [Qemu-devel] [PATCH v2 2/5] target-m68k: fix gen_flush_flags(), Laurent Vivier, 2017/01/13
- Re: [Qemu-devel] [PATCH v2 2/5] target-m68k: fix gen_flush_flags(),
Richard Henderson <=
- [Qemu-devel] [PATCH v2 4/5] target-m68k: CAS doesn't need aligned access, Laurent Vivier, 2017/01/13
- [Qemu-devel] [PATCH v2 5/5] target-m68k: increment/decrement with SP, Laurent Vivier, 2017/01/13
- [Qemu-devel] [PATCH v2 3/5] target-m68k: manage pre-dec et post-inc in CAS, Laurent Vivier, 2017/01/13