@samitouri / QOSamiQemu / commits / b1d5cbf56a

disas/riscv: Handle aliases of csrrw during decode

Reviewed-by: Alistair Francis <alistair.francis@wdc.com> Signed-off-by: Richard Henderson <richard.henderson@linaro.org> Message-ID: <20260812223142.349142-23-richard.henderson@linaro.org> Signed-off-by: Alistair Francis <alistair.francis@wdc.com>

Richard Henderson committed Aug 12, 2026 at 15:31 UTC b1d5cbf56abc5604c6848350926271e7f865f988
1 file changed +9 -13
disas/riscv.c
+9 -13
@@ -1081,9 +1081,6 @@ static const rvc_constraint rvcc_frrm[] = { rvc_rs1_eq_x0, rvc_csr_eq_0x002,
1081 rvc_end };
1082 static const rvc_constraint rvcc_frflags[] = { rvc_rs1_eq_x0, rvc_csr_eq_0x001,
1083 rvc_end };
1084 -static const rvc_constraint rvcc_fscsr[] = { rvc_csr_eq_0x003, rvc_end };
1085 -static const rvc_constraint rvcc_fsrm[] = { rvc_csr_eq_0x002, rvc_end };
1086 -static const rvc_constraint rvcc_fsflags[] = { rvc_csr_eq_0x001, rvc_end };
1084 static const rvc_constraint rvcc_fsrmi[] = { rvc_csr_eq_0x002, rvc_end };
1085 static const rvc_constraint rvcc_fsflagsi[] = { rvc_csr_eq_0x001, rvc_end };
1086
@@ -1166,14 +1163,6 @@ static const rv_comp_data rvcp_subw[] = {
1163 { rv_op_illegal, NULL }
1164 };
1165
1169 -static const rv_comp_data rvcp_csrrw[] = {
1170 - { rv_op_fscsr, rvcc_fscsr },
1171 - { rv_op_fsrm, rvcc_fsrm },
1172 - { rv_op_fsflags, rvcc_fsflags },
1173 - { rv_op_illegal, NULL }
1174 -};
1175 -
1176 -
1166 static const rv_comp_data rvcp_csrrs[] = {
1167 { rv_op_rdcycle, rvcc_rdcycle },
1168 { rv_op_rdtime, rvcc_rdtime },
@@ -1761,7 +1750,7 @@ static const rv_opcode_data rvi_opcode_data[] = {
1750 { "sfence.vm", rv_codec_r, rv_fmt_rs1 },
1751 { "sfence.vma", rv_codec_r, rv_fmt_rs1_rs2 },
1752 { "wfi", rv_codec_none, rv_fmt_none },
1764 - { "csrrw", rv_codec_i_csr, rv_fmt_rd_csr_rs1, rvcp_csrrw },
1753 + { "csrrw", rv_codec_i_csr, rv_fmt_rd_csr_rs1 },
1754 { "csrrs", rv_codec_i_csr, rv_fmt_rd_csr_rs1, rvcp_csrrs },
1755 { "csrrc", rv_codec_i_csr, rv_fmt_rd_csr_rs1 },
1756 { "csrrwi", rv_codec_i_csr, rv_fmt_rd_csr_zimm, rvcp_csrrwi },
@@ -4443,7 +4432,14 @@ static void decode_inst_opcode(rv_decode *dec, rv_isa isa)
4432 break;
4433 }
4434 break;
4446 - case 1: op = rv_op_csrrw; break;
4435 + case 1:
4436 + switch (operand_csr12(inst)) {
4437 + case 1: op = rv_op_fsflags; break;
4438 + case 2: op = rv_op_fsrm; break;
4439 + case 3: op = rv_op_fscsr; break;
4440 + default: op = rv_op_csrrw; break;
4441 + }
4442 + break;
4443 case 2: op = rv_op_csrrs; break;
4444 case 3: op = rv_op_csrrc; break;
4445 case 4: