Home Original page

[PATCH 1/1] doc/riscv: Add description of disassembler options

Jan Beulich jbeulich@suse.com
Wed Mar 26 13:02:31 GMT 2025 
On 26.03.2025 13:29, Marek Pikula wrote:
> On 26.03.2025 12:50, Jan Beulich wrote:
>> On 26.03.2025 03:33, Nelson Chu wrote:
>>> This is okay from my side, thanks.  Btw, the privileged spec 1.9.1 support
>>> was removed since binutils 2.43, so the option of objdump should also
>>> remove it.  The linker only recognizes it and then reports a warning that
>>> it may conflict with other spec versions.
>> It's imo okay to go in as-is. I'd like to note though that it looks as if
>> originally the architecture specific sections were sorted alphabetically
>> (and hence the addition might better go between PPC and VAX). That sorting
>> was partly broken by ix86 having got renamed to x86 (but left in place),
>> by aarch64 having got inserted after Arm, and by MIPS having ended up
>> between PPC and VAX.
>>
> Indeed, I noticed the semi-alphabetic order but in the end I didn't 
> figure out the best place to insert it, so I just placed it at the end. 
> Shall I push v2 of this patch, or will you move it to the (semi) correct 
> spot before merging?

Well, if I was the one to commit this I probably would. It being a RISC-V
patch, I kind of expect Nelson to be the one to ultimately put this in, though.
Nelson - unless you tell me otherwise, of course.

Jan
More information about the Binutils mailing list