Mercurial > hg > graal-compiler
comparison test/compiler/rangechecks/TestRangeCheckSmearingLoopOpts.java @ 20701:b6585ac86988
8048170: Test closed/java/text/Normalizer/ConformanceTest.java failed
Summary: PhaseIdealLoop::split_if_with_blocks_post() shoulnd't reorder range checks adjusted by range check smearing
Reviewed-by: kvn, jrose
author | roland |
---|---|
date | Thu, 04 Dec 2014 14:34:11 +0100 |
parents | |
children |
comparison
equal
deleted
inserted
replaced
20700:0558eb13dcf3 | 20701:b6585ac86988 |
---|---|
1 /* | |
2 * Copyright (c) 2014, Oracle and/or its affiliates. All rights reserved. | |
3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. | |
4 * | |
5 * This code is free software; you can redistribute it and/or modify it | |
6 * under the terms of the GNU General Public License version 2 only, as | |
7 * published by the Free Software Foundation. | |
8 * | |
9 * This code is distributed in the hope that it will be useful, but WITHOUT | |
10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or | |
11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License | |
12 * version 2 for more details (a copy is included in the LICENSE file that | |
13 * accompanied this code). | |
14 * | |
15 * You should have received a copy of the GNU General Public License version | |
16 * 2 along with this work; if not, write to the Free Software Foundation, | |
17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. | |
18 * | |
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA | |
20 * or visit www.oracle.com if you need additional information or have any | |
21 * questions. | |
22 */ | |
23 | |
24 /* | |
25 * @test | |
26 * @bug 8048170 | |
27 * @summary Following range check smearing, range check cannot be replaced by dominating identical test. | |
28 * @run main/othervm -XX:-BackgroundCompilation -XX:-UseOnStackReplacement TestRangeCheckSmearingLoopOpts | |
29 * | |
30 */ | |
31 public class TestRangeCheckSmearingLoopOpts { | |
32 | |
33 static int dummy; | |
34 | |
35 static int m1(int[] array, int i) { | |
36 for (;;) { | |
37 for (;;) { | |
38 if (array[i] < 0) { // range check (i+0) dominates equivalent check below | |
39 break; | |
40 } | |
41 i++; | |
42 } | |
43 | |
44 // A control flow that stops IfNode::up_one_dom() | |
45 if ((i % 2)== 0) { | |
46 if ((array[i] % 2) == 0) { | |
47 dummy = i; | |
48 } | |
49 } | |
50 | |
51 // IfNode::Ideal will rewrite some range checks if Compile::allow_range_check_smearing | |
52 if (array[i-1] == 9) { // range check (i-1) unchanged | |
53 int res = array[i-3]; // range check (i-3) unchanged | |
54 res += array[i]; // range check (i+0) unchanged | |
55 res += array[i-2]; // removed redundant range check | |
56 // the previous access might be hoisted by | |
57 // PhaseIdealLoop::split_if_with_blocks_post because | |
58 // it appears to have the same guard, but it also | |
59 // depends on the previous guards | |
60 return res; | |
61 } | |
62 i++; | |
63 } | |
64 } | |
65 | |
66 static public void main(String[] args) { | |
67 int[] array = { 0, 1, 2, -3, 4, 5, -2, 7, 8, 9, -1 }; | |
68 for (int i = 0; i < 20000; i++) { | |
69 m1(array, 0); | |
70 } | |
71 array[0] = -1; | |
72 try { | |
73 m1(array, 0); | |
74 } catch(ArrayIndexOutOfBoundsException aioobe) {} | |
75 } | |
76 } |