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