[llvm] [lld] [RISCV] Support printing immediate of RISCV MCInst in hexadecimal format (PR #74053)

[Jessica Clarke via llvm-commits]

jrtc27 wrote:

> > > gnu objdump for RISC-V is inconsistent. Looking at a dump from 2.41 I see Shift by immediate uses hex. addi/andi/ori/xori/li use decimal lui uses hex. load/store offset use decimal. label offsets on branches use hex vmv.v.i uses decimal
> > 
> > 
> > Thanks for your correction, and do you think we should conform to gnu's behavior? Or just be the same as other architecture in llvm.
> 
> We can conform to other architectures.

AArch64 uses decimal, as a data point, as do the various MIPS targets, PowerPC and x86, at least for a simple load with an offset.

To me this makes the output less user-friendly, and it's quite annoying churn for downstream. I've been perfectly happy with the output as it stands... Not to mention that it now *deviates* from what appears to be the standard practice.

So personally I'm extremely anti this patch.

https://github.com/llvm/llvm-project/pull/74053