[lworld] RFR: 8374800: [lworld] Add a NULLABLE_NON_ATOMIC_FLAT layout [v2]
Coleen Phillimore
coleenp at openjdk.org
Wed Jan 14 20:25:56 UTC 2026
On Wed, 14 Jan 2026 20:14:07 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/oops/inlineKlass.hpp line 95:
>
>> 93: int _null_free_atomic_size_in_bytes; // size and alignment requirement for a null-free atomic layout, -1 if no atomic flat layout is possible
>> 94: int _nullable_atomic_size_in_bytes; // size and alignment requirement for a nullable layout (always atomic), -1 if no nullable flat layout is possible
>> 95: int _nullable_non_atomic_size_in_bytes; // size and alignment requirement for a nullable non-atomic layout, -1 if not available
>
> Here is something that I'm confused about. Does an InlineKlass have multiple possible layouts and sizes depending on the container? Is that why the InlineKlass needs to save all these size non-static data members?
Can you align the // s please?
-------------
PR Review Comment: https://git.openjdk.org/valhalla/pull/1407#discussion_r2691895437
More information about the valhalla-dev
mailing list