On Tue, 25 Feb 2025 10:02:51 GMT, Per Minborg <pminb...@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
