Integrated: 8316582: Minor startup regression in 22-b15 due JDK-8310929
Claes Redestad
redestad at openjdk.org
Thu Sep 21 09:41:38 UTC 2023
On Wed, 20 Sep 2023 09:12:48 GMT, Claes Redestad <redestad at 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.
This pull request has now been integrated.
Changeset: 913e43fe
Author: Claes Redestad <redestad at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/913e43fea995b746fb9e1b25587d254396c7c3c9
Stats: 31 lines in 2 files changed: 9 ins; 4 del; 18 mod
8316582: Minor startup regression in 22-b15 due JDK-8310929
Reviewed-by: liach, rriggs
-------------
PR: https://git.openjdk.org/jdk/pull/15836
More information about the core-libs-dev
mailing list