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