Hi Vladimir,<div><br></div><div><font face="courier new, monospace">foo(Integer intobj) {</font></div><div><font face="courier new, monospace">  String intstr = intobj.toString();</font></div><div><font face="courier new, monospace">  String str = intstr + &quot;, &quot; + intstr;</font></div>
<div><br></div><div>What should be the desired behavior for the case above?</div><div>Is it all right to skip all Integer.toString() calls, and let later optimizations handle it?</div><div><br></div><div>And, two typos:</div>
<div><br></div><div>541               // chain. It&#39;s node could be eliminated only if it&#39;s result</div><div><br></div><div>It&#39;s -&gt; Its. Other occurrences of &quot;it&#39;s&quot; should also be replaced with &quot;its&quot;.</div>
<div><br></div><div>543               // An other limitation: it should be used only once because</div><div><br></div><div>An other -&gt; Another.</div><div><br></div><div>- Kris<br><br><div class="gmail_quote">On Fri, Jul 6, 2012 at 4:37 AM, Vladimir Kozlov <span dir="ltr">&lt;<a href="mailto:vladimir.kozlov@oracle.com" target="_blank">vladimir.kozlov@oracle.com</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">Vladimir Kozlov wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<a href="http://cr.openjdk.java.net/~kvn/7123926/webrev" target="_blank">http://cr.openjdk.java.net/~<u></u>kvn/7123926/webrev</a><br>
<br>
7123926: Some CTW test crash: !_control.contains(ctrl)<br>
<br>
Don&#39;t try to eliminate Integer::toString() call node during String concatenation optimization if it&#39;s result has several uses.<br>
<br>
Eliminate case:<br>
  foo (Integer intob) {<br>
    String str = &quot;int: &quot; + intobj;<br>
<br>
Keep case:<br>
  foo (Integer intob) {<br>
    String intstr = intobj.toString();<br>
    String str = &quot;int: &quot; + intobj; // second use is in SB allocation <br>
</blockquote>
<br></div>
Sorry, it should be: &quot;int: &quot; + intstr;<br>
<br>
Vladimir<div class="HOEnZb"><div class="h5"><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
debug info<br>
<br>
Tested with failed test.<br>
<br>
Thanks,<br>
Vladimir<br>
</blockquote>
</div></div></blockquote></div><br></div>