RFR: 8338544: Dedicated Array class descriptor implementation
Chen Liang
liach at openjdk.org
Wed Sep 25 20:03:36 UTC 2024
On Wed, 25 Sep 2024 20:00:12 GMT, Chen Liang <liach at openjdk.org> wrote:
>> src/java.base/share/classes/java/lang/invoke/InvokerBytecodeGenerator.java line 79:
>>
>>> 77: private static final ClassDesc CD_LambdaForm_Name = ReferenceClassDescImpl.ofValidated("Ljava/lang/invoke/LambdaForm$Name;");
>>> 78: private static final ClassDesc CD_LoopClauses = ReferenceClassDescImpl.ofValidated("Ljava/lang/invoke/MethodHandleImpl$LoopClauses;");
>>> 79: private static final ClassDesc CD_Object_array = CD_Object.arrayType();
>>
>> I guess `CD_Object.arrayType()` shows up often enough now - even once _in_ `java.lang.constant.ConstantDescs` - that we might as well pin it down as a constant somewhere (`ConstantDescs` is a candidate location, but that will take a CSR).
>
> This patch already has a CSR for trivial signature changes. The real difficulty lies in how we should name our new array class descriptors, `Object_array` or `ObjectArray` or what else?
That said, can you leave a quick review on CSR https://bugs.openjdk.org/browse/JDK-8340963 too?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20665#discussion_r1775925133
More information about the core-libs-dev
mailing list