comparison test/compiler/8007294/Test8007294.java @ 8116:6931f425c517

8007294: ReduceFieldZeroing doesn't check for dependent load and can lead to incorrect execution Summary: InitializeNode::can_capture_store() must check that the captured store doesn't overwrite a memory location that is loaded before the store. Reviewed-by: kvn
author roland
date Mon, 25 Feb 2013 14:13:04 +0100
parents
children
comparison
equal deleted inserted replaced
8097:be1fbee20765 8116:6931f425c517
1 /*
2 * Copyright (c) 2013, 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 8007294
27 * @summary ReduceFieldZeroing doesn't check for dependent load and can lead to incorrect execution
28 * @run main/othervm -XX:+IgnoreUnrecognizedVMOptions -XX:+AlwaysIncrementalInline -XX:-UseOnStackReplacement -XX:-BackgroundCompilation Test8007294
29 *
30 */
31
32 public class Test8007294 {
33
34 int i1;
35 int i2;
36
37 Test8007294(int i1, int i2) {
38 this.i1 = i1;
39 this.i2 = i2;
40 }
41
42 static int m(int v) {
43 return v;
44 }
45
46 static Test8007294 test1() {
47 Test8007294 obj = new Test8007294(10, 100);
48 int v1 = obj.i1;
49
50 int v3 = m(v1);
51 int v2 = obj.i2;
52 obj.i2 = v3;
53 obj.i1 = v2;
54
55 return obj;
56 }
57
58 static int test2(int i) {
59 int j = 0;
60 if (i > 0) {
61 j = 1;
62 }
63
64 int[] arr = new int[10];
65 arr[0] = 1;
66 arr[1] = 2;
67 int v1 = arr[j];
68 arr[0] = 3;
69 arr[1] = 4;
70
71 return v1;
72 }
73
74 static public void main(String[] args) {
75 boolean failed = false;
76 for (int i = 0; i < 20000; i++) {
77 Test8007294 obj = test1();
78 if (obj.i1 != 100 || obj.i2 != 10) {
79 System.out.println("FAILED test1 obj.i1 = " + obj.i1 +", obj.i2 = " + obj.i2);
80 failed = true;
81 break;
82 }
83 }
84 for (int i = 0; i < 20000; i++) {
85 int res = test2(1);
86 if (res != 2) {
87 System.out.println("FAILED test2 = " + res);
88 failed = true;
89 break;
90 }
91 }
92 if (failed) {
93 System.exit(97);
94 } else {
95 System.out.println("PASSED");
96 }
97 }
98 }