[lworld] RFR: 8357474: [lworld] Consolidate load/store flat inline types [v9]

Quan Anh Mai qamai at openjdk.org
Mon May 26 15:20:21 UTC 2025


On Mon, 26 May 2025 09:09:37 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:

>> Quan Anh Mai has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 11 additional commits since the last revision:
>> 
>>  - Merge branch 'lworld' into loadstoreflat
>>  - cast_to_flat_array
>>  - weird layout
>>  - typo
>>  - generalize weird layout
>>  - fix array of nullable empty type
>>  - fix array field calculations
>>  - array
>>  - convert_from_payload
>>  - add tests
>>  - ... and 1 more: https://git.openjdk.org/valhalla/compare/d1759c70...e49bbc8c
>
> src/hotspot/share/opto/inlinetypenode.cpp line 488:
> 
>> 486:       if (base->is_Con() && oop_ptr->is_inlinetypeptr() && !is_array && !mismatched) {
>> 487:         // If the oop to the inline type is constant (static final field), we can
>> 488:         // also treat the fields as constants because the inline type is immutable.
> 
> Why is this not needed anymore?

Because the loads we generate can be constant-folded, there is no need for an explicit path here.

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

PR Review Comment: https://git.openjdk.org/valhalla/pull/1470#discussion_r2107554051


More information about the valhalla-dev mailing list