[lworld] RFR: 8376813: [lworld] Add print for dummy field reused as null-marker [v2]
duke
duke at openjdk.org
Wed Feb 4 08:11:49 UTC 2026
On Mon, 2 Feb 2026 09:27:33 GMT, Joel Sikström <jsikstro at openjdk.org> wrote:
>> Hello,
>>
>> Right now it's not clear that the dummy field that is injected into empty inline klasses can be/is reused for the null-marker when looking at the printed layout from -XX:+PrintInlineLayout. I suggest we enhance this print to indicate if the dummy field has been reused for the null-marker.
>>
>> I've tested that the added comment in the print is there when nullability is turned on (default), and not there when turned off (`-XX:-UseNullableValueFlattening -XX:-UseNullableNonAtomicValueFlattening`).
>>
>>
>> Before:
>> Instance fields:
>> @0 RESERVED 12/-
>> @12 REGULAR 1/1 ".empty" B
>>
>> After:
>> Instance fields:
>> @0 RESERVED 12/-
>> @12 REGULAR 1/1 ".empty" B (reused as null-marker)
>>
>>
>> Testing:
>> * GHA
>> * Oracle's tier1
>
> Joel Sikström has updated the pull request incrementally with one additional commit since the last revision:
>
> no dummy field in static layout printing
@jsikstro
Your change (at version 87849f24d507c2cb52fe30bbc357d47b3fb115ac) is now ready to be sponsored by a Committer.
-------------
PR Comment: https://git.openjdk.org/valhalla/pull/2001#issuecomment-3845941963
More information about the valhalla-dev
mailing list