[foreign-abi] Integrated: Turn on downcall intrinsics by default
Jorn Vernee
jvernee at openjdk.java.net
Tue Sep 22 17:58:23 UTC 2020
On Tue, 22 Sep 2020 13:59:02 GMT, Jorn Vernee <jvernee at openjdk.org> wrote:
> Hi,
>
> This PR turns on down call intrinsics by default.
>
> Since the time they were turned off some fixes have been done in the area of GC handling of optimized native calls, and
> no further problems in that area have been found. What currently works is:
> 1. All TestDowncall and TestUpcall tests, where each function is invoked in a 20_000 iteration loop with -Xbatch,
> making sure these calls are intrinsified. 2. TestIntrinsics tests which intrinsifies calls the same way.
> 3. All jextract samples with intrinsics turned on, which from the logs also leads to calls being intrinsified,
> especially on large extraction runs like Windows.h.
> There is one more case that is known to be problematic: TestUpcallHighArity crashes/fails when the call is intrinsified
> (using same method as 1 and 2). This seems to be caused by a known problem with the handling on stack arguments for
> intrinsified calls. So, for now, intrinsification of calls that pass arguments on the stack will be turned off. If
> there are other cases you would like to make sure work, please take the test for a spin.
> Thanks,
> Jorn
This pull request has now been integrated.
Changeset: 6ab5d40b
Author: Jorn Vernee <jvernee at openjdk.org>
URL: https://git.openjdk.java.net/panama-foreign/commit/6ab5d40b
Stats: 161 lines in 3 files changed: 71 ins; 54 del; 36 mod
Turn on downcall intrinsics by default
Reviewed-by: mcimadamore
-------------
PR: https://git.openjdk.java.net/panama-foreign/pull/354
More information about the panama-dev
mailing list