[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