[PATCH v2] RISC-V: Accept version, supervisor extension and more than one NSE for -march.

2 views
Skip to first unread message

Kito Cheng

unread,
Dec 3, 2018, 10:03:04 AM12/3/18
to binu...@sourceware.org, pat...@groups.riscv.org, Jim Wilson, Palmer Dabbelt
Hi Jim:

I've addressed most issues in you mail exclude newer spec update like
Z* and RV32E with F/D :)

Tested on CentOS 7 (GCC 4.8.5).

Changes since v1:
- Fixed several typos.
- Minor variable naming fixes (ext -> subset, subset to subset_list).
- Remove riscv_next_arch_char.
- Refine riscv_parsing_subset_version, remove last one argument, spec
only allow underscore between each ISA subset components[1].
- Add comment to explain argument and return value for
riscv_parsing_subset_version, riscv_parse_std_ext,
riscv_parse_sv_or_non_std_ext and riscv_parse_subset.
- Fix ChangeLog.
- Fix build error.
- Fix opcodes/riscv-dis.c.
- Two more testcase march-ok-g2_p1.d and march-ok-i2p0m2_a2f2.d

[1] https://github.com/riscv/riscv-isa-manual/commit/87d431236b5a01ff80e3567e602fdeaa797e97e8
0001-RISC-V-Accept-version-supervisor-extension-and-more-.patch

Jim Wilson

unread,
Dec 3, 2018, 5:15:44 PM12/3/18
to Kito Cheng, Binutils, pat...@groups.riscv.org, Palmer Dabbelt
On Mon, Dec 3, 2018 at 7:03 AM Kito Cheng <kito....@gmail.com> wrote:
> I've addressed most issues in you mail exclude newer spec update like
> Z* and RV32E with F/D :)

Thanks. This looks OK to me. I committed it with some minor
ChangeLog entry edits.

Jim
Reply all
Reply to author
Forward
0 new messages