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

[David Holmes]

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

>> --enable-native-access command line option
>
> Athijegannathan Sundararajan has updated the pull request incrementally with one additional commit since the last revision:
> 
>   missed Method.setAccessible route.

src/hotspot/share/interpreter/linkResolver.cpp line 827:

> 825:     check_method_loader_constraints(link_info, resolved_method, "method", CHECK_NULL);
> 826: 
> 827:     check_restricted_method(link_info, resolved_method, CHECK_NULL);

Not at all clear why this is guarded by `check_loader_constraints`. ?? Otherwise the placement of the check seems fine.

src/hotspot/share/interpreter/linkResolver.cpp line 925:

> 923:     check_method_loader_constraints(link_info, resolved_method, "interface method", CHECK_NULL);
> 924: 
> 925:     check_restricted_method(link_info, resolved_method, CHECK_NULL);

Same comment as in `resolve_method`.

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

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