On Mon, 2 Feb 2026 09:53:04 GMT, Joel Sikström <[email protected]> wrote:
>> Hello, >> >> Please refer to the JBS issue for a more detailed description of the >> background of this change. In summary, I suggest we only keep the array of >> InlineLayoutInfo for InstanceKlasses which need it, which are Klasses that >> have fields that have been inlined. >> >> To make the transition to this easier, I suggest we change the following >> properties in FieldLayoutBuilder: >> >> _has_inline_type_fields >> _has_flattening_information >> >> to >> >> _has_inlineable_fields >> _has_inlined_fields >> >> The `_has_inlineable_fields` property is only used for printing and >> `_has_inlined_fields` is the property we expose out to the ClassFileParser, >> telling us that this class has inlined fields, so the array of >> InlineLayoutInfo must be "preserved" and is possible to read from. Hence, >> the array is now only safe to access if `InstanceKlass::has_inlined_fields` >> is true, or simply if the actual field being accessed is flat >> (`fieldDescriptor::is_flat`). >> >> I only found one place (in ciReplay.cpp) where we access the array of >> InlineLayoutInfo even though we might not have any inlined fields and only >> fields that are inlineable. I've changed this to use the normal "reference" >> path for fields that aren't flat. >> >> Testing: >> * Oracle's tier1-5, hotspot_valhalla and jdk_valhalla > > Joel Sikström has updated the pull request incrementally with two additional > commits since the last revision: > > - Move InlineKlass::cast() to callers > - Add abstract check to field_is_inlineable Thank you for the reviews everyone! I merged with the tip of lworld and reran tier1-4 which is green. The failing test in GHA is due to an unrelated issue (solved in [JDK-8376358](https://bugs.openjdk.org/browse/JDK-8376358)). ------------- PR Comment: https://git.openjdk.org/valhalla/pull/1966#issuecomment-3840812186
