On Tue, 6 Aug 2024 23:41:09 GMT, Shaojin Wen <d...@openjdk.org> wrote:

>> This PR implements the same algorithm as the current generateMHInlineCopy 
>> based on bytecode to improve startup performance.
>
> Shaojin Wen has updated the pull request incrementally with six additional 
> commits since the last revision:
> 
>  - checkOverflow
>  - highArity default 0
>  - use MethodTypeDesc instead of MethodType
>  - inlineThreshold parameter
>  - use more idiomatic class-file API
>  - rename SimpleStringBuilderStrategy to InlineHiddenClassStrategy

src/java.base/share/classes/java/lang/invoke/StringConcatFactory.java line 1584:

> 1582:                         }
> 1583:                         cb.iadd()
> 1584:                           .invokestatic(CD_StringConcatHelper, 
> "checkOverflow", MTD_int_int);

Good, but wouldn't we generate slightly more compact code if this check was 
baked into the `stringSize` (when we call that) and only emitted in this 
generated for `char` parameters?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20273#discussion_r1706229674

Reply via email to