RFR: 8318072: DonwcallLinker does not acquire/release segments in interpreter

Maurizio Cimadamore mcimadamore at openjdk.org
Fri Oct 13 12:02:49 UTC 2023


On Fri, 13 Oct 2023 08:00:07 GMT, Jorn Vernee <jvernee at openjdk.org> wrote:

> Implement missing by-reference argument acquire/release functionality in DowncallLinker::invokeInterpBindings.
> 
> I've also simplified the related code a bit:
> - `retIndexMap` was not used. I've removed it
> - `BindingInterpreter.StoreFunc::store`'s type argument was not used. Removed
> - UpcallLinker was redundantly collecting the move bindings for return values. Removed
> 
> I've added runs without specialization to the failing tests as well, so that we keep testing this.
> 
> Testing: `jdk_foreign` suite.

Looks good

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

Marked as reviewed by mcimadamore (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/16177#pullrequestreview-1676254470


More information about the core-libs-dev mailing list