[foreign-memaccess+abi] RFR: 8263459: Add better support for restricted methods

Maurizio Cimadamore mcimadamore at openjdk.java.net
Fri Mar 19 11:55:54 UTC 2021


On Fri, 19 Mar 2021 11:39:13 GMT, Athijegannathan Sundararajan <sundar at openjdk.org> wrote:

> --enable-native-access command line option

I've already reviewed the code separately and provided some commments which have been addressed here. It would be great if somebody from the VM team could go over the VM changes. Added some minor comments.

src/java.base/share/classes/jdk/internal/module/IllegalNativeAccessChecker.java line 2:

> 1: /*
> 2:  *  Copyright (c) 2019, Oracle and/or its affiliates. All rights reserved.

Copyright year seems off ?

src/jdk.incubator.foreign/share/classes/jdk/internal/foreign/Utils.java line 111:

> 109: 
> 110:     public static void checkRestrictedAccess(String method) {
> 111:         switch (foreignRestrictedAccess) {

is this "foreignRestrictedAccess" field removed in the patch?

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

Marked as reviewed by mcimadamore (Committer).

PR: https://git.openjdk.java.net/panama-foreign/pull/471


More information about the panama-dev mailing list