RFC (S) 8076758: new StringBuilder().append(String).toString() should be recognized by OptimizeStringConcat

Vladimir Kozlov vladimir.kozlov at oracle.com
Fri Aug 28 16:45:02 UTC 2015


Tobias should look on this. I am fine with the idea.
I think these are related to0:

https://bugs.openjdk.java.net/browse/JDK-6969165
https://bugs.openjdk.java.net/browse/JDK-7179968

Thanks,
Vladimir

On 8/28/15 3:12 AM, Aleksey Shipilev wrote:
> Hi,
>
> I would like to see this one fixed, because it touches on pending String
> improvements in JDK:
>    https://bugs.openjdk.java.net/browse/JDK-8076758
>
> Here is a proof-of-concept patch:
>    http://cr.openjdk.java.net/~shade/8076758/webrev.00/
>
> It passes JPRT and fixes the performance issues in microbenchmarks, but
> I'm not sure the code change is completely correct, given the history of
> OptimizeStringConcat bugs. Could anyone from a compiler team chime in?
>
> Thanks!
> -Aleksey
>


More information about the hotspot-compiler-dev mailing list