RFR: 8357822: C2: Multiple string optimization tests are no longer testing string concatenation optimizations [v2]
Roberto Castañeda Lozano
rcastanedalo at openjdk.org
Thu Jun 5 12:56:51 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!
-------------
Marked as reviewed by rcastanedalo (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/25610#pullrequestreview-2900209239
More information about the hotspot-compiler-dev
mailing list