Re: [PATCH] RISC-V: Add h extension support

[Kito Cheng <kito.cheng@gmail.com>]

Thanks for the review, committed :)

On Mon, Oct 24, 2022 at 11:05 PM Jeff Law via Gcc-patches
<gcc-patches@gcc.gnu.org> wrote:
>
>
> On 10/24/22 03:55, Kito Cheng wrote:
> > `h` was the prefix of multi-letter extension name, but it become a
> > extension in later RISC-V isa spec.
> >
> > Fortunately we don't have any extension really defined is prefixed
> > with `h`, so we can just change that.
> >
> > gcc/ChangeLog:
> >
> >       * common/config/riscv/riscv-common.cc (riscv_ext_version_table):
> >       Add `h`.
> >       (riscv_supported_std_ext): Ditto.
> >       (multi_letter_subset_rank): Remove `h`.
> >       (riscv_subset_list::parse_std_ext): Handle `h` as single letter
> >       extension.
> >       (riscv_subset_list::parse): Ditto.
> >
> > gcc/testsuite/ChangeLog:
> >
> >       * testsuite/gcc.target/riscv/arch-18.c: New.
> >       * testsuite/gcc.target/riscv/arch-5.c: Remove test for prefixed
> >       with `h`.
> >       * testsuite/gcc.target/riscv/predef-23.c: New.
>
> OK.
>
>
> Jeff
>
>