[PATCH] D151309: [RISCV][NFC] Simplify decoding code of disassembler

[Craig Topper via Phabricator via llvm-commits]

craig.topper added a comment.

In D151309#4370646 <https://reviews.llvm.org/D151309#4370646>, @jrtc27 wrote:

> Downstream we have a separate encoding mode that replaces some standard instructions with alternative ones; even if the order doesn't matter upstream, it would be preferable for us if the standard table was the last one not the first one.

Well my thought doesn't work anyway because Zcmp and Zcmt are in a separate decoder namespace but the instructions they replace from C aren't in a namespace. So they have to be checked first. Same with Zfinx/Zdinx vs F/D.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D151309/new/

https://reviews.llvm.org/D151309