[foreign-memaccess+abi] RFR: 8302346: Lift upcall sharing mechanism to AbstractLinker [v2]

Maurizio Cimadamore mcimadamore at openjdk.org
Fri Feb 17 21:54:46 UTC 2023


On Fri, 17 Feb 2023 20:54:29 GMT, Jorn Vernee <jvernee at openjdk.org> wrote:

>> Lift the upcall sharing mechanism to AbstractLinker, where it can live next to the similar mechanism for down calls. This also allows the fallback linker to use this mechanism.
>> 
>> Instead of an upcall stub, AbstractLinker::arrangeDowncall now return an UpcallStubFactory, which is a callback accepting a target MethodHandle and Arena, which are then used to construct the actual upcall stub.
>> 
>> Perhaps the trickiest part of this patch is that we have to simulate the effects of `SharedUtils::adaptUpcallForIMR` on the target method type. I've added a new method in SharedUtils, called `computeUpcallIMRType` for that, which mimics the shape of the `adaptUpcallForIMR` method.
>> 
>> Since the shape of most of the arrangeUpcall methods was very similar, I factored them out into a helper method in SharedUtils as well.
>
> Jorn Vernee has updated the pull request incrementally with one additional commit since the last revision:
> 
>   use fake MH for adaptation

Nice simplification!

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

Marked as reviewed by mcimadamore (Committer).

PR: https://git.openjdk.org/panama-foreign/pull/791


More information about the panama-dev mailing list