On Mon, 31 Aug 2026 13:50:18 GMT, Frederic Parain <[email protected]> wrote:

>> Johan Sjölen has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Self-review
>
> src/hotspot/share/oops/layoutKind.hpp line 142:
> 
>> 140:   int _null_marker_offset;
>> 141:   // Size of each LayoutKind. For atomic layouts, the size also acts as 
>> alignment.
>> 142:   int _sizes[static_cast<size_t>(LayoutKind::COUNT) - 1]; // REFERENCE 
>> has no size, so we remove 1
> 
> REFERENCE has a size of `oopSize` or `uint_32` if compressed oops are used.
> This could be used to avoid the irregularity REFERENCE is causing here.

The best choice seems to be `sizeof(narrowOop)` and `oopSize`.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/32276#discussion_r3902354142

Reply via email to