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

Hamlin Li mli at openjdk.org
Mon Jun 3 11:42:28 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:

  move

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

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

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

  Stats: 443 lines in 1 file changed: 220 ins; 223 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