RFR: 8350118: Simplify the layout access VarHandle [v2]
Chen Liang
liach at openjdk.org
Tue Feb 25 13:03:57 UTC 2025
On Tue, 25 Feb 2025 10:02:51 GMT, Per Minborg <pminborg 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/java/lang/invoke/SegmentVarHandle.java line 50:
>
>> 48: final MemoryLayout enclosing;
>> 49: /** The fixed offset value, if exists. vform decides if the fixed offset is used. */
>> 50: final long offset;
>
> These fields are already trusted by virtue of residing in the `j.l.i` package?
Yes. See https://github.com/openjdk/jdk/blob/8cfebc41dc8ec7b0d24d9c467b91de82d28b73fc/src/hotspot/share/ci/ciField.cpp#L223
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23720#discussion_r1969738602
More information about the build-dev
mailing list