On Thu, 29 Aug 2024 22:40:43 GMT, Dean Long <[email protected]> wrote:
>> src/hotspot/share/oops/klass.hpp line 436:
>>
>>> 434: #endif
>>> 435: static ByteSize bitmap_offset() { return
>>> byte_offset_of(Klass, _bitmap); }
>>> 436: static ByteSize misc_flags_offset() { return
>>> byte_offset_of(Klass, _misc_flags); }
>>
>> Suggestion:
>>
>> static ByteSize misc_flags_offset() { return
>> byte_offset_of(Klass, _misc_flags._flags); }
>
> We probably shouldn't assume the _flags field starts at offset 0.
Yes, this was wrong. Thanks for seeing this.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20719#discussion_r1738643187
