RFR: 8350118: Simplify the layout access VarHandle [v2]

Jorn Vernee jvernee at openjdk.org
Wed Feb 26 17:02:00 UTC 2025


On Fri, 21 Feb 2025 10:05:36 GMT, Maurizio Cimadamore <mcimadamore 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/VarForm.java line 150:
> 
>> 148:         String methodName = value.methodName();
>> 149:         MethodType type = methodType_table[value.at.ordinal()].insertParameterTypes(0, VarHandle.class);
>> 150:         assert !UNSAFE.shouldBeInitialized(implClass) : implClass;
> 
> Thanks - took a long time to figure this one out -- hopefully it will save time in the future ;-)

TBH, I thought that resolving the member name would implicitly initialize the class

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23720#discussion_r1971985978


More information about the build-dev mailing list