RFR: 8316694: Implement relocation of nmethod within CodeCache [v18]

Wed May 28 20:45:57 UTC 2025

On Wed, 28 May 2025 20:35:12 GMT, Evgeny Astigeevich <eastigeevich at openjdk.org> wrote:

> The patched call instruction will be incorrect.

That's not entirely correct. The null trampoline check is needed because on debug builds branches of distance >2M will fall into the `if (!Assembler::reachable_from_branch_at(addr(), x))` block but Graal would not have generated a trampoline for that call because it is still <128M. It is still safe to use that distance but it is just different than what HotSpot expects

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23573#discussion_r2112716099