diff options
author | Song Gao <gaosong@loongson.cn> | 2024-03-20 09:39:55 +0800 |
---|---|---|
committer | Song Gao <gaosong@loongson.cn> | 2024-03-20 10:20:08 +0800 |
commit | 77642f92c0b71a105aba2a4d03bc62328eae703b (patch) | |
tree | d861520acf8180a02f2e294f8cd413346e0ea2dd /target | |
parent | 9c70db9a433886c4647bfe01bf256e426c698561 (diff) | |
download | qemu-77642f92c0b71a105aba2a4d03bc62328eae703b.zip qemu-77642f92c0b71a105aba2a4d03bc62328eae703b.tar.gz qemu-77642f92c0b71a105aba2a4d03bc62328eae703b.tar.bz2 |
target/loongarch: Fix qemu-loongarch64 hang when executing 'll.d $t0, $t0, 0'
On gen_ll, if a->imm is zero, make_address_x return src1,
but the load to destination may clobber src1. We use a new
destination to fix this problem.
Fixes: c5af6628f4be (target/loongarch: Extract make_address_i() helper)
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Suggested-by: Richard Henderson <richard.henderson@linaro.org>
Signed-off-by: Song Gao <gaosong@loongson.cn>
Message-Id: <20240320013955.1561311-1-gaosong@loongson.cn>
Diffstat (limited to 'target')
-rw-r--r-- | target/loongarch/tcg/insn_trans/trans_atomic.c.inc | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/target/loongarch/tcg/insn_trans/trans_atomic.c.inc b/target/loongarch/tcg/insn_trans/trans_atomic.c.inc index 80c2e28..974bc2a 100644 --- a/target/loongarch/tcg/insn_trans/trans_atomic.c.inc +++ b/target/loongarch/tcg/insn_trans/trans_atomic.c.inc @@ -5,14 +5,14 @@ static bool gen_ll(DisasContext *ctx, arg_rr_i *a, MemOp mop) { - TCGv dest = gpr_dst(ctx, a->rd, EXT_NONE); + TCGv t1 = tcg_temp_new(); TCGv src1 = gpr_src(ctx, a->rj, EXT_NONE); TCGv t0 = make_address_i(ctx, src1, a->imm); - tcg_gen_qemu_ld_i64(dest, t0, ctx->mem_idx, mop); + tcg_gen_qemu_ld_i64(t1, t0, ctx->mem_idx, mop); tcg_gen_st_tl(t0, tcg_env, offsetof(CPULoongArchState, lladdr)); - tcg_gen_st_tl(dest, tcg_env, offsetof(CPULoongArchState, llval)); - gen_set_gpr(a->rd, dest, EXT_NONE); + tcg_gen_st_tl(t1, tcg_env, offsetof(CPULoongArchState, llval)); + gen_set_gpr(a->rd, t1, EXT_NONE); return true; } |