On Mon, 16 Mar 2026 19:49:31 GMT, Frederic Parain <[email protected]> wrote:
>> This patch removes the mandatory preloading triggered by the @NullRestricted
>> annotated fields, and falls back to the speculative pre-loadings allowed by
>> the LoadableDescriptors attribute.
>> See the CR for more details about the rational and the new semantic of the
>> @NullRestricted annotation.
>>
>> Tested with Mach5 tier1-4.
>>
>> Thank you.
>
> Frederic Parain has updated the pull request incrementally with one
> additional commit since the last revision:
>
> Update log messages when poking the system dictionary
src/hotspot/share/classfile/classFileParser.cpp line 6323:
> 6321: void ClassFileParser::preload_classes(ConstantPool* cp, TRAPS) {
> 6322: for (GrowableArrayIterator<FieldInfo> it = _temp_field_info->begin();
> it != _temp_field_info->end(); ++it) {
> 6323: FieldInfo fieldinfo = *it;
Suggestion:
for (FieldInfo fieldinfo : *_temp_field_info) {
-------------
PR Review Comment:
https://git.openjdk.org/valhalla/pull/2228#discussion_r2944563299
