RFR: 8222484: Specialize generation of simple String concatenation expressions

Jim Laskey james.laskey at oracle.com
Mon Apr 15 12:37:05 UTC 2019


The StringConcatHelper::valueOf really should be StringConcatHelper::stringOf to clarify the result. Use here is unlike String::valueOf context where the result is logically String.

StringConcatHelper::mix comments really should describe the encoding. If does a lot of handwaving re checking for overflow and mixing coder.

Cheers,

-- Jim







> On Apr 15, 2019, at 7:49 AM, Claes Redestad <claes.redestad at oracle.com> wrote:
> 
> Hi,
> 
> please review this enhancement to specialize a few simple and common
> String concatenation patterns in the default strategy.
> 
> Bug:    https://bugs.openjdk.java.net/browse/JDK-8222484
> Webrev: http://cr.openjdk.java.net/~redestad/8222484/open.00/
> 
> This reduces first-time bootstrap times for the targeted patterns from
> ~20ms to be lost in the noise on my machine (so ~0.5ms), while being
> peak performance neutral and not regressing bootstrap characteristics on
> more complex patterns.
> 
> This also consolidates the String::concat method to reuse the optimized
> method in StringConcatHelper, which speeds up that method by ~1.4x for a
> few sampled inputs.
> 
> Testing: tier1-3
> 
> Thanks!
> 
> /Claes



More information about the core-libs-dev mailing list