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

[Aleksey Shipilev]

Good, thanks! Waiting for Tobias.

-Aleksey

On 08/28/2015 07:45 PM, Vladimir Kozlov wrote:
> 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
>>


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20150828/7af9352d/signature.asc>