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

Coleen Phillimore coleenp at openjdk.org
Mon Oct 27 19:35:30 UTC 2025


On Mon, 27 Oct 2025 19:27:57 GMT, Frederic Parain <fparain at openjdk.org> wrote:

>> This is an alternate version of the substitutability method.
>> To use it, add -Xshare:off -XX:+UseAltSubstitutabilityMethod to the command line of the Valhalla VM in preview mode.
>
> Frederic Parain has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Add comment

Marked as reviewed by coleenp (Committer).

src/hotspot/share/classfile/fieldLayoutBuilder.cpp line 1365:

> 1363:   // Local fields are processed in ascending offset order, so an insertion is very likely be performed
> 1364:   // next to the previous insertion. However, in some cases local fields and inherited fields can be
> 1365:   // interleaved, in which case the search of the insertion position cannot depend on the previous insertion.

thanks.

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

PR Review: https://git.openjdk.org/valhalla/pull/1695#pullrequestreview-3385146100
PR Review Comment: https://git.openjdk.org/valhalla/pull/1695#discussion_r2466845979


More information about the valhalla-dev mailing list