[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 15/17] target/riscv: convert to DisasJumpType
From: |
Richard Henderson |
Subject: |
Re: [Qemu-devel] [PATCH v2 15/17] target/riscv: convert to DisasJumpType |
Date: |
Thu, 12 Apr 2018 18:24:50 -1000 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 |
> Cc: Michael Clark <address@hidden>
> Cc: Palmer Dabbelt <address@hidden>
> Cc: Sagar Karandikar <address@hidden>
> Cc: Bastian Koppelmann <address@hidden>
>
> Signed-off-by: Emilio G. Cota <address@hidden>
> ---
> target/riscv/translate.c | 72
> +++++++++++++++++++-----------------------------
> 1 file changed, 28 insertions(+), 44 deletions(-)
Reviewed-by: Richard Henderson <address@hidden>
> @@ -1345,12 +1339,12 @@ static void gen_system(CPURISCVState *env,
> DisasContext *ctx, uint32_t opc,
> /* always generates U-level ECALL, fixed in do_interrupt handler
> */
> generate_exception(ctx, RISCV_EXCP_U_ECALL);
> tcg_gen_exit_tb(0); /* no chaining */
> - ctx->bstate = BS_BRANCH;
> + ctx->is_jmp = DISAS_NORETURN;
> break;
> case 0x1: /* EBREAK */
> generate_exception(ctx, RISCV_EXCP_BREAKPOINT);
> tcg_gen_exit_tb(0); /* no chaining */
> - ctx->bstate = BS_BRANCH;
> + ctx->is_jmp = DISAS_NORETURN;
> break;
Not for Emilio, but for the RISCV guys as a follow-up, exit_tb after
generate_exception is dead code -- we have already exited via longjmp. There
are more than these two instances.
r~
- Re: [Qemu-devel] [PATCH v2 06/17] target/mips: convert to DisasJumpType, (continued)
- [Qemu-devel] [PATCH v2 08/17] target/mips: use *ctx for DisasContext, Emilio G. Cota, 2018/04/06
- [Qemu-devel] [PATCH v2 11/17] target/s390x: convert to DisasContextBase, Emilio G. Cota, 2018/04/06
- [Qemu-devel] [PATCH v2 01/17] translator: merge max_insns into DisasContextBase, Emilio G. Cota, 2018/04/06
- [Qemu-devel] [PATCH v2 09/17] target/mips: convert to TranslatorOps, Emilio G. Cota, 2018/04/06
- [Qemu-devel] [PATCH v2 15/17] target/riscv: convert to DisasJumpType, Emilio G. Cota, 2018/04/06
- [Qemu-devel] [PATCH v2 17/17] target/riscv: convert to TranslatorOps, Emilio G. Cota, 2018/04/06
- Re: [Qemu-devel] [PATCH v2 17/17] target/riscv: convert to TranslatorOps, Richard Henderson, 2018/04/13
[Qemu-devel] [PATCH v2 12/17] target/s390x: convert to TranslatorOps, Emilio G. Cota, 2018/04/06
[Qemu-devel] [PATCH v2 16/17] target/riscv: convert to DisasContextBase, Emilio G. Cota, 2018/04/06