RFR: 8254231: Implementation of Foreign Linker API (Incubator) [v7]
Jorn Vernee
jvernee at openjdk.java.net
Thu Oct 22 16:57:31 UTC 2020
On Thu, 22 Oct 2020 16:31:00 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:
>> Some of this is familiar to me from reviews in the `panama-foreign` repository, but less so than the memory API. I focused on the Java code, and ignored changes that are common with the memory API PR.
>>
>> If it helps in can provide a PR in the `panama-foreign` repository addressing editorial comments to the linker API.
>
> I've just uploaded a new iteration which addresses most of @PaulSandoz comments on the API/Java impl.
The most recent changes address Coleen's review comments on: #1
The changes I've made are:
- Use TempNewSymbol when creating a symbol in field_offset & find_InstanceKlass
- change JVM_* macros -> JNI_*
- rename parseABIDescritpor & parseBufferLayout -> parse_abi_descriptor & parse_buffer_layout
- remove ref to global function using '::', by merging the 2 functions together
- make 2 functions static, instead of ForeignGlobals class members
-------------
PR: https://git.openjdk.java.net/jdk/pull/634
More information about the security-dev
mailing list