Mercurial > hg > graal-jvmci-8
annotate graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/CompilationTask.java @ 14543:12eaf1a47a90
removed ResolvedJavaMethod.getCompilerStorage(); moved last compiled graph node count into profiling info (i.e. MethodData metadata)
author | Doug Simon <doug.simon@oracle.com> |
---|---|
date | Fri, 14 Mar 2014 22:05:50 +0100 |
parents | cd22c8dbda4f |
children | 4877b0cb446f |
rev | line source |
---|---|
5136
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
1 /* |
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
2 * Copyright (c) 2012, Oracle and/or its affiliates. All rights reserved. |
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
4 * |
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
5 * This code is free software; you can redistribute it and/or modify it |
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
6 * under the terms of the GNU General Public License version 2 only, as |
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
7 * published by the Free Software Foundation. |
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
8 * |
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
9 * This code is distributed in the hope that it will be useful, but WITHOUT |
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
12 * version 2 for more details (a copy is included in the LICENSE file that |
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
13 * accompanied this code). |
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
14 * |
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
15 * You should have received a copy of the GNU General Public License version |
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
16 * 2 along with this work; if not, write to the Free Software Foundation, |
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
18 * |
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
20 * or visit www.oracle.com if you need additional information or have any |
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
21 * questions. |
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
22 */ |
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
23 package com.oracle.graal.hotspot; |
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
24 |
13912
766de6735435
Setup the OSR calling convention before calling compileGraph rather than patching it in the LIRGenerator
Gilles Duboscq <duboscq@ssw.jku.at>
parents:
13859
diff
changeset
|
25 import static com.oracle.graal.api.code.CallingConvention.Type.*; |
9612
66db0353f55a
compilation is explicitly given a CallingConvention for the graph being compiled instead of deriving it from a method
Doug Simon <doug.simon@oracle.com>
parents:
9041
diff
changeset
|
26 import static com.oracle.graal.api.code.CodeUtil.*; |
13239 | 27 import static com.oracle.graal.compiler.GraalCompiler.*; |
13464
2236d18302e0
made -G:PrintCompRate incompatible with -XX:+CITime and -XX:+CITimeEach
Doug Simon <doug.simon@oracle.com>
parents:
13452
diff
changeset
|
28 import static com.oracle.graal.hotspot.bridge.VMToCompilerImpl.*; |
8636
ac4dbfecec8f
passed a Replacements object to inlining utility methods that need one instead of the GraalRuntime API
Doug Simon <doug.simon@oracle.com>
parents:
8627
diff
changeset
|
29 import static com.oracle.graal.nodes.StructuredGraph.*; |
9864
063a712fe8d8
converted remaining options in GraalOptions to new system (GRAAL-27)
Doug Simon <doug.simon@oracle.com>
parents:
9849
diff
changeset
|
30 import static com.oracle.graal.phases.GraalOptions.*; |
12779
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12614
diff
changeset
|
31 import static com.oracle.graal.phases.common.InliningUtil.*; |
7361
b79ad92d5a26
compile the intrinsic graph for a method if the method is scheduled for compilation (in addition to intrinsifiying it when it is called)
Doug Simon <doug.simon@oracle.com>
parents:
6963
diff
changeset
|
32 |
14027
aba77882e314
be more careful with clinit of CompilationTask
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
14016
diff
changeset
|
33 import java.io.*; |
9813
c010c12ab682
put disassembling of installed code under the control of Debug.log(); use -G:Log=CodeInstall to show disassembly for all installed code
Doug Simon <doug.simon@oracle.com>
parents:
9807
diff
changeset
|
34 import java.lang.reflect.*; |
14172
cd22c8dbda4f
Made graph caching compilation local
Doug Simon <doug.simon@oracle.com>
parents:
14163
diff
changeset
|
35 import java.util.*; |
5136
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
36 import java.util.concurrent.*; |
10056
a323a9e20f9d
Fixed a few race conditions in the compilation queue.
Christian Haeubl <haeubl@ssw.jku.at>
parents:
10019
diff
changeset
|
37 import java.util.concurrent.atomic.*; |
5136
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
38 |
5510
426c605c9d3c
Move cri.ci to api.code.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
5507
diff
changeset
|
39 import com.oracle.graal.api.code.*; |
9813
c010c12ab682
put disassembling of installed code under the control of Debug.log(); use -G:Log=CodeInstall to show disassembly for all installed code
Doug Simon <doug.simon@oracle.com>
parents:
9807
diff
changeset
|
40 import com.oracle.graal.api.code.CallingConvention.Type; |
5507
dc71b06d09f8
Moving classes from cri.ri to api.meta.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
5432
diff
changeset
|
41 import com.oracle.graal.api.meta.*; |
12779
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12614
diff
changeset
|
42 import com.oracle.graal.compiler.CompilerThreadFactory.CompilerThread; |
5136
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
43 import com.oracle.graal.debug.*; |
13197
8569b9e047cd
change debug scopes implementation to prevent extra frames related to mechanism being added to call stack
Doug Simon <doug.simon@oracle.com>
parents:
13106
diff
changeset
|
44 import com.oracle.graal.debug.Debug.Scope; |
7558
223f645acb9b
added compilation rate measurements on top of existing metric/timer facility - enabled with -Dgraal.benchmark.compilation=true
Doug Simon <doug.simon@oracle.com>
parents:
7552
diff
changeset
|
45 import com.oracle.graal.debug.internal.*; |
12779
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12614
diff
changeset
|
46 import com.oracle.graal.hotspot.bridge.*; |
5548
8872bc0eebdf
Renaming hotspot.ri => hotspot.meta.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
5547
diff
changeset
|
47 import com.oracle.graal.hotspot.meta.*; |
13452
8275a0d0c90a
create profiling info, phase plan and optimistic opts when running a CompilationTask, not when creating it (GRAAL-640)
Doug Simon <doug.simon@oracle.com>
parents:
13366
diff
changeset
|
48 import com.oracle.graal.hotspot.phases.*; |
13229
325b4e4efb60
added CompilationResultBuilderFactory to support peep-hole instrumentation of methods as their code is emitted
Doug Simon <doug.simon@oracle.com>
parents:
13228
diff
changeset
|
49 import com.oracle.graal.lir.asm.*; |
5138 | 50 import com.oracle.graal.nodes.*; |
8637
ce5750014c3d
moved Replacements and MacroSubstitution from the graal.api.replacements project to graal.nodes project and reversed the dependency between these two projects (the latter now/again depends on the former)
Doug Simon <doug.simon@oracle.com>
parents:
8636
diff
changeset
|
51 import com.oracle.graal.nodes.spi.*; |
6525
2c913b643422
rename packages in graal.phases to match project name
Doug Simon <doug.simon@oracle.com>
parents:
6500
diff
changeset
|
52 import com.oracle.graal.phases.*; |
9968
3df534c97af1
Create Suites instance in runtime.
Roland Schatz <roland.schatz@oracle.com>
parents:
9920
diff
changeset
|
53 import com.oracle.graal.phases.tiers.*; |
5136
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
54 |
13859
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
55 public class CompilationTask implements Runnable, Comparable { |
5176
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
5138
diff
changeset
|
56 |
14027
aba77882e314
be more careful with clinit of CompilationTask
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
14016
diff
changeset
|
57 // Keep static finals in a group with withinEnqueue as the last one. CompilationTask can be |
aba77882e314
be more careful with clinit of CompilationTask
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
14016
diff
changeset
|
58 // called from within it's own clinit so it needs to be careful about accessing state. Once |
aba77882e314
be more careful with clinit of CompilationTask
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
14016
diff
changeset
|
59 // withinEnqueue is non-null we assume that CompilationTask is fully initialized. |
aba77882e314
be more careful with clinit of CompilationTask
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
14016
diff
changeset
|
60 private static final AtomicLong uniqueTaskIds = new AtomicLong(); |
aba77882e314
be more careful with clinit of CompilationTask
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
14016
diff
changeset
|
61 |
14016
555867401850
Make Debug.metric objects static
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13912
diff
changeset
|
62 private static final DebugMetric BAILOUTS = Debug.metric("Bailouts"); |
555867401850
Make Debug.metric objects static
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13912
diff
changeset
|
63 |
14027
aba77882e314
be more careful with clinit of CompilationTask
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
14016
diff
changeset
|
64 private static final ThreadLocal<Boolean> withinEnqueue = new ThreadLocal<Boolean>() { |
5183
e1e681a5558e
fix PriorityQueue, enable PriorityQueue and CacheGraphs
Lukas Stadler <lukas.stadler@jku.at>
parents:
5177
diff
changeset
|
65 |
e1e681a5558e
fix PriorityQueue, enable PriorityQueue and CacheGraphs
Lukas Stadler <lukas.stadler@jku.at>
parents:
5177
diff
changeset
|
66 @Override |
e1e681a5558e
fix PriorityQueue, enable PriorityQueue and CacheGraphs
Lukas Stadler <lukas.stadler@jku.at>
parents:
5177
diff
changeset
|
67 protected Boolean initialValue() { |
e1e681a5558e
fix PriorityQueue, enable PriorityQueue and CacheGraphs
Lukas Stadler <lukas.stadler@jku.at>
parents:
5177
diff
changeset
|
68 return Boolean.valueOf(Thread.currentThread() instanceof CompilerThread); |
e1e681a5558e
fix PriorityQueue, enable PriorityQueue and CacheGraphs
Lukas Stadler <lukas.stadler@jku.at>
parents:
5177
diff
changeset
|
69 } |
e1e681a5558e
fix PriorityQueue, enable PriorityQueue and CacheGraphs
Lukas Stadler <lukas.stadler@jku.at>
parents:
5177
diff
changeset
|
70 }; |
e1e681a5558e
fix PriorityQueue, enable PriorityQueue and CacheGraphs
Lukas Stadler <lukas.stadler@jku.at>
parents:
5177
diff
changeset
|
71 |
14027
aba77882e314
be more careful with clinit of CompilationTask
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
14016
diff
changeset
|
72 public static final boolean isWithinEnqueue() { |
aba77882e314
be more careful with clinit of CompilationTask
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
14016
diff
changeset
|
73 // It's possible this can be called before the <clinit> has completed so check for null |
aba77882e314
be more careful with clinit of CompilationTask
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
14016
diff
changeset
|
74 return withinEnqueue == null || withinEnqueue.get(); |
aba77882e314
be more careful with clinit of CompilationTask
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
14016
diff
changeset
|
75 } |
aba77882e314
be more careful with clinit of CompilationTask
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
14016
diff
changeset
|
76 |
14102
43a80ad2730a
rename BeginEnqueue to Enqueueing
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
14027
diff
changeset
|
77 public static class Enqueueing implements Closeable { |
43a80ad2730a
rename BeginEnqueue to Enqueueing
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
14027
diff
changeset
|
78 public Enqueueing() { |
14027
aba77882e314
be more careful with clinit of CompilationTask
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
14016
diff
changeset
|
79 assert !withinEnqueue.get(); |
aba77882e314
be more careful with clinit of CompilationTask
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
14016
diff
changeset
|
80 withinEnqueue.set(Boolean.TRUE); |
aba77882e314
be more careful with clinit of CompilationTask
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
14016
diff
changeset
|
81 } |
aba77882e314
be more careful with clinit of CompilationTask
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
14016
diff
changeset
|
82 |
aba77882e314
be more careful with clinit of CompilationTask
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
14016
diff
changeset
|
83 public void close() { |
aba77882e314
be more careful with clinit of CompilationTask
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
14016
diff
changeset
|
84 withinEnqueue.set(Boolean.FALSE); |
aba77882e314
be more careful with clinit of CompilationTask
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
14016
diff
changeset
|
85 } |
aba77882e314
be more careful with clinit of CompilationTask
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
14016
diff
changeset
|
86 } |
aba77882e314
be more careful with clinit of CompilationTask
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
14016
diff
changeset
|
87 |
10056
a323a9e20f9d
Fixed a few race conditions in the compilation queue.
Christian Haeubl <haeubl@ssw.jku.at>
parents:
10019
diff
changeset
|
88 private enum CompilationStatus { |
13859
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
89 Queued, Running, Finished |
10056
a323a9e20f9d
Fixed a few race conditions in the compilation queue.
Christian Haeubl <haeubl@ssw.jku.at>
parents:
10019
diff
changeset
|
90 } |
5136
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
91 |
12456
f87c68d79f07
improved support for co-existing, multiple backends (GRAAL-363)
Doug Simon <doug.simon@oracle.com>
parents:
12431
diff
changeset
|
92 private final HotSpotBackend backend; |
5554
70f715dfbb41
Bring Java renamings and restructurings to the C++ part.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
5548
diff
changeset
|
93 private final HotSpotResolvedJavaMethod method; |
6677
3dddb311395f
hotspot infrastructure for OnStackReplacement
Lukas Stadler <lukas.stadler@jku.at>
parents:
6676
diff
changeset
|
94 private final int entryBCI; |
5176
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
5138
diff
changeset
|
95 private final int id; |
10076
25de9c96a032
Minor CompilationTask refactoring.
Christian Haeubl <haeubl@ssw.jku.at>
parents:
10056
diff
changeset
|
96 private final AtomicReference<CompilationStatus> status; |
5136
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
97 |
9813
c010c12ab682
put disassembling of installed code under the control of Debug.log(); use -G:Log=CodeInstall to show disassembly for all installed code
Doug Simon <doug.simon@oracle.com>
parents:
9807
diff
changeset
|
98 private StructuredGraph graph; |
c010c12ab682
put disassembling of installed code under the control of Debug.log(); use -G:Log=CodeInstall to show disassembly for all installed code
Doug Simon <doug.simon@oracle.com>
parents:
9807
diff
changeset
|
99 |
13859
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
100 /** |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
101 * A long representing the sequence number of this task. Used for sorting the compile queue. |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
102 */ |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
103 private long taskId; |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
104 |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
105 private boolean blocking; |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
106 |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
107 public CompilationTask(HotSpotBackend backend, HotSpotResolvedJavaMethod method, int entryBCI, boolean blocking) { |
12456
f87c68d79f07
improved support for co-existing, multiple backends (GRAAL-363)
Doug Simon <doug.simon@oracle.com>
parents:
12431
diff
changeset
|
108 this.backend = backend; |
5136
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
109 this.method = method; |
6677
3dddb311395f
hotspot infrastructure for OnStackReplacement
Lukas Stadler <lukas.stadler@jku.at>
parents:
6676
diff
changeset
|
110 this.entryBCI = entryBCI; |
14104 | 111 this.id = method.allocateCompileId(entryBCI); |
13859
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
112 this.blocking = blocking; |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
113 this.taskId = uniqueTaskIds.incrementAndGet(); |
10078
7bcc4bf839fe
Bugfix for compilation queue.
Christian Haeubl <haeubl@ssw.jku.at>
parents:
10076
diff
changeset
|
114 this.status = new AtomicReference<>(CompilationStatus.Queued); |
5176
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
5138
diff
changeset
|
115 } |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
5138
diff
changeset
|
116 |
6721
98d691bc23da
make osr compilations asynchronous
Lukas Stadler <lukas.stadler@jku.at>
parents:
6684
diff
changeset
|
117 public ResolvedJavaMethod getMethod() { |
5176
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
5138
diff
changeset
|
118 return method; |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
5138
diff
changeset
|
119 } |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
5138
diff
changeset
|
120 |
10640
88672775a26c
Compilation policy fixes and changed default compilation policy.
Christian Haeubl <haeubl@ssw.jku.at>
parents:
10577
diff
changeset
|
121 public int getId() { |
88672775a26c
Compilation policy fixes and changed default compilation policy.
Christian Haeubl <haeubl@ssw.jku.at>
parents:
10577
diff
changeset
|
122 return id; |
88672775a26c
Compilation policy fixes and changed default compilation policy.
Christian Haeubl <haeubl@ssw.jku.at>
parents:
10577
diff
changeset
|
123 } |
88672775a26c
Compilation policy fixes and changed default compilation policy.
Christian Haeubl <haeubl@ssw.jku.at>
parents:
10577
diff
changeset
|
124 |
6721
98d691bc23da
make osr compilations asynchronous
Lukas Stadler <lukas.stadler@jku.at>
parents:
6684
diff
changeset
|
125 public int getEntryBCI() { |
98d691bc23da
make osr compilations asynchronous
Lukas Stadler <lukas.stadler@jku.at>
parents:
6684
diff
changeset
|
126 return entryBCI; |
98d691bc23da
make osr compilations asynchronous
Lukas Stadler <lukas.stadler@jku.at>
parents:
6684
diff
changeset
|
127 } |
98d691bc23da
make osr compilations asynchronous
Lukas Stadler <lukas.stadler@jku.at>
parents:
6684
diff
changeset
|
128 |
5136
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
129 public void run() { |
5183
e1e681a5558e
fix PriorityQueue, enable PriorityQueue and CacheGraphs
Lukas Stadler <lukas.stadler@jku.at>
parents:
5177
diff
changeset
|
130 withinEnqueue.set(Boolean.FALSE); |
e1e681a5558e
fix PriorityQueue, enable PriorityQueue and CacheGraphs
Lukas Stadler <lukas.stadler@jku.at>
parents:
5177
diff
changeset
|
131 try { |
13366
1480cfe97462
CTWCompilationTask should not be removed from compilation queue
Doug Simon <doug.simon@oracle.com>
parents:
13353
diff
changeset
|
132 runCompilation(true); |
8525
54f0a88e4523
Be a little bit more careful around compilation task queuing
Gilles Duboscq <duboscq@ssw.jku.at>
parents:
8508
diff
changeset
|
133 } finally { |
5183
e1e681a5558e
fix PriorityQueue, enable PriorityQueue and CacheGraphs
Lukas Stadler <lukas.stadler@jku.at>
parents:
5177
diff
changeset
|
134 withinEnqueue.set(Boolean.TRUE); |
13859
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
135 status.set(CompilationStatus.Finished); |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
136 synchronized (this) { |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
137 notifyAll(); |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
138 } |
5176
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
5138
diff
changeset
|
139 } |
5183
e1e681a5558e
fix PriorityQueue, enable PriorityQueue and CacheGraphs
Lukas Stadler <lukas.stadler@jku.at>
parents:
5177
diff
changeset
|
140 } |
e1e681a5558e
fix PriorityQueue, enable PriorityQueue and CacheGraphs
Lukas Stadler <lukas.stadler@jku.at>
parents:
5177
diff
changeset
|
141 |
7558
223f645acb9b
added compilation rate measurements on top of existing metric/timer facility - enabled with -Dgraal.benchmark.compilation=true
Doug Simon <doug.simon@oracle.com>
parents:
7552
diff
changeset
|
142 /** |
13859
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
143 * Block waiting till the compilation completes. |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
144 */ |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
145 public synchronized void block() { |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
146 while (status.get() != CompilationStatus.Finished) { |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
147 try { |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
148 wait(); |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
149 } catch (InterruptedException e) { |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
150 // Ignore and retry |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
151 } |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
152 } |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
153 } |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
154 |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
155 /** |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
156 * Sort blocking tasks before non-blocking ones and then by lowest taskId within the group. |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
157 */ |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
158 public int compareTo(Object o) { |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
159 if (!(o instanceof CompilationTask)) { |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
160 return 1; |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
161 } |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
162 CompilationTask task2 = (CompilationTask) o; |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
163 if (this.blocking != task2.blocking) { |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
164 // Blocking CompilationTasks are always higher than CompilationTasks |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
165 return task2.blocking ? 1 : -1; |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
166 } |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
167 // Within the two groups sort by sequence id, so they are processed in insertion order. |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
168 return this.taskId > task2.taskId ? 1 : -1; |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
169 } |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
170 |
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
171 /** |
7558
223f645acb9b
added compilation rate measurements on top of existing metric/timer facility - enabled with -Dgraal.benchmark.compilation=true
Doug Simon <doug.simon@oracle.com>
parents:
7552
diff
changeset
|
172 * Time spent in compilation. |
223f645acb9b
added compilation rate measurements on top of existing metric/timer facility - enabled with -Dgraal.benchmark.compilation=true
Doug Simon <doug.simon@oracle.com>
parents:
7552
diff
changeset
|
173 */ |
223f645acb9b
added compilation rate measurements on top of existing metric/timer facility - enabled with -Dgraal.benchmark.compilation=true
Doug Simon <doug.simon@oracle.com>
parents:
7552
diff
changeset
|
174 public static final DebugTimer CompilationTime = Debug.timer("CompilationTime"); |
223f645acb9b
added compilation rate measurements on top of existing metric/timer facility - enabled with -Dgraal.benchmark.compilation=true
Doug Simon <doug.simon@oracle.com>
parents:
7552
diff
changeset
|
175 |
11487
21738e530332
added metrics for timing front-end, back-end and code installation (GRAAL-448)
Doug Simon <doug.simon@oracle.com>
parents:
10672
diff
changeset
|
176 public static final DebugTimer CodeInstallationTime = Debug.timer("CodeInstallation"); |
21738e530332
added metrics for timing front-end, back-end and code installation (GRAAL-448)
Doug Simon <doug.simon@oracle.com>
parents:
10672
diff
changeset
|
177 |
13353
0e5c4f9fa9a5
enabled non-hosted CompileTheWorld execution with complete bootstrapping and the ability to override compilation options separately for CTW compilations
Doug Simon <doug.simon@oracle.com>
parents:
13239
diff
changeset
|
178 protected Suites getSuites(HotSpotProviders providers) { |
0e5c4f9fa9a5
enabled non-hosted CompileTheWorld execution with complete bootstrapping and the ability to override compilation options separately for CTW compilations
Doug Simon <doug.simon@oracle.com>
parents:
13239
diff
changeset
|
179 return providers.getSuites().getDefaultSuites(); |
0e5c4f9fa9a5
enabled non-hosted CompileTheWorld execution with complete bootstrapping and the ability to override compilation options separately for CTW compilations
Doug Simon <doug.simon@oracle.com>
parents:
13239
diff
changeset
|
180 } |
0e5c4f9fa9a5
enabled non-hosted CompileTheWorld execution with complete bootstrapping and the ability to override compilation options separately for CTW compilations
Doug Simon <doug.simon@oracle.com>
parents:
13239
diff
changeset
|
181 |
13585 | 182 protected PhaseSuite<HighTierContext> getGraphBuilderSuite(HotSpotProviders providers) { |
183 PhaseSuite<HighTierContext> suite = providers.getSuites().getDefaultGraphBuilderSuite(); | |
184 | |
13452
8275a0d0c90a
create profiling info, phase plan and optimistic opts when running a CompilationTask, not when creating it (GRAAL-640)
Doug Simon <doug.simon@oracle.com>
parents:
13366
diff
changeset
|
185 boolean osrCompilation = entryBCI != StructuredGraph.INVOCATION_ENTRY_BCI; |
8275a0d0c90a
create profiling info, phase plan and optimistic opts when running a CompilationTask, not when creating it (GRAAL-640)
Doug Simon <doug.simon@oracle.com>
parents:
13366
diff
changeset
|
186 if (osrCompilation) { |
13585 | 187 suite = suite.copy(); |
188 suite.appendPhase(new OnStackReplacementPhase()); | |
13452
8275a0d0c90a
create profiling info, phase plan and optimistic opts when running a CompilationTask, not when creating it (GRAAL-640)
Doug Simon <doug.simon@oracle.com>
parents:
13366
diff
changeset
|
189 } |
13585 | 190 return suite; |
13452
8275a0d0c90a
create profiling info, phase plan and optimistic opts when running a CompilationTask, not when creating it (GRAAL-640)
Doug Simon <doug.simon@oracle.com>
parents:
13366
diff
changeset
|
191 } |
8275a0d0c90a
create profiling info, phase plan and optimistic opts when running a CompilationTask, not when creating it (GRAAL-640)
Doug Simon <doug.simon@oracle.com>
parents:
13366
diff
changeset
|
192 |
8275a0d0c90a
create profiling info, phase plan and optimistic opts when running a CompilationTask, not when creating it (GRAAL-640)
Doug Simon <doug.simon@oracle.com>
parents:
13366
diff
changeset
|
193 protected OptimisticOptimizations getOptimisticOpts(ProfilingInfo profilingInfo) { |
8275a0d0c90a
create profiling info, phase plan and optimistic opts when running a CompilationTask, not when creating it (GRAAL-640)
Doug Simon <doug.simon@oracle.com>
parents:
13366
diff
changeset
|
194 return new OptimisticOptimizations(profilingInfo); |
8275a0d0c90a
create profiling info, phase plan and optimistic opts when running a CompilationTask, not when creating it (GRAAL-640)
Doug Simon <doug.simon@oracle.com>
parents:
13366
diff
changeset
|
195 } |
8275a0d0c90a
create profiling info, phase plan and optimistic opts when running a CompilationTask, not when creating it (GRAAL-640)
Doug Simon <doug.simon@oracle.com>
parents:
13366
diff
changeset
|
196 |
8275a0d0c90a
create profiling info, phase plan and optimistic opts when running a CompilationTask, not when creating it (GRAAL-640)
Doug Simon <doug.simon@oracle.com>
parents:
13366
diff
changeset
|
197 protected ProfilingInfo getProfilingInfo() { |
8275a0d0c90a
create profiling info, phase plan and optimistic opts when running a CompilationTask, not when creating it (GRAAL-640)
Doug Simon <doug.simon@oracle.com>
parents:
13366
diff
changeset
|
198 boolean osrCompilation = entryBCI != StructuredGraph.INVOCATION_ENTRY_BCI; |
8275a0d0c90a
create profiling info, phase plan and optimistic opts when running a CompilationTask, not when creating it (GRAAL-640)
Doug Simon <doug.simon@oracle.com>
parents:
13366
diff
changeset
|
199 return method.getCompilationProfilingInfo(osrCompilation); |
8275a0d0c90a
create profiling info, phase plan and optimistic opts when running a CompilationTask, not when creating it (GRAAL-640)
Doug Simon <doug.simon@oracle.com>
parents:
13366
diff
changeset
|
200 } |
8275a0d0c90a
create profiling info, phase plan and optimistic opts when running a CompilationTask, not when creating it (GRAAL-640)
Doug Simon <doug.simon@oracle.com>
parents:
13366
diff
changeset
|
201 |
13366
1480cfe97462
CTWCompilationTask should not be removed from compilation queue
Doug Simon <doug.simon@oracle.com>
parents:
13353
diff
changeset
|
202 public void runCompilation(boolean clearFromCompilationQueue) { |
10672
e7d07c9bb779
Removed priority compilation queue.
Christian Haeubl <haeubl@ssw.jku.at>
parents:
10640
diff
changeset
|
203 /* |
e7d07c9bb779
Removed priority compilation queue.
Christian Haeubl <haeubl@ssw.jku.at>
parents:
10640
diff
changeset
|
204 * no code must be outside this try/finally because it could happen otherwise that |
e7d07c9bb779
Removed priority compilation queue.
Christian Haeubl <haeubl@ssw.jku.at>
parents:
10640
diff
changeset
|
205 * clearQueuedForCompilation() is not executed |
e7d07c9bb779
Removed priority compilation queue.
Christian Haeubl <haeubl@ssw.jku.at>
parents:
10640
diff
changeset
|
206 */ |
12779
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12614
diff
changeset
|
207 |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12614
diff
changeset
|
208 HotSpotVMConfig config = backend.getRuntime().getConfig(); |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12614
diff
changeset
|
209 long previousInlinedBytecodes = InlinedBytecodes.getCurrentValue(); |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12614
diff
changeset
|
210 long previousCompilationTime = CompilationTime.getCurrentValue(); |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12614
diff
changeset
|
211 HotSpotInstalledCode installedCode = null; |
10672
e7d07c9bb779
Removed priority compilation queue.
Christian Haeubl <haeubl@ssw.jku.at>
parents:
10640
diff
changeset
|
212 try (TimerCloseable a = CompilationTime.start()) { |
14107
800057208a2c
enable C1 + Graal tiered
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
14104
diff
changeset
|
213 if (!tryToChangeStatus(CompilationStatus.Queued, CompilationStatus.Running)) { |
800057208a2c
enable C1 + Graal tiered
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
14104
diff
changeset
|
214 return; |
800057208a2c
enable C1 + Graal tiered
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
14104
diff
changeset
|
215 } |
800057208a2c
enable C1 + Graal tiered
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
14104
diff
changeset
|
216 boolean isOSR = entryBCI != StructuredGraph.INVOCATION_ENTRY_BCI; |
800057208a2c
enable C1 + Graal tiered
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
14104
diff
changeset
|
217 |
800057208a2c
enable C1 + Graal tiered
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
14104
diff
changeset
|
218 // If there is already compiled code for this method on our level we simply return. |
800057208a2c
enable C1 + Graal tiered
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
14104
diff
changeset
|
219 // Graal compiles are always at the highest compile level, even in non-tiered mode so we |
800057208a2c
enable C1 + Graal tiered
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
14104
diff
changeset
|
220 // only need to check for that value. |
800057208a2c
enable C1 + Graal tiered
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
14104
diff
changeset
|
221 if (method.hasCodeAtLevel(entryBCI, config.compilationLevelFullOptimization)) { |
10672
e7d07c9bb779
Removed priority compilation queue.
Christian Haeubl <haeubl@ssw.jku.at>
parents:
10640
diff
changeset
|
222 return; |
e7d07c9bb779
Removed priority compilation queue.
Christian Haeubl <haeubl@ssw.jku.at>
parents:
10640
diff
changeset
|
223 } |
10640
88672775a26c
Compilation policy fixes and changed default compilation policy.
Christian Haeubl <haeubl@ssw.jku.at>
parents:
10577
diff
changeset
|
224 |
14107
800057208a2c
enable C1 + Graal tiered
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
14104
diff
changeset
|
225 CompilationStatistics stats = CompilationStatistics.create(method, isOSR); |
9864
063a712fe8d8
converted remaining options in GraalOptions to new system (GRAAL-27)
Doug Simon <doug.simon@oracle.com>
parents:
9849
diff
changeset
|
226 final boolean printCompilation = PrintCompilation.getValue() && !TTY.isSuppressed(); |
5136
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
227 if (printCompilation) { |
11828
1d64bfb3f481
new PrintAfterCompilation option, simplify PrintCompilation output
Lukas Stadler <lukas.stadler@jku.at>
parents:
11487
diff
changeset
|
228 TTY.println(getMethodDescription() + "..."); |
5136
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
229 } |
9864
063a712fe8d8
converted remaining options in GraalOptions to new system (GRAAL-27)
Doug Simon <doug.simon@oracle.com>
parents:
9849
diff
changeset
|
230 if (HotSpotPrintCompilation.getValue()) { |
8996
6d86ce1297bc
GRAAL-213: add HotSpot-style PrintCompilation and PrintInlining
twisti
parents:
8637
diff
changeset
|
231 printCompilation(); |
6d86ce1297bc
GRAAL-213: add HotSpot-style PrintCompilation and PrintInlining
twisti
parents:
8637
diff
changeset
|
232 } |
5136
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
233 |
5546
e42c0df7212a
Rename CiTargetMethod => CompilationResult.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
5541
diff
changeset
|
234 CompilationResult result = null; |
9864
063a712fe8d8
converted remaining options in GraalOptions to new system (GRAAL-27)
Doug Simon <doug.simon@oracle.com>
parents:
9849
diff
changeset
|
235 TTY.Filter filter = new TTY.Filter(PrintFilter.getValue(), method); |
6684
28a158024338
Add OSR marker in -G:+PrintCompilation output
Gilles Duboscq <duboscq@ssw.jku.at>
parents:
6679
diff
changeset
|
236 long start = System.currentTimeMillis(); |
13197
8569b9e047cd
change debug scopes implementation to prevent extra frames related to mechanism being added to call stack
Doug Simon <doug.simon@oracle.com>
parents:
13106
diff
changeset
|
237 try (Scope s = Debug.scope("Compiling", new DebugDumpScope(String.valueOf(id), true))) { |
14172
cd22c8dbda4f
Made graph caching compilation local
Doug Simon <doug.simon@oracle.com>
parents:
14163
diff
changeset
|
238 Map<ResolvedJavaMethod, StructuredGraph> graphCache = null; |
cd22c8dbda4f
Made graph caching compilation local
Doug Simon <doug.simon@oracle.com>
parents:
14163
diff
changeset
|
239 if (GraalOptions.CacheGraphs.getValue()) { |
cd22c8dbda4f
Made graph caching compilation local
Doug Simon <doug.simon@oracle.com>
parents:
14163
diff
changeset
|
240 graphCache = new HashMap<>(); |
13197
8569b9e047cd
change debug scopes implementation to prevent extra frames related to mechanism being added to call stack
Doug Simon <doug.simon@oracle.com>
parents:
13106
diff
changeset
|
241 } |
12488
1a4dc163cd0a
abstracted HotSpotGraalRuntime as a RuntimeProvider which removes the recent selector addition to the Graal capabilities API and also makes Truffle independent of the graal.hotspot project (GRAAL-363)
Doug Simon <doug.simon@oracle.com>
parents:
12465
diff
changeset
|
242 |
13197
8569b9e047cd
change debug scopes implementation to prevent extra frames related to mechanism being added to call stack
Doug Simon <doug.simon@oracle.com>
parents:
13106
diff
changeset
|
243 HotSpotProviders providers = backend.getProviders(); |
8569b9e047cd
change debug scopes implementation to prevent extra frames related to mechanism being added to call stack
Doug Simon <doug.simon@oracle.com>
parents:
13106
diff
changeset
|
244 Replacements replacements = providers.getReplacements(); |
8569b9e047cd
change debug scopes implementation to prevent extra frames related to mechanism being added to call stack
Doug Simon <doug.simon@oracle.com>
parents:
13106
diff
changeset
|
245 graph = replacements.getMethodSubstitution(method); |
8569b9e047cd
change debug scopes implementation to prevent extra frames related to mechanism being added to call stack
Doug Simon <doug.simon@oracle.com>
parents:
13106
diff
changeset
|
246 if (graph == null || entryBCI != INVOCATION_ENTRY_BCI) { |
8569b9e047cd
change debug scopes implementation to prevent extra frames related to mechanism being added to call stack
Doug Simon <doug.simon@oracle.com>
parents:
13106
diff
changeset
|
247 graph = new StructuredGraph(method, entryBCI); |
8569b9e047cd
change debug scopes implementation to prevent extra frames related to mechanism being added to call stack
Doug Simon <doug.simon@oracle.com>
parents:
13106
diff
changeset
|
248 } else { |
8569b9e047cd
change debug scopes implementation to prevent extra frames related to mechanism being added to call stack
Doug Simon <doug.simon@oracle.com>
parents:
13106
diff
changeset
|
249 // Compiling method substitution - must clone the graph |
8569b9e047cd
change debug scopes implementation to prevent extra frames related to mechanism being added to call stack
Doug Simon <doug.simon@oracle.com>
parents:
13106
diff
changeset
|
250 graph = graph.copy(); |
8569b9e047cd
change debug scopes implementation to prevent extra frames related to mechanism being added to call stack
Doug Simon <doug.simon@oracle.com>
parents:
13106
diff
changeset
|
251 } |
8569b9e047cd
change debug scopes implementation to prevent extra frames related to mechanism being added to call stack
Doug Simon <doug.simon@oracle.com>
parents:
13106
diff
changeset
|
252 InlinedBytecodes.add(method.getCodeSize()); |
8569b9e047cd
change debug scopes implementation to prevent extra frames related to mechanism being added to call stack
Doug Simon <doug.simon@oracle.com>
parents:
13106
diff
changeset
|
253 CallingConvention cc = getCallingConvention(providers.getCodeCache(), Type.JavaCallee, graph.method(), false); |
13912
766de6735435
Setup the OSR calling convention before calling compileGraph rather than patching it in the LIRGenerator
Gilles Duboscq <duboscq@ssw.jku.at>
parents:
13859
diff
changeset
|
254 if (graph.getEntryBCI() != StructuredGraph.INVOCATION_ENTRY_BCI) { |
766de6735435
Setup the OSR calling convention before calling compileGraph rather than patching it in the LIRGenerator
Gilles Duboscq <duboscq@ssw.jku.at>
parents:
13859
diff
changeset
|
255 // for OSR, only a pointer is passed to the method. |
766de6735435
Setup the OSR calling convention before calling compileGraph rather than patching it in the LIRGenerator
Gilles Duboscq <duboscq@ssw.jku.at>
parents:
13859
diff
changeset
|
256 JavaType[] parameterTypes = new JavaType[]{providers.getMetaAccess().lookupJavaType(long.class)}; |
766de6735435
Setup the OSR calling convention before calling compileGraph rather than patching it in the LIRGenerator
Gilles Duboscq <duboscq@ssw.jku.at>
parents:
13859
diff
changeset
|
257 CallingConvention tmp = providers.getCodeCache().getRegisterConfig().getCallingConvention(JavaCallee, providers.getMetaAccess().lookupJavaType(void.class), parameterTypes, |
766de6735435
Setup the OSR calling convention before calling compileGraph rather than patching it in the LIRGenerator
Gilles Duboscq <duboscq@ssw.jku.at>
parents:
13859
diff
changeset
|
258 backend.getTarget(), false); |
766de6735435
Setup the OSR calling convention before calling compileGraph rather than patching it in the LIRGenerator
Gilles Duboscq <duboscq@ssw.jku.at>
parents:
13859
diff
changeset
|
259 cc = new CallingConvention(cc.getStackSize(), cc.getReturn(), tmp.getArgument(0)); |
766de6735435
Setup the OSR calling convention before calling compileGraph rather than patching it in the LIRGenerator
Gilles Duboscq <duboscq@ssw.jku.at>
parents:
13859
diff
changeset
|
260 } |
13353
0e5c4f9fa9a5
enabled non-hosted CompileTheWorld execution with complete bootstrapping and the ability to override compilation options separately for CTW compilations
Doug Simon <doug.simon@oracle.com>
parents:
13239
diff
changeset
|
261 Suites suites = getSuites(providers); |
13452
8275a0d0c90a
create profiling info, phase plan and optimistic opts when running a CompilationTask, not when creating it (GRAAL-640)
Doug Simon <doug.simon@oracle.com>
parents:
13366
diff
changeset
|
262 ProfilingInfo profilingInfo = getProfilingInfo(); |
8275a0d0c90a
create profiling info, phase plan and optimistic opts when running a CompilationTask, not when creating it (GRAAL-640)
Doug Simon <doug.simon@oracle.com>
parents:
13366
diff
changeset
|
263 OptimisticOptimizations optimisticOpts = getOptimisticOpts(profilingInfo); |
14142
4eac66a9b87d
Remove reference to graph in LIRGenerator.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
14107
diff
changeset
|
264 result = compileGraph(graph, null, cc, method, providers, backend, backend.getTarget(), graphCache, getGraphBuilderSuite(providers), optimisticOpts, profilingInfo, |
14143
8836f566b0bc
Remove withScope parameter on GraalCompiler.compileGraph.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
14142
diff
changeset
|
265 method.getSpeculationLog(), suites, new CompilationResult(), CompilationResultBuilderFactory.Default); |
13630
b1838411e896
Use compile ids assigned by hotspot
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13585
diff
changeset
|
266 result.setId(getId()); |
b1838411e896
Use compile ids assigned by hotspot
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13585
diff
changeset
|
267 result.setEntryBCI(entryBCI); |
13197
8569b9e047cd
change debug scopes implementation to prevent extra frames related to mechanism being added to call stack
Doug Simon <doug.simon@oracle.com>
parents:
13106
diff
changeset
|
268 } catch (Throwable e) { |
8569b9e047cd
change debug scopes implementation to prevent extra frames related to mechanism being added to call stack
Doug Simon <doug.simon@oracle.com>
parents:
13106
diff
changeset
|
269 throw Debug.handle(e); |
5136
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
270 } finally { |
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
271 filter.remove(); |
11828
1d64bfb3f481
new PrintAfterCompilation option, simplify PrintCompilation output
Lukas Stadler <lukas.stadler@jku.at>
parents:
11487
diff
changeset
|
272 final boolean printAfterCompilation = PrintAfterCompilation.getValue() && !TTY.isSuppressed(); |
1d64bfb3f481
new PrintAfterCompilation option, simplify PrintCompilation output
Lukas Stadler <lukas.stadler@jku.at>
parents:
11487
diff
changeset
|
273 if (printAfterCompilation) { |
1d64bfb3f481
new PrintAfterCompilation option, simplify PrintCompilation output
Lukas Stadler <lukas.stadler@jku.at>
parents:
11487
diff
changeset
|
274 TTY.println(getMethodDescription() + String.format(" | %4dms %5dB", System.currentTimeMillis() - start, (result != null ? result.getTargetCodeSize() : -1))); |
1d64bfb3f481
new PrintAfterCompilation option, simplify PrintCompilation output
Lukas Stadler <lukas.stadler@jku.at>
parents:
11487
diff
changeset
|
275 } else if (printCompilation) { |
6684
28a158024338
Add OSR marker in -G:+PrintCompilation output
Gilles Duboscq <duboscq@ssw.jku.at>
parents:
6679
diff
changeset
|
276 TTY.println(String.format("%-6d Graal %-70s %-45s %-50s | %4dms %5dB", id, "", "", "", System.currentTimeMillis() - start, (result != null ? result.getTargetCodeSize() : -1))); |
5136
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
277 } |
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
278 } |
5275
290b3025b66f
added support for disassembling code after installation (so that the result of patching and relocation can be seen)
Doug Simon <doug.simon@oracle.com>
parents:
5226
diff
changeset
|
279 |
11487
21738e530332
added metrics for timing front-end, back-end and code installation (GRAAL-448)
Doug Simon <doug.simon@oracle.com>
parents:
10672
diff
changeset
|
280 try (TimerCloseable b = CodeInstallationTime.start()) { |
12779
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12614
diff
changeset
|
281 installedCode = installMethod(result); |
14543
12eaf1a47a90
removed ResolvedJavaMethod.getCompilerStorage(); moved last compiled graph node count into profiling info (i.e. MethodData metadata)
Doug Simon <doug.simon@oracle.com>
parents:
14172
diff
changeset
|
282 if (!isOSR) { |
12eaf1a47a90
removed ResolvedJavaMethod.getCompilerStorage(); moved last compiled graph node count into profiling info (i.e. MethodData metadata)
Doug Simon <doug.simon@oracle.com>
parents:
14172
diff
changeset
|
283 ProfilingInfo profile = method.getProfilingInfo(); |
12eaf1a47a90
removed ResolvedJavaMethod.getCompilerStorage(); moved last compiled graph node count into profiling info (i.e. MethodData metadata)
Doug Simon <doug.simon@oracle.com>
parents:
14172
diff
changeset
|
284 profile.setCompilerIRSize(StructuredGraph.class, graph.getNodeCount()); |
12eaf1a47a90
removed ResolvedJavaMethod.getCompilerStorage(); moved last compiled graph node count into profiling info (i.e. MethodData metadata)
Doug Simon <doug.simon@oracle.com>
parents:
14172
diff
changeset
|
285 } |
11487
21738e530332
added metrics for timing front-end, back-end and code installation (GRAAL-448)
Doug Simon <doug.simon@oracle.com>
parents:
10672
diff
changeset
|
286 } |
10672
e7d07c9bb779
Removed priority compilation queue.
Christian Haeubl <haeubl@ssw.jku.at>
parents:
10640
diff
changeset
|
287 stats.finish(method); |
5547
b6617d13ea44
Dropping Ci* prefixes. Some additional renamings in the api.code interface.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
5546
diff
changeset
|
288 } catch (BailoutException bailout) { |
14016
555867401850
Make Debug.metric objects static
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13912
diff
changeset
|
289 BAILOUTS.increment(); |
9864
063a712fe8d8
converted remaining options in GraalOptions to new system (GRAAL-27)
Doug Simon <doug.simon@oracle.com>
parents:
9849
diff
changeset
|
290 if (ExitVMOnBailout.getValue()) { |
5719
429accae15aa
moved some methods from CodeUtil to MetaUtil
Doug Simon <doug.simon@oracle.com>
parents:
5603
diff
changeset
|
291 TTY.cachedOut.println(MetaUtil.format("Bailout in %H.%n(%p)", method)); |
5136
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
292 bailout.printStackTrace(TTY.cachedOut); |
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
293 System.exit(-1); |
9864
063a712fe8d8
converted remaining options in GraalOptions to new system (GRAAL-27)
Doug Simon <doug.simon@oracle.com>
parents:
9849
diff
changeset
|
294 } else if (PrintBailout.getValue()) { |
5719
429accae15aa
moved some methods from CodeUtil to MetaUtil
Doug Simon <doug.simon@oracle.com>
parents:
5603
diff
changeset
|
295 TTY.cachedOut.println(MetaUtil.format("Bailout in %H.%n(%p)", method)); |
5201
891399c54706
Add a PrintBailout option
Gilles Duboscq <duboscq@ssw.jku.at>
parents:
5188
diff
changeset
|
296 bailout.printStackTrace(TTY.cachedOut); |
5136
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
297 } |
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
298 } catch (Throwable t) { |
9864
063a712fe8d8
converted remaining options in GraalOptions to new system (GRAAL-27)
Doug Simon <doug.simon@oracle.com>
parents:
9849
diff
changeset
|
299 if (PrintStackTraceOnException.getValue() || ExitVMOnException.getValue()) { |
9802
9dcd6f8a843b
CompileTask: add option to print exception on compile error
Bernhard Urban <bernhard.urban@jku.at>
parents:
9790
diff
changeset
|
300 t.printStackTrace(TTY.cachedOut); |
9dcd6f8a843b
CompileTask: add option to print exception on compile error
Bernhard Urban <bernhard.urban@jku.at>
parents:
9790
diff
changeset
|
301 } |
9864
063a712fe8d8
converted remaining options in GraalOptions to new system (GRAAL-27)
Doug Simon <doug.simon@oracle.com>
parents:
9849
diff
changeset
|
302 if (ExitVMOnException.getValue()) { |
5136
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
303 System.exit(-1); |
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
304 } |
10056
a323a9e20f9d
Fixed a few race conditions in the compilation queue.
Christian Haeubl <haeubl@ssw.jku.at>
parents:
10019
diff
changeset
|
305 } finally { |
13464
2236d18302e0
made -G:PrintCompRate incompatible with -XX:+CITime and -XX:+CITimeEach
Doug Simon <doug.simon@oracle.com>
parents:
13452
diff
changeset
|
306 if ((config.ciTime || config.ciTimeEach || PrintCompRate.getValue() != 0) && installedCode != null) { |
12779
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12614
diff
changeset
|
307 long processedBytes = InlinedBytecodes.getCurrentValue() - previousInlinedBytecodes; |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12614
diff
changeset
|
308 long time = CompilationTime.getCurrentValue() - previousCompilationTime; |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12614
diff
changeset
|
309 TimeUnit timeUnit = CompilationTime.getTimeUnit(); |
13103
c0b0974dd509
moved notification of Graal compilation statistics from VMToCompiler to CompilerToVM
Doug Simon <doug.simon@oracle.com>
parents:
12779
diff
changeset
|
310 long timeUnitsPerSecond = timeUnit.convert(1, TimeUnit.SECONDS); |
c0b0974dd509
moved notification of Graal compilation statistics from VMToCompiler to CompilerToVM
Doug Simon <doug.simon@oracle.com>
parents:
12779
diff
changeset
|
311 CompilerToVM c2vm = backend.getRuntime().getCompilerToVM(); |
c0b0974dd509
moved notification of Graal compilation statistics from VMToCompiler to CompilerToVM
Doug Simon <doug.simon@oracle.com>
parents:
12779
diff
changeset
|
312 c2vm.notifyCompilationStatistics(id, method, entryBCI != INVOCATION_ENTRY_BCI, (int) processedBytes, time, timeUnitsPerSecond, installedCode); |
12779
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12614
diff
changeset
|
313 } |
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12614
diff
changeset
|
314 |
13366
1480cfe97462
CTWCompilationTask should not be removed from compilation queue
Doug Simon <doug.simon@oracle.com>
parents:
13353
diff
changeset
|
315 if (clearFromCompilationQueue) { |
1480cfe97462
CTWCompilationTask should not be removed from compilation queue
Doug Simon <doug.simon@oracle.com>
parents:
13353
diff
changeset
|
316 assert method.isQueuedForCompilation(); |
1480cfe97462
CTWCompilationTask should not be removed from compilation queue
Doug Simon <doug.simon@oracle.com>
parents:
13353
diff
changeset
|
317 method.clearQueuedForCompilation(); |
1480cfe97462
CTWCompilationTask should not be removed from compilation queue
Doug Simon <doug.simon@oracle.com>
parents:
13353
diff
changeset
|
318 } |
5136
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
319 } |
5176
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
5138
diff
changeset
|
320 } |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
5138
diff
changeset
|
321 |
11828
1d64bfb3f481
new PrintAfterCompilation option, simplify PrintCompilation output
Lukas Stadler <lukas.stadler@jku.at>
parents:
11487
diff
changeset
|
322 private String getMethodDescription() { |
1d64bfb3f481
new PrintAfterCompilation option, simplify PrintCompilation output
Lukas Stadler <lukas.stadler@jku.at>
parents:
11487
diff
changeset
|
323 return String.format("%-6d Graal %-70s %-45s %-50s %s", id, method.getDeclaringClass().getName(), method.getName(), method.getSignature().getMethodDescriptor(), |
1d64bfb3f481
new PrintAfterCompilation option, simplify PrintCompilation output
Lukas Stadler <lukas.stadler@jku.at>
parents:
11487
diff
changeset
|
324 entryBCI == StructuredGraph.INVOCATION_ENTRY_BCI ? "" : "(OSR@" + entryBCI + ") "); |
1d64bfb3f481
new PrintAfterCompilation option, simplify PrintCompilation output
Lukas Stadler <lukas.stadler@jku.at>
parents:
11487
diff
changeset
|
325 } |
1d64bfb3f481
new PrintAfterCompilation option, simplify PrintCompilation output
Lukas Stadler <lukas.stadler@jku.at>
parents:
11487
diff
changeset
|
326 |
8996
6d86ce1297bc
GRAAL-213: add HotSpot-style PrintCompilation and PrintInlining
twisti
parents:
8637
diff
changeset
|
327 /** |
6d86ce1297bc
GRAAL-213: add HotSpot-style PrintCompilation and PrintInlining
twisti
parents:
8637
diff
changeset
|
328 * Print a HotSpot-style compilation message to the console. |
6d86ce1297bc
GRAAL-213: add HotSpot-style PrintCompilation and PrintInlining
twisti
parents:
8637
diff
changeset
|
329 */ |
6d86ce1297bc
GRAAL-213: add HotSpot-style PrintCompilation and PrintInlining
twisti
parents:
8637
diff
changeset
|
330 private void printCompilation() { |
6d86ce1297bc
GRAAL-213: add HotSpot-style PrintCompilation and PrintInlining
twisti
parents:
8637
diff
changeset
|
331 final boolean isOSR = entryBCI != StructuredGraph.INVOCATION_ENTRY_BCI; |
6d86ce1297bc
GRAAL-213: add HotSpot-style PrintCompilation and PrintInlining
twisti
parents:
8637
diff
changeset
|
332 final int mod = method.getModifiers(); |
6d86ce1297bc
GRAAL-213: add HotSpot-style PrintCompilation and PrintInlining
twisti
parents:
8637
diff
changeset
|
333 TTY.println(String.format("%7d %4d %c%c%c%c%c %s %s(%d bytes)", 0, id, isOSR ? '%' : ' ', Modifier.isSynchronized(mod) ? 's' : ' ', ' ', ' ', Modifier.isNative(mod) ? 'n' : ' ', |
6d86ce1297bc
GRAAL-213: add HotSpot-style PrintCompilation and PrintInlining
twisti
parents:
8637
diff
changeset
|
334 MetaUtil.format("%H::%n(%p)", method), isOSR ? "@ " + entryBCI + " " : "", method.getCodeSize())); |
6d86ce1297bc
GRAAL-213: add HotSpot-style PrintCompilation and PrintInlining
twisti
parents:
8637
diff
changeset
|
335 } |
6d86ce1297bc
GRAAL-213: add HotSpot-style PrintCompilation and PrintInlining
twisti
parents:
8637
diff
changeset
|
336 |
12779
f6c511451e4a
made Graal report its compilation info under -XX:+CITime in the same format as c1 and c2
Doug Simon <doug.simon@oracle.com>
parents:
12614
diff
changeset
|
337 private HotSpotInstalledCode installMethod(final CompilationResult compResult) { |
12456
f87c68d79f07
improved support for co-existing, multiple backends (GRAAL-363)
Doug Simon <doug.simon@oracle.com>
parents:
12431
diff
changeset
|
338 final HotSpotCodeCacheProvider codeCache = backend.getProviders().getCodeCache(); |
13197
8569b9e047cd
change debug scopes implementation to prevent extra frames related to mechanism being added to call stack
Doug Simon <doug.simon@oracle.com>
parents:
13106
diff
changeset
|
339 HotSpotInstalledCode installedCode = null; |
8569b9e047cd
change debug scopes implementation to prevent extra frames related to mechanism being added to call stack
Doug Simon <doug.simon@oracle.com>
parents:
13106
diff
changeset
|
340 try (Scope s = Debug.scope("CodeInstall", new DebugDumpScope(String.valueOf(id), true), codeCache, method)) { |
13630
b1838411e896
Use compile ids assigned by hotspot
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13585
diff
changeset
|
341 installedCode = codeCache.installMethod(method, compResult); |
13197
8569b9e047cd
change debug scopes implementation to prevent extra frames related to mechanism being added to call stack
Doug Simon <doug.simon@oracle.com>
parents:
13106
diff
changeset
|
342 } catch (Throwable e) { |
8569b9e047cd
change debug scopes implementation to prevent extra frames related to mechanism being added to call stack
Doug Simon <doug.simon@oracle.com>
parents:
13106
diff
changeset
|
343 throw Debug.handle(e); |
8569b9e047cd
change debug scopes implementation to prevent extra frames related to mechanism being added to call stack
Doug Simon <doug.simon@oracle.com>
parents:
13106
diff
changeset
|
344 } |
8569b9e047cd
change debug scopes implementation to prevent extra frames related to mechanism being added to call stack
Doug Simon <doug.simon@oracle.com>
parents:
13106
diff
changeset
|
345 return installedCode; |
5275
290b3025b66f
added support for disassembling code after installation (so that the result of patching and relocation can be seen)
Doug Simon <doug.simon@oracle.com>
parents:
5226
diff
changeset
|
346 } |
290b3025b66f
added support for disassembling code after installation (so that the result of patching and relocation can be seen)
Doug Simon <doug.simon@oracle.com>
parents:
5226
diff
changeset
|
347 |
10056
a323a9e20f9d
Fixed a few race conditions in the compilation queue.
Christian Haeubl <haeubl@ssw.jku.at>
parents:
10019
diff
changeset
|
348 private boolean tryToChangeStatus(CompilationStatus from, CompilationStatus to) { |
10076
25de9c96a032
Minor CompilationTask refactoring.
Christian Haeubl <haeubl@ssw.jku.at>
parents:
10056
diff
changeset
|
349 return status.compareAndSet(from, to); |
10056
a323a9e20f9d
Fixed a few race conditions in the compilation queue.
Christian Haeubl <haeubl@ssw.jku.at>
parents:
10019
diff
changeset
|
350 } |
a323a9e20f9d
Fixed a few race conditions in the compilation queue.
Christian Haeubl <haeubl@ssw.jku.at>
parents:
10019
diff
changeset
|
351 |
5176
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
5138
diff
changeset
|
352 @Override |
7382
31d1cc9219d8
gave CompilationTask a toString() method
Doug Simon <doug.simon@oracle.com>
parents:
7377
diff
changeset
|
353 public String toString() { |
10672
e7d07c9bb779
Removed priority compilation queue.
Christian Haeubl <haeubl@ssw.jku.at>
parents:
10640
diff
changeset
|
354 return "Compilation[id=" + id + ", " + MetaUtil.format("%H.%n(%p)", method) + (entryBCI == StructuredGraph.INVOCATION_ENTRY_BCI ? "" : "@" + entryBCI) + "]"; |
7382
31d1cc9219d8
gave CompilationTask a toString() method
Doug Simon <doug.simon@oracle.com>
parents:
7377
diff
changeset
|
355 } |
5136
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
356 } |