RFR: 8273100: Improve AbstractStringBuilder.append(String) when using CompactStrings [v3]

Mon Aug 30 13:33:30 UTC 2021

On Mon, 30 Aug 2021 11:49:49 GMT, Claes Redestad <redestad at openjdk.org> wrote:

>> Refactor to improve inlining, which helps some microbenchmarks exer StringBuilder.append(String)
>
> Claes Redestad has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Simplify and call getBytes(String, int, byte) when possible

src/java.base/share/classes/java/lang/AbstractStringBuilder.java line 1730:

> 1728:     }
> 1729: 
> 1730:     private void putStringAt(int index, String str) {

Can we replace all the calls to this method with calls to previous method as `putStringAt(index, str, 0, str.length())`?

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

PR: https://git.openjdk.java.net/jdk/pull/5291