Re: [PATCH v2 36/52] disas/riscv: Skip post-processing of illegal insns

Alistair <[email protected]>
Newsgroups gmane.comp.emulators.qemu
Message-ID <[email protected]>
On Wed, 2026-08-12 at 15:31 -0700, Richard Henderson wrote:
> Signed-off-by: Richard Henderson <[email protected]>

Reviewed-by: Alistair Francis <[email protected]>

Alistair

> ---
>  disas/riscv.c | 8 +++++---
>  1 file changed, 5 insertions(+), 3 deletions(-)
> 
> diff --git a/disas/riscv.c b/disas/riscv.c
> index bc86cb866a..75c2fa11e6 100644
> --- a/disas/riscv.c
> +++ b/disas/riscv.c
> @@ -5321,12 +5321,14 @@ static GString *disasm_inst(rv_isa isa,
> uint64_t pc, rv_inst inst,
>              }
>          }
>      }
> -    if (!op) {
> +
> +    if (op) {
> +        decode_inst_operands(&dec, isa, op);
> +        op = decode_inst_lift_pseudo(&dec, op);
> +    } else {
>          op = &rvi_opcode_data[rv_op_illegal];
>      }
>  
> -    decode_inst_operands(&dec, isa, op);
> -    op = decode_inst_lift_pseudo(&dec, op);
>      return format_inst(24, &dec, op);
>  }
>  
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.