On Mon, 26 Aug 2024 13:30:18 GMT, Claes Redestad <redes...@openjdk.org> wrote:
> #14404 caused some startup regressions, where the main cause of startup > increase in that change was due the use of - and the not very optimized state > of - runtime bootstrapped switches. This was remedied by a series of > optimizations to the switch bootstrap methods and finally a desugaring of the > switch added by #14404. Now @shipilev reports that there appears to be some > lingering startup issue from #14404. > > One thing that stands out is a couple of caches which are being eagerly > initialized regardless of user locale. This PR makes those caches lazily > initialized and slightly more efficient (no need to implement `UnaryOperator` > after inlining some code). Avoids loading 4 classes if running the > HelloStream startup test in a locale that's in the set of constant base > locales, 2 otherwise. This doesn't really do much to move the needle for > startup on my setup, but tentatively it's the only remaining inefficiency > I've found that came from #14404. This pull request has now been integrated. Changeset: a98ecad0 Author: Claes Redestad <redes...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/a98ecad0a920f12d81386de3d0f549d542014773 Stats: 37 lines in 2 files changed: 12 ins; 16 del; 9 mod 8338897: Small startup regression remains after JDK-8309622 and JDK-8331932 Reviewed-by: liach, naoto ------------- PR: https://git.openjdk.org/jdk/pull/20713
