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

[Coleen Phillimore]

On Tue, 12 Aug 2025 20:04:31 GMT, Dan Heidinga <heidinga at openjdk.org> wrote:

>> 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?

That's a good idea.  I didn't know a better C++ way to initialize these.  I'm sure it exists.

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

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