[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