On Sat, 27 Apr 2024 00:14:23 GMT, Chen Liang <li...@openjdk.org> 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

Reply via email to