Re: [7u-dev] Request for approval for 8066103: C2's range check smearing allows out of bound array accesses

2015-01-07 Thread Roland Westrelin
Thanks Vladimir Sean. Roland. On Jan 5, 2015, at 7:17 PM, Vladimir Kozlov vladimir.koz...@oracle.com wrote: 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

Re: [7u-dev] Request for approval for 8048170: Test closed/java/text/Normalizer/ConformanceTest.java failed

2015-01-07 Thread Roland Westrelin
Thanks Vladimir Sean. Roland. On Jan 5, 2015, at 7:10 PM, Vladimir Kozlov vladimir.koz...@oracle.com wrote: Changes looks good. They match changes in 9 and 8u. Thanks, Vladimir On 1/5/15 8:57 AM, Roland Westrelin wrote: Hi Vladimir, Can you review this? Thanks, Roland

hg: jdk7u/jdk7u-dev/hotspot: 2 new changesets

2015-01-07 Thread roland . westrelin
Changeset: 2581c454c7cc Author:roland Date: 2014-12-22 14:53 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/hotspot/rev/2581c454c7cc 8048170: Test closed/java/text/Normalizer/ConformanceTest.java failed Summary: PhaseIdealLoop::split_if_with_blocks_post() shoulnd't reorder

[7u-dev] Request for approval for 8048170: Test closed/java/text/Normalizer/ConformanceTest.java failed

2014-12-22 Thread Roland Westrelin
Backport to 7u request. Changes were pushed to jdk9 a bit more than 2 weeks ago and to 8u a bit less than 2 weeks ago. Nightly testing is fine. The change doesn’t apply cleanly to 7u sources but the conflict is straightforward to solve. https://bugs.openjdk.java.net/browse/JDK-8048170

[7u-dev] Request for approval for 8066103: C2's range check smearing allows out of bound array accesses

2014-12-22 Thread Roland Westrelin
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