RISC-V encoding bug

[petabyt]

Some if statements in RISCVAsmBackend.cpp were intended to have multiple statements, but are missing braces. This causes the beq instruction to be encoded as -1 (FF FF FF FF)

keystone/llvm/lib/Target/RISCV/MCTargetDesc/RISCVAsmBackend.cpp

Line 201 in fb92f32

if (!isInt<21>(Value))

[RossComputerGuy]

That code is from LLVM 3.9.0, LLVM's current release is 21.1.0. This project really should update LLVM.

[petabyt]

I don't see any problem with the old LLVM, it's just that the person who backported RISC-V support made a dumb mistake lol

https://github.com/keystone-engine/keystone/pull/549/files

[RossComputerGuy]

Says:

• adds the support for RISC-V architecture (refers to Any plans for RISCV ? #491) from LLVM 9.0.1 version - the first official release of RISC-V to LLVM

Surely, the RISC-V support from LLVM 9 isn't as stable as something like 21. That's about 6 years of improvements.