RFR: 8287394: AArch64: Remove cbuf parameter from far_call/far_jump/trampoline_call

Evgeny Astigeevich eastigeevich at openjdk.org
Thu Sep 15 11:43:47 UTC 2022


On Mon, 12 Sep 2022 18:50:23 GMT, Evgeny Astigeevich <eastigeevich at openjdk.org> wrote:

> `far_call`/`far_jump`/`trampoline_call` have parameter `cbuf` with the default value `NULL`. We always call `far_call`/`far_jump` with `NULL`. We call  `trampoline_call` with either `NULL` or the `CodeBuffer` currently used by `MacroAssembler`. If not `NULL` we mark a trampoline call position.
> 
> Andrew Haley(@theRealAph) in https://github.com/openjdk/jdk/pull/8564#discussion_r871062342 suggests to remove it.
> 
> This PR removes the parameter. In the case of `trampoline_call` we explicitly return the position of the generated trampoline calls. All places using the position of the generated trampoline call are updated.
> 
> Tested with release and fastdebug builds:
> - `gtest`: Passed.
> - `tier1`/`tier2`: Passed.

I created https://bugs.openjdk.org/browse/JDK-8293856.

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

PR: https://git.openjdk.org/jdk/pull/10244


More information about the hotspot-dev mailing list