[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