RFR: 8332689: RISC-V: Use load instead of trampolines [v16]

[Robbin Ehn]

On Wed, 26 Jun 2024 08:22:40 GMT, Fei Yang <fyang at openjdk.org> wrote:

>> As there was no users of jump_link, before this merge, except trampoline call, it always emit JAL as that is the only encoding we need. Hence we only have one place where we use JAL for calls, trampoline call.
>> 
>> If we think we should use JAL for very short calls, manully adding it to one place is not the right approach IMHO.
>> Instead we should have rt_call emit JAL for runtime addresses, i.e. fixed address in libjvm.so or fixed address in code cache if reachable.
>> 
>> For exampel, I think this site, in gen_continuation_enter():
>> `__ rt_call(CAST_FROM_FN_PTR(address, StubRoutines::cont_thaw()));`
>> Could be benficial to use JAL in.
>
> Sounds reasonable. I guess this change won't affect performance much as it's call to the slow path stub. I will do JMH for this case to see.

Thanks!

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19453#discussion_r1654413923