RFR: 8357823: Changes in StringBuilder (JDK-8351443) caused a 1-3% regression in biojava

[Chen Liang]

On Fri, 30 May 2025 14:01:04 GMT, Roger Riggs <rriggs at openjdk.org> wrote:

> Comment out assertions added in JDK-8351443 from AbstractStringBuilder.ensureCapacityNewCoder that increase the codesize, preventing some inlining, and reducing performance
> 
>        assert coder == newCoder || newCoder == UTF16 : "bad new coder UTF16 -> LATIN1";
>        assert count <= newCapacity : "count exceeds new capacity";

In javap output, BCI 0-28 and 59-81 are the two assertions. Total code size is 125, and the no-assertion size is 75. Is there any hotspot limit in these two number reached as a result of these two assertions?

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

PR Comment: https://git.openjdk.org/jdk/pull/25550#issuecomment-2923667909