RFR: 8332900: RISC-V: refactor nativeInst_riscv.cpp and macroAssembler_riscv.cpp [v4]

Hamlin Li mli at openjdk.org
Fri May 31 14:40:15 UTC 2024


> Hi,
> Can you help to review the patch?
> Currently, code in nativeInst_riscv.cpp and macroAssembler_riscv.cpp call each other, which is not right for readability and maintainance.
> After refactoring, basically only code in nativeInst_riscv.cpp calls code in macroAssembler_riscv.cpp, but not in reverse direction.
> 
> Thanks!
> 
> * Tests are still running, so far so good.

Hamlin Li has updated the pull request incrementally with one additional commit since the last revision:

  adjust accessibility

-------------

Changes:
  - all: https://git.openjdk.org/jdk/pull/19459/files
  - new: https://git.openjdk.org/jdk/pull/19459/files/fe345dd3..ef11d6a7

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=19459&range=03
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=19459&range=02-03

  Stats: 17 lines in 1 file changed: 6 ins; 11 del; 0 mod
  Patch: https://git.openjdk.org/jdk/pull/19459.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/19459/head:pull/19459

PR: https://git.openjdk.org/jdk/pull/19459


More information about the hotspot-dev mailing list