Request for reviews (S): 7052494: Eclipse test fails on JDK 7 b142
Tom Rodriguez
tom.rodriguez at oracle.com
Mon Jun 20 16:30:08 PDT 2011
So the first file is just anti-deltaing those two changes that disallowed other forms which seems fine. The second file looks ok but I think the comment should say "can" instead of "could".
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.
More information about the hotspot-compiler-dev
mailing list