RFR: 8350118: Simplify the layout access VarHandle [v2]
Jorn Vernee
jvernee at openjdk.org
Wed Feb 26 17:26:01 UTC 2025
On Wed, 26 Feb 2025 17:22:13 GMT, Jorn Vernee <jvernee at openjdk.org> wrote:
>> Chen Liang has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Review remarks, dates, some more simplifications
>
> src/java.base/share/classes/jdk/internal/foreign/LayoutPath.java line 216:
>
>> 214: } else {
>> 215: // simpler adaptation
>> 216: handle = MethodHandles.insertCoordinates(handle, 2, offset); // (MS, long)
>
> Okay, looking at this code I think I get it: when the offset into the enclosing layout is fixed, we don't have strides (from `sequenceElement()`), so we don't need the extra adaptation.
I suggest maybe renaming `noStride` to something like `fixedOffsetInEnclosing`
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23720#discussion_r1972026669
More information about the build-dev
mailing list