Mercurial > hg > graal-compiler

annotate graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/CompilationTask.java @ 15470:c55f44b3c5e5

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
remove NodesToDoubles, refactoring of node probability and inlining relevance computation
author Lukas Stadler <lukas.stadler@oracle.com>
date Fri, 02 May 2014 12:02:27 +0200
parents a20be10ad437
children 7340fe377764
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
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
Doug Simon <doug.simon@oracle.com>
parents: 13233 13217
diff changeset
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.*;
13464
2236d18302e0 made -G:PrintCompRate incompatible with -XX:+CITime and -XX:+CITimeEach
Doug Simon <doug.simon@oracle.com>
parents: 13452
diff changeset
30 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
31 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
32 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
33
14027
aba77882e314 be more careful with clinit of CompilationTask
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 14016
diff changeset
34 import java.io.*;
14753
b5cdbf6d793a print total allocated bytes for compilations
twisti
parents: 14724
diff changeset
35 import java.lang.management.*;
14172
cd22c8dbda4f Made graph caching compilation local
Doug Simon <doug.simon@oracle.com>
parents: 14163
diff changeset
36 import java.util.*;
5136
8f4f0ebffca2 Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff changeset
37 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
38 import java.util.concurrent.atomic.*;
5136
8f4f0ebffca2 Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff changeset
39
5510
426c605c9d3c Move cri.ci to api.code.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents: 5507
diff changeset
40 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
41 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
42 import com.oracle.graal.api.meta.*;
14831
cbf616a24600 Added baseline path in runCompilation
Niclas Adlertz <niclas.adlertz@oracle.com>
parents: 14552
diff changeset
43 import com.oracle.graal.baseline.*;
14724
c3ec1e4494b8 update BenchmarkCounters (add TimedDynamicCounters, fix -XX:+GraalCountersExcludeCompiler)
Lukas Stadler <lukas.stadler@oracle.com>
parents: 14552
diff changeset
44 import com.oracle.graal.compiler.*;
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.*;
5548
8872bc0eebdf Renaming hotspot.ri => hotspot.meta.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents: 5547
diff changeset
50 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
51 import com.oracle.graal.hotspot.phases.*;
14831
cbf616a24600 Added baseline path in runCompilation
Niclas Adlertz <niclas.adlertz@oracle.com>
parents: 14552
diff changeset
52 import com.oracle.graal.java.*;
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
53 import com.oracle.graal.lir.asm.*;
5138
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents: 5137
diff changeset
54 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
55 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
56 import com.oracle.graal.phases.*;
9968
3df534c97af1 Create Suites instance in runtime.
Roland Schatz <roland.schatz@oracle.com>
parents: 9920
diff changeset
57 import com.oracle.graal.phases.tiers.*;
5136
8f4f0ebffca2 Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff changeset
58
14991
64dcb92ee75a Truffle: Change signature for Truffle calls from (PackedFrame, Arguments) to (Object[]).
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents: 14919
diff changeset
59 import edu.umd.cs.findbugs.annotations.*;
14776
07dd21c7d533 revived use of FindBugs annotations
Doug Simon <doug.simon@oracle.com>
parents: 14753
diff changeset
60
15040
288c23143d47 Fix most raw type references.
Josef Eisl <josef.eisl@jku.at>
parents: 14991
diff changeset
61 public class CompilationTask implements Runnable, Comparable<Object> {
5176
af59b4dfc9e4 compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents: 5138
diff changeset
62
14027
aba77882e314 be more careful with clinit of CompilationTask
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 14016
diff changeset
63 // 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
64 // 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
65 // 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
66 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
67
14016
555867401850 Make Debug.metric objects static
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13912
diff changeset
68 private static final DebugMetric BAILOUTS = Debug.metric("Bailouts");
555867401850 Make Debug.metric objects static
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13912
diff changeset
69
14027
aba77882e314 be more careful with clinit of CompilationTask
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 14016
diff changeset
70 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
71
e1e681a5558e fix PriorityQueue, enable PriorityQueue and CacheGraphs
Lukas Stadler <lukas.stadler@jku.at>
parents: 5177
diff changeset
72 @Override
e1e681a5558e fix PriorityQueue, enable PriorityQueue and CacheGraphs
Lukas Stadler <lukas.stadler@jku.at>
parents: 5177
diff changeset
73 protected Boolean initialValue() {
e1e681a5558e fix PriorityQueue, enable PriorityQueue and CacheGraphs
Lukas Stadler <lukas.stadler@jku.at>
parents: 5177
diff changeset
74 return Boolean.valueOf(Thread.currentThread() instanceof CompilerThread);
e1e681a5558e fix PriorityQueue, enable PriorityQueue and CacheGraphs
Lukas Stadler <lukas.stadler@jku.at>
parents: 5177
diff changeset
75 }
e1e681a5558e fix PriorityQueue, enable PriorityQueue and CacheGraphs
Lukas Stadler <lukas.stadler@jku.at>
parents: 5177
diff changeset
76 };
e1e681a5558e fix PriorityQueue, enable PriorityQueue and CacheGraphs
Lukas Stadler <lukas.stadler@jku.at>
parents: 5177
diff changeset
77
14027
aba77882e314 be more careful with clinit of CompilationTask
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 14016
diff changeset
78 public static final boolean isWithinEnqueue() {
aba77882e314 be more careful with clinit of CompilationTask
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 14016
diff changeset
79 // 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
80 return withinEnqueue == null || withinEnqueue.get();
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
14102
43a80ad2730a rename BeginEnqueue to Enqueueing
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 14027
diff changeset
83 public static class Enqueueing implements Closeable {
43a80ad2730a rename BeginEnqueue to Enqueueing
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 14027
diff changeset
84 public Enqueueing() {
14027
aba77882e314 be more careful with clinit of CompilationTask
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 14016
diff changeset
85 assert !withinEnqueue.get();
aba77882e314 be more careful with clinit of CompilationTask
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 14016
diff changeset
86 withinEnqueue.set(Boolean.TRUE);
aba77882e314 be more careful with clinit of CompilationTask
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 14016
diff changeset
87 }
aba77882e314 be more careful with clinit of CompilationTask
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 14016
diff changeset
88
aba77882e314 be more careful with clinit of CompilationTask
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 14016
diff changeset
89 public void close() {
aba77882e314 be more careful with clinit of CompilationTask
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 14016
diff changeset
90 withinEnqueue.set(Boolean.FALSE);
aba77882e314 be more careful with clinit of CompilationTask
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 14016
diff changeset
91 }
aba77882e314 be more careful with clinit of CompilationTask
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 14016
diff changeset
92 }
aba77882e314 be more careful with clinit of CompilationTask
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 14016
diff changeset
93
10056
a323a9e20f9d Fixed a few race conditions in the compilation queue.
Christian Haeubl <haeubl@ssw.jku.at>
parents: 10019
diff changeset
94 private enum CompilationStatus {
14908
8db6e76cb658 Formatter: Keep one enum constant per line
Gilles Duboscq <duboscq@ssw.jku.at>
parents: 14852
diff changeset
95 Queued,
8db6e76cb658 Formatter: Keep one enum constant per line
Gilles Duboscq <duboscq@ssw.jku.at>
parents: 14852
diff changeset
96 Running,
8db6e76cb658 Formatter: Keep one enum constant per line
Gilles Duboscq <duboscq@ssw.jku.at>
parents: 14852
diff changeset
97 Finished
10056
a323a9e20f9d Fixed a few race conditions in the compilation queue.
Christian Haeubl <haeubl@ssw.jku.at>
parents: 10019
diff changeset
98 }
5136
8f4f0ebffca2 Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff changeset
99
12456
f87c68d79f07 improved support for co-existing, multiple backends (GRAAL-363)
Doug Simon <doug.simon@oracle.com>
parents: 12431
diff changeset
100 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
101 private final HotSpotResolvedJavaMethod method;
6677
3dddb311395f hotspot infrastructure for OnStackReplacement
Lukas Stadler <lukas.stadler@jku.at>
parents: 6676
diff changeset
102 private final int entryBCI;
5176
af59b4dfc9e4 compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents: 5138
diff changeset
103 private final int id;
10076
25de9c96a032 Minor CompilationTask refactoring.
Christian Haeubl <haeubl@ssw.jku.at>
parents: 10056
diff changeset
104 private final AtomicReference<CompilationStatus> status;
5136
8f4f0ebffca2 Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff changeset
105
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
106 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
107
13859
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
108 /**
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
109 * 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
110 */
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
111 private long taskId;
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
112
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
113 private boolean blocking;
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
114
14753
b5cdbf6d793a print total allocated bytes for compilations
twisti
parents: 14724
diff changeset
115 /**
b5cdbf6d793a print total allocated bytes for compilations
twisti
parents: 14724
diff changeset
116 * A {@link com.sun.management.ThreadMXBean} to be able to query some information about the
b5cdbf6d793a print total allocated bytes for compilations
twisti
parents: 14724
diff changeset
117 * current compiler thread, e.g. total allocated bytes.
b5cdbf6d793a print total allocated bytes for compilations
twisti
parents: 14724
diff changeset
118 */
b5cdbf6d793a print total allocated bytes for compilations
twisti
parents: 14724
diff changeset
119 private final com.sun.management.ThreadMXBean threadMXBean = (com.sun.management.ThreadMXBean) ManagementFactory.getThreadMXBean();
b5cdbf6d793a print total allocated bytes for compilations
twisti
parents: 14724
diff changeset
120
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
121 /**
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
122 * The address of the native CompileTask associated with this compilation. If 0L, then this
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
123 * compilation is being managed by a Graal compilation queue otherwise its managed by a native
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
124 * HotSpot compilation queue.
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
125 */
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
126 private final long ctask;
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
127
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
128 public CompilationTask(HotSpotBackend backend, HotSpotResolvedJavaMethod method, int entryBCI, long ctask, boolean blocking) {
12456
f87c68d79f07 improved support for co-existing, multiple backends (GRAAL-363)
Doug Simon <doug.simon@oracle.com>
parents: 12431
diff changeset
129 this.backend = backend;
5136
8f4f0ebffca2 Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff changeset
130 this.method = method;
6677
3dddb311395f hotspot infrastructure for OnStackReplacement
Lukas Stadler <lukas.stadler@jku.at>
parents: 6676
diff changeset
131 this.entryBCI = entryBCI;
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
132 if (ctask == 0L) {
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
133 this.id = method.allocateCompileId(entryBCI);
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
134 } else {
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
135 this.id = unsafe.getInt(ctask + backend.getRuntime().getConfig().compileTaskCompileIdOffset);
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
136 }
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
137 this.ctask = ctask;
13859
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
138 this.blocking = blocking;
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
139 this.taskId = uniqueTaskIds.incrementAndGet();
10078
7bcc4bf839fe Bugfix for compilation queue.
Christian Haeubl <haeubl@ssw.jku.at>
parents: 10076
diff changeset
140 this.status = new AtomicReference<>(CompilationStatus.Queued);
5176
af59b4dfc9e4 compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents: 5138
diff changeset
141 }
af59b4dfc9e4 compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents: 5138
diff changeset
142
6721
98d691bc23da make osr compilations asynchronous
Lukas Stadler <lukas.stadler@jku.at>
parents: 6684
diff changeset
143 public ResolvedJavaMethod getMethod() {
5176
af59b4dfc9e4 compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents: 5138
diff changeset
144 return method;
af59b4dfc9e4 compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents: 5138
diff changeset
145 }
af59b4dfc9e4 compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents: 5138
diff changeset
146
10640
88672775a26c Compilation policy fixes and changed default compilation policy.
Christian Haeubl <haeubl@ssw.jku.at>
parents: 10577
diff changeset
147 public int getId() {
88672775a26c Compilation policy fixes and changed default compilation policy.
Christian Haeubl <haeubl@ssw.jku.at>
parents: 10577
diff changeset
148 return id;
88672775a26c Compilation policy fixes and changed default compilation policy.
Christian Haeubl <haeubl@ssw.jku.at>
parents: 10577
diff changeset
149 }
88672775a26c Compilation policy fixes and changed default compilation policy.
Christian Haeubl <haeubl@ssw.jku.at>
parents: 10577
diff changeset
150
6721
98d691bc23da make osr compilations asynchronous
Lukas Stadler <lukas.stadler@jku.at>
parents: 6684
diff changeset
151 public int getEntryBCI() {
98d691bc23da make osr compilations asynchronous
Lukas Stadler <lukas.stadler@jku.at>
parents: 6684
diff changeset
152 return entryBCI;
98d691bc23da make osr compilations asynchronous
Lukas Stadler <lukas.stadler@jku.at>
parents: 6684
diff changeset
153 }
98d691bc23da make osr compilations asynchronous
Lukas Stadler <lukas.stadler@jku.at>
parents: 6684
diff changeset
154
14776
07dd21c7d533 revived use of FindBugs annotations
Doug Simon <doug.simon@oracle.com>
parents: 14753
diff changeset
155 @SuppressFBWarnings(value = "NN_NAKED_NOTIFY")
5136
8f4f0ebffca2 Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff changeset
156 public void run() {
5183
e1e681a5558e fix PriorityQueue, enable PriorityQueue and CacheGraphs
Lukas Stadler <lukas.stadler@jku.at>
parents: 5177
diff changeset
157 withinEnqueue.set(Boolean.FALSE);
e1e681a5558e fix PriorityQueue, enable PriorityQueue and CacheGraphs
Lukas Stadler <lukas.stadler@jku.at>
parents: 5177
diff changeset
158 try {
13366
1480cfe97462 CTWCompilationTask should not be removed from compilation queue
Doug Simon <doug.simon@oracle.com>
parents: 13353
diff changeset
159 runCompilation(true);
8525
54f0a88e4523 Be a little bit more careful around compilation task queuing
Gilles Duboscq <duboscq@ssw.jku.at>
parents: 8508
diff changeset
160 } finally {
5183
e1e681a5558e fix PriorityQueue, enable PriorityQueue and CacheGraphs
Lukas Stadler <lukas.stadler@jku.at>
parents: 5177
diff changeset
161 withinEnqueue.set(Boolean.TRUE);
13859
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
162 status.set(CompilationStatus.Finished);
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
163 synchronized (this) {
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
164 notifyAll();
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
165 }
5176
af59b4dfc9e4 compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents: 5138
diff changeset
166 }
5183
e1e681a5558e fix PriorityQueue, enable PriorityQueue and CacheGraphs
Lukas Stadler <lukas.stadler@jku.at>
parents: 5177
diff changeset
167 }
e1e681a5558e fix PriorityQueue, enable PriorityQueue and CacheGraphs
Lukas Stadler <lukas.stadler@jku.at>
parents: 5177
diff changeset
168
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
169 /**
13859
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
170 * Block waiting till the compilation completes.
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
171 */
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
172 public synchronized void block() {
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
173 while (status.get() != CompilationStatus.Finished) {
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
174 try {
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
175 wait();
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
176 } catch (InterruptedException e) {
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
177 // Ignore and retry
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
178 }
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
179 }
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
180 }
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
181
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
182 /**
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
183 * 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
184 */
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
185 public int compareTo(Object o) {
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
186 if (!(o instanceof CompilationTask)) {
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
187 return 1;
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
188 }
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
189 CompilationTask task2 = (CompilationTask) o;
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
190 if (this.blocking != task2.blocking) {
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
191 // Blocking CompilationTasks are always higher than CompilationTasks
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
192 return task2.blocking ? 1 : -1;
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
193 }
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
194 // 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
195 return this.taskId > task2.taskId ? 1 : -1;
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
196 }
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
197
1e01e2644a5d Make blocking compiles safe
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13630
diff changeset
198 /**
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
199 * 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
200 */
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
201 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
202
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
203 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
204
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
205 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
206 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
207 }
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
208
13585
f4f0a8a01ce0 remove PhasePlan
Lukas Stadler <lukas.stadler@jku.at>
parents: 13584
diff changeset
209 protected PhaseSuite<HighTierContext> getGraphBuilderSuite(HotSpotProviders providers) {
f4f0a8a01ce0 remove PhasePlan
Lukas Stadler <lukas.stadler@jku.at>
parents: 13584
diff changeset
210 PhaseSuite<HighTierContext> suite = providers.getSuites().getDefaultGraphBuilderSuite();
f4f0a8a01ce0 remove PhasePlan
Lukas Stadler <lukas.stadler@jku.at>
parents: 13584
diff changeset
211
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
212 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
213 if (osrCompilation) {
13585
f4f0a8a01ce0 remove PhasePlan
Lukas Stadler <lukas.stadler@jku.at>
parents: 13584
diff changeset
214 suite = suite.copy();
f4f0a8a01ce0 remove PhasePlan
Lukas Stadler <lukas.stadler@jku.at>
parents: 13584
diff changeset
215 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
216 }
13585
f4f0a8a01ce0 remove PhasePlan
Lukas Stadler <lukas.stadler@jku.at>
parents: 13584
diff changeset
217 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
218 }
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
219
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
220 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
221 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
222 }
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
223
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
224 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
225 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
226 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
227 }
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
228
13366
1480cfe97462 CTWCompilationTask should not be removed from compilation queue
Doug Simon <doug.simon@oracle.com>
parents: 13353
diff changeset
229 public void runCompilation(boolean clearFromCompilationQueue) {
10672
e7d07c9bb779 Removed priority compilation queue.
Christian Haeubl <haeubl@ssw.jku.at>
parents: 10640
diff changeset
230 /*
e7d07c9bb779 Removed priority compilation queue.
Christian Haeubl <haeubl@ssw.jku.at>
parents: 10640
diff changeset
231 * 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
232 * clearQueuedForCompilation() is not executed
e7d07c9bb779 Removed priority compilation queue.
Christian Haeubl <haeubl@ssw.jku.at>
parents: 10640
diff changeset
233 */
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
234
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
235 HotSpotVMConfig config = backend.getRuntime().getConfig();
14753
b5cdbf6d793a print total allocated bytes for compilations
twisti
parents: 14724
diff changeset
236 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
237 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
238 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
239 HotSpotInstalledCode installedCode = null;
14753
b5cdbf6d793a print total allocated bytes for compilations
twisti
parents: 14724
diff changeset
240
10672
e7d07c9bb779 Removed priority compilation queue.
Christian Haeubl <haeubl@ssw.jku.at>
parents: 10640
diff changeset
241 try (TimerCloseable a = CompilationTime.start()) {
14107
800057208a2c enable C1 + Graal tiered
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 14104
diff changeset
242 if (!tryToChangeStatus(CompilationStatus.Queued, CompilationStatus.Running)) {
800057208a2c enable C1 + Graal tiered
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 14104
diff changeset
243 return;
800057208a2c enable C1 + Graal tiered
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 14104
diff changeset
244 }
800057208a2c enable C1 + Graal tiered
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 14104
diff changeset
245 boolean isOSR = entryBCI != StructuredGraph.INVOCATION_ENTRY_BCI;
800057208a2c enable C1 + Graal tiered
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 14104
diff changeset
246
800057208a2c enable C1 + Graal tiered
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 14104
diff changeset
247 // 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
248 // 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
249 // only need to check for that value.
800057208a2c enable C1 + Graal tiered
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 14104
diff changeset
250 if (method.hasCodeAtLevel(entryBCI, config.compilationLevelFullOptimization)) {
10672
e7d07c9bb779 Removed priority compilation queue.
Christian Haeubl <haeubl@ssw.jku.at>
parents: 10640
diff changeset
251 return;
e7d07c9bb779 Removed priority compilation queue.
Christian Haeubl <haeubl@ssw.jku.at>
parents: 10640
diff changeset
252 }
10640
88672775a26c Compilation policy fixes and changed default compilation policy.
Christian Haeubl <haeubl@ssw.jku.at>
parents: 10577
diff changeset
253
14107
800057208a2c enable C1 + Graal tiered
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 14104
diff changeset
254 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
255 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
256 if (printCompilation) {
11828
1d64bfb3f481 new PrintAfterCompilation option, simplify PrintCompilation output
Lukas Stadler <lukas.stadler@jku.at>
parents: 11487
diff changeset
257 TTY.println(getMethodDescription() + "...");
5136
8f4f0ebffca2 Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff changeset
258 }
9864
063a712fe8d8 converted remaining options in GraalOptions to new system (GRAAL-27)
Doug Simon <doug.simon@oracle.com>
parents: 9849
diff changeset
259 if (HotSpotPrintCompilation.getValue()) {
8996
6d86ce1297bc GRAAL-213: add HotSpot-style PrintCompilation and PrintInlining
twisti
parents: 8637
diff changeset
260 printCompilation();
6d86ce1297bc GRAAL-213: add HotSpot-style PrintCompilation and PrintInlining
twisti
parents: 8637
diff changeset
261 }
5136
8f4f0ebffca2 Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff changeset
262
5546
e42c0df7212a Rename CiTargetMethod => CompilationResult.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents: 5541
diff changeset
263 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
264 TTY.Filter filter = new TTY.Filter(PrintFilter.getValue(), method);
14753
b5cdbf6d793a print total allocated bytes for compilations
twisti
parents: 14724
diff changeset
265 final long start = System.currentTimeMillis();
b5cdbf6d793a print total allocated bytes for compilations
twisti
parents: 14724
diff changeset
266 final long allocatedBytesBefore = threadMXBean.getThreadAllocatedBytes(threadId);
b5cdbf6d793a print total allocated bytes for compilations
twisti
parents: 14724
diff changeset
267
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 try (Scope s = Debug.scope("Compiling", new DebugDumpScope(String.valueOf(id), true))) {
14831
cbf616a24600 Added baseline path in runCompilation
Niclas Adlertz <niclas.adlertz@oracle.com>
parents: 14552
diff changeset
269
cbf616a24600 Added baseline path in runCompilation
Niclas Adlertz <niclas.adlertz@oracle.com>
parents: 14552
diff changeset
270 if (UseBaselineCompiler.getValue() == true) {
cbf616a24600 Added baseline path in runCompilation
Niclas Adlertz <niclas.adlertz@oracle.com>
parents: 14552
diff changeset
271 HotSpotProviders providers = backend.getProviders();
cbf616a24600 Added baseline path in runCompilation
Niclas Adlertz <niclas.adlertz@oracle.com>
parents: 14552
diff changeset
272 BaselineCompiler baselineCompiler = new BaselineCompiler(GraphBuilderConfiguration.getDefault(), providers.getMetaAccess());
14919
bd08e610e6f3 BaselineCompiler: create BytecodeParser.
Josef Eisl <josef.eisl@jku.at>
parents: 14908
diff changeset
273 OptimisticOptimizations optimisticOpts = OptimisticOptimizations.ALL;
bd08e610e6f3 BaselineCompiler: create BytecodeParser.
Josef Eisl <josef.eisl@jku.at>
parents: 14908
diff changeset
274 result = baselineCompiler.generate(method, -1, backend, new CompilationResult(), method, CompilationResultBuilderFactory.Default, optimisticOpts);
14831
cbf616a24600 Added baseline path in runCompilation
Niclas Adlertz <niclas.adlertz@oracle.com>
parents: 14552
diff changeset
275 } else {
cbf616a24600 Added baseline path in runCompilation
Niclas Adlertz <niclas.adlertz@oracle.com>
parents: 14552
diff changeset
276 Map<ResolvedJavaMethod, StructuredGraph> graphCache = null;
cbf616a24600 Added baseline path in runCompilation
Niclas Adlertz <niclas.adlertz@oracle.com>
parents: 14552
diff changeset
277 if (GraalOptions.CacheGraphs.getValue()) {
cbf616a24600 Added baseline path in runCompilation
Niclas Adlertz <niclas.adlertz@oracle.com>
parents: 14552
diff changeset
278 graphCache = new HashMap<>();
cbf616a24600 Added baseline path in runCompilation
Niclas Adlertz <niclas.adlertz@oracle.com>
parents: 14552
diff changeset
279 }
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
280
14831
cbf616a24600 Added baseline path in runCompilation
Niclas Adlertz <niclas.adlertz@oracle.com>
parents: 14552
diff changeset
281 HotSpotProviders providers = backend.getProviders();
cbf616a24600 Added baseline path in runCompilation
Niclas Adlertz <niclas.adlertz@oracle.com>
parents: 14552
diff changeset
282 Replacements replacements = providers.getReplacements();
cbf616a24600 Added baseline path in runCompilation
Niclas Adlertz <niclas.adlertz@oracle.com>
parents: 14552
diff changeset
283 graph = replacements.getMethodSubstitution(method);
cbf616a24600 Added baseline path in runCompilation
Niclas Adlertz <niclas.adlertz@oracle.com>
parents: 14552
diff changeset
284 if (graph == null || entryBCI != INVOCATION_ENTRY_BCI) {
cbf616a24600 Added baseline path in runCompilation
Niclas Adlertz <niclas.adlertz@oracle.com>
parents: 14552
diff changeset
285 graph = new StructuredGraph(method, entryBCI);
cbf616a24600 Added baseline path in runCompilation
Niclas Adlertz <niclas.adlertz@oracle.com>
parents: 14552
diff changeset
286 } else {
cbf616a24600 Added baseline path in runCompilation
Niclas Adlertz <niclas.adlertz@oracle.com>
parents: 14552
diff changeset
287 // Compiling method substitution - must clone the graph
cbf616a24600 Added baseline path in runCompilation
Niclas Adlertz <niclas.adlertz@oracle.com>
parents: 14552
diff changeset
288 graph = graph.copy();
cbf616a24600 Added baseline path in runCompilation
Niclas Adlertz <niclas.adlertz@oracle.com>
parents: 14552
diff changeset
289 }
cbf616a24600 Added baseline path in runCompilation
Niclas Adlertz <niclas.adlertz@oracle.com>
parents: 14552
diff changeset
290 InlinedBytecodes.add(method.getCodeSize());
cbf616a24600 Added baseline path in runCompilation
Niclas Adlertz <niclas.adlertz@oracle.com>
parents: 14552
diff changeset
291 CallingConvention cc = getCallingConvention(providers.getCodeCache(), Type.JavaCallee, graph.method(), false);
cbf616a24600 Added baseline path in runCompilation
Niclas Adlertz <niclas.adlertz@oracle.com>
parents: 14552
diff changeset
292 if (graph.getEntryBCI() != StructuredGraph.INVOCATION_ENTRY_BCI) {
cbf616a24600 Added baseline path in runCompilation
Niclas Adlertz <niclas.adlertz@oracle.com>
parents: 14552
diff changeset
293 // for OSR, only a pointer is passed to the method.
cbf616a24600 Added baseline path in runCompilation
Niclas Adlertz <niclas.adlertz@oracle.com>
parents: 14552
diff changeset
294 JavaType[] parameterTypes = new JavaType[]{providers.getMetaAccess().lookupJavaType(long.class)};
cbf616a24600 Added baseline path in runCompilation
Niclas Adlertz <niclas.adlertz@oracle.com>
parents: 14552
diff changeset
295 CallingConvention tmp = providers.getCodeCache().getRegisterConfig().getCallingConvention(JavaCallee, providers.getMetaAccess().lookupJavaType(void.class), parameterTypes,
cbf616a24600 Added baseline path in runCompilation
Niclas Adlertz <niclas.adlertz@oracle.com>
parents: 14552
diff changeset
296 backend.getTarget(), false);
cbf616a24600 Added baseline path in runCompilation
Niclas Adlertz <niclas.adlertz@oracle.com>
parents: 14552
diff changeset
297 cc = new CallingConvention(cc.getStackSize(), cc.getReturn(), tmp.getArgument(0));
cbf616a24600 Added baseline path in runCompilation
Niclas Adlertz <niclas.adlertz@oracle.com>
parents: 14552
diff changeset
298 }
cbf616a24600 Added baseline path in runCompilation
Niclas Adlertz <niclas.adlertz@oracle.com>
parents: 14552
diff changeset
299 Suites suites = getSuites(providers);
cbf616a24600 Added baseline path in runCompilation
Niclas Adlertz <niclas.adlertz@oracle.com>
parents: 14552
diff changeset
300 ProfilingInfo profilingInfo = getProfilingInfo();
cbf616a24600 Added baseline path in runCompilation
Niclas Adlertz <niclas.adlertz@oracle.com>
parents: 14552
diff changeset
301 OptimisticOptimizations optimisticOpts = getOptimisticOpts(profilingInfo);
cbf616a24600 Added baseline path in runCompilation
Niclas Adlertz <niclas.adlertz@oracle.com>
parents: 14552
diff changeset
302 result = compileGraph(graph, null, cc, method, providers, backend, backend.getTarget(), graphCache, getGraphBuilderSuite(providers), optimisticOpts, profilingInfo,
cbf616a24600 Added baseline path in runCompilation
Niclas Adlertz <niclas.adlertz@oracle.com>
parents: 14552
diff changeset
303 method.getSpeculationLog(), suites, new CompilationResult(), CompilationResultBuilderFactory.Default);
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
304 }
13630
b1838411e896 Use compile ids assigned by hotspot
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13585
diff changeset
305 result.setId(getId());
b1838411e896 Use compile ids assigned by hotspot
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13585
diff changeset
306 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
307 } 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
308 throw Debug.handle(e);
5136
8f4f0ebffca2 Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff changeset
309 } finally {
8f4f0ebffca2 Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff changeset
310 filter.remove();
11828
1d64bfb3f481 new PrintAfterCompilation option, simplify PrintCompilation output
Lukas Stadler <lukas.stadler@jku.at>
parents: 11487
diff changeset
311 final boolean printAfterCompilation = PrintAfterCompilation.getValue() && !TTY.isSuppressed();
14753
b5cdbf6d793a print total allocated bytes for compilations
twisti
parents: 14724
diff changeset
312
b5cdbf6d793a print total allocated bytes for compilations
twisti
parents: 14724
diff changeset
313 if (printAfterCompilation || printCompilation) {
b5cdbf6d793a print total allocated bytes for compilations
twisti
parents: 14724
diff changeset
314 final long stop = System.currentTimeMillis();
b5cdbf6d793a print total allocated bytes for compilations
twisti
parents: 14724
diff changeset
315 final int targetCodeSize = result != null ? result.getTargetCodeSize() : -1;
b5cdbf6d793a print total allocated bytes for compilations
twisti
parents: 14724
diff changeset
316 final long allocatedBytesAfter = threadMXBean.getThreadAllocatedBytes(threadId);
b5cdbf6d793a print total allocated bytes for compilations
twisti
parents: 14724
diff changeset
317 final long allocatedBytes = (allocatedBytesAfter - allocatedBytesBefore) / 1024;
b5cdbf6d793a print total allocated bytes for compilations
twisti
parents: 14724
diff changeset
318
b5cdbf6d793a print total allocated bytes for compilations
twisti
parents: 14724
diff changeset
319 if (printAfterCompilation) {
b5cdbf6d793a print total allocated bytes for compilations
twisti
parents: 14724
diff changeset
320 TTY.println(getMethodDescription() + String.format(" | %4dms %5dB %5dkB", stop - start, targetCodeSize, allocatedBytes));
b5cdbf6d793a print total allocated bytes for compilations
twisti
parents: 14724
diff changeset
321 } else if (printCompilation) {
b5cdbf6d793a print total allocated bytes for compilations
twisti
parents: 14724
diff changeset
322 TTY.println(String.format("%-6d Graal %-70s %-45s %-50s | %4dms %5dB %5dkB", id, "", "", "", stop - start, targetCodeSize, allocatedBytes));
b5cdbf6d793a print total allocated bytes for compilations
twisti
parents: 14724
diff changeset
323 }
5136
8f4f0ebffca2 Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff changeset
324 }
8f4f0ebffca2 Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff changeset
325 }
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
326
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
327 try (TimerCloseable 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
328 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
329 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
330 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
331 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
332 }
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
333 }
14545
4877b0cb446f removed ResolvedJavaMethod.getCompiledCodeSize()
Doug Simon <doug.simon@oracle.com>
parents: 14543
diff changeset
334 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
335 } catch (BailoutException bailout) {
14016
555867401850 Make Debug.metric objects static
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 13912
diff changeset
336 BAILOUTS.increment();
9864
063a712fe8d8 converted remaining options in GraalOptions to new system (GRAAL-27)
Doug Simon <doug.simon@oracle.com>
parents: 9849
diff changeset
337 if (ExitVMOnBailout.getValue()) {
5719
429accae15aa moved some methods from CodeUtil to MetaUtil
Doug Simon <doug.simon@oracle.com>
parents: 5603
diff changeset
338 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
339 bailout.printStackTrace(TTY.cachedOut);
8f4f0ebffca2 Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff changeset
340 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
341 } else if (PrintBailout.getValue()) {
5719
429accae15aa moved some methods from CodeUtil to MetaUtil
Doug Simon <doug.simon@oracle.com>
parents: 5603
diff changeset
342 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
343 bailout.printStackTrace(TTY.cachedOut);
5136
8f4f0ebffca2 Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff changeset
344 }
8f4f0ebffca2 Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff changeset
345 } 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
346 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
347 t.printStackTrace(TTY.cachedOut);
9dcd6f8a843b CompileTask: add option to print exception on compile error
Bernhard Urban <bernhard.urban@jku.at>
parents: 9790
diff changeset
348 }
9864
063a712fe8d8 converted remaining options in GraalOptions to new system (GRAAL-27)
Doug Simon <doug.simon@oracle.com>
parents: 9849
diff changeset
349 if (ExitVMOnException.getValue()) {
5136
8f4f0ebffca2 Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff changeset
350 System.exit(-1);
8f4f0ebffca2 Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff changeset
351 }
10056
a323a9e20f9d Fixed a few race conditions in the compilation queue.
Christian Haeubl <haeubl@ssw.jku.at>
parents: 10019
diff changeset
352 } finally {
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
353 int processedBytes = (int) (InlinedBytecodes.getCurrentValue() - previousInlinedBytecodes);
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
354 if (ctask != 0L) {
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
355 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
356 }
13464
2236d18302e0 made -G:PrintCompRate incompatible with -XX:+CITime and -XX:+CITimeEach
Doug Simon <doug.simon@oracle.com>
parents: 13452
diff changeset
357 if ((config.ciTime || config.ciTimeEach || PrintCompRate.getValue() != 0) && installedCode != null) {
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
358
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
359 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
360 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
361 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
362 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
363 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
364 }
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
365
13366
1480cfe97462 CTWCompilationTask should not be removed from compilation queue
Doug Simon <doug.simon@oracle.com>
parents: 13353
diff changeset
366 if (clearFromCompilationQueue) {
1480cfe97462 CTWCompilationTask should not be removed from compilation queue
Doug Simon <doug.simon@oracle.com>
parents: 13353
diff changeset
367 assert method.isQueuedForCompilation();
1480cfe97462 CTWCompilationTask should not be removed from compilation queue
Doug Simon <doug.simon@oracle.com>
parents: 13353
diff changeset
368 method.clearQueuedForCompilation();
1480cfe97462 CTWCompilationTask should not be removed from compilation queue
Doug Simon <doug.simon@oracle.com>
parents: 13353
diff changeset
369 }
5136
8f4f0ebffca2 Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff changeset
370 }
5176
af59b4dfc9e4 compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents: 5138
diff changeset
371 }
af59b4dfc9e4 compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents: 5138
diff changeset
372
11828
1d64bfb3f481 new PrintAfterCompilation option, simplify PrintCompilation output
Lukas Stadler <lukas.stadler@jku.at>
parents: 11487
diff changeset
373 private String getMethodDescription() {
1d64bfb3f481 new PrintAfterCompilation option, simplify PrintCompilation output
Lukas Stadler <lukas.stadler@jku.at>
parents: 11487
diff changeset
374 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
375 entryBCI == StructuredGraph.INVOCATION_ENTRY_BCI ? "" : "(OSR@" + entryBCI + ") ");
1d64bfb3f481 new PrintAfterCompilation option, simplify PrintCompilation output
Lukas Stadler <lukas.stadler@jku.at>
parents: 11487
diff changeset
376 }
1d64bfb3f481 new PrintAfterCompilation option, simplify PrintCompilation output
Lukas Stadler <lukas.stadler@jku.at>
parents: 11487
diff changeset
377
8996
6d86ce1297bc GRAAL-213: add HotSpot-style PrintCompilation and PrintInlining
twisti
parents: 8637
diff changeset
378 /**
6d86ce1297bc GRAAL-213: add HotSpot-style PrintCompilation and PrintInlining
twisti
parents: 8637
diff changeset
379 * Print a HotSpot-style compilation message to the console.
6d86ce1297bc GRAAL-213: add HotSpot-style PrintCompilation and PrintInlining
twisti
parents: 8637
diff changeset
380 */
6d86ce1297bc GRAAL-213: add HotSpot-style PrintCompilation and PrintInlining
twisti
parents: 8637
diff changeset
381 private void printCompilation() {
6d86ce1297bc GRAAL-213: add HotSpot-style PrintCompilation and PrintInlining
twisti
parents: 8637
diff changeset
382 final boolean isOSR = entryBCI != StructuredGraph.INVOCATION_ENTRY_BCI;
14549
36d39c597c79 Respect CIPrintCompilerName and output a timestamp in Graal's PrintCompilation implementation
Gilles Duboscq <duboscq@ssw.jku.at>
parents: 14545
diff changeset
383 String compilerName = "";
36d39c597c79 Respect CIPrintCompilerName and output a timestamp in Graal's PrintCompilation implementation
Gilles Duboscq <duboscq@ssw.jku.at>
parents: 14545
diff changeset
384 if (HotSpotCIPrintCompilerName.getValue()) {
36d39c597c79 Respect CIPrintCompilerName and output a timestamp in Graal's PrintCompilation implementation
Gilles Duboscq <duboscq@ssw.jku.at>
parents: 14545
diff changeset
385 compilerName = "Graal:";
36d39c597c79 Respect CIPrintCompilerName and output a timestamp in Graal's PrintCompilation implementation
Gilles Duboscq <duboscq@ssw.jku.at>
parents: 14545
diff changeset
386 }
14551
bf7d914b8ef7 Print compilation level in Graal's PrintCompilation output
Gilles Duboscq <duboscq@ssw.jku.at>
parents: 14549
diff changeset
387 HotSpotVMConfig config = backend.getRuntime().getConfig();
bf7d914b8ef7 Print compilation level in Graal's PrintCompilation output
Gilles Duboscq <duboscq@ssw.jku.at>
parents: 14549
diff changeset
388 int compLevel = config.compilationLevelFullOptimization;
15045
0286888f792b fix PrintCompilation formatting and use same time source
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 15040
diff changeset
389 String compLevelString;
14551
bf7d914b8ef7 Print compilation level in Graal's PrintCompilation output
Gilles Duboscq <duboscq@ssw.jku.at>
parents: 14549
diff changeset
390 if (config.tieredCompilation) {
15045
0286888f792b fix PrintCompilation formatting and use same time source
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 15040
diff changeset
391 compLevelString = "- ";
14551
bf7d914b8ef7 Print compilation level in Graal's PrintCompilation output
Gilles Duboscq <duboscq@ssw.jku.at>
parents: 14549
diff changeset
392 if (compLevel != -1) {
15045
0286888f792b fix PrintCompilation formatting and use same time source
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 15040
diff changeset
393 compLevelString = (char) ('0' + compLevel) + " ";
14551
bf7d914b8ef7 Print compilation level in Graal's PrintCompilation output
Gilles Duboscq <duboscq@ssw.jku.at>
parents: 14549
diff changeset
394 }
bf7d914b8ef7 Print compilation level in Graal's PrintCompilation output
Gilles Duboscq <duboscq@ssw.jku.at>
parents: 14549
diff changeset
395 } else {
15045
0286888f792b fix PrintCompilation formatting and use same time source
Tom Rodriguez <tom.rodriguez@oracle.com>
parents: 15040
diff changeset
396 compLevelString = "";
14551
bf7d914b8ef7 Print compilation level in Graal's PrintCompilation output
Gilles Duboscq <duboscq@ssw.jku.at>
parents: 14549
diff changeset
397 }
14552
2c78ee2a890f Graal PrintCompilation: print exception handlers and blocking flags
Gilles Duboscq <duboscq@ssw.jku.at>
parents: 14551
diff changeset
398 boolean hasExceptionHandlers = method.getExceptionHandlers().length > 0;
15311
820c6d353358 added ModifiersProvider as superinterface for ResolvedJava[Method|Field|Type] and implemented all checks against modifiers as default methods
Doug Simon <doug.simon@oracle.com>
parents: 15259
diff changeset
399 TTY.println(String.format("%s%7d %4d %c%c%c%c%c %s %s %s(%d bytes)", compilerName, backend.getRuntime().compilerToVm.getTimeStamp(), id, isOSR ? '%' : ' ', method.isSynchronized() ? 's'
820c6d353358 added ModifiersProvider as superinterface for ResolvedJava[Method|Field|Type] and implemented all checks against modifiers as default methods
Doug Simon <doug.simon@oracle.com>
parents: 15259
diff changeset
400 : ' ', hasExceptionHandlers ? '!' : ' ', blocking ? 'b' : ' ', method.isNative() ? 'n' : ' ', compLevelString, MetaUtil.format("%H::%n(%p)", method), isOSR ? "@ " + entryBCI +
820c6d353358 added ModifiersProvider as superinterface for ResolvedJava[Method|Field|Type] and implemented all checks against modifiers as default methods
Doug Simon <doug.simon@oracle.com>
parents: 15259
diff changeset
401 " " : "", method.getCodeSize()));
8996
6d86ce1297bc GRAAL-213: add HotSpot-style PrintCompilation and PrintInlining
twisti
parents: 8637
diff changeset
402 }
6d86ce1297bc GRAAL-213: add HotSpot-style PrintCompilation and PrintInlining
twisti
parents: 8637
diff changeset
403
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
404 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
405 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
406 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
407 try (Scope s = Debug.scope("CodeInstall", new DebugDumpScope(String.valueOf(id), true), codeCache, method)) {
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
408 installedCode = codeCache.installMethod(method, compResult, ctask);
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
409 } 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
410 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
411 }
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
412 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
413 }
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
414
10056
a323a9e20f9d Fixed a few race conditions in the compilation queue.
Christian Haeubl <haeubl@ssw.jku.at>
parents: 10019
diff changeset
415 private boolean tryToChangeStatus(CompilationStatus from, CompilationStatus to) {
10076
25de9c96a032 Minor CompilationTask refactoring.
Christian Haeubl <haeubl@ssw.jku.at>
parents: 10056
diff changeset
416 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
417 }
a323a9e20f9d Fixed a few race conditions in the compilation queue.
Christian Haeubl <haeubl@ssw.jku.at>
parents: 10019
diff changeset
418
5176
af59b4dfc9e4 compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents: 5138
diff changeset
419 @Override
7382
31d1cc9219d8 gave CompilationTask a toString() method
Doug Simon <doug.simon@oracle.com>
parents: 7377
diff changeset
420 public String toString() {
10672
e7d07c9bb779 Removed priority compilation queue.
Christian Haeubl <haeubl@ssw.jku.at>
parents: 10640
diff changeset
421 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
422 }
5136
8f4f0ebffca2 Move compilation task logic to separate class.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
diff changeset
423 }