RFR: 8357822: C2: Multiple string optimization tests are no longer testing string concatenation optimizations [v2]
Daniel Skantz
dskantz at openjdk.org
Mon Jun 9 06:13:53 UTC 2025
On Thu, 5 Jun 2025 08:51:11 GMT, Daniel Skantz <dskantz at openjdk.org> wrote:
>> This PR updates a few tests to reintroduce testing of string concatenation optimizations since a few bugs have recently been identified in this area.
>>
>> Selection criteria: performed a text search on the test suite and identified tests for string concatenations or string optimizations that are not currently compiled with `-XDstringConcat=inline` and are not using StringBuilders explicitly.
>>
>> Testing: T1-4.
>>
>> Extra testing: ran the tests manually with `-XX:+PrintOptimizeStringConcat` and verified that the tests are exercising string optimizations after the fix.
>
> Daniel Skantz has updated the pull request incrementally with two additional commits since the last revision:
>
> - revert change to TestStringIntrinsics.java
> - Update test/hotspot/jtreg/compiler/c2/Test7046096.java
>
> Co-authored-by: Emanuel Peter <emanuel.peter at oracle.com>
Thanks for the reviews and suggestions!
-------------
PR Comment: https://git.openjdk.org/jdk/pull/25610#issuecomment-2954752738
More information about the hotspot-compiler-dev
mailing list