This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
[PATCH v5 0/3] RISC-V: Support more rigorous check for CSR
- From: Nelson Chu <nelson dot chu at sifive dot com>
- To: binutils at sourceware dot org, jimw at sifive dot com, andrew dot burgess at embecosm dot com
- Date: Wed, 12 Feb 2020 02:18:48 -0800
- Subject: [PATCH v5 0/3] RISC-V: Support more rigorous check for CSR
Hi binutils,
I get some feedbacks from Andrew Burgess of gdb side. Therefroe I update the
series of patches. There are some differences between v5 version and v3 version
that Jim reviewed, shown as follows:
1. Implement the draft of CSR spec is inadvisable, so I remove the first patch
of v3 version.
2. Reorder the patches. Move the patch "RISC-V: Disable the CSR checking by
default" to the second patch.
The detailed of patches:
* [PATCH v5 1/3] RISC-V: Support the ISA-dependent CSR checking.
a. Add missing comments for each function.
b. Add -march=rv32if option to the csr-dw-regnum test case.
c. Report warning messages rather than error for the CSR cheking.
d. Keep the consistent usage in gas/config/tc-riscv.c (from Andrew's suggestion)
d-1. Change the 'typedef struct {...} riscv_csr_extra' to
'struct riscv_csr_extra {...}'.
d-2. Use 'if (hash_error != NULL)' in init_opcode_hash and riscv_init_csr_hash, since
hash_error is not a boolean.
* [PATCH v5 2/3] RISC-V: Disable the CSR checking by default.
* [PATCH v5 3/3] RISC-V: Support the read-only CSR checking.
Thanks
Nelson