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