RFR: 8257882: Implement linkToNative intrinsic on AArch64 [v2]
Nick Gasson
ngasson at openjdk.java.net
Fri Dec 11 07:42:21 UTC 2020
On Thu, 10 Dec 2020 13:45:21 GMT, Jorn Vernee <jvernee at openjdk.org> wrote:
>> Nick Gasson has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Review comments
>
> src/hotspot/cpu/aarch64/sharedRuntime_aarch64.cpp line 3104:
>
>> 3102: const GrowableArray<VMReg>& output_registers) {
>> 3103: BufferBlob* _invoke_native_blob =
>> 3104: BufferBlob::create("nep_invoker_blob", MethodHandles::adapter_code_size);
>
> That reminds me; this should _not_ use MethodHandles::adapter_code_size, but a separate constant, since the former is tailored specifically to method handle stubs (and is too large for this case). I still need to update the one for x86 as well (looks like I forgot to do that one before when I changed them for invoker/upcall handler). I think 1024 bytes should be more than enough, but would need to test it.
I've changed it to 1024 for AArch64, definitely large enough.
-------------
PR: https://git.openjdk.java.net/jdk/pull/1711
More information about the core-libs-dev
mailing list