RFR: 8338930: StringConcatFactory hardCoded string concatenation strategy [v2]
ExE Boss
duke at openjdk.org
Sun Aug 25 15:09:06 UTC 2024
On Sun, 25 Aug 2024 12:12:42 GMT, Shaojin Wen <duke at openjdk.org> wrote:
>> This is a follow-up to PR #20273, which improves performance when the number of parameters exceeds 20.
>>
>> When the number of parameters is large, the possibility of reuse will be lower, so we can use the static concat method and write the length and coder directly into the bytecode to solve the performance regression problem.
>
> Shaojin Wen has updated the pull request incrementally with one additional commit since the last revision:
>
> Revert "Optimize the construction of MethodType and MethodTypeDesc to reduce memory allocation"
>
> This reverts commit 3bed7290f5cb987e86407f698fb0598f19d65628.
src/java.base/share/classes/java/lang/invoke/StringConcatFactory.java line 1169:
> 1167: paramTypes[i] = cl;
> 1168: }
> 1169: return changed ? MethodType.methodType(args.returnType(), paramTypes) : args;
Suggestion:
return changed ? MethodType.methodType(args.returnType(), paramTypes, true) : args;
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20675#discussion_r1730375667
More information about the core-libs-dev
mailing list