[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