On Wed, 14 Jan 2026 20:14:07 GMT, Coleen Phillimore <[email protected]> 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
