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

[Nelson Chu]

Hi Jan

Yeah thanks, I will, but this seems to need a v2 for some minor changes,
so...

Hi Marek,

Do you mind sending a v2 patch with these two minor changes,
1. The semi-alphabetic order issue
2. Remove 1.9.1 in the document.  Btw, it is also good to remove 1.9.1 in
the dis-asembler's help description, that probably just forgot to remove at
that time.  But just keep it in the cpu-riscv.c since the linker still need
to recognize it.

Thanks
Nelson

On Wed, Mar 26, 2025 at 9:02 PM Jan Beulich <jbeulich@suse.com> wrote:

> 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
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://sourceware.org/pipermail/binutils/attachments/20250327/2d0bf62a/attachment-0001.htm>