author | alanb |
Tue, 03 May 2016 09:09:20 +0100 | |
changeset 37773 | e5b3e9732c3c |
parent 36508 | 5f9eee6b383b |
child 38207 | 2ed792c9481d |
permissions | -rw-r--r-- |
35086
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
1 |
/* |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
2 |
* Copyright (c) 2015, Oracle and/or its affiliates. All rights reserved. |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
3 |
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
4 |
* |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
5 |
* This code is free software; you can redistribute it and/or modify it |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
6 |
* under the terms of the GNU General Public License version 2 only, as |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
7 |
* published by the Free Software Foundation. |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
8 |
* |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
9 |
* This code is distributed in the hope that it will be useful, but WITHOUT |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
10 |
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
11 |
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
12 |
* version 2 for more details (a copy is included in the LICENSE file that |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
13 |
* accompanied this code). |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
14 |
* |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
15 |
* You should have received a copy of the GNU General Public License version |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
16 |
* 2 along with this work; if not, write to the Free Software Foundation, |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
17 |
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
18 |
* |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
20 |
* or visit www.oracle.com if you need additional information or have any |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
21 |
* questions. |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
22 |
*/ |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
23 |
|
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
24 |
/* |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
25 |
* @test |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
26 |
* @bug 8072008 |
36508 | 27 |
* @library /testlibrary /test/lib ../patches |
28 |
* @modules java.base/jdk.internal.vm.annotation |
|
29 |
* @build java.base/java.lang.invoke.MethodHandleHelper |
|
30 |
* @build sun.hotspot.WhiteBox |
|
37773
e5b3e9732c3c
8154956: Module system implementation refresh (4/2016)
alanb
parents:
36508
diff
changeset
|
31 |
* @run main/bootclasspath/othervm -XX:+IgnoreUnrecognizedVMOptions |
36508 | 32 |
* -XX:+UnlockDiagnosticVMOptions -XX:+WhiteBoxAPI |
33 |
* -Xbatch -XX:-TieredCompilation -XX:CICompilerCount=1 |
|
34 |
* -XX:+FoldStableValues |
|
35 |
* compiler.jsr292.NonInlinedCall.GCTest |
|
35086
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
36 |
*/ |
36508 | 37 |
|
38 |
package compiler.jsr292.NonInlinedCall; |
|
39 |
||
40 |
import java.lang.invoke.MethodHandleHelper; |
|
41 |
import java.lang.invoke.MethodHandleHelper.NonInlinedReinvoker; |
|
42 |
import java.lang.invoke.MethodHandles; |
|
43 |
import java.lang.invoke.MethodHandle; |
|
44 |
||
45 |
import java.lang.invoke.MethodType; |
|
46 |
import java.lang.ref.PhantomReference; |
|
47 |
import java.lang.ref.Reference; |
|
48 |
import java.lang.ref.ReferenceQueue; |
|
49 |
||
50 |
import jdk.internal.vm.annotation.DontInline; |
|
51 |
import jdk.internal.vm.annotation.Stable; |
|
35086
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
52 |
|
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
53 |
import sun.hotspot.WhiteBox; |
36508 | 54 |
|
35086
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
55 |
import static jdk.test.lib.Asserts.*; |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
56 |
|
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
57 |
public class GCTest { |
36508 | 58 |
static final MethodHandles.Lookup LOOKUP = MethodHandleHelper.IMPL_LOOKUP; |
35086
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
59 |
|
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
60 |
static class T { |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
61 |
static int f1() { return 0; } |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
62 |
static int f2() { return 1; } |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
63 |
} |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
64 |
|
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
65 |
static @Stable MethodHandle mh; |
36508 | 66 |
static PhantomReference<Object> lform; |
35086
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
67 |
|
36508 | 68 |
static final ReferenceQueue<Object> rq = new ReferenceQueue<>(); |
35086
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
69 |
static final WhiteBox WB = WhiteBox.getWhiteBox(); |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
70 |
|
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
71 |
@DontInline |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
72 |
static int invokeBasic() { |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
73 |
try { |
36508 | 74 |
return MethodHandleHelper.invokeBasicI(mh); |
35086
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
75 |
} catch (Throwable e) { |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
76 |
throw new Error(e); |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
77 |
} |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
78 |
} |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
79 |
|
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
80 |
static void test(int expected) { |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
81 |
for (int i = 0; i < 20_000; i++) { |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
82 |
invokeBasic(); |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
83 |
} |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
84 |
assertEquals(invokeBasic(), expected); |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
85 |
} |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
86 |
|
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
87 |
public static void main(String[] args) throws Exception { |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
88 |
mh = NonInlinedReinvoker.make( |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
89 |
LOOKUP.findStatic(T.class, "f1", MethodType.methodType(int.class))); |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
90 |
|
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
91 |
// Monitor LambdaForm GC |
36508 | 92 |
lform = new PhantomReference<>(MethodHandleHelper.getLambdaForm(mh), rq); |
35086
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
93 |
|
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
94 |
test(0); |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
95 |
WB.clearInlineCaches(); |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
96 |
test(0); |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
97 |
|
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
98 |
mh = NonInlinedReinvoker.make( |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
99 |
LOOKUP.findStatic(T.class, "f2", MethodType.methodType(int.class))); |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
100 |
|
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
101 |
Reference<?> ref = null; |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
102 |
while (ref == null) { |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
103 |
WB.fullGC(); |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
104 |
try { |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
105 |
ref = rq.remove(1000); |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
106 |
} catch (InterruptedException e) { /*ignore*/ } |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
107 |
} |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
108 |
|
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
109 |
test(1); |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
110 |
WB.clearInlineCaches(); |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
111 |
test(1); |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
112 |
|
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
113 |
System.out.println("TEST PASSED"); |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
114 |
} |
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
diff
changeset
|
115 |
} |