RFR: 8337788: RISC-V: Cleanup code in MacroAssembler::reserved_stack_check
Gui Cao
gcao at openjdk.org
Sun Aug 4 02:48:13 UTC 2024
Hi,
In the MacroAssembler::reserved_stack_check() function:
RuntimeAddress target(StubRoutines::throw_delayed_StackOverflowError_entry());
relocate(target.rspec(), [&] {
int32_t offset;
movptr(t0, target.target(), offset);
jr(t0, offset);
});
can be simplified to:
la(t0, RuntimeAddress(StubRoutines::throw_delayed_StackOverflowError_entry()));
jr(t0);
In addition, the code formatting has been modified to remove the extra spaces before the code.
Please take a look and have some reviews. Thanks a lot.
### Testing
- [x] Run tier1-3 tests on SOPHON SG2042 (fastdebug)
-------------
Commit messages:
- JDK-8337788: RISC-V: Cleanup code in MacroAssembler::reserved_stack_check
Changes: https://git.openjdk.org/jdk/pull/20458/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=20458&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8337788
Stats: 19 lines in 1 file changed: 0 ins; 4 del; 15 mod
Patch: https://git.openjdk.org/jdk/pull/20458.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/20458/head:pull/20458
PR: https://git.openjdk.org/jdk/pull/20458
More information about the hotspot-dev
mailing list