On Thu, 23 Oct 2025 20:26:54 GMT, Coleen Phillimore <[email protected]> wrote:
>> Frederic Parain has updated the pull request with a new target base due to a
>> merge or a rebase. The pull request now contains two commits:
>>
>> - Merge remote-tracking branch 'upstream/lworld' into new_acmp
>> - Add alternate version of the substitutability method
>
> src/hotspot/share/classfile/fieldLayoutBuilder.cpp line 1381:
>
>> 1379: case LayoutRawBlock::REGULAR:
>> 1380: {
>> 1381: FieldInfo* fi = _field_info->adr_at(b->field_index());
>
> Is there an assumption that the block field indexes are in ascending order?
I'm not sure I understand the question.
LayoutRawBlocks represent the fields layout, they are chained in ascending
offset order.
The field_index is the index in the FieldInfo array (from the class file) where
this field is declared.
-------------
PR Review Comment:
https://git.openjdk.org/valhalla/pull/1695#discussion_r2460605982
