Integrated: 8332900: RISC-V: refactor nativeInst_riscv.cpp and macroAssembler_riscv.cpp

Hamlin Li mli at openjdk.org
Tue Jun 4 07:08:07 UTC 2024


On Wed, 29 May 2024 15:47:05 GMT, Hamlin Li <mli at openjdk.org> wrote:

> 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.

This pull request has now been integrated.

Changeset: 454660d3
Author:    Hamlin Li <mli at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/454660d361e39f362ff0e10a5c2389af910cca23
Stats:     775 lines in 6 files changed: 354 ins; 341 del; 80 mod

8332900: RISC-V: refactor nativeInst_riscv.cpp and macroAssembler_riscv.cpp

Reviewed-by: fyang, luhenry

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

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


More information about the hotspot-dev mailing list