[lworld] RFR: 8374800: [lworld] Add a NULLABLE_NON_ATOMIC_FLAT layout [v2]
Coleen Phillimore
coleenp at openjdk.org
Fri Jan 16 19:06:48 UTC 2026
On Fri, 16 Jan 2026 17:41:39 GMT, Frederic Parain <fparain at openjdk.org> 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
More information about the valhalla-dev
mailing list