[lworld] RFR: 8370450: [lworld] Alternate implementation of the substitutability test method [v2]

[Frederic Parain]

On Thu, 23 Oct 2025 20:26:54 GMT, Coleen Phillimore <coleenp at openjdk.org> 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