On Mon, 26 Aug 2024 13:57:46 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. > > Claes Redestad has updated the pull request incrementally with one additional > commit since the last revision: > > final Marked as reviewed by liach (Reviewer). ------------- PR Review: https://git.openjdk.org/jdk/pull/20713#pullrequestreview-2260759254
