Request for reviews (S): 7052494: Eclipse test fails on JDK 7 b142

[Vladimir Kozlov]

Thank you, Tom

On 6/20/11 4:30 PM, Tom Rodriguez wrote:
> So the first file is just anti-deltaing those two changes that disallowed other forms which seems fine.

Correct.

>  The second file looks ok but I think the comment should say "can" instead of "could".

Agree, I will fix the comment.

Thanks,
Vladimir

>
> tom
>
> On Jun 17, 2011, at 4:57 PM, Vladimir Kozlov wrote:
>
>> This is for 7u2.
>>
>> http://cr.openjdk.java.net/~kvn/7052494/webrev
>>
>> Fixed 7052494: Eclipse test fails on JDK 7 b142
>>
>> New code in 5091921 fix incorrectly replaces loop test 'ne' with 'lt' (and with 'gt' for negative stride). As result loop's body is not executed when initial value>  limit. Note, such loops are terminated usually by other checks inside loop's body (for example, range checks).
>>
>> Fix:
>> Keep 'ne' test when we can't guarantee during compilation that init<  limit.
>> Regression test is added.
>>
>> Tested with Eclipse tests.
>