RFR: 8317269: Store old classes in linked state in AOT cache [v2]

Ioi Lam iklam at openjdk.org
Tue Aug 19 23:54:41 UTC 2025


On Tue, 19 Aug 2025 21:07:27 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> Ioi Lam has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Fixed bugs found in JCK testing
>
> src/hotspot/share/classfile/systemDictionaryShared.cpp line 264:
> 
>> 262:     }
>> 263: 
>> 264: 
> 
> extra blank lines.

Fixed.

> src/hotspot/share/runtime/mutexLocker.cpp line 309:
> 
>> 307:   MUTEX_DEFN(DumpTimeTable_lock              , PaddedMutex  , nosafepoint);
>> 308:   MUTEX_DEFN(CDSLambda_lock                  , PaddedMutex  , nosafepoint);
>> 309:   MUTEX_DEFN(DumpRegion_lock                 , PaddedMutex  , nosafepoint-1); // Holds DumpTimeTable_lock
> 
> There's a MUTEX_DEFL where you can define the mutex in terms of the locks that it takes out.

Fixed.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26754#discussion_r2286641037
PR Review Comment: https://git.openjdk.org/jdk/pull/26754#discussion_r2286640872


More information about the hotspot-dev mailing list