jdk17u-dev icon indicating copy to clipboard operation
jdk17u-dev copied to clipboard

8357822: C2: Multiple string optimization tests are no longer testing string concatenation optimizations

Open rm-gh-8 opened this issue 3 weeks ago • 7 comments
trafficstars

Backporting JDK-8357822: C2: Multiple string optimization tests are no longer testing string concatenation optimizations. Adds several tests for string concatenation optimization in C2. Ran GHA Sanity Checks (passed) and adjusted tests (passed). Patch is clean.


Progress

  • [x] Change must not contain extraneous whitespace
  • [x] Commit message must refer to an issue
  • [ ] JDK-8357822 needs maintainer approval

Issue

  • JDK-8357822: C2: Multiple string optimization tests are no longer testing string concatenation optimizations (Bug - P4)

Reviewing

Using git

Checkout this PR locally:
$ git fetch https://git.openjdk.org/jdk17u-dev.git pull/4130/head:pull/4130
$ git checkout pull/4130

Update a local copy of the PR:
$ git checkout pull/4130
$ git pull https://git.openjdk.org/jdk17u-dev.git pull/4130/head

Using Skara CLI tools

Checkout this PR locally:
$ git pr checkout 4130

View PR using the GUI difftool:
$ git pr show -t 4130

Using diff file

Download this PR as a diff file:
https://git.openjdk.org/jdk17u-dev/pull/4130.diff

rm-gh-8 avatar Oct 28 '25 19:10 rm-gh-8