Commit 8e9ae00c21 for qemu.org
commit 8e9ae00c217f1c66f13cf8eec584be58b84c501c
Author: wangyang <wangyang25@otcaix.iscas.ac.cn>
Date: Fri Sep 4 09:26:08 2026 +0800
target/riscv: write back fsgnjn.h and fsgnjx.h results
Under Zfinx/Zhinx, trans_fsgnjn_h() and trans_fsgnjx_h() compute their
results into a temporary but return without calling gen_set_fpr_hs().
The instructions consequently retire without updating the corresponding
integer register. Add the missing writeback calls, matching
trans_fsgnj_h().
Resolves: https://gitlab.com/qemu-project/qemu/-/work_items/4407
Signed-off-by: wangyang <wangyang25@otcaix.iscas.ac.cn>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-ID: <ebb7f8a1c374783dd585ee838e1d30f4d2bfbccc.20260904092608.git.wangyang25.otcaix.iscas.ac.cn>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
diff --git a/target/riscv/tcg/insn_trans/trans_rvzfh.c.inc b/target/riscv/tcg/insn_trans/trans_rvzfh.c.inc
index b17ef9541c..4af717ce54 100644
--- a/target/riscv/tcg/insn_trans/trans_rvzfh.c.inc
+++ b/target/riscv/tcg/insn_trans/trans_rvzfh.c.inc
@@ -319,6 +319,7 @@ static bool trans_fsgnjn_h(DisasContext *ctx, arg_fsgnjn_h *a)
if (ctx->cfg_ptr->ext_zfinx) {
tcg_gen_ext16s_i64(dest, dest);
}
+ gen_set_fpr_hs(ctx, a->rd, dest);
mark_fs_dirty(ctx);
return true;
}
@@ -363,6 +364,7 @@ static bool trans_fsgnjx_h(DisasContext *ctx, arg_fsgnjx_h *a)
if (ctx->cfg_ptr->ext_zfinx) {
tcg_gen_ext16s_i64(dest, dest);
}
+ gen_set_fpr_hs(ctx, a->rd, dest);
mark_fs_dirty(ctx);
return true;
}