diff options
| author | Richard Henderson <richard.henderson@linaro.org> | 2023-09-14 07:39:07 -0700 |
|---|---|---|
| committer | Richard Henderson <richard.henderson@linaro.org> | 2023-09-28 10:07:32 -0700 |
| commit | f47a90dacca8f74210a2675bdde7ab3856872b94 (patch) | |
| tree | 97e5534dd1bd299727ef1bff3e5c6a220a03d998 | |
| parent | 36e9aab3c569d4c9ad780473596e18479838d1aa (diff) | |
| download | qemu-f47a90dacca8f74210a2675bdde7ab3856872b94.zip qemu-f47a90dacca8f74210a2675bdde7ab3856872b94.tar.gz qemu-f47a90dacca8f74210a2675bdde7ab3856872b94.tar.bz2 | |
accel/tcg: Avoid load of icount_decr if unused
With CF_NOIRQ and without !CF_USE_ICOUNT, the load isn't used.
Avoid emitting it.
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
| -rw-r--r-- | accel/tcg/translator.c | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/accel/tcg/translator.c b/accel/tcg/translator.c index 1a6a544..a398301 100644 --- a/accel/tcg/translator.c +++ b/accel/tcg/translator.c @@ -49,12 +49,15 @@ bool translator_io_start(DisasContextBase *db) static TCGOp *gen_tb_start(uint32_t cflags) { - TCGv_i32 count = tcg_temp_new_i32(); + TCGv_i32 count = NULL; TCGOp *icount_start_insn = NULL; - tcg_gen_ld_i32(count, cpu_env, - offsetof(ArchCPU, neg.icount_decr.u32) - - offsetof(ArchCPU, env)); + if ((cflags & CF_USE_ICOUNT) || !(cflags & CF_NOIRQ)) { + count = tcg_temp_new_i32(); + tcg_gen_ld_i32(count, cpu_env, + offsetof(ArchCPU, neg.icount_decr.u32) - + offsetof(ArchCPU, env)); + } if (cflags & CF_USE_ICOUNT) { /* |