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

Coleen Phillimore coleenp at openjdk.org
Fri Aug 29 21:54:49 UTC 2025


On Tue, 19 Aug 2025 23:31:55 GMT, Ioi Lam <iklam at openjdk.org> wrote:

>> src/hotspot/share/cds/metaspaceShared.cpp line 636:
>> 
>>> 634: 
>>> 635: void VM_PopulateDumpSharedSpace::doit() {
>>> 636:   CDSConfig::set_is_at_cds_safepoint(true);
>> 
>> Can you just use SafepointSynchronize::is_at_safepoint()?  Why new flag?
>
> We need to know if are are at the CDS safepoint, not any safepoint.

I don't think we share safepoints anymore (?)

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

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


More information about the hotspot-dev mailing list