[lworld] RFR: 8376813: [lworld] Add print for dummy field reused as null-marker [v2]

Joel Sikström jsikstro at openjdk.org
Mon Feb 2 09:27:36 UTC 2026


On Fri, 30 Jan 2026 22:02:09 GMT, Chen Liang <liach at openjdk.org> wrote:

>> Joel Sikström has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   no dummy field in static layout printing
>
> src/hotspot/share/classfile/fieldLayoutBuilder.cpp line 1587:
> 
>> 1585:     _layout->print(&st, false, _super_klass, _inline_layout_info_array, dummy_field_is_reused_as_null_marker);
>> 1586:     st.print_cr("Static fields:");
>> 1587:     _static_layout->print(&st, true, nullptr, _inline_layout_info_array, dummy_field_is_reused_as_null_marker);
> 
> Does the static layout ever have a dummy field? I would have anticipated this to be simply `false`.

No, the dummy field is nonstatic. I'll change the _static_layout print to use `false` instead.

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

PR Review Comment: https://git.openjdk.org/valhalla/pull/2001#discussion_r2753353383


More information about the valhalla-dev mailing list