On Thu, 28 May 2026 15:09:03 GMT, Dan Heidinga <[email protected]> wrote:
>> Casper Norrbin has updated the pull request with a new target base due to a
>> merge or a rebase. The incremental webrev excludes the unrelated changes
>> brought in by the merge/rebase. The pull request contains five additional
>> commits since the last revision:
>>
>> - create symbol only when logging
>> - Merge branch 'lworld' into systemdict-nullrestricted-loading
>> - Validate archived inline field dependencies without LoadableDescriptors
>> - re-add failed marker
>> - removed forced loading for nullrestricted
>
> src/hotspot/share/classfile/systemDictionary.cpp line 1208:
>
>> 1206: if (ik->has_inlined_fields()) {
>> 1207: for (AllFieldStream fs(ik); !fs.done(); fs.next()) {
>> 1208: if (fs.access_flags().is_static()) continue;
>
> @fparain As per our discussion about nullresticted fields, this will need to
> be updated as well to not skip static fields
Hmm, does this mean NR static fields may have non-reference layout?
-------------
PR Review Comment:
https://git.openjdk.org/valhalla/pull/2409#discussion_r3318863217
