On Sat, 27 Apr 2024 00:14:23 GMT, Chen Liang <[email protected]> wrote:
>> Vicente Romero has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> code refactoring
>
> src/java.base/share/classes/java/lang/reflect/Executable.java line 348:
>
>> 346: final Type[] out = new
>> Type[nonGenericParamTypes.length];
>> 347: final Parameter[] params = getParameters();
>> 348: if (getDeclaringClass().isRecord() && this instanceof
>> Constructor) {
>
> Sorry for late notice but we can move the 2 variable allocation to the else
> block, to speed up the record constructor path.
yep true, will do
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/17070#discussion_r1583416245