[lworld] RFR: 8360530: [lworld] CDS does not preload classes listed in the LoadableDescriptors attribute [v2]

[Tobias Hartmann]

On Mon, 30 Jun 2025 13:19:19 GMT, Quan Anh Mai <qamai at openjdk.org> wrote:

>> Matias Saavedra Silva has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Fix copyright
>
> src/hotspot/share/classfile/systemDictionary.cpp line 1105:
> 
>> 1103:       if (fs.access_flags().is_static()) continue;
>> 1104:       Symbol* sig = fs.signature();
>> 1105:       if (fs.is_null_free_inline_type()) {
> 
> Why is it being null-free significant here, I thought nullable fields can be flattened, too?

Good point, that seems wrong as well.

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

PR Review Comment: https://git.openjdk.org/valhalla/pull/1492#discussion_r2175100105