[lworld] RFR: 8364034: [lworld] Pre-register boxing classes in class loaders [v5]

[Coleen Phillimore]

On Fri, 25 Jul 2025 18:29:03 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> Frederic Parain has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Restore multiple returns
>
> src/hotspot/share/classfile/systemDictionary.cpp line 1786:
> 
>> 1784:   assert(k->is_loaded(), "must be");
>> 1785:   assert(k->class_loader_data() != loader_data, "only for classes defined by a parent loader");
>> 1786:   if (dictionary->find_class(current, name) == nullptr) {
> 
> You could also keep the assert and have it be:
> assert(!CDSConfig::is_using_aot_linked_classes() || dictionary->find_class() == nullptr, "Ioi would like this probably");

Oh, no this assert isn't right because you add wrapper classes when using_aot_linked_classes too.  Never mind this.

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

PR Review Comment: https://git.openjdk.org/valhalla/pull/1517#discussion_r2232042966