author | erikj |
Tue, 09 Oct 2018 14:17:50 -0700 | |
branch | ihse-runtestprebuilt-branch |
changeset 56945 | 35914fe36a9c |
parent 47216 | 71c04702a3d5 |
permissions | -rw-r--r-- |
42086
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
1 |
/* |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
2 |
* Copyright (c) 2016, SAP SE and/or its affiliates. All rights reserved. |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
3 |
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
4 |
* |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
5 |
* This code is free software; you can redistribute it and/or modify it |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
6 |
* under the terms of the GNU General Public License version 2 only, as |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
7 |
* published by the Free Software Foundation. |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
8 |
* |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
9 |
* This code is distributed in the hope that it will be useful, but WITHOUT |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
10 |
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
11 |
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
12 |
* version 2 for more details (a copy is included in the LICENSE file that |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
13 |
* accompanied this code). |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
14 |
* |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
15 |
* You should have received a copy of the GNU General Public License version |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
16 |
* 2 along with this work; if not, write to the Free Software Foundation, |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
17 |
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
18 |
* |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
20 |
* or visit www.oracle.com if you need additional information or have any |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
21 |
* questions. |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
22 |
*/ |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
23 |
|
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
24 |
/* |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
25 |
* @test |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
26 |
* @bug 8159611 |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
27 |
* @summary The elimination of System.arraycopy by EscapeAnalysis prevents |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
28 |
* an IndexOutOfBoundsException from being thrown if the arraycopy |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
29 |
* is called with a negative length argument. |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
30 |
* @modules java.base/jdk.internal.misc |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
31 |
* @library /testlibrary /test/lib |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
32 |
* @build sun.hotspot.WhiteBox |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
33 |
* @run driver ClassFileInstaller sun.hotspot.WhiteBox |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
34 |
* sun.hotspot.WhiteBox$WhiteBoxPermission |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
35 |
* |
56945
35914fe36a9c
Handle space in JTREG_KEYWORDS. Introduce TEST_OPTS_JAVA_OPTIONS. Set individual test timeouts to 300. Fix InstrumentationTest to handle deep work dir paths better.
erikj
parents:
47216
diff
changeset
|
36 |
* @run main/othervm/timeout=300 |
42086
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
37 |
* -Xbootclasspath/a:. |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
38 |
* -XX:+UnlockDiagnosticVMOptions |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
39 |
* -XX:+WhiteBoxAPI |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
40 |
* -XX:-UseOnStackReplacement |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
41 |
* compiler.escapeAnalysis.TestArrayCopy |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
42 |
* |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
43 |
* @author Volker Simonis |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
44 |
*/ |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
45 |
|
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
46 |
package compiler.escapeAnalysis; |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
47 |
|
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
48 |
import sun.hotspot.WhiteBox; |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
49 |
import java.lang.reflect.Method; |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
50 |
|
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
51 |
public class TestArrayCopy { |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
52 |
|
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
53 |
private static final WhiteBox WB = WhiteBox.getWhiteBox(); |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
54 |
// DST_LEN Must be const, otherwise EliminateAllocations won't work |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
55 |
static final int DST_LEN = 4; |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
56 |
static final int SRC_LEN = 8; |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
57 |
|
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
58 |
public static boolean do_test1(Object src, int src_pos, int dst_pos, int cpy_len) { |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
59 |
try { |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
60 |
System.arraycopy(src, src_pos, new Object[DST_LEN], dst_pos, cpy_len); |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
61 |
return false; |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
62 |
} catch (IndexOutOfBoundsException e) { |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
63 |
return true; |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
64 |
} |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
65 |
} |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
66 |
|
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
67 |
public static int do_test2(Object src, int src_pos, int dst_pos, int cpy_len) { |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
68 |
try { |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
69 |
System.arraycopy(src, src_pos, new Object[DST_LEN], dst_pos, cpy_len); |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
70 |
return 0; |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
71 |
} catch (IndexOutOfBoundsException e) { |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
72 |
return 1; |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
73 |
} catch (ArrayStoreException e) { |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
74 |
return 2; |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
75 |
} |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
76 |
} |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
77 |
|
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
78 |
static final int COUNT = 100_000; |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
79 |
static final int[] src_pos = { 0, -1, -1, 0, 0, 0, 1, 1, 1, 1, 1 }; |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
80 |
static final int[] dst_pos = { 0, -1, 0, -1, 0, 1, 0, 1, 1, 1, 1 }; |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
81 |
static final int[] cpy_len = { 0, 0, 0, 0, -1, -1, -1, -1, 8, 4, 2 }; |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
82 |
|
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
83 |
public static void main(String args[]) throws Exception { |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
84 |
int length = args.length > 0 ? Integer.parseInt(args[0]) : -1; |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
85 |
int[] int_arr = new int[SRC_LEN]; |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
86 |
Object[] obj_arr = new Object[SRC_LEN]; |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
87 |
|
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
88 |
Method test1 = TestArrayCopy.class.getMethod("do_test1", Object.class, int.class, int.class, int.class); |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
89 |
Method test2 = TestArrayCopy.class.getMethod("do_test2", Object.class, int.class, int.class, int.class); |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
90 |
|
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
91 |
for (int i = 0; i < src_pos.length; i++) { |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
92 |
int sp = src_pos[i]; |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
93 |
int dp = dst_pos[i]; |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
94 |
int cl = cpy_len[i]; |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
95 |
String version1 = String.format("System.arraycopy(Object[8], %d, new Object[%d], %d, %d)", sp, DST_LEN, dp, cl); |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
96 |
String version2 = String.format("System.arraycopy(int[8], %d, new Object[%d], %d, %d)", sp, DST_LEN, dp, cl); |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
97 |
System.out.format("Testing " + version1 + "\nand " + version2).flush(); |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
98 |
for (int x = 0; x < COUNT; x++) { |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
99 |
if (!do_test1(obj_arr, sp, dp, cl) && |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
100 |
(sp < 0 || dp < 0 || cl < 0 || (sp + cl >= SRC_LEN) || (dp + cl >= DST_LEN))) { |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
101 |
throw new RuntimeException("Expected IndexOutOfBoundsException for " + version1); |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
102 |
} |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
103 |
int res = do_test2(int_arr, sp, dp, cl); |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
104 |
if (res == 0 || res == 1) { |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
105 |
throw new RuntimeException("Expected ArrayStoreException for " + version2); |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
106 |
} |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
107 |
} |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
108 |
WB.deoptimizeMethod(test1); |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
109 |
WB.clearMethodState(test1); |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
110 |
WB.deoptimizeMethod(test2); |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
111 |
WB.clearMethodState(test2); |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
112 |
} |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
113 |
|
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
114 |
} |
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
diff
changeset
|
115 |
} |