RFR: 8365926: RISC-V: Performance regression in renaissance (chi-square) [v2]
Robbin Ehn
rehn at openjdk.org
Wed Sep 3 07:56:44 UTC 2025
On Tue, 2 Sep 2025 12:47:23 GMT, Hamlin Li <mli at openjdk.org> wrote:
>> Maybe we can give it a new name to avoid possible confusion? `jmp_pc` or simply `pc`?
>
>> We only change the instruction at "instruction_address() + 2 * NativeInstruction::instruction_size".
>
> Right!
>
>> Note that jal_pos and jal_pc means a "jump and link instruction", not specifically jal or jalr.
>
> As we're patching either `jal` or `jalr` instruction, so jal is misleading, I agree `jmp_xxx` is a better name.
fixed
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26944#discussion_r2318105646
More information about the hotspot-compiler-dev
mailing list