RFR: 8254693: Add Panama feature to pass heap segments to native code [v12]
Jorn Vernee
jvernee at openjdk.org
Thu Nov 9 15:43:08 UTC 2023
On Wed, 8 Nov 2023 19:52:00 GMT, Vladimir Ivanov <vlivanov at openjdk.org> wrote:
>> Jorn Vernee has updated the pull request incrementally with two additional commits since the last revision:
>>
>> - a -> an
>> - add note to downcallHandle about passing heap segments by-reference
>
> src/hotspot/cpu/aarch64/downcallLinker_aarch64.cpp line 182:
>
>> 180: ArgumentShuffle arg_shuffle(filtered_java_regs, out_regs, shuffle_reg);
>> 181:
>> 182: #ifndef PRODUCT
>
> Any particular reason to exclude the logging in product builds? `ArgumentShuffle::print_on()` is unconditionally available there.
This is partly historical. The log output is only intended for debugging, not for end-user eyes. So, I think I originally excluded it as a way of trimming fat from the product build.
Either way, `ArgumentShuffle::print_on` should probably be excluded/included on the same basis.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/16201#discussion_r1388189136
More information about the core-libs-dev
mailing list