RFR: 8245959: Extend String concat testing to account for folded constants

Paul Sandoz paul.sandoz at oracle.com
Wed May 27 19:20:05 UTC 2020


Looks good.

Paul.

> On May 27, 2020, at 5:54 AM, Claes Redestad <claes.redestad at oracle.com> wrote:
> 
> Hi,
> 
> Since JDK-8222852 the default String concat strategy aggressively folds
> string constants into prepender method handles, which means an
> expression like foo + bar and one like "x" + foo + "y" + bar + "z" will
> have similar shape, but take slightly different paths through generated
> code.
> 
> This patch extends the ImplicitStringConcatShapes test to add more
> shape variants with surrounding String constants, which should improve
> coverage and put a bit more stress on the implementation strategy.
> 
> The changes of interest are in ImplicitStringConcatShapesTestGen -
> since that's what is used to generate ImplicitStringConcatShapes
> 
> Bug:    https://bugs.openjdk.java.net/browse/JDK-8245959
> Webrev: http://cr.openjdk.java.net/~redestad/8245959/open.00/
> 
> Testing: tier1
> 
> Thanks!
> 
> /Claes



More information about the core-libs-dev mailing list