[lworld] RFR: 8364483: [lworld] Pre-register migrated classes in class loaders [v2]

Dan Heidinga heidinga at openjdk.org
Tue Aug 12 20:07:30 UTC 2025


On Wed, 6 Aug 2025 21:48:03 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> This change preloads and preregisters all classes in the migrated classes lists, in all class loaders.
>> Tested with tier1 locally and with -XX:+PrintSystemDictionaryAtExit.
>
> Coleen Phillimore has updated the pull request incrementally with one additional commit since the last revision:
> 
>   remove commented out code.

src/hotspot/share/classfile/vmSymbols.cpp line 339:

> 337:   _migrated_class_names[i++] = java_time_chrono_HijrahDate();
> 338:   _migrated_class_names[i++] = java_time_chrono_JapaneseDate();
> 339:   _migrated_class_names[i++] = java_time_chrono_ThaiBuddhistDate();

Should this `assert(i == _migrated_class_names_length);` as a handshake with the size of the set of migrated classes declared in vmSymbols.hpp?

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

PR Review Comment: https://git.openjdk.org/valhalla/pull/1520#discussion_r2271062996


More information about the valhalla-dev mailing list