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

Jan Beulich jbeulich@suse.com
Mon Jan 6 08:22:32 GMT 2025


On 03.01.2025 07:22, 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.

As to a testcase: Yes. As to where to put it: Almost all disassembler
tests really live in the gas testsuite, afaict.

Jan



More information about the Binutils mailing list