RFR: 8231349: Move intrinsic stubs generation to compiler runtime initialization code [v2]

[Vladimir Kozlov]

On Wed, 22 Mar 2023 21:47:01 GMT, Vladimir Ivanov <vlivanov at openjdk.org> wrote:

>> Vladimir Kozlov has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Address Claes comments
>
> src/hotspot/cpu/aarch64/globals_aarch64.hpp line 39:
> 
>> 37: define_pd_global(bool, UncommonNullCast,         true);  // Uncommon-trap NULLs past to check cast
>> 38: 
>> 39: #if COMPILER2_OR_JVMCI
> 
> I find it easier to read when only the default value is guarded. In that respect, `COMPILER2_OR_JVMCI` code diverges from the rest of the code base.

So you want something similar to `COMPILER1_AND_COMPILER2_PRESENT()` used for `CodeCacheSegmentSize` in following line?
Yes, we have similar macro for JVMCI and C2. How about this:

define_pd_global(bool, MoveIntrinsicStubsGen,   false COMPILER2_OR_JVMCI_PRESENT( || true));

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/13096#discussion_r1145475473