Mercurial > hg > graal-compiler
annotate graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/CompilationTask.java @ 19965:9bd252b8e3ad
Check for null installedCode when reporting events
author | Tom Rodriguez <tom.rodriguez@oracle.com> |
---|---|
date | Thu, 19 Mar 2015 17:25:16 -0700 |
parents | 31fac91ea3b7 |
children | cc3131ff7ce2 |
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.*; |
15259
d90e5c22ba55
Move GraalOptions to graal.compiler.common.
Josef Eisl <josef.eisl@jku.at>
parents:
15162
diff
changeset
|
28 import static com.oracle.graal.compiler.common.GraalOptions.*; |
15463
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
15311
diff
changeset
|
29 import static com.oracle.graal.compiler.common.UnsafeAccess.*; |
18690
abcff66a23b0
Add ability to programmatically set the dump level
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
18222
diff
changeset
|
30 import static com.oracle.graal.debug.Debug.*; |
16006
66a9286203a2
decoupled Graal runtime initialization and Graal compilation queue initialization
Doug Simon <doug.simon@oracle.com>
parents:
15702
diff
changeset
|
31 import static com.oracle.graal.hotspot.HotSpotGraalRuntime.*; |
66a9286203a2
decoupled Graal runtime initialization and Graal compilation queue initialization
Doug Simon <doug.simon@oracle.com>
parents:
15702
diff
changeset
|
32 import static com.oracle.graal.hotspot.InitTimer.*; |
17153
646ddd52d79a
only generate extra debug info if on HotSpot *and* compiling for HotSpot execution
Doug Simon <doug.simon@oracle.com>
parents:
17113
diff
changeset
|
33 import static com.oracle.graal.hotspot.meta.HotSpotSuitesProvider.*; |
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
|
34 import static com.oracle.graal.nodes.StructuredGraph.*; |
15470
c55f44b3c5e5
remove NodesToDoubles, refactoring of node probability and inlining relevance computation
Lukas Stadler <lukas.stadler@oracle.com>
parents:
15463
diff
changeset
|
35 import static com.oracle.graal.phases.common.inlining.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
|
36 |
14753 | 37 import java.lang.management.*; |
14172
cd22c8dbda4f
Made graph caching compilation local
Doug Simon <doug.simon@oracle.com>
parents:
14163
diff
changeset
|
38 import java.util.*; |
5136
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
39 import java.util.concurrent.*; |
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
40 |
5510
426c605c9d3c
Move cri.ci to api.code.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
5507
diff
changeset
|
41 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
|
42 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
|
43 import com.oracle.graal.api.meta.*; |
15663 | 44 import com.oracle.graal.api.runtime.*; |
15259
d90e5c22ba55
Move GraalOptions to graal.compiler.common.
Josef Eisl <josef.eisl@jku.at>
parents:
15162
diff
changeset
|
45 import com.oracle.graal.compiler.common.*; |
5136
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
46 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
|
47 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
|
48 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
|
49 import com.oracle.graal.hotspot.bridge.*; |
15663 | 50 import com.oracle.graal.hotspot.events.*; |
51 import com.oracle.graal.hotspot.events.EventProvider.CompilationEvent; | |
52 import com.oracle.graal.hotspot.events.EventProvider.CompilerFailureEvent; | |
5548
8872bc0eebdf
Renaming hotspot.ri => hotspot.meta.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
5547
diff
changeset
|
53 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
|
54 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
|
55 import com.oracle.graal.lir.asm.*; |
19232
66c60942c06c
GraalCompiler.emitLowLevel: use LowLevelSuites instead of LowLevelCompilerConfiguration.
Josef Eisl <josef.eisl@jku.at>
parents:
19050
diff
changeset
|
56 import com.oracle.graal.lir.phases.*; |
5138 | 57 import com.oracle.graal.nodes.*; |
19774
31fac91ea3b7
disallow method substitution to be compilation root
Doug Simon <doug.simon@oracle.com>
parents:
19693
diff
changeset
|
58 import com.oracle.graal.nodes.StructuredGraph.AllowAssumptions; |
6525
2c913b643422
rename packages in graal.phases to match project name
Doug Simon <doug.simon@oracle.com>
parents:
6500
diff
changeset
|
59 import com.oracle.graal.phases.*; |
18920
4af661af76fd
Restructuring in the GraphBuilder to have less fields in the phase instance.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
18690
diff
changeset
|
60 import com.oracle.graal.phases.OptimisticOptimizations.Optimization; |
9968
3df534c97af1
Create Suites instance in runtime.
Roland Schatz <roland.schatz@oracle.com>
parents:
9920
diff
changeset
|
61 import com.oracle.graal.phases.tiers.*; |
16106
5d84706642ce
ensure a DebugEnvironment is initialized for native compiler threads
Doug Simon <doug.simon@oracle.com>
parents:
16015
diff
changeset
|
62 import com.oracle.graal.printer.*; |
5136
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
63 |
16015
19a48d9834e1
exclude CompilationTask from JaCoCo
Doug Simon <doug.simon@oracle.com>
parents:
16006
diff
changeset
|
64 //JaCoCo Exclude |
19a48d9834e1
exclude CompilationTask from JaCoCo
Doug Simon <doug.simon@oracle.com>
parents:
16006
diff
changeset
|
65 |
16399
4481cf549cfc
removed (Java based) CompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
16243
diff
changeset
|
66 public class CompilationTask { |
5176
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
5138
diff
changeset
|
67 |
16006
66a9286203a2
decoupled Graal runtime initialization and Graal compilation queue initialization
Doug Simon <doug.simon@oracle.com>
parents:
15702
diff
changeset
|
68 static { |
66a9286203a2
decoupled Graal runtime initialization and Graal compilation queue initialization
Doug Simon <doug.simon@oracle.com>
parents:
15702
diff
changeset
|
69 try (InitTimer t = timer("initialize CompilationTask")) { |
66a9286203a2
decoupled Graal runtime initialization and Graal compilation queue initialization
Doug Simon <doug.simon@oracle.com>
parents:
15702
diff
changeset
|
70 // Must be first to ensure any options accessed by the rest of the class |
66a9286203a2
decoupled Graal runtime initialization and Graal compilation queue initialization
Doug Simon <doug.simon@oracle.com>
parents:
15702
diff
changeset
|
71 // initializer are initialized from the command line. |
66a9286203a2
decoupled Graal runtime initialization and Graal compilation queue initialization
Doug Simon <doug.simon@oracle.com>
parents:
15702
diff
changeset
|
72 HotSpotOptions.initialize(); |
66a9286203a2
decoupled Graal runtime initialization and Graal compilation queue initialization
Doug Simon <doug.simon@oracle.com>
parents:
15702
diff
changeset
|
73 } |
66a9286203a2
decoupled Graal runtime initialization and Graal compilation queue initialization
Doug Simon <doug.simon@oracle.com>
parents:
15702
diff
changeset
|
74 } |
66a9286203a2
decoupled Graal runtime initialization and Graal compilation queue initialization
Doug Simon <doug.simon@oracle.com>
parents:
15702
diff
changeset
|
75 |
14016
555867401850
Make Debug.metric objects static
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13912
diff
changeset
|
76 private static final DebugMetric BAILOUTS = Debug.metric("Bailouts"); |
555867401850
Make Debug.metric objects static
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13912
diff
changeset
|
77 |
12456
f87c68d79f07
improved support for co-existing, multiple backends (GRAAL-363)
Doug Simon <doug.simon@oracle.com>
parents:
12431
diff
changeset
|
78 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
|
79 private final HotSpotResolvedJavaMethod method; |
6677
3dddb311395f
hotspot infrastructure for OnStackReplacement
Lukas Stadler <lukas.stadler@jku.at>
parents:
6676
diff
changeset
|
80 private final int entryBCI; |
5176
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
5138
diff
changeset
|
81 private final int id; |
15695
128359d7cddc
once the Graal compilation queue has been shutdown, don't process any pending compilations and be more defensive about preventing future compilations to be queued
Doug Simon <doug.simon@oracle.com>
parents:
15663
diff
changeset
|
82 |
17371
58f45b63b802
CompileTheWorld compilations are no longer installed as default nmethods
Doug Simon <doug.simon@oracle.com>
parents:
17153
diff
changeset
|
83 /** |
58f45b63b802
CompileTheWorld compilations are no longer installed as default nmethods
Doug Simon <doug.simon@oracle.com>
parents:
17153
diff
changeset
|
84 * Specifies whether the compilation result is installed as the |
58f45b63b802
CompileTheWorld compilations are no longer installed as default nmethods
Doug Simon <doug.simon@oracle.com>
parents:
17153
diff
changeset
|
85 * {@linkplain HotSpotNmethod#isDefault() default} nmethod for the compiled method. |
58f45b63b802
CompileTheWorld compilations are no longer installed as default nmethods
Doug Simon <doug.simon@oracle.com>
parents:
17153
diff
changeset
|
86 */ |
58f45b63b802
CompileTheWorld compilations are no longer installed as default nmethods
Doug Simon <doug.simon@oracle.com>
parents:
17153
diff
changeset
|
87 private final boolean installAsDefault; |
58f45b63b802
CompileTheWorld compilations are no longer installed as default nmethods
Doug Simon <doug.simon@oracle.com>
parents:
17153
diff
changeset
|
88 |
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
|
89 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
|
90 |
13859
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
91 /** |
14753 | 92 * A {@link com.sun.management.ThreadMXBean} to be able to query some information about the |
93 * current compiler thread, e.g. total allocated bytes. | |
94 */ | |
15696
5f1373b3527d
make CompilationTask.threadMXBean static
Doug Simon <doug.simon@oracle.com>
parents:
15695
diff
changeset
|
95 private static final com.sun.management.ThreadMXBean threadMXBean = (com.sun.management.ThreadMXBean) ManagementFactory.getThreadMXBean(); |
14753 | 96 |
15463
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
15311
diff
changeset
|
97 /** |
19321
98592ae4b1fa
only record method dependencies if JVMTI hotswapping or breakpointing is enabled
Doug Simon <doug.simon@oracle.com>
parents:
19306
diff
changeset
|
98 * The address of the GraalEnv associated with this compilation or 0L if no such object exists. |
15463
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
15311
diff
changeset
|
99 */ |
19321
98592ae4b1fa
only record method dependencies if JVMTI hotswapping or breakpointing is enabled
Doug Simon <doug.simon@oracle.com>
parents:
19306
diff
changeset
|
100 private final long graalEnv; |
15463
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
15311
diff
changeset
|
101 |
19321
98592ae4b1fa
only record method dependencies if JVMTI hotswapping or breakpointing is enabled
Doug Simon <doug.simon@oracle.com>
parents:
19306
diff
changeset
|
102 public CompilationTask(HotSpotBackend backend, HotSpotResolvedJavaMethod method, int entryBCI, long graalEnv, int id, boolean installAsDefault) { |
12456
f87c68d79f07
improved support for co-existing, multiple backends (GRAAL-363)
Doug Simon <doug.simon@oracle.com>
parents:
12431
diff
changeset
|
103 this.backend = backend; |
5136
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
104 this.method = method; |
6677
3dddb311395f
hotspot infrastructure for OnStackReplacement
Lukas Stadler <lukas.stadler@jku.at>
parents:
6676
diff
changeset
|
105 this.entryBCI = entryBCI; |
16399
4481cf549cfc
removed (Java based) CompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
16243
diff
changeset
|
106 this.id = id; |
19321
98592ae4b1fa
only record method dependencies if JVMTI hotswapping or breakpointing is enabled
Doug Simon <doug.simon@oracle.com>
parents:
19306
diff
changeset
|
107 this.graalEnv = graalEnv; |
17371
58f45b63b802
CompileTheWorld compilations are no longer installed as default nmethods
Doug Simon <doug.simon@oracle.com>
parents:
17153
diff
changeset
|
108 this.installAsDefault = installAsDefault; |
5176
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
5138
diff
changeset
|
109 } |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
5138
diff
changeset
|
110 |
6721
98d691bc23da
make osr compilations asynchronous
Lukas Stadler <lukas.stadler@jku.at>
parents:
6684
diff
changeset
|
111 public ResolvedJavaMethod getMethod() { |
5176
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
5138
diff
changeset
|
112 return method; |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
5138
diff
changeset
|
113 } |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
5138
diff
changeset
|
114 |
15663 | 115 /** |
116 * Returns the compilation id of this task. | |
117 * | |
118 * @return compile id | |
119 */ | |
10640
88672775a26c
Compilation policy fixes and changed default compilation policy.
Christian Haeubl <haeubl@ssw.jku.at>
parents:
10577
diff
changeset
|
120 public int getId() { |
88672775a26c
Compilation policy fixes and changed default compilation policy.
Christian Haeubl <haeubl@ssw.jku.at>
parents:
10577
diff
changeset
|
121 return id; |
88672775a26c
Compilation policy fixes and changed default compilation policy.
Christian Haeubl <haeubl@ssw.jku.at>
parents:
10577
diff
changeset
|
122 } |
88672775a26c
Compilation policy fixes and changed default compilation policy.
Christian Haeubl <haeubl@ssw.jku.at>
parents:
10577
diff
changeset
|
123 |
6721
98d691bc23da
make osr compilations asynchronous
Lukas Stadler <lukas.stadler@jku.at>
parents:
6684
diff
changeset
|
124 public int getEntryBCI() { |
98d691bc23da
make osr compilations asynchronous
Lukas Stadler <lukas.stadler@jku.at>
parents:
6684
diff
changeset
|
125 return entryBCI; |
98d691bc23da
make osr compilations asynchronous
Lukas Stadler <lukas.stadler@jku.at>
parents:
6684
diff
changeset
|
126 } |
98d691bc23da
make osr compilations asynchronous
Lukas Stadler <lukas.stadler@jku.at>
parents:
6684
diff
changeset
|
127 |
13859
1e01e2644a5d
Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13630
diff
changeset
|
128 /** |
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
|
129 * 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
|
130 */ |
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
|
131 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
|
132 |
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
|
133 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
|
134 |
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
|
135 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
|
136 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
|
137 } |
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
|
138 |
19270
292442bed972
Rename LowLevelSuites to LIRSuites.
Josef Eisl <josef.eisl@jku.at>
parents:
19232
diff
changeset
|
139 protected LIRSuites getLIRSuites(HotSpotProviders providers) { |
292442bed972
Rename LowLevelSuites to LIRSuites.
Josef Eisl <josef.eisl@jku.at>
parents:
19232
diff
changeset
|
140 return providers.getSuites().getDefaultLIRSuites(); |
19232
66c60942c06c
GraalCompiler.emitLowLevel: use LowLevelSuites instead of LowLevelCompilerConfiguration.
Josef Eisl <josef.eisl@jku.at>
parents:
19050
diff
changeset
|
141 } |
66c60942c06c
GraalCompiler.emitLowLevel: use LowLevelSuites instead of LowLevelCompilerConfiguration.
Josef Eisl <josef.eisl@jku.at>
parents:
19050
diff
changeset
|
142 |
13585 | 143 protected PhaseSuite<HighTierContext> getGraphBuilderSuite(HotSpotProviders providers) { |
17153
646ddd52d79a
only generate extra debug info if on HotSpot *and* compiling for HotSpot execution
Doug Simon <doug.simon@oracle.com>
parents:
17113
diff
changeset
|
144 PhaseSuite<HighTierContext> suite = withSimpleDebugInfoIfRequested(providers.getSuites().getDefaultGraphBuilderSuite()); |
646ddd52d79a
only generate extra debug info if on HotSpot *and* compiling for HotSpot execution
Doug Simon <doug.simon@oracle.com>
parents:
17113
diff
changeset
|
145 |
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
|
146 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
|
147 if (osrCompilation) { |
13585 | 148 suite = suite.copy(); |
149 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
|
150 } |
13585 | 151 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
|
152 } |
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
|
153 |
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
|
154 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
|
155 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
|
156 } |
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
|
157 |
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
|
158 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
|
159 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
|
160 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
|
161 } |
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
|
162 |
15695
128359d7cddc
once the Graal compilation queue has been shutdown, don't process any pending compilations and be more defensive about preventing future compilations to be queued
Doug Simon <doug.simon@oracle.com>
parents:
15663
diff
changeset
|
163 public void runCompilation() { |
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
|
164 HotSpotVMConfig config = backend.getRuntime().getConfig(); |
14753 | 165 final long threadId = Thread.currentThread().getId(); |
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
|
166 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
|
167 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
|
168 HotSpotInstalledCode installedCode = null; |
15663 | 169 final boolean isOSR = entryBCI != StructuredGraph.INVOCATION_ENTRY_BCI; |
170 | |
171 // Log a compilation event. | |
172 EventProvider eventProvider = Graal.getRequiredCapability(EventProvider.class); | |
173 CompilationEvent compilationEvent = eventProvider.newCompilationEvent(); | |
14753 | 174 |
19693
fa75218e3942
Fix nested Timer and MemUse tracking logic
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
19346
diff
changeset
|
175 try (DebugCloseable a = CompilationTime.start()) { |
14107
800057208a2c
enable C1 + Graal tiered
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
14104
diff
changeset
|
176 // 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
|
177 // 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
|
178 // only need to check for that value. |
800057208a2c
enable C1 + Graal tiered
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
14104
diff
changeset
|
179 if (method.hasCodeAtLevel(entryBCI, config.compilationLevelFullOptimization)) { |
10672
e7d07c9bb779
Removed priority compilation queue.
Christian Haeubl <haeubl@ssw.jku.at>
parents:
10640
diff
changeset
|
180 return; |
e7d07c9bb779
Removed priority compilation queue.
Christian Haeubl <haeubl@ssw.jku.at>
parents:
10640
diff
changeset
|
181 } |
10640
88672775a26c
Compilation policy fixes and changed default compilation policy.
Christian Haeubl <haeubl@ssw.jku.at>
parents:
10577
diff
changeset
|
182 |
14107
800057208a2c
enable C1 + Graal tiered
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
14104
diff
changeset
|
183 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
|
184 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
|
185 if (printCompilation) { |
11828
1d64bfb3f481
new PrintAfterCompilation option, simplify PrintCompilation output
Lukas Stadler <lukas.stadler@jku.at>
parents:
11487
diff
changeset
|
186 TTY.println(getMethodDescription() + "..."); |
5136
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
187 } |
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
188 |
5546
e42c0df7212a
Rename CiTargetMethod => CompilationResult.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
5541
diff
changeset
|
189 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
|
190 TTY.Filter filter = new TTY.Filter(PrintFilter.getValue(), method); |
14753 | 191 final long start = System.currentTimeMillis(); |
192 final long allocatedBytesBefore = threadMXBean.getThreadAllocatedBytes(threadId); | |
193 | |
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
|
194 try (Scope s = Debug.scope("Compiling", new DebugDumpScope(String.valueOf(id), true))) { |
15663 | 195 // Begin the compilation event. |
196 compilationEvent.begin(); | |
14831
cbf616a24600
Added baseline path in runCompilation
Niclas Adlertz <niclas.adlertz@oracle.com>
parents:
14552
diff
changeset
|
197 |
19332
833d0361c3e2
Temporarily remove the baseline compiler experiment.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
19321
diff
changeset
|
198 Map<ResolvedJavaMethod, StructuredGraph> graphCache = null; |
833d0361c3e2
Temporarily remove the baseline compiler experiment.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
19321
diff
changeset
|
199 if (GraalOptions.CacheGraphs.getValue()) { |
833d0361c3e2
Temporarily remove the baseline compiler experiment.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
19321
diff
changeset
|
200 graphCache = new HashMap<>(); |
833d0361c3e2
Temporarily remove the baseline compiler experiment.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
19321
diff
changeset
|
201 } |
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
|
202 |
19332
833d0361c3e2
Temporarily remove the baseline compiler experiment.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
19321
diff
changeset
|
203 boolean recordEvolMethodDeps = graalEnv == 0 || unsafe.getByte(graalEnv + config.graalEnvJvmtiCanHotswapOrPostBreakpointOffset) != 0; |
19321
98592ae4b1fa
only record method dependencies if JVMTI hotswapping or breakpointing is enabled
Doug Simon <doug.simon@oracle.com>
parents:
19306
diff
changeset
|
204 |
19332
833d0361c3e2
Temporarily remove the baseline compiler experiment.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
19321
diff
changeset
|
205 HotSpotProviders providers = backend.getProviders(); |
19774
31fac91ea3b7
disallow method substitution to be compilation root
Doug Simon <doug.simon@oracle.com>
parents:
19693
diff
changeset
|
206 graph = new StructuredGraph(method, entryBCI, AllowAssumptions.from(OptAssumptions.getValue())); |
31fac91ea3b7
disallow method substitution to be compilation root
Doug Simon <doug.simon@oracle.com>
parents:
19693
diff
changeset
|
207 if (!recordEvolMethodDeps) { |
31fac91ea3b7
disallow method substitution to be compilation root
Doug Simon <doug.simon@oracle.com>
parents:
19693
diff
changeset
|
208 graph.disableInlinedMethodRecording(); |
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
|
209 } |
19332
833d0361c3e2
Temporarily remove the baseline compiler experiment.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
19321
diff
changeset
|
210 InlinedBytecodes.add(method.getCodeSize()); |
833d0361c3e2
Temporarily remove the baseline compiler experiment.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
19321
diff
changeset
|
211 CallingConvention cc = getCallingConvention(providers.getCodeCache(), Type.JavaCallee, graph.method(), false); |
833d0361c3e2
Temporarily remove the baseline compiler experiment.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
19321
diff
changeset
|
212 if (graph.getEntryBCI() != StructuredGraph.INVOCATION_ENTRY_BCI) { |
833d0361c3e2
Temporarily remove the baseline compiler experiment.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
19321
diff
changeset
|
213 // for OSR, only a pointer is passed to the method. |
833d0361c3e2
Temporarily remove the baseline compiler experiment.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
19321
diff
changeset
|
214 JavaType[] parameterTypes = new JavaType[]{providers.getMetaAccess().lookupJavaType(long.class)}; |
833d0361c3e2
Temporarily remove the baseline compiler experiment.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
19321
diff
changeset
|
215 CallingConvention tmp = providers.getCodeCache().getRegisterConfig().getCallingConvention(JavaCallee, providers.getMetaAccess().lookupJavaType(void.class), parameterTypes, |
833d0361c3e2
Temporarily remove the baseline compiler experiment.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
19321
diff
changeset
|
216 backend.getTarget(), false); |
833d0361c3e2
Temporarily remove the baseline compiler experiment.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
19321
diff
changeset
|
217 cc = new CallingConvention(cc.getStackSize(), cc.getReturn(), tmp.getArgument(0)); |
833d0361c3e2
Temporarily remove the baseline compiler experiment.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
19321
diff
changeset
|
218 } |
833d0361c3e2
Temporarily remove the baseline compiler experiment.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
19321
diff
changeset
|
219 Suites suites = getSuites(providers); |
833d0361c3e2
Temporarily remove the baseline compiler experiment.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
19321
diff
changeset
|
220 LIRSuites lirSuites = getLIRSuites(providers); |
833d0361c3e2
Temporarily remove the baseline compiler experiment.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
19321
diff
changeset
|
221 ProfilingInfo profilingInfo = getProfilingInfo(); |
833d0361c3e2
Temporarily remove the baseline compiler experiment.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
19321
diff
changeset
|
222 OptimisticOptimizations optimisticOpts = getOptimisticOpts(profilingInfo); |
833d0361c3e2
Temporarily remove the baseline compiler experiment.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
19321
diff
changeset
|
223 if (isOSR) { |
833d0361c3e2
Temporarily remove the baseline compiler experiment.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
19321
diff
changeset
|
224 // In OSR compiles, we cannot rely on never executed code profiles, because |
833d0361c3e2
Temporarily remove the baseline compiler experiment.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
19321
diff
changeset
|
225 // all code after the OSR loop is never executed. |
833d0361c3e2
Temporarily remove the baseline compiler experiment.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
19321
diff
changeset
|
226 optimisticOpts.remove(Optimization.RemoveNeverExecutedCode); |
833d0361c3e2
Temporarily remove the baseline compiler experiment.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
19321
diff
changeset
|
227 } |
833d0361c3e2
Temporarily remove the baseline compiler experiment.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
19321
diff
changeset
|
228 result = compileGraph(graph, cc, method, providers, backend, backend.getTarget(), graphCache, getGraphBuilderSuite(providers), optimisticOpts, profilingInfo, |
833d0361c3e2
Temporarily remove the baseline compiler experiment.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
19321
diff
changeset
|
229 method.getSpeculationLog(), suites, lirSuites, new CompilationResult(), CompilationResultBuilderFactory.Default); |
13630
b1838411e896
Use compile ids assigned by hotspot
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13585
diff
changeset
|
230 result.setId(getId()); |
b1838411e896
Use compile ids assigned by hotspot
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13585
diff
changeset
|
231 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
|
232 } 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
|
233 throw Debug.handle(e); |
5136
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
234 } finally { |
15663 | 235 // End the compilation event. |
236 compilationEvent.end(); | |
237 | |
5136
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
238 filter.remove(); |
11828
1d64bfb3f481
new PrintAfterCompilation option, simplify PrintCompilation output
Lukas Stadler <lukas.stadler@jku.at>
parents:
11487
diff
changeset
|
239 final boolean printAfterCompilation = PrintAfterCompilation.getValue() && !TTY.isSuppressed(); |
14753 | 240 |
241 if (printAfterCompilation || printCompilation) { | |
242 final long stop = System.currentTimeMillis(); | |
243 final int targetCodeSize = result != null ? result.getTargetCodeSize() : -1; | |
244 final long allocatedBytesAfter = threadMXBean.getThreadAllocatedBytes(threadId); | |
245 final long allocatedBytes = (allocatedBytesAfter - allocatedBytesBefore) / 1024; | |
246 | |
247 if (printAfterCompilation) { | |
248 TTY.println(getMethodDescription() + String.format(" | %4dms %5dB %5dkB", stop - start, targetCodeSize, allocatedBytes)); | |
249 } else if (printCompilation) { | |
250 TTY.println(String.format("%-6d Graal %-70s %-45s %-50s | %4dms %5dB %5dkB", id, "", "", "", stop - start, targetCodeSize, allocatedBytes)); | |
251 } | |
5136
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
252 } |
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
253 } |
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
|
254 |
19693
fa75218e3942
Fix nested Timer and MemUse tracking logic
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
19346
diff
changeset
|
255 try (DebugCloseable b = CodeInstallationTime.start()) { |
15162
dbb1fe8348e6
Allow to supply predefined installed code object when adding a method to the code cache.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
15045
diff
changeset
|
256 installedCode = (HotSpotInstalledCode) 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
|
257 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
|
258 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
|
259 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
|
260 } |
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
|
261 } |
14545
4877b0cb446f
removed ResolvedJavaMethod.getCompiledCodeSize()
Doug Simon <doug.simon@oracle.com>
parents:
14543
diff
changeset
|
262 stats.finish(method, installedCode); |
5547
b6617d13ea44
Dropping Ci* prefixes. Some additional renamings in the api.code interface.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
5546
diff
changeset
|
263 } catch (BailoutException bailout) { |
14016
555867401850
Make Debug.metric objects static
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
13912
diff
changeset
|
264 BAILOUTS.increment(); |
9864
063a712fe8d8
converted remaining options in GraalOptions to new system (GRAAL-27)
Doug Simon <doug.simon@oracle.com>
parents:
9849
diff
changeset
|
265 if (ExitVMOnBailout.getValue()) { |
16480
10c12d09a8d2
moved format(String format, JavaMethod method) from MetaUtil to be a default method in JavaMethod
Doug Simon <doug.simon@oracle.com>
parents:
16405
diff
changeset
|
266 TTY.cachedOut.println(method.format("Bailout in %H.%n(%p)")); |
5136
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
267 bailout.printStackTrace(TTY.cachedOut); |
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
268 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
|
269 } else if (PrintBailout.getValue()) { |
16480
10c12d09a8d2
moved format(String format, JavaMethod method) from MetaUtil to be a default method in JavaMethod
Doug Simon <doug.simon@oracle.com>
parents:
16405
diff
changeset
|
270 TTY.cachedOut.println(method.format("Bailout in %H.%n(%p)")); |
5201
891399c54706
Add a PrintBailout option
Gilles Duboscq <duboscq@ssw.jku.at>
parents:
5188
diff
changeset
|
271 bailout.printStackTrace(TTY.cachedOut); |
5136
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
272 } |
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
273 } 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
|
274 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
|
275 t.printStackTrace(TTY.cachedOut); |
9dcd6f8a843b
CompileTask: add option to print exception on compile error
Bernhard Urban <bernhard.urban@jku.at>
parents:
9790
diff
changeset
|
276 } |
15663 | 277 |
278 // Log a failure event. | |
279 CompilerFailureEvent event = eventProvider.newCompilerFailureEvent(); | |
280 if (event.shouldWrite()) { | |
281 event.setCompileId(getId()); | |
282 event.setMessage(t.getMessage()); | |
283 event.commit(); | |
284 } | |
285 | |
9864
063a712fe8d8
converted remaining options in GraalOptions to new system (GRAAL-27)
Doug Simon <doug.simon@oracle.com>
parents:
9849
diff
changeset
|
286 if (ExitVMOnException.getValue()) { |
5136
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
287 System.exit(-1); |
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
288 } |
10056
a323a9e20f9d
Fixed a few race conditions in the compilation queue.
Christian Haeubl <haeubl@ssw.jku.at>
parents:
10019
diff
changeset
|
289 } finally { |
15663 | 290 final int processedBytes = (int) (InlinedBytecodes.getCurrentValue() - previousInlinedBytecodes); |
291 | |
292 // Log a compilation event. | |
19965
9bd252b8e3ad
Check for null installedCode when reporting events
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
19774
diff
changeset
|
293 if (compilationEvent.shouldWrite() && installedCode != null) { |
16480
10c12d09a8d2
moved format(String format, JavaMethod method) from MetaUtil to be a default method in JavaMethod
Doug Simon <doug.simon@oracle.com>
parents:
16405
diff
changeset
|
294 compilationEvent.setMethod(method.format("%H.%n(%p)")); |
15663 | 295 compilationEvent.setCompileId(getId()); |
296 compilationEvent.setCompileLevel(config.compilationLevelFullOptimization); | |
297 compilationEvent.setSucceeded(true); | |
298 compilationEvent.setIsOsr(isOSR); | |
299 compilationEvent.setCodeSize(installedCode.getSize()); | |
300 compilationEvent.setInlinedBytes(processedBytes); | |
301 compilationEvent.commit(); | |
302 } | |
303 | |
19321
98592ae4b1fa
only record method dependencies if JVMTI hotswapping or breakpointing is enabled
Doug Simon <doug.simon@oracle.com>
parents:
19306
diff
changeset
|
304 if (graalEnv != 0) { |
98592ae4b1fa
only record method dependencies if JVMTI hotswapping or breakpointing is enabled
Doug Simon <doug.simon@oracle.com>
parents:
19306
diff
changeset
|
305 long ctask = unsafe.getAddress(graalEnv + config.graalEnvTaskOffset); |
98592ae4b1fa
only record method dependencies if JVMTI hotswapping or breakpointing is enabled
Doug Simon <doug.simon@oracle.com>
parents:
19306
diff
changeset
|
306 assert ctask != 0L; |
15463
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
15311
diff
changeset
|
307 unsafe.putInt(ctask + config.compileTaskNumInlinedBytecodesOffset, processedBytes); |
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
15311
diff
changeset
|
308 } |
16399
4481cf549cfc
removed (Java based) CompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
16243
diff
changeset
|
309 if ((config.ciTime || config.ciTimeEach) && 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
|
310 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
|
311 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
|
312 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
|
313 CompilerToVM c2vm = backend.getRuntime().getCompilerToVM(); |
15463
a20be10ad437
made Graal work with the HotSpot compiler queue and compiler threads, enabled by -XX:-UseGraalCompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
15311
diff
changeset
|
314 c2vm.notifyCompilationStatistics(id, method, entryBCI != INVOCATION_ENTRY_BCI, 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
|
315 } |
5136
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
316 } |
5176
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
5138
diff
changeset
|
317 } |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
5138
diff
changeset
|
318 |
11828
1d64bfb3f481
new PrintAfterCompilation option, simplify PrintCompilation output
Lukas Stadler <lukas.stadler@jku.at>
parents:
11487
diff
changeset
|
319 private String getMethodDescription() { |
16483
3c3cd36a3836
moved signatureToMethodDescriptor(Signature sig) from MetaUtil to be a default method in Signature
Doug Simon <doug.simon@oracle.com>
parents:
16480
diff
changeset
|
320 return String.format("%-6d Graal %-70s %-45s %-50s %s", id, method.getDeclaringClass().getName(), method.getName(), method.getSignature().toMethodDescriptor(), |
11828
1d64bfb3f481
new PrintAfterCompilation option, simplify PrintCompilation output
Lukas Stadler <lukas.stadler@jku.at>
parents:
11487
diff
changeset
|
321 entryBCI == StructuredGraph.INVOCATION_ENTRY_BCI ? "" : "(OSR@" + entryBCI + ") "); |
1d64bfb3f481
new PrintAfterCompilation option, simplify PrintCompilation output
Lukas Stadler <lukas.stadler@jku.at>
parents:
11487
diff
changeset
|
322 } |
1d64bfb3f481
new PrintAfterCompilation option, simplify PrintCompilation output
Lukas Stadler <lukas.stadler@jku.at>
parents:
11487
diff
changeset
|
323 |
15162
dbb1fe8348e6
Allow to supply predefined installed code object when adding a method to the code cache.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
15045
diff
changeset
|
324 private InstalledCode 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
|
325 final HotSpotCodeCacheProvider codeCache = backend.getProviders().getCodeCache(); |
15162
dbb1fe8348e6
Allow to supply predefined installed code object when adding a method to the code cache.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
15045
diff
changeset
|
326 InstalledCode installedCode = null; |
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
|
327 try (Scope s = Debug.scope("CodeInstall", new DebugDumpScope(String.valueOf(id), true), codeCache, method)) { |
19321
98592ae4b1fa
only record method dependencies if JVMTI hotswapping or breakpointing is enabled
Doug Simon <doug.simon@oracle.com>
parents:
19306
diff
changeset
|
328 installedCode = codeCache.installMethod(method, compResult, graalEnv, installAsDefault); |
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
|
329 } 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
|
330 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
|
331 } |
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
|
332 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
|
333 } |
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
|
334 |
5176
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
5138
diff
changeset
|
335 @Override |
7382
31d1cc9219d8
gave CompilationTask a toString() method
Doug Simon <doug.simon@oracle.com>
parents:
7377
diff
changeset
|
336 public String toString() { |
16480
10c12d09a8d2
moved format(String format, JavaMethod method) from MetaUtil to be a default method in JavaMethod
Doug Simon <doug.simon@oracle.com>
parents:
16405
diff
changeset
|
337 return "Compilation[id=" + id + ", " + method.format("%H.%n(%p)") + (entryBCI == StructuredGraph.INVOCATION_ENTRY_BCI ? "" : "@" + entryBCI) + "]"; |
7382
31d1cc9219d8
gave CompilationTask a toString() method
Doug Simon <doug.simon@oracle.com>
parents:
7377
diff
changeset
|
338 } |
16006
66a9286203a2
decoupled Graal runtime initialization and Graal compilation queue initialization
Doug Simon <doug.simon@oracle.com>
parents:
15702
diff
changeset
|
339 |
66a9286203a2
decoupled Graal runtime initialization and Graal compilation queue initialization
Doug Simon <doug.simon@oracle.com>
parents:
15702
diff
changeset
|
340 /** |
16106
5d84706642ce
ensure a DebugEnvironment is initialized for native compiler threads
Doug Simon <doug.simon@oracle.com>
parents:
16015
diff
changeset
|
341 * Schedules compilation of a metaspace Method. |
16006
66a9286203a2
decoupled Graal runtime initialization and Graal compilation queue initialization
Doug Simon <doug.simon@oracle.com>
parents:
15702
diff
changeset
|
342 * |
66a9286203a2
decoupled Graal runtime initialization and Graal compilation queue initialization
Doug Simon <doug.simon@oracle.com>
parents:
15702
diff
changeset
|
343 * Called from the VM. |
16399
4481cf549cfc
removed (Java based) CompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
16243
diff
changeset
|
344 * |
4481cf549cfc
removed (Java based) CompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
16243
diff
changeset
|
345 * @param metaspaceMethod |
4481cf549cfc
removed (Java based) CompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
16243
diff
changeset
|
346 * @param entryBCI |
19321
98592ae4b1fa
only record method dependencies if JVMTI hotswapping or breakpointing is enabled
Doug Simon <doug.simon@oracle.com>
parents:
19306
diff
changeset
|
347 * @param graalEnv address of native GraalEnv object |
16399
4481cf549cfc
removed (Java based) CompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
16243
diff
changeset
|
348 * @param id CompileTask::_compile_id |
16006
66a9286203a2
decoupled Graal runtime initialization and Graal compilation queue initialization
Doug Simon <doug.simon@oracle.com>
parents:
15702
diff
changeset
|
349 */ |
66a9286203a2
decoupled Graal runtime initialization and Graal compilation queue initialization
Doug Simon <doug.simon@oracle.com>
parents:
15702
diff
changeset
|
350 @SuppressWarnings("unused") |
19321
98592ae4b1fa
only record method dependencies if JVMTI hotswapping or breakpointing is enabled
Doug Simon <doug.simon@oracle.com>
parents:
19306
diff
changeset
|
351 private static void compileMetaspaceMethod(long metaspaceMethod, int entryBCI, long graalEnv, int id) { |
16399
4481cf549cfc
removed (Java based) CompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
16243
diff
changeset
|
352 // Ensure a Graal runtime is initialized prior to Debug being initialized as the former |
4481cf549cfc
removed (Java based) CompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
16243
diff
changeset
|
353 // may include processing command line options used by the latter. |
4481cf549cfc
removed (Java based) CompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
16243
diff
changeset
|
354 Graal.getRuntime(); |
16106
5d84706642ce
ensure a DebugEnvironment is initialized for native compiler threads
Doug Simon <doug.simon@oracle.com>
parents:
16015
diff
changeset
|
355 |
16399
4481cf549cfc
removed (Java based) CompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
16243
diff
changeset
|
356 // Ensure a debug configuration for this thread is initialized |
4481cf549cfc
removed (Java based) CompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
16243
diff
changeset
|
357 if (Debug.isEnabled() && DebugScope.getConfig() == null) { |
16405
9bfc4247262f
send log output to native tty
Lukas Stadler <lukas.stadler@oracle.com>
parents:
16399
diff
changeset
|
358 DebugEnvironment.initialize(TTY.cachedOut); |
16006
66a9286203a2
decoupled Graal runtime initialization and Graal compilation queue initialization
Doug Simon <doug.simon@oracle.com>
parents:
15702
diff
changeset
|
359 } |
16399
4481cf549cfc
removed (Java based) CompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
16243
diff
changeset
|
360 |
18222
a8cff27ca2e1
converted HotSpotResolvedJavaMethod to an interface
Doug Simon <doug.simon@oracle.com>
parents:
17371
diff
changeset
|
361 HotSpotResolvedJavaMethod method = HotSpotResolvedJavaMethodImpl.fromMetaspace(metaspaceMethod); |
19321
98592ae4b1fa
only record method dependencies if JVMTI hotswapping or breakpointing is enabled
Doug Simon <doug.simon@oracle.com>
parents:
19306
diff
changeset
|
362 compileMethod(method, entryBCI, graalEnv, id); |
16006
66a9286203a2
decoupled Graal runtime initialization and Graal compilation queue initialization
Doug Simon <doug.simon@oracle.com>
parents:
15702
diff
changeset
|
363 } |
66a9286203a2
decoupled Graal runtime initialization and Graal compilation queue initialization
Doug Simon <doug.simon@oracle.com>
parents:
15702
diff
changeset
|
364 |
66a9286203a2
decoupled Graal runtime initialization and Graal compilation queue initialization
Doug Simon <doug.simon@oracle.com>
parents:
15702
diff
changeset
|
365 /** |
66a9286203a2
decoupled Graal runtime initialization and Graal compilation queue initialization
Doug Simon <doug.simon@oracle.com>
parents:
15702
diff
changeset
|
366 * Compiles a method to machine code. |
66a9286203a2
decoupled Graal runtime initialization and Graal compilation queue initialization
Doug Simon <doug.simon@oracle.com>
parents:
15702
diff
changeset
|
367 */ |
19321
98592ae4b1fa
only record method dependencies if JVMTI hotswapping or breakpointing is enabled
Doug Simon <doug.simon@oracle.com>
parents:
19306
diff
changeset
|
368 static void compileMethod(HotSpotResolvedJavaMethod method, int entryBCI, long graalEnv, int id) { |
16399
4481cf549cfc
removed (Java based) CompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
16243
diff
changeset
|
369 HotSpotBackend backend = runtime().getHostBackend(); |
19321
98592ae4b1fa
only record method dependencies if JVMTI hotswapping or breakpointing is enabled
Doug Simon <doug.simon@oracle.com>
parents:
19306
diff
changeset
|
370 CompilationTask task = new CompilationTask(backend, method, entryBCI, graalEnv, id, true); |
18690
abcff66a23b0
Add ability to programmatically set the dump level
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
18222
diff
changeset
|
371 try (DebugConfigScope dcs = setConfig(new TopLevelDebugConfig())) { |
abcff66a23b0
Add ability to programmatically set the dump level
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
18222
diff
changeset
|
372 task.runCompilation(); |
abcff66a23b0
Add ability to programmatically set the dump level
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
18222
diff
changeset
|
373 } |
16399
4481cf549cfc
removed (Java based) CompilationQueue
Doug Simon <doug.simon@oracle.com>
parents:
16243
diff
changeset
|
374 return; |
16006
66a9286203a2
decoupled Graal runtime initialization and Graal compilation queue initialization
Doug Simon <doug.simon@oracle.com>
parents:
15702
diff
changeset
|
375 } |
5136
8f4f0ebffca2
Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff
changeset
|
376 } |