On Fri, 16 Jan 2026 17:41:39 GMT, Frederic Parain <[email protected]> wrote:
>> src/hotspot/share/classfile/fieldLayoutBuilder.cpp line 1212:
>>
>>> 1210: _null_free_non_atomic_layout_alignment = _payload_alignment;
>>> 1211: }
>>> 1212: if (UseNullableValueFlattening) {
>>
>> Could this be else if (UseNullableValueFlattening)? These are mutually
>> exclusive iirc so the else if would make that explicit.
>
> They are not mutually exclusive. Each layout can be enabled/disabled
> individually, mostly for testing/debugging purpose.
okay.
-------------
PR Review Comment:
https://git.openjdk.org/valhalla/pull/1407#discussion_r2699643418
