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