Re: [PATCH 0/7] RISC-V: Support version controling for ISA standard extensions and CSR

Kito Cheng
2020-04-30T16:10:39Z

>
> * -misa-spec = [2p0|2p1|2p2|20190608|20191213]
> You can simply choose the ISA spec by this option, and then assembler will set
> the version for the standard extensions if you don’t set in the ELF arch
> attributes or -march option.

Could you change this into 2.0|2.1|... instead of 2p0|2p1|... ?
Unlike arch string has specified by spec, it must use `p` instead of `.`.
so I think there is no necessary to use `p` here.

> * --with-isa-spec = [2p0|2p1|2p2|20190608|20191213]
> The syntax is same as -misa-spec option. Assembler will check this if
> -misa-spec option isn’t set.

Same comment as -misa-spec.

> * -mpriv-spec=[1p9|1p9p1|1p10|1p11]
> This is used to set the privileged spec’s version, and we can decide whether
> the CSR is valid or not.

Change to 1.9 instead of 1p9.

>
> === New Default Configure Option ===
>
> * --with-priv-spec = [1p9|1p9p1|1p10|1p11]

Same comment as -mpriv-spec=