On Wed, 20 Sep 2023 09:12:48 GMT, Claes Redestad <redes...@openjdk.org> wrote:

> This patch reverts the use of `ByteArrayLittleEndian` in `StringLatin1`. 
> 
> This use is the cause of a small (~1.5ms) startup regression in 22-b15. While 
> a manageable startup regression in and of itself, the use of `VarHandles` in 
> core utility classes brings an increased risk of bootstrap circularity 
> issues, for example disqualifying the use of things like `Integers.toString` 
> in some places.
> 
> Reverting this partially rolls back the performance improvement gained by 
> JDK-8310929. It seems reasonable that the compiler can be enhanced to gain 
> that loss back.

Thanks for tracking down the regression.

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

Marked as reviewed by rriggs (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/15836#pullrequestreview-1636230022

Reply via email to