On Fri, 30 Jan 2026 22:02:09 GMT, Chen Liang <[email protected]> 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
