Looks good.
Thanks,
Vladimir
On 1/5/15 8:57 AM, Roland Westrelin wrote:
Vladimir,
Can you review this one as well?
Thanks,
Roland.
Begin forwarded message:
Date: December 22, 2014 at 4:56:40 PM GMT+1
From: Seán Coffey <sean.cof...@oracle.com>
To: Roland Westrelin <roland.westre...@oracle.com>, jdk7u-dev@openjdk.java.net
Subject: Re: [7u-dev] Request for approval for 8066103: C2's range check
smearing allows out of bound array accesses
As per the 8048170 request, please get a peer review before pushing to
jdk7u-dev.
Approved.
regards,
Sean.
On 22/12/14 14:51, Roland Westrelin wrote:
Backport to 7u request. Changes were pushed to jdk9 2 weeks ago and to 8u a bit
less than 2 weeks ago. Nightly testing is fine. Changes apply cleanly to 7u
sources but the test case cannot be run and I had to remove its use of the
whitebox API.
https://bugs.openjdk.java.net/browse/JDK-8066103
http://cr.openjdk.java.net/~roland/8066103/webrev.7u/
Review thread:
http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2014-December/016405.html
jdk9 changeset:
http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/287f46592168
jdk8u changeset:
http://hg.openjdk.java.net/jdk8u/hs-dev/hotspot/rev/e3d0aaab84aa
Thanks,
Roland.
