RFR: 8341696: C2: Non-fluid StringBuilder pattern bails out in OptoStringConcat [v9]
Vladimir Kozlov
kvn at openjdk.org
Tue Jan 21 18:26:41 UTC 2025
On Tue, 21 Jan 2025 14:02:00 GMT, Theo Weidmann <tweidmann at openjdk.org> wrote:
>> Extends stringopts to also recognize non-fluid uses of StringBuilder and optimize them the same way.
>>
>> For example, this basic case was not optimized before and is optimized with this PR:
>>
>>
>> StringBuilder sb = new StringBuilder();
>> sb.append("a");
>> sb.append(a);
>> return sb.toString();
>
> Theo Weidmann has updated the pull request incrementally with one additional commit since the last revision:
>
> Fix merge gone wrong
Re-approved after merge.
@theoweidmannoracle, I see failures in your latest internal testing.
-------------
Marked as reviewed by kvn (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/22537#pullrequestreview-2565395118
Changes requested by kvn (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/22537#pullrequestreview-2565401076
More information about the hotspot-compiler-dev
mailing list