@samitouri / QOSamiQemu / commits / 4a5a8434bb

disas/riscv: Drop codec from rv_decode

This is mostly write-only, only used in one place; other updates are ignored. Reviewed-by: Alistair Francis <alistair.francis@wdc.com> Signed-off-by: Richard Henderson <richard.henderson@linaro.org> Message-ID: <20260812223142.349142-13-richard.henderson@linaro.org> Signed-off-by: Alistair Francis <alistair.francis@wdc.com>

Richard Henderson committed Aug 12, 2026 at 15:31 UTC 4a5a8434bb8d71cbed6e71a7355e59eb514a97fb
2 files changed +3 -5
disas/riscv.c
+3 -4
@@ -4603,8 +4603,9 @@ static void decode_inst_operands(rv_decode *dec, rv_isa isa,
4603 {
4604 rv_inst inst = dec->inst;
4605
4606 - dec->codec = op->codec;
4607 - switch (dec->codec) {
4606 + switch (op->codec) {
4607 + case rv_codec_illegal:
4608 + break;
4609 case rv_codec_none:
4610 dec->rd = dec->rs1 = dec->rs2 = rv_ireg_zero;
4611 dec->imm = 0;
@@ -5369,7 +5370,6 @@ static const rv_opcode_data *decode_inst_lift_pseudo(rv_decode *dec,
5370 if (check_constraints(dec, comp_data->constraints)) {
5371 dec->op = comp_data->op;
5372 op = &dec->opcode_data[dec->op];
5372 - dec->codec = op->codec;
5373 break;
5374 }
5375 comp_data++;
@@ -5407,7 +5407,6 @@ static const rv_opcode_data *decode_inst_decompress(rv_decode *dec, rv_isa isa,
5407 dec->op = decomp_op;
5408 }
5409 op = &dec->opcode_data[decomp_op];
5410 - dec->codec = op->codec;
5410 }
5411 return op;
5412 }
disas/riscv.h
-1
@@ -197,7 +197,6 @@ typedef struct {
197 int32_t imm;
198 int32_t imm1;
199 uint16_t op;
200 - uint8_t codec;
200 uint8_t rd;
201 uint8_t rs1;
202 uint8_t rs2;