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