RFR: 8336831: Optimize StringConcatHelper.simpleConcat [v2]
Chen Liang
liach at openjdk.org
Fri Jul 19 20:58:31 UTC 2024
On Fri, 19 Jul 2024 20:55:03 GMT, Shaojin Wen <duke at openjdk.org> wrote:
>> src/java.base/share/classes/java/lang/StringConcatHelper.java line 371:
>>
>>> 369: return new String(s1);
>>> 370: }
>>> 371: byte coder = (byte) (s1.coder() | s2.coder());
>>
>> Suggestion:
>>
>> int coder = s1.coder() | s2.coder();
>
> Converting coder to int type requires processing when calling new String(byte[], byte)
You are right, I ignored that `coder` is reused again below. My bad!
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20253#discussion_r1684964634
More information about the core-libs-dev
mailing list