Re: RFR: 8330802: Desugar switch in Locale::createLocale [v2]

2024-04-23 Thread Claes Redestad
On Mon, 22 Apr 2024 14:11:41 GMT, Claes Redestad wrote: >> This switch expression in `Locale::createLocale` is causing a somewhat large >> startup regression on my local system. Desugaring to if statements seem like >> the right thing to do while we investigate ways to further reduce overheads

Re: RFR: 8330802: Desugar switch in Locale::createLocale [v2]

2024-04-22 Thread Mandy Chung
On Mon, 22 Apr 2024 14:11:41 GMT, Claes Redestad wrote: >> This switch expression in `Locale::createLocale` is causing a somewhat large >> startup regression on my local system. Desugaring to if statements seem like >> the right thing to do while we investigate ways to further reduce overheads

Re: RFR: 8330802: Desugar switch in Locale::createLocale [v2]

2024-04-22 Thread Naoto Sato
On Mon, 22 Apr 2024 14:11:41 GMT, Claes Redestad wrote: >> This switch expression in `Locale::createLocale` is causing a somewhat large >> startup regression on my local system. Desugaring to if statements seem like >> the right thing to do while we investigate ways to further reduce overheads

Re: RFR: 8330802: Desugar switch in Locale::createLocale [v2]

2024-04-22 Thread Roger Riggs
On Mon, 22 Apr 2024 14:11:41 GMT, Claes Redestad wrote: >> This switch expression in `Locale::createLocale` is causing a somewhat large >> startup regression on my local system. Desugaring to if statements seem like >> the right thing to do while we investigate ways to further reduce overheads

Re: RFR: 8330802: Desugar switch in Locale::createLocale [v2]

2024-04-22 Thread Chen Liang
On Mon, 22 Apr 2024 14:11:41 GMT, Claes Redestad wrote: >> This switch expression in `Locale::createLocale` is causing a somewhat large >> startup regression on my local system. Desugaring to if statements seem like >> the right thing to do while we investigate ways to further reduce overheads

Re: RFR: 8330802: Desugar switch in Locale::createLocale [v2]

2024-04-22 Thread Claes Redestad
> This switch expression in `Locale::createLocale` is causing a somewhat large > startup regression on my local system. Desugaring to if statements seem like > the right thing to do while we investigate ways to further reduce overheads > in `SwitchBootstraps`. > > These numbers are against a

Re: RFR: 8330802: Desugar switch in Locale::createLocale

2024-04-22 Thread Chen Liang
On Mon, 22 Apr 2024 13:38:58 GMT, Claes Redestad wrote: >> src/java.base/share/classes/java/util/Locale.java line 1003: >> >>> 1001: return new Locale(lk.base, lk.exts); >>> 1002: } else { >>> 1003: throw new InternalError("should not happen"); >> >> The default

Re: RFR: 8330802: Desugar switch in Locale::createLocale

2024-04-22 Thread Claes Redestad
On Mon, 22 Apr 2024 13:05:47 GMT, Chen Liang wrote: >> This switch expression in `Locale::createLocale` is causing a somewhat large >> startup regression on my local system. Desugaring to if statements seem like >> the right thing to do while we investigate ways to further reduce overheads >>

Re: RFR: 8330802: Desugar switch in Locale::createLocale

2024-04-22 Thread Chen Liang
On Mon, 22 Apr 2024 10:34:29 GMT, Claes Redestad wrote: > This switch expression in `Locale::createLocale` is causing a somewhat large > startup regression on my local system. Desugaring to if statements seem like > the right thing to do while we investigate ways to further reduce overheads >

Re: RFR: 8330802: Desugar switch in Locale::createLocale

2024-04-22 Thread Alan Bateman
On Mon, 22 Apr 2024 10:34:29 GMT, Claes Redestad wrote: > This switch expression in `Locale::createLocale` is causing a somewhat large > startup regression on my local system. Desugaring to if statements seem like > the right thing to do while we investigate ways to further reduce overheads >

RFR: 8330802: Desugar switch in Locale::createLocale

2024-04-22 Thread Claes Redestad
This switch expression in `Locale::createLocale` is causing a somewhat large startup regression on my local system. Desugaring to if statements seem like the right thing to do while we investigate ways to further reduce overheads in `SwitchBootstraps`. These numbers are against a baseline