author | jcm |
Fri, 06 May 2016 05:57:43 -0700 | |
changeset 38243 | f6cedfdd4721 |
parent 38188 | e76af590ea2f |
child 38245 | c6c2ac29463b |
permissions | -rw-r--r-- |
31857
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
1 |
/* |
36508 | 2 |
* Copyright (c) 2015, 2016, Oracle and/or its affiliates. All rights reserved. |
31857
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
3 |
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
4 |
* |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
5 |
* This code is free software; you can redistribute it and/or modify it |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
6 |
* under the terms of the GNU General Public License version 2 only, as |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
7 |
* published by the Free Software Foundation. Oracle designates this |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
8 |
* particular file as subject to the "Classpath" exception as provided |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
9 |
* by Oracle in the LICENSE file that accompanied this code. |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
10 |
* |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
11 |
* This code is distributed in the hope that it will be useful, but WITHOUT |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
12 |
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
13 |
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
14 |
* version 2 for more details (a copy is included in the LICENSE file that |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
15 |
* accompanied this code). |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
16 |
* |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
17 |
* You should have received a copy of the GNU General Public License version |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
18 |
* 2 along with this work; if not, write to the Free Software Foundation, |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
19 |
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
20 |
* |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
21 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
22 |
* or visit www.oracle.com if you need additional information or have any |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
23 |
* questions. |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
24 |
*/ |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
25 |
|
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
26 |
/* |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
27 |
* @test |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
28 |
* @summary tests on constant folding of unsafe get operations |
36508 | 29 |
* @library /testlibrary |
36337
d4b2f60ff5a9
8150543: Mismatched access detection is inaccurate
vlivanov
parents:
35099
diff
changeset
|
30 |
* |
d4b2f60ff5a9
8150543: Mismatched access detection is inaccurate
vlivanov
parents:
35099
diff
changeset
|
31 |
* @requires vm.flavor != "client" |
d4b2f60ff5a9
8150543: Mismatched access detection is inaccurate
vlivanov
parents:
35099
diff
changeset
|
32 |
* |
36508 | 33 |
* @modules java.base/jdk.internal.org.objectweb.asm |
34 |
* java.base/jdk.internal.vm.annotation |
|
35 |
* java.base/jdk.internal.misc |
|
38030
93f24e7b3c43
8152590: C2: @Stable support doesn't always work w/ incremental inlining
vlivanov
parents:
36838
diff
changeset
|
36 |
* |
38188
e76af590ea2f
8155751: Some tests miss othervm for main/bootclasspath mode
vlivanov
parents:
38030
diff
changeset
|
37 |
* @run main/bootclasspath/othervm -XX:+UnlockDiagnosticVMOptions |
e76af590ea2f
8155751: Some tests miss othervm for main/bootclasspath mode
vlivanov
parents:
38030
diff
changeset
|
38 |
* -Xbatch -XX:-TieredCompilation |
e76af590ea2f
8155751: Some tests miss othervm for main/bootclasspath mode
vlivanov
parents:
38030
diff
changeset
|
39 |
* -XX:+FoldStableValues |
e76af590ea2f
8155751: Some tests miss othervm for main/bootclasspath mode
vlivanov
parents:
38030
diff
changeset
|
40 |
* -XX:CompileCommand=dontinline,UnsafeGetConstantField.checkGetAddress() |
e76af590ea2f
8155751: Some tests miss othervm for main/bootclasspath mode
vlivanov
parents:
38030
diff
changeset
|
41 |
* -XX:CompileCommand=dontinline,*.test* |
e76af590ea2f
8155751: Some tests miss othervm for main/bootclasspath mode
vlivanov
parents:
38030
diff
changeset
|
42 |
* -XX:+UseUnalignedAccesses |
e76af590ea2f
8155751: Some tests miss othervm for main/bootclasspath mode
vlivanov
parents:
38030
diff
changeset
|
43 |
* compiler.unsafe.UnsafeGetConstantField |
36508 | 44 |
* |
38188
e76af590ea2f
8155751: Some tests miss othervm for main/bootclasspath mode
vlivanov
parents:
38030
diff
changeset
|
45 |
* @run main/bootclasspath/othervm -XX:+UnlockDiagnosticVMOptions |
e76af590ea2f
8155751: Some tests miss othervm for main/bootclasspath mode
vlivanov
parents:
38030
diff
changeset
|
46 |
* -Xbatch -XX:-TieredCompilation |
e76af590ea2f
8155751: Some tests miss othervm for main/bootclasspath mode
vlivanov
parents:
38030
diff
changeset
|
47 |
* -XX:+FoldStableValues |
e76af590ea2f
8155751: Some tests miss othervm for main/bootclasspath mode
vlivanov
parents:
38030
diff
changeset
|
48 |
* -XX:CompileCommand=dontinline,UnsafeGetConstantField.checkGetAddress() |
e76af590ea2f
8155751: Some tests miss othervm for main/bootclasspath mode
vlivanov
parents:
38030
diff
changeset
|
49 |
* -XX:CompileCommand=dontinline,*.test* |
e76af590ea2f
8155751: Some tests miss othervm for main/bootclasspath mode
vlivanov
parents:
38030
diff
changeset
|
50 |
* -XX:CompileCommand=inline,*Unsafe.get* |
e76af590ea2f
8155751: Some tests miss othervm for main/bootclasspath mode
vlivanov
parents:
38030
diff
changeset
|
51 |
* -XX:-UseUnalignedAccesses |
e76af590ea2f
8155751: Some tests miss othervm for main/bootclasspath mode
vlivanov
parents:
38030
diff
changeset
|
52 |
* compiler.unsafe.UnsafeGetConstantField |
31857
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
53 |
*/ |
36508 | 54 |
package compiler.unsafe; |
31857
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
55 |
|
35099
982950884444
8144223: Move j.l.invoke.{ForceInline, DontInline, Stable} to jdk.internal.vm.annotation package
psandoz
parents:
33730
diff
changeset
|
56 |
import jdk.internal.org.objectweb.asm.ClassWriter; |
982950884444
8144223: Move j.l.invoke.{ForceInline, DontInline, Stable} to jdk.internal.vm.annotation package
psandoz
parents:
33730
diff
changeset
|
57 |
import jdk.internal.org.objectweb.asm.FieldVisitor; |
982950884444
8144223: Move j.l.invoke.{ForceInline, DontInline, Stable} to jdk.internal.vm.annotation package
psandoz
parents:
33730
diff
changeset
|
58 |
import jdk.internal.org.objectweb.asm.MethodVisitor; |
982950884444
8144223: Move j.l.invoke.{ForceInline, DontInline, Stable} to jdk.internal.vm.annotation package
psandoz
parents:
33730
diff
changeset
|
59 |
import jdk.internal.org.objectweb.asm.Opcodes; |
982950884444
8144223: Move j.l.invoke.{ForceInline, DontInline, Stable} to jdk.internal.vm.annotation package
psandoz
parents:
33730
diff
changeset
|
60 |
import jdk.internal.org.objectweb.asm.Type; |
36508 | 61 |
import jdk.internal.vm.annotation.Stable; |
31857
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
62 |
import jdk.test.lib.Asserts; |
36617
390e8993b88e
8152483: Fix a couple of tests that are being incorrectly run on C1 after jigsaw M3
amurillo
parents:
36508
diff
changeset
|
63 |
import jdk.test.lib.Platform; |
36508 | 64 |
import jdk.internal.misc.Unsafe; |
65 |
||
66 |
import java.io.IOException; |
|
67 |
import java.nio.file.Files; |
|
68 |
import java.nio.file.Path; |
|
69 |
import java.nio.file.Paths; |
|
35099
982950884444
8144223: Move j.l.invoke.{ForceInline, DontInline, Stable} to jdk.internal.vm.annotation package
psandoz
parents:
33730
diff
changeset
|
70 |
|
31857
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
71 |
import static jdk.internal.org.objectweb.asm.Opcodes.*; |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
72 |
|
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
73 |
public class UnsafeGetConstantField { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
74 |
static final Class<?> THIS_CLASS = UnsafeGetConstantField.class; |
33606 | 75 |
static final Unsafe U = Unsafe.getUnsafe(); |
31857
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
76 |
|
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
77 |
public static void main(String[] args) { |
36617
390e8993b88e
8152483: Fix a couple of tests that are being incorrectly run on C1 after jigsaw M3
amurillo
parents:
36508
diff
changeset
|
78 |
if (Platform.isServer()) { |
390e8993b88e
8152483: Fix a couple of tests that are being incorrectly run on C1 after jigsaw M3
amurillo
parents:
36508
diff
changeset
|
79 |
testUnsafeGetAddress(); |
390e8993b88e
8152483: Fix a couple of tests that are being incorrectly run on C1 after jigsaw M3
amurillo
parents:
36508
diff
changeset
|
80 |
testUnsafeGetField(); |
390e8993b88e
8152483: Fix a couple of tests that are being incorrectly run on C1 after jigsaw M3
amurillo
parents:
36508
diff
changeset
|
81 |
testUnsafeGetFieldUnaligned(); |
390e8993b88e
8152483: Fix a couple of tests that are being incorrectly run on C1 after jigsaw M3
amurillo
parents:
36508
diff
changeset
|
82 |
} |
31857
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
83 |
System.out.println("TEST PASSED"); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
84 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
85 |
|
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
86 |
static final long nativeAddr = U.allocateMemory(16); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
87 |
static void testUnsafeGetAddress() { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
88 |
long cookie = 0x12345678L; |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
89 |
U.putAddress(nativeAddr, cookie); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
90 |
for (int i = 0; i < 20_000; i++) { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
91 |
Asserts.assertEquals(checkGetAddress(), cookie); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
92 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
93 |
} |
36508 | 94 |
|
31857
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
95 |
static long checkGetAddress() { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
96 |
return U.getAddress(nativeAddr); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
97 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
98 |
|
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
99 |
static void testUnsafeGetField() { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
100 |
int[] testedFlags = new int[] { 0, ACC_STATIC, ACC_FINAL, (ACC_STATIC | ACC_FINAL) }; |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
101 |
boolean[] boolValues = new boolean[] { false, true }; |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
102 |
String[] modes = new String[] { "", "Volatile" }; |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
103 |
|
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
104 |
for (JavaType t : JavaType.values()) { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
105 |
for (int flags : testedFlags) { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
106 |
for (boolean stable : boolValues) { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
107 |
for (boolean hasDefaultValue : boolValues) { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
108 |
for (String suffix : modes) { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
109 |
runTest(t, flags, stable, hasDefaultValue, suffix); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
110 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
111 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
112 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
113 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
114 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
115 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
116 |
|
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
117 |
static void testUnsafeGetFieldUnaligned() { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
118 |
JavaType[] types = new JavaType[] { JavaType.S, JavaType.C, JavaType.I, JavaType.J }; |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
119 |
int[] testedFlags = new int[] { 0, ACC_STATIC, ACC_FINAL, (ACC_STATIC | ACC_FINAL) }; |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
120 |
boolean[] boolValues = new boolean[] { false, true }; |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
121 |
|
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
122 |
for (JavaType t : types) { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
123 |
for (int flags : testedFlags) { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
124 |
for (boolean stable : boolValues) { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
125 |
for (boolean hasDefaultValue : boolValues) { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
126 |
runTest(t, flags, stable, hasDefaultValue, "Unaligned"); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
127 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
128 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
129 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
130 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
131 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
132 |
|
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
133 |
static void runTest(JavaType t, int flags, boolean stable, boolean hasDefaultValue, String postfix) { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
134 |
Generator g = new Generator(t, flags, stable, hasDefaultValue, postfix); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
135 |
Test test = g.generate(); |
36508 | 136 |
System.err.printf("type=%s flags=%d stable=%b default=%b post=%s\n", |
31857
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
137 |
t.typeName, flags, stable, hasDefaultValue, postfix); |
36508 | 138 |
try { |
139 |
Object expected = hasDefaultValue ? t.defaultValue : t.value; |
|
140 |
// Trigger compilation |
|
141 |
for (int i = 0; i < 20_000; i++) { |
|
142 |
Asserts.assertEQ(expected, test.testDirect(), "i = "+ i +" direct read returns wrong value"); |
|
143 |
Asserts.assertEQ(expected, test.testUnsafe(), "i = "+ i +" unsafe read returns wrong value"); |
|
144 |
} |
|
145 |
||
146 |
test.changeToDefault(); |
|
147 |
if (!hasDefaultValue && (stable || g.isFinal())) { |
|
148 |
Asserts.assertEQ(t.value, test.testDirect(), |
|
149 |
"direct read doesn't return prev value"); |
|
38243
f6cedfdd4721
8156124: Update compiler/unsafe/UnsafeGetConstantField after JDK-8148518 is fixed
jcm
parents:
38188
diff
changeset
|
150 |
Asserts.assertEQ(test.testDirect(), test.testUnsafe()); |
36508 | 151 |
} else { |
152 |
Asserts.assertEQ(t.defaultValue, test.testDirect(), |
|
153 |
"direct read doesn't return default value"); |
|
154 |
Asserts.assertEQ(test.testDirect(), test.testUnsafe(), |
|
155 |
"direct and unsafe reads return different values"); |
|
156 |
} |
|
157 |
} catch (Throwable e) { |
|
158 |
try { |
|
159 |
g.dump(); |
|
160 |
} catch (IOException io) { |
|
161 |
io.printStackTrace(); |
|
162 |
} |
|
163 |
throw e; |
|
31857
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
164 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
165 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
166 |
|
36508 | 167 |
public interface Test { |
31857
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
168 |
Object testDirect(); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
169 |
Object testUnsafe(); |
36508 | 170 |
void changeToDefault(); |
31857
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
171 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
172 |
|
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
173 |
enum JavaType { |
36508 | 174 |
Z("Boolean", true, false), |
175 |
B("Byte", new Byte((byte) -1), new Byte((byte) 0)), |
|
176 |
S("Short", new Short((short) -1), new Short((short) 0)), |
|
177 |
C("Char", Character.MAX_VALUE, '\0'), |
|
178 |
I("Int", -1, 0), |
|
179 |
J("Long", -1L, 0L), |
|
180 |
F("Float", -1F, 0F), |
|
181 |
D("Double", -1D, 0D), |
|
182 |
L("Object", "", null); |
|
31857
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
183 |
|
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
184 |
String typeName; |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
185 |
Object value; |
36508 | 186 |
Object defaultValue; |
31857
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
187 |
String wrapper; |
36508 | 188 |
JavaType(String name, Object value, Object defaultValue) { |
31857
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
189 |
this.typeName = name; |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
190 |
this.value = value; |
36508 | 191 |
this.defaultValue = defaultValue; |
31857
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
192 |
this.wrapper = internalName(value.getClass()); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
193 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
194 |
|
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
195 |
String desc() { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
196 |
if (this == JavaType.L) { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
197 |
return "Ljava/lang/Object;"; |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
198 |
} else { |
36508 | 199 |
return name(); |
31857
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
200 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
201 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
202 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
203 |
|
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
204 |
static String internalName(Class cls) { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
205 |
return cls.getName().replace('.', '/'); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
206 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
207 |
static String descriptor(Class cls) { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
208 |
return String.format("L%s;", internalName(cls)); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
209 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
210 |
|
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
211 |
/** |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
212 |
* Sample generated class: |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
213 |
* static class T1 implements Test { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
214 |
* final int f = -1; |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
215 |
* static final long FIELD_OFFSET; |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
216 |
* static final T1 t = new T1(); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
217 |
* static { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
218 |
* FIELD_OFFSET = U.objectFieldOffset(T1.class.getDeclaredField("f")); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
219 |
* } |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
220 |
* public Object testDirect() { return t.f; } |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
221 |
* public Object testUnsafe() { return U.getInt(t, FIELD_OFFSET); } |
36508 | 222 |
* public void changeToDefault() { U.putInt(t, 0, FIELD_OFFSET); } |
31857
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
223 |
* } |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
224 |
*/ |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
225 |
static class Generator { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
226 |
static final String FIELD_NAME = "f"; |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
227 |
static final String UNSAFE_NAME = internalName(Unsafe.class); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
228 |
static final String UNSAFE_DESC = descriptor(Unsafe.class); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
229 |
|
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
230 |
final JavaType type; |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
231 |
final int flags; |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
232 |
final boolean stable; |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
233 |
final boolean hasDefaultValue; |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
234 |
final String nameSuffix; |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
235 |
|
36508 | 236 |
final String name; |
31857
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
237 |
final String className; |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
238 |
final String classDesc; |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
239 |
final String fieldDesc; |
36508 | 240 |
final byte[] classFile; |
31857
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
241 |
|
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
242 |
Generator(JavaType t, int flags, boolean stable, boolean hasDefaultValue, String suffix) { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
243 |
this.type = t; |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
244 |
this.flags = flags; |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
245 |
this.stable = stable; |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
246 |
this.hasDefaultValue = hasDefaultValue; |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
247 |
this.nameSuffix = suffix; |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
248 |
|
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
249 |
fieldDesc = type.desc(); |
36508 | 250 |
name = String.format("Test%s%s__f=%d__s=%b__d=%b", |
251 |
type.typeName, suffix, flags, stable, hasDefaultValue); |
|
252 |
className = "java/lang/invoke/" + name; |
|
31857
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
253 |
classDesc = String.format("L%s;", className); |
36508 | 254 |
classFile = generateClassFile(); |
31857
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
255 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
256 |
|
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
257 |
byte[] generateClassFile() { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
258 |
ClassWriter cw = new ClassWriter(ClassWriter.COMPUTE_MAXS | ClassWriter.COMPUTE_FRAMES); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
259 |
cw.visit(Opcodes.V1_8, Opcodes.ACC_PUBLIC | Opcodes.ACC_SUPER, className, null, "java/lang/Object", |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
260 |
new String[]{ internalName(Test.class) }); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
261 |
|
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
262 |
// Declare fields |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
263 |
cw.visitField(ACC_FINAL | ACC_STATIC, "t", classDesc, null, null).visitEnd(); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
264 |
cw.visitField(ACC_FINAL | ACC_STATIC, "FIELD_OFFSET", "J", null, null).visitEnd(); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
265 |
cw.visitField(ACC_FINAL | ACC_STATIC, "U", UNSAFE_DESC, null, null).visitEnd(); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
266 |
if (isStatic()) { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
267 |
cw.visitField(ACC_FINAL | ACC_STATIC, "STATIC_BASE", "Ljava/lang/Object;", null, null).visitEnd(); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
268 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
269 |
|
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
270 |
FieldVisitor fv = cw.visitField(flags, FIELD_NAME, fieldDesc, null, null); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
271 |
if (stable) { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
272 |
fv.visitAnnotation(descriptor(Stable.class), true); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
273 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
274 |
fv.visitEnd(); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
275 |
|
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
276 |
// Methods |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
277 |
{ // <init> |
36508 | 278 |
MethodVisitor mv = cw.visitMethod(ACC_PUBLIC, "<init>", "()V", null, null); |
31857
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
279 |
mv.visitCode(); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
280 |
|
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
281 |
mv.visitVarInsn(ALOAD, 0); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
282 |
mv.visitMethodInsn(INVOKESPECIAL, "java/lang/Object", "<init>", "()V", false); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
283 |
if (!isStatic()) { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
284 |
initField(mv); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
285 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
286 |
mv.visitInsn(RETURN); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
287 |
|
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
288 |
mv.visitMaxs(0, 0); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
289 |
mv.visitEnd(); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
290 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
291 |
|
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
292 |
{ // public Object testDirect() { return t.f; } |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
293 |
MethodVisitor mv = cw.visitMethod(ACC_PUBLIC, "testDirect", "()Ljava/lang/Object;", null, null); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
294 |
mv.visitCode(); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
295 |
|
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
296 |
getFieldValue(mv); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
297 |
wrapResult(mv); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
298 |
mv.visitInsn(ARETURN); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
299 |
|
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
300 |
mv.visitMaxs(0, 0); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
301 |
mv.visitEnd(); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
302 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
303 |
|
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
304 |
{ // public Object testUnsafe() { return U.getInt(t, FIELD_OFFSET); } |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
305 |
MethodVisitor mv = cw.visitMethod(ACC_PUBLIC, "testUnsafe", "()Ljava/lang/Object;", null, null); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
306 |
mv.visitCode(); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
307 |
|
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
308 |
getFieldValueUnsafe(mv); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
309 |
wrapResult(mv); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
310 |
mv.visitInsn(ARETURN); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
311 |
|
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
312 |
mv.visitMaxs(0, 0); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
313 |
mv.visitEnd(); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
314 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
315 |
|
36508 | 316 |
{ // public void changeToDefault() { U.putInt(t, FIELD_OFFSET, 0); } |
317 |
MethodVisitor mv = cw.visitMethod(ACC_PUBLIC, "changeToDefault", "()V", null, null); |
|
318 |
mv.visitCode(); |
|
319 |
getUnsafe(mv); |
|
320 |
if (isStatic()) { |
|
321 |
mv.visitFieldInsn(GETSTATIC, className, "STATIC_BASE", "Ljava/lang/Object;"); |
|
322 |
} else { |
|
323 |
mv.visitFieldInsn(GETSTATIC, className, "t", classDesc); |
|
324 |
} |
|
325 |
mv.visitFieldInsn(GETSTATIC, className, "FIELD_OFFSET", "J"); |
|
326 |
||
327 |
if (type.defaultValue != null) { |
|
328 |
mv.visitLdcInsn(type.defaultValue); |
|
329 |
} else { |
|
330 |
mv.visitInsn(ACONST_NULL); |
|
331 |
} |
|
332 |
String name = "put" + type.typeName + nameSuffix; |
|
333 |
mv.visitMethodInsn(INVOKEVIRTUAL, UNSAFE_NAME, name, "(Ljava/lang/Object;J" + type.desc()+ ")V", false); |
|
334 |
mv.visitInsn(RETURN); |
|
335 |
||
336 |
mv.visitMaxs(0, 0); |
|
337 |
mv.visitEnd(); |
|
338 |
} |
|
339 |
||
31857
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
340 |
{ // <clinit> |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
341 |
MethodVisitor mv = cw.visitMethod(ACC_STATIC, "<clinit>", "()V", null, null); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
342 |
mv.visitCode(); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
343 |
|
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
344 |
// Cache Unsafe instance |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
345 |
mv.visitMethodInsn(INVOKESTATIC, UNSAFE_NAME, "getUnsafe", "()"+UNSAFE_DESC, false); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
346 |
mv.visitFieldInsn(PUTSTATIC, className, "U", UNSAFE_DESC); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
347 |
|
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
348 |
// Create test object instance |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
349 |
mv.visitTypeInsn(NEW, className); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
350 |
mv.visitInsn(DUP); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
351 |
mv.visitMethodInsn(INVOKESPECIAL, className, "<init>", "()V", false); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
352 |
mv.visitFieldInsn(PUTSTATIC, className, "t", classDesc); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
353 |
|
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
354 |
// Compute field offset |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
355 |
getUnsafe(mv); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
356 |
getField(mv); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
357 |
mv.visitMethodInsn(INVOKEVIRTUAL, UNSAFE_NAME, (isStatic() ? "staticFieldOffset" : "objectFieldOffset"), |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
358 |
"(Ljava/lang/reflect/Field;)J", false); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
359 |
mv.visitFieldInsn(PUTSTATIC, className, "FIELD_OFFSET", "J"); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
360 |
|
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
361 |
// Compute base offset for static field |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
362 |
if (isStatic()) { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
363 |
getUnsafe(mv); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
364 |
getField(mv); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
365 |
mv.visitMethodInsn(INVOKEVIRTUAL, UNSAFE_NAME, "staticFieldBase", "(Ljava/lang/reflect/Field;)Ljava/lang/Object;", false); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
366 |
mv.visitFieldInsn(PUTSTATIC, className, "STATIC_BASE", "Ljava/lang/Object;"); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
367 |
initField(mv); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
368 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
369 |
|
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
370 |
mv.visitInsn(RETURN); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
371 |
mv.visitMaxs(0, 0); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
372 |
mv.visitEnd(); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
373 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
374 |
|
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
375 |
return cw.toByteArray(); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
376 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
377 |
|
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
378 |
Test generate() { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
379 |
Class<?> c = U.defineClass(className, classFile, 0, classFile.length, THIS_CLASS.getClassLoader(), null); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
380 |
try { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
381 |
return (Test) c.newInstance(); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
382 |
} catch(Exception e) { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
383 |
throw new Error(e); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
384 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
385 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
386 |
|
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
387 |
boolean isStatic() { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
388 |
return (flags & ACC_STATIC) > 0; |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
389 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
390 |
boolean isFinal() { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
391 |
return (flags & ACC_FINAL) > 0; |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
392 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
393 |
void getUnsafe(MethodVisitor mv) { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
394 |
mv.visitFieldInsn(GETSTATIC, className, "U", UNSAFE_DESC); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
395 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
396 |
void getField(MethodVisitor mv) { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
397 |
mv.visitLdcInsn(Type.getType(classDesc)); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
398 |
mv.visitLdcInsn(FIELD_NAME); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
399 |
mv.visitMethodInsn(INVOKEVIRTUAL, "java/lang/Class", "getDeclaredField", "(Ljava/lang/String;)Ljava/lang/reflect/Field;", false); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
400 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
401 |
void getFieldValue(MethodVisitor mv) { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
402 |
if (isStatic()) { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
403 |
mv.visitFieldInsn(GETSTATIC, className, FIELD_NAME, fieldDesc); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
404 |
} else { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
405 |
mv.visitFieldInsn(GETSTATIC, className, "t", classDesc); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
406 |
mv.visitFieldInsn(GETFIELD, className, FIELD_NAME, fieldDesc); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
407 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
408 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
409 |
void getFieldValueUnsafe(MethodVisitor mv) { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
410 |
getUnsafe(mv); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
411 |
if (isStatic()) { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
412 |
mv.visitFieldInsn(GETSTATIC, className, "STATIC_BASE", "Ljava/lang/Object;"); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
413 |
} else { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
414 |
mv.visitFieldInsn(GETSTATIC, className, "t", classDesc); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
415 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
416 |
mv.visitFieldInsn(GETSTATIC, className, "FIELD_OFFSET", "J"); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
417 |
String name = "get" + type.typeName + nameSuffix; |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
418 |
mv.visitMethodInsn(INVOKEVIRTUAL, UNSAFE_NAME, name, "(Ljava/lang/Object;J)" + type.desc(), false); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
419 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
420 |
void wrapResult(MethodVisitor mv) { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
421 |
if (type != JavaType.L) { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
422 |
String desc = String.format("(%s)L%s;", type.desc(), type.wrapper); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
423 |
mv.visitMethodInsn(INVOKESTATIC, type.wrapper, "valueOf", desc, false); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
424 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
425 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
426 |
void initField(MethodVisitor mv) { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
427 |
if (hasDefaultValue) { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
428 |
return; // Nothing to do |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
429 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
430 |
if (!isStatic()) { |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
431 |
mv.visitVarInsn(ALOAD, 0); |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
432 |
} |
36508 | 433 |
mv.visitLdcInsn(type.value); |
434 |
mv.visitFieldInsn((isStatic() ? PUTSTATIC : PUTFIELD), className, FIELD_NAME, fieldDesc); |
|
435 |
} |
|
31857
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
436 |
|
36508 | 437 |
public void dump() throws IOException { |
438 |
Path path = Paths.get(".", name + ".class").toAbsolutePath(); |
|
439 |
System.err.println("dumping test class to " + path); |
|
440 |
Files.write(path, classFile); |
|
31857
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
441 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
442 |
} |
adbf29d9ca43
8078629: VM should constant fold Unsafe.get*() loads from final fields
vlivanov
parents:
diff
changeset
|
443 |
} |