[PATCH v2] RISC-V: Fix disassembly of partial instructions

Charlie Jenkins charlie@rivosinc.com
Fri Jan 3 19:23:09 GMT 2025


On Fri, Jan 03, 2025 at 06:22:34AM +0000, Maciej W. Rozycki wrote:
> On Thu, 2 Jan 2025, Charlie Jenkins wrote:
> 
> > > That or "display" instead of "disassembly", if you ask me.
> > 
> > Sounds good, thank you for letting me know. Is there anything beyond
> > this wording that is of concern in this patch? Unless there is anything
> > else that needs to be changed, I can send a new version with the same
> > diff (containing Nelson's comments) but change "disassembly" to
> > "display" in the title and message.
> 
>  I think relevant test cases need to be added to the testsuite, I guess 
> under binutils/testsuite/binutils-all/riscv/ as this covers `objdump' 
> rather than any of the other tools.  This is to guarantee things won't 
> regress again by accident.

Yes that is a good point, I will add a test for this.

- Charlie

> 
>   Maciej


More information about the Binutils mailing list