[PATCH 1/2] RISC-V: Add 'H' to canonical extension ordering

Nelson Chu nelson.chu@sifive.com
Wed Jun 22 10:39:30 GMT 2022


Sorry for being late, I have been busy recently.  I didn't notice that
the isa spec had been updated, anyway, the h is a single extension
after v, rather than the multi-letter extension for now.  So LGTM,
committed.

Besides, please give me some time to review other patches, I've been
busy with other things for a while...

Thanks
Nelson

On Sun, Jun 5, 2022 at 12:43 PM Tsukasa OI <research_trasio@irq.a4lg.com> wrote:
>
> This commit adds 'H' to canonical extension ordering based on current
> consensus (not officially ratified as a new ISA specification manual
> but discussion for software compatibility is made).
>
> bfd/ChangeLog
>
>         * elfxx-riscv.c (riscv_ext_canonical_order): Add 'H' for
>         canonical extension ordering based on current consensus.
> ---
>  bfd/elfxx-riscv.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/bfd/elfxx-riscv.c b/bfd/elfxx-riscv.c
> index 92ad03feea0..5c2c616a760 100644
> --- a/bfd/elfxx-riscv.c
> +++ b/bfd/elfxx-riscv.c
> @@ -1345,7 +1345,7 @@ riscv_recognized_prefixed_ext (const char *ext)
>  }
>
>  /* Canonical order for single letter extensions.  */
> -static const char riscv_ext_canonical_order[] = "eigmafdqlcbkjtpvn";
> +static const char riscv_ext_canonical_order[] = "eigmafdqlcbkjtpvnh";
>
>  /* Array is used to compare the orders of standard extensions quickly.  */
>  static int riscv_ext_order[26] = {0};
> --
> 2.34.1
>


More information about the Binutils mailing list