mbox series

[v2,0/2] RISC-V: Introduce new architecture extension test macros

Message ID 20210107095544.20845-1-kito.cheng@sifive.com
Headers show
Series RISC-V: Introduce new architecture extension test macros | expand

Message

Kito Cheng Jan. 7, 2021, 9:55 a.m. UTC 
This patch set introduce new set of architecture extension test macros
which is accept on riscv-c-api-doc[1] recently.

The motivation of this scheme is have an unify naming scheme for
extension macro and add the capability to checking version.

V2 Changes:
- Fix MacOS build issue.
- Create new header file: riscv-subset.h

[1] https://github.com/riscv/riscv-c-api-doc/blob/master/riscv-c-api.md#architecture-extension-test-macro

Comments

Jim Wilson Jan. 7, 2021, 7:48 p.m. UTC | #1 
On Thu, Jan 7, 2021 at 1:55 AM Kito Cheng <kito.cheng@sifive.com> wrote:

> This patch set introduce new set of architecture extension test macros
> which is accept on riscv-c-api-doc[1] recently.
>
> The motivation of this scheme is have an unify naming scheme for
> extension macro and add the capability to checking version.
>
> V2 Changes:
> - Fix MacOS build issue.
> - Create new header file: riscv-subset.h
>

This patch series looks good to me.

Jim
Kito Cheng Jan. 8, 2021, 3:34 a.m. UTC | #2 
Committed, thanks :)

On Fri, Jan 8, 2021 at 3:49 AM Jim Wilson <jimw@sifive.com> wrote:
>
> On Thu, Jan 7, 2021 at 1:55 AM Kito Cheng <kito.cheng@sifive.com> wrote:
>
> > This patch set introduce new set of architecture extension test macros
> > which is accept on riscv-c-api-doc[1] recently.
> >
> > The motivation of this scheme is have an unify naming scheme for
> > extension macro and add the capability to checking version.
> >
> > V2 Changes:
> > - Fix MacOS build issue.
> > - Create new header file: riscv-subset.h
> >
>
> This patch series looks good to me.
>
> Jim