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