RFR: 8361449: RISC-V: Code cleanup for native call [v2]

Dingli Zhang dzhang at openjdk.org
Mon Jul 14 04:27:46 UTC 2025


On Sat, 12 Jul 2025 08:41:52 GMT, Feilong Jiang <fjiang at openjdk.org> wrote:

>> src/hotspot/cpu/riscv/relocInfo_riscv.cpp line 76:
>> 
>>> 74: address Relocation::pd_call_destination(address orig_addr) {
>>> 75:   assert(is_call(), "should be a call here");
>>> 76:   if (orig_addr == nullptr) {
>> 
>> IIUC, it is synchronized with [JDK-8321509](https://bugs.openjdk.org/browse/JDK-8321509)? Should we add `USE_TRAMPOLINE_STUB_FIX_OWNER` and the other stuff?
>
> [JDK-8343430](https://bugs.openjdk.org/browse/JDK-8343430) already removed the trampoline call for RISCV, so we are good, right?

Correct,  it is partially synchronized with [JDK-8321509](https://bugs.openjdk.org/browse/JDK-8321509), mainly 4 and 6 [here](https://github.com/openjdk/jdk/pull/19796#issuecomment-2188094970). 
We do not really need `USE_TRAMPOLINE_STUB_FIX_OWNER` after [JDK-8343430](https://bugs.openjdk.org/browse/JDK-8343430).

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26150#discussion_r2203821966


More information about the hotspot-compiler-dev mailing list