On Wed, 30 Apr 2025 15:40:35 GMT, Ioi Lam <ik...@openjdk.org> wrote:
>> src/java.base/share/classes/java/lang/ref/Reference.java line 313:
>>
>>> 311: }
>>> 312:
>>> 313: private static void runtimeSetup() {
>>
>> I don't know if you are planning an annotation or something to mark these
>> "runtimeSetup" methods but each one will minimally need a comment so that
>> it's clear to anyone touching this code that it may be called by the VM when
>> loading the AOT cache.
>
> Yes, the plan is to use annotation to mark the `runtimeSetup` and
> `assemblySetup` methods.
I added a comment above the new `runtimeSetup()` method in Reference.java. For
the other existing occurrences, I plan to address them in a separate RFE,
probably [JDK-8342481](https://bugs.openjdk.org/browse/JDK-8342481)
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24757#discussion_r2069174481
