riscv_dis_shift_xlen_2

Disassembler: Check shift amount against XLEN (Idea 2)

• Status: Pending (RFC)
• Branch: riscv-dis-shift-xlen-2
• Tracking PR: #25 (view Pull Request and Diff)
• Mailing List:
  • Not yet

Conflicts With

• Pending (RFC): Disassembler: Check shift amount against XLEN (Idea 1)
  This is mutually exclusive idea of fixing shift amount issue.
• Disassembler: Fix printf types on riscv-dis.c
  It also touches the same location. Merging is easy (this patch takes precedence).

Unlike idea 1, conflicts are minimal.

Issue Solved

In the disassembler, there's no validation whether shift amount is valid on specific architecture.

Idea 1 patchset fixes that by adding xlen argument to match_func (in opcode entries).

Idea 2 patchset (this patchset) fixes that by printing invalid0x[SHAMT] instead of actual shift width (SHAMT in hexadecimal).