[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