This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[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


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]