Request for reviews (XS): 7123926: Some CTW test crash: !_control.contains(ctrl)
Krystal Mok
rednaxelafx at gmail.com
Thu Jul 5 15:52:08 PDT 2012
Hi Vladimir,
foo(Integer intobj) {
String intstr = intobj.toString();
String str = intstr + ", " + intstr;
What should be the desired behavior for the case above?
Is it all right to skip all Integer.toString() calls, and let later
optimizations handle it?
And, two typos:
541 // chain. It's node could be eliminated only if it's
result
It's -> Its. Other occurrences of "it's" should also be replaced with "its".
543 // An other limitation: it should be used only once
because
An other -> Another.
- Kris
On Fri, Jul 6, 2012 at 4:37 AM, Vladimir Kozlov
<vladimir.kozlov at oracle.com>wrote:
> Vladimir Kozlov wrote:
>
>> http://cr.openjdk.java.net/~**kvn/7123926/webrev<http://cr.openjdk.java.net/~kvn/7123926/webrev>
>>
>> 7123926: Some CTW test crash: !_control.contains(ctrl)
>>
>> Don't try to eliminate Integer::toString() call node during String
>> concatenation optimization if it's result has several uses.
>>
>> Eliminate case:
>> foo (Integer intob) {
>> String str = "int: " + intobj;
>>
>> Keep case:
>> foo (Integer intob) {
>> String intstr = intobj.toString();
>> String str = "int: " + intobj; // second use is in SB allocation
>>
>
> Sorry, it should be: "int: " + intstr;
>
> Vladimir
>
>
> debug info
>>
>> Tested with failed test.
>>
>> Thanks,
>> Vladimir
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20120706/a50af946/attachment.html
More information about the hotspot-compiler-dev
mailing list