RFR: 8254693: Add Panama feature to pass heap segments to native code [v12]
Jorn Vernee
jvernee at openjdk.org
Mon Nov 13 12:51:36 UTC 2023
On Thu, 9 Nov 2023 15:39:54 GMT, Jorn Vernee <jvernee at openjdk.org> wrote:
>> 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.
Either way, this seems like something that should be addressed in a separate patch
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/16201#discussion_r1391072872
More information about the core-libs-dev
mailing list