[lworld] RFR: 8374800: [lworld] Add a NULLABLE_NON_ATOMIC_FLAT layout [v2]
Frederic Parain
fparain at openjdk.org
Fri Jan 16 17:45:06 UTC 2026
On Wed, 14 Jan 2026 20:05:40 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:
>> Frederic Parain has updated the pull request incrementally with one additional commit since the last revision:
>>
>> More renaming
>
> 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.
-------------
PR Review Comment: https://git.openjdk.org/valhalla/pull/1407#discussion_r2699412900
More information about the valhalla-dev
mailing list