review(XS) 7032696: Fix for 7029152 broke VM

[Vladimir Kozlov]

thank you, Tom

Vladimir

Tom Rodriguez wrote:
> Ok.
> 
> tom
> 
> On Mar 31, 2011, at 9:51 AM, Vladimir Kozlov wrote:
> 
>> I updated webrev. As John pointed me, Ideal() should return 'this' when memory is updated. I also added a band-aid for one iteration loops problem which causes the second failure in the bug report. I have RFE 7004555 to fix it.
>>
>> Vladimir
>>
>> Vladimir Kozlov wrote:
>>> http://cr.openjdk.java.net/~kvn/7032696/webrev
>>> Fixed: 7032696: Fix for 7029152 broke VM
>>> StrIntrinsicNode::Ideal() should not optimize memory during Parse.
>>> Tested with full CTW.
>