Mercurial > hg > truffle
annotate src/share/vm/runtime/compilationPolicy.cpp @ 6290:01d274503562
canonicalize reads and writes on null objects to deopts
author | Lukas Stadler <lukas.stadler@jku.at> |
---|---|
date | Tue, 28 Aug 2012 12:15:10 +0200 |
parents | d87155082c4d |
children | fd71ca8c5f88 |
rev | line source |
---|---|
0 | 1 /* |
2348
5d8f5a6dced7
7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
1972
diff
changeset
|
2 * Copyright (c) 2000, 2011, Oracle and/or its affiliates. All rights reserved. |
0 | 3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
4 * | |
5 * This code is free software; you can redistribute it and/or modify it | |
6 * under the terms of the GNU General Public License version 2 only, as | |
7 * published by the Free Software Foundation. | |
8 * | |
9 * This code is distributed in the hope that it will be useful, but WITHOUT | |
10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or | |
11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License | |
12 * version 2 for more details (a copy is included in the LICENSE file that | |
13 * accompanied this code). | |
14 * | |
15 * You should have received a copy of the GNU General Public License version | |
16 * 2 along with this work; if not, write to the Free Software Foundation, | |
17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. | |
18 * | |
1552
c18cbe5936b8
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
1202
diff
changeset
|
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
c18cbe5936b8
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
1202
diff
changeset
|
20 * or visit www.oracle.com if you need additional information or have any |
c18cbe5936b8
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
1202
diff
changeset
|
21 * questions. |
0 | 22 * |
23 */ | |
24 | |
1972 | 25 #include "precompiled.hpp" |
26 #include "code/compiledIC.hpp" | |
27 #include "code/nmethod.hpp" | |
28 #include "code/scopeDesc.hpp" | |
29 #include "compiler/compilerOracle.hpp" | |
30 #include "interpreter/interpreter.hpp" | |
31 #include "oops/methodDataOop.hpp" | |
32 #include "oops/methodOop.hpp" | |
33 #include "oops/oop.inline.hpp" | |
34 #include "prims/nativeLookup.hpp" | |
2348
5d8f5a6dced7
7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
1972
diff
changeset
|
35 #include "runtime/advancedThresholdPolicy.hpp" |
1972 | 36 #include "runtime/compilationPolicy.hpp" |
37 #include "runtime/frame.hpp" | |
38 #include "runtime/handles.inline.hpp" | |
39 #include "runtime/rframe.hpp" | |
40 #include "runtime/simpleThresholdPolicy.hpp" | |
41 #include "runtime/stubRoutines.hpp" | |
42 #include "runtime/thread.hpp" | |
43 #include "runtime/timer.hpp" | |
44 #include "runtime/vframe.hpp" | |
45 #include "runtime/vm_operations.hpp" | |
46 #include "utilities/events.hpp" | |
47 #include "utilities/globalDefinitions.hpp" | |
0 | 48 |
49 CompilationPolicy* CompilationPolicy::_policy; | |
50 elapsedTimer CompilationPolicy::_accumulated_time; | |
51 bool CompilationPolicy::_in_vm_startup; | |
52 | |
53 // Determine compilation policy based on command line argument | |
54 void compilationPolicy_init() { | |
55 CompilationPolicy::set_in_vm_startup(DelayCompilationDuringStartup); | |
56 | |
57 switch(CompilationPolicyChoice) { | |
58 case 0: | |
59 CompilationPolicy::set_policy(new SimpleCompPolicy()); | |
60 break; | |
61 | |
62 case 1: | |
4977
532be189cf09
Reducing diff to hsx24
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
4970
diff
changeset
|
63 #if defined(COMPILER2) || defined(GRAAL) |
0 | 64 CompilationPolicy::set_policy(new StackWalkCompPolicy()); |
4977
532be189cf09
Reducing diff to hsx24
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
4970
diff
changeset
|
65 #else |
532be189cf09
Reducing diff to hsx24
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
4970
diff
changeset
|
66 Unimplemented(); |
532be189cf09
Reducing diff to hsx24
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
4970
diff
changeset
|
67 #endif |
0 | 68 break; |
1783 | 69 case 2: |
70 #ifdef TIERED | |
71 CompilationPolicy::set_policy(new SimpleThresholdPolicy()); | |
72 #else | |
73 Unimplemented(); | |
74 #endif | |
75 break; | |
2348
5d8f5a6dced7
7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
1972
diff
changeset
|
76 case 3: |
5d8f5a6dced7
7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
1972
diff
changeset
|
77 #ifdef TIERED |
5d8f5a6dced7
7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
1972
diff
changeset
|
78 CompilationPolicy::set_policy(new AdvancedThresholdPolicy()); |
5d8f5a6dced7
7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
1972
diff
changeset
|
79 #else |
5d8f5a6dced7
7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
1972
diff
changeset
|
80 Unimplemented(); |
5d8f5a6dced7
7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
1972
diff
changeset
|
81 #endif |
5d8f5a6dced7
7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
1972
diff
changeset
|
82 break; |
5176
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
83 case 4: |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
84 #if defined(GRAAL) |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
85 CompilationPolicy::set_policy(new GraalCompPolicy()); |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
86 #else |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
87 Unimplemented(); |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
88 #endif |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
89 break; |
0 | 90 default: |
5176
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
91 fatal("CompilationPolicyChoice must be in the range: [0-4]"); |
0 | 92 } |
1783 | 93 CompilationPolicy::policy()->initialize(); |
0 | 94 } |
95 | |
96 void CompilationPolicy::completed_vm_startup() { | |
97 if (TraceCompilationPolicy) { | |
98 tty->print("CompilationPolicy: completed vm startup.\n"); | |
99 } | |
100 _in_vm_startup = false; | |
101 } | |
102 | |
103 // Returns true if m must be compiled before executing it | |
104 // This is intended to force compiles for methods (usually for | |
105 // debugging) that would otherwise be interpreted for some reason. | |
1783 | 106 bool CompilationPolicy::must_be_compiled(methodHandle m, int comp_level) { |
0 | 107 if (m->has_compiled_code()) return false; // already compiled |
1783 | 108 if (!can_be_compiled(m, comp_level)) return false; |
0 | 109 |
110 return !UseInterpreter || // must compile all methods | |
1202 | 111 (UseCompiler && AlwaysCompileLoopMethods && m->has_loops() && CompileBroker::should_compile_new_jobs()); // eagerly compile loop methods |
0 | 112 } |
113 | |
114 // Returns true if m is allowed to be compiled | |
1783 | 115 bool CompilationPolicy::can_be_compiled(methodHandle m, int comp_level) { |
0 | 116 if (m->is_abstract()) return false; |
117 if (DontCompileHugeMethods && m->code_size() > HugeMethodLimit) return false; | |
118 | |
1174
ddb7834449d0
6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents:
0
diff
changeset
|
119 // Math intrinsics should never be compiled as this can lead to |
ddb7834449d0
6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents:
0
diff
changeset
|
120 // monotonicity problems because the interpreter will prefer the |
ddb7834449d0
6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents:
0
diff
changeset
|
121 // compiled code to the intrinsic version. This can't happen in |
ddb7834449d0
6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents:
0
diff
changeset
|
122 // production because the invocation counter can't be incremented |
ddb7834449d0
6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents:
0
diff
changeset
|
123 // but we shouldn't expose the system to this problem in testing |
ddb7834449d0
6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents:
0
diff
changeset
|
124 // modes. |
ddb7834449d0
6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents:
0
diff
changeset
|
125 if (!AbstractInterpreter::can_be_compiled(m)) { |
ddb7834449d0
6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents:
0
diff
changeset
|
126 return false; |
ddb7834449d0
6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents:
0
diff
changeset
|
127 } |
1783 | 128 if (comp_level == CompLevel_all) { |
129 return !m->is_not_compilable(CompLevel_simple) && !m->is_not_compilable(CompLevel_full_optimization); | |
130 } else { | |
131 return !m->is_not_compilable(comp_level); | |
132 } | |
133 } | |
1174
ddb7834449d0
6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents:
0
diff
changeset
|
134 |
1783 | 135 bool CompilationPolicy::is_compilation_enabled() { |
136 // NOTE: CompileBroker::should_compile_new_jobs() checks for UseCompiler | |
137 return !delay_compilation_during_startup() && CompileBroker::should_compile_new_jobs(); | |
0 | 138 } |
139 | |
140 #ifndef PRODUCT | |
141 void CompilationPolicy::print_time() { | |
142 tty->print_cr ("Accumulated compilationPolicy times:"); | |
143 tty->print_cr ("---------------------------"); | |
144 tty->print_cr (" Total: %3.3f sec.", _accumulated_time.seconds()); | |
145 } | |
146 | |
1783 | 147 void NonTieredCompPolicy::trace_osr_completion(nmethod* osr_nm) { |
0 | 148 if (TraceOnStackReplacement) { |
149 if (osr_nm == NULL) tty->print_cr("compilation failed"); | |
150 else tty->print_cr("nmethod " INTPTR_FORMAT, osr_nm); | |
151 } | |
152 } | |
153 #endif // !PRODUCT | |
154 | |
1783 | 155 void NonTieredCompPolicy::initialize() { |
156 // Setup the compiler thread numbers | |
157 if (CICompilerCountPerCPU) { | |
158 // Example: if CICompilerCountPerCPU is true, then we get | |
159 // max(log2(8)-1,1) = 2 compiler threads on an 8-way machine. | |
160 // May help big-app startup time. | |
161 _compiler_count = MAX2(log2_intptr(os::active_processor_count())-1,1); | |
162 } else { | |
163 _compiler_count = CICompilerCount; | |
164 } | |
165 } | |
166 | |
1821
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
167 // Note: this policy is used ONLY if TieredCompilation is off. |
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
168 // compiler_count() behaves the following way: |
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
169 // - with TIERED build (with both COMPILER1 and COMPILER2 defined) it should return |
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
170 // zero for the c1 compilation levels, hence the particular ordering of the |
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
171 // statements. |
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
172 // - the same should happen when COMPILER2 is defined and COMPILER1 is not |
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
173 // (server build without TIERED defined). |
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
174 // - if only COMPILER1 is defined (client build), zero should be returned for |
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
175 // the c2 level. |
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
176 // - if neither is defined - always return zero. |
1783 | 177 int NonTieredCompPolicy::compiler_count(CompLevel comp_level) { |
1821
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
178 assert(!TieredCompilation, "This policy should not be used with TieredCompilation"); |
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
179 #ifdef COMPILER2 |
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
180 if (is_c2_compile(comp_level)) { |
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
181 return _compiler_count; |
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
182 } else { |
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
183 return 0; |
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
184 } |
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
185 #endif |
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
186 |
1783 | 187 #ifdef COMPILER1 |
188 if (is_c1_compile(comp_level)) { | |
189 return _compiler_count; | |
1821
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
190 } else { |
df015ec64052
6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents:
1783
diff
changeset
|
191 return 0; |
1783 | 192 } |
193 #endif | |
194 | |
195 return 0; | |
196 } | |
197 | |
198 void NonTieredCompPolicy::reset_counter_for_invocation_event(methodHandle m) { | |
0 | 199 // Make sure invocation and backedge counter doesn't overflow again right away |
200 // as would be the case for native methods. | |
201 | |
202 // BUT also make sure the method doesn't look like it was never executed. | |
203 // Set carry bit and reduce counter's value to min(count, CompileThreshold/2). | |
204 m->invocation_counter()->set_carry(); | |
205 m->backedge_counter()->set_carry(); | |
206 | |
207 assert(!m->was_never_executed(), "don't reset to 0 -- could be mistaken for never-executed"); | |
208 } | |
209 | |
1783 | 210 void NonTieredCompPolicy::reset_counter_for_back_branch_event(methodHandle m) { |
5176
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
211 GRAAL_ONLY(assert(false, "unexpected")); |
0 | 212 // Delay next back-branch event but pump up invocation counter to triger |
213 // whole method compilation. | |
214 InvocationCounter* i = m->invocation_counter(); | |
215 InvocationCounter* b = m->backedge_counter(); | |
216 | |
217 // Don't set invocation_counter's value too low otherwise the method will | |
218 // look like immature (ic < ~5300) which prevents the inlining based on | |
219 // the type profiling. | |
220 i->set(i->state(), CompileThreshold); | |
221 // Don't reset counter too low - it is used to check if OSR method is ready. | |
222 b->set(b->state(), CompileThreshold / 2); | |
223 } | |
224 | |
1783 | 225 // |
226 // CounterDecay | |
227 // | |
228 // Interates through invocation counters and decrements them. This | |
229 // is done at each safepoint. | |
230 // | |
231 class CounterDecay : public AllStatic { | |
232 static jlong _last_timestamp; | |
233 static void do_method(methodOop m) { | |
234 m->invocation_counter()->decay(); | |
235 } | |
236 public: | |
237 static void decay(); | |
238 static bool is_decay_needed() { | |
239 return (os::javaTimeMillis() - _last_timestamp) > CounterDecayMinIntervalLength; | |
240 } | |
241 }; | |
242 | |
243 jlong CounterDecay::_last_timestamp = 0; | |
244 | |
245 void CounterDecay::decay() { | |
246 _last_timestamp = os::javaTimeMillis(); | |
247 | |
248 // This operation is going to be performed only at the end of a safepoint | |
249 // and hence GC's will not be going on, all Java mutators are suspended | |
250 // at this point and hence SystemDictionary_lock is also not needed. | |
251 assert(SafepointSynchronize::is_at_safepoint(), "can only be executed at a safepoint"); | |
252 int nclasses = SystemDictionary::number_of_classes(); | |
253 double classes_per_tick = nclasses * (CounterDecayMinIntervalLength * 1e-3 / | |
254 CounterHalfLifeTime); | |
255 for (int i = 0; i < classes_per_tick; i++) { | |
256 klassOop k = SystemDictionary::try_get_next_class(); | |
257 if (k != NULL && k->klass_part()->oop_is_instance()) { | |
258 instanceKlass::cast(k)->methods_do(do_method); | |
259 } | |
260 } | |
261 } | |
262 | |
263 // Called at the end of the safepoint | |
264 void NonTieredCompPolicy::do_safepoint_work() { | |
265 if(UseCounterDecay && CounterDecay::is_decay_needed()) { | |
266 CounterDecay::decay(); | |
267 } | |
268 } | |
269 | |
270 void NonTieredCompPolicy::reprofile(ScopeDesc* trap_scope, bool is_osr) { | |
271 ScopeDesc* sd = trap_scope; | |
272 for (; !sd->is_top(); sd = sd->sender()) { | |
273 // Reset ICs of inlined methods, since they can trigger compilations also. | |
274 sd->method()->invocation_counter()->reset(); | |
275 } | |
276 InvocationCounter* c = sd->method()->invocation_counter(); | |
277 if (is_osr) { | |
278 // It was an OSR method, so bump the count higher. | |
279 c->set(c->state(), CompileThreshold); | |
280 } else { | |
281 c->reset(); | |
282 } | |
283 sd->method()->backedge_counter()->reset(); | |
284 } | |
285 | |
286 // This method can be called by any component of the runtime to notify the policy | |
287 // that it's recommended to delay the complation of this method. | |
288 void NonTieredCompPolicy::delay_compilation(methodOop method) { | |
289 method->invocation_counter()->decay(); | |
290 method->backedge_counter()->decay(); | |
291 } | |
292 | |
293 void NonTieredCompPolicy::disable_compilation(methodOop method) { | |
294 method->invocation_counter()->set_state(InvocationCounter::wait_for_nothing); | |
295 method->backedge_counter()->set_state(InvocationCounter::wait_for_nothing); | |
296 } | |
297 | |
298 CompileTask* NonTieredCompPolicy::select_task(CompileQueue* compile_queue) { | |
299 return compile_queue->first(); | |
300 } | |
301 | |
302 bool NonTieredCompPolicy::is_mature(methodOop method) { | |
303 methodDataOop mdo = method->method_data(); | |
304 assert(mdo != NULL, "Should be"); | |
305 uint current = mdo->mileage_of(method); | |
306 uint initial = mdo->creation_mileage(); | |
307 if (current < initial) | |
308 return true; // some sort of overflow | |
309 uint target; | |
310 if (ProfileMaturityPercentage <= 0) | |
311 target = (uint) -ProfileMaturityPercentage; // absolute value | |
312 else | |
313 target = (uint)( (ProfileMaturityPercentage * CompileThreshold) / 100 ); | |
314 return (current >= initial + target); | |
315 } | |
316 | |
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
317 nmethod* NonTieredCompPolicy::event(methodHandle method, methodHandle inlinee, int branch_bci, |
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
318 int bci, CompLevel comp_level, nmethod* nm, JavaThread* thread) { |
1783 | 319 assert(comp_level == CompLevel_none, "This should be only called from the interpreter"); |
320 NOT_PRODUCT(trace_frequency_counter_overflow(method, branch_bci, bci)); | |
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
321 if (JvmtiExport::can_post_interpreter_events() && thread->is_interp_only_mode()) { |
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
322 // If certain JVMTI events (e.g. frame pop event) are requested then the |
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
323 // thread is forced to remain in interpreted code. This is |
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
324 // implemented partly by a check in the run_compiled_code |
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
325 // section of the interpreter whether we should skip running |
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
326 // compiled code, and partly by skipping OSR compiles for |
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
327 // interpreted-only threads. |
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
328 if (bci != InvocationEntryBci) { |
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
329 reset_counter_for_back_branch_event(method); |
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
330 return NULL; |
1783 | 331 } |
332 } | |
333 if (bci == InvocationEntryBci) { | |
334 // when code cache is full, compilation gets switched off, UseCompiler | |
335 // is set to false | |
336 if (!method->has_compiled_code() && UseCompiler) { | |
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
337 method_invocation_event(method, thread); |
1783 | 338 } else { |
339 // Force counter overflow on method entry, even if no compilation | |
340 // happened. (The method_invocation_event call does this also.) | |
341 reset_counter_for_invocation_event(method); | |
342 } | |
343 // compilation at an invocation overflow no longer goes and retries test for | |
344 // compiled method. We always run the loser of the race as interpreted. | |
345 // so return NULL | |
346 return NULL; | |
347 } else { | |
348 // counter overflow in a loop => try to do on-stack-replacement | |
349 nmethod* osr_nm = method->lookup_osr_nmethod_for(bci, CompLevel_highest_tier, true); | |
350 NOT_PRODUCT(trace_osr_request(method, osr_nm, bci)); | |
351 // when code cache is full, we should not compile any more... | |
352 if (osr_nm == NULL && UseCompiler) { | |
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
353 method_back_branch_event(method, bci, thread); |
1783 | 354 osr_nm = method->lookup_osr_nmethod_for(bci, CompLevel_highest_tier, true); |
355 } | |
356 if (osr_nm == NULL) { | |
357 reset_counter_for_back_branch_event(method); | |
358 return NULL; | |
359 } | |
360 return osr_nm; | |
361 } | |
362 return NULL; | |
363 } | |
364 | |
365 #ifndef PRODUCT | |
366 void NonTieredCompPolicy::trace_frequency_counter_overflow(methodHandle m, int branch_bci, int bci) { | |
367 if (TraceInvocationCounterOverflow) { | |
368 InvocationCounter* ic = m->invocation_counter(); | |
369 InvocationCounter* bc = m->backedge_counter(); | |
370 ResourceMark rm; | |
371 const char* msg = | |
372 bci == InvocationEntryBci | |
373 ? "comp-policy cntr ovfl @ %d in entry of " | |
374 : "comp-policy cntr ovfl @ %d in loop of "; | |
375 tty->print(msg, bci); | |
376 m->print_value(); | |
377 tty->cr(); | |
378 ic->print(); | |
379 bc->print(); | |
380 if (ProfileInterpreter) { | |
381 if (bci != InvocationEntryBci) { | |
382 methodDataOop mdo = m->method_data(); | |
383 if (mdo != NULL) { | |
384 int count = mdo->bci_to_data(branch_bci)->as_JumpData()->taken(); | |
385 tty->print_cr("back branch count = %d", count); | |
386 } | |
387 } | |
388 } | |
389 } | |
390 } | |
391 | |
392 void NonTieredCompPolicy::trace_osr_request(methodHandle method, nmethod* osr, int bci) { | |
393 if (TraceOnStackReplacement) { | |
394 ResourceMark rm; | |
395 tty->print(osr != NULL ? "Reused OSR entry for " : "Requesting OSR entry for "); | |
396 method->print_short_name(tty); | |
397 tty->print_cr(" at bci %d", bci); | |
398 } | |
399 } | |
400 #endif // !PRODUCT | |
401 | |
0 | 402 // SimpleCompPolicy - compile current method |
403 | |
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
404 void SimpleCompPolicy::method_invocation_event(methodHandle m, JavaThread* thread) { |
0 | 405 int hot_count = m->invocation_count(); |
406 reset_counter_for_invocation_event(m); | |
407 const char* comment = "count"; | |
408 | |
1783 | 409 if (is_compilation_enabled() && can_be_compiled(m)) { |
0 | 410 nmethod* nm = m->code(); |
411 if (nm == NULL ) { | |
412 const char* comment = "count"; | |
1783 | 413 CompileBroker::compile_method(m, InvocationEntryBci, CompLevel_highest_tier, |
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
414 m, hot_count, comment, thread); |
0 | 415 } |
416 } | |
417 } | |
418 | |
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
419 void SimpleCompPolicy::method_back_branch_event(methodHandle m, int bci, JavaThread* thread) { |
0 | 420 int hot_count = m->backedge_count(); |
421 const char* comment = "backedge_count"; | |
422 | |
1783 | 423 if (is_compilation_enabled() && !m->is_not_osr_compilable() && can_be_compiled(m)) { |
424 CompileBroker::compile_method(m, bci, CompLevel_highest_tier, | |
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
425 m, hot_count, comment, thread); |
1783 | 426 NOT_PRODUCT(trace_osr_completion(m->lookup_osr_nmethod_for(bci, CompLevel_highest_tier, true));) |
0 | 427 } |
428 } | |
5176
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
429 |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
430 // GraalCompPolicy - compile current method |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
431 |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
432 #ifdef GRAAL |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
433 |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
434 void GraalCompPolicy::method_invocation_event(methodHandle m, JavaThread* thread) { |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
435 int hot_count = m->invocation_count(); |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
436 jlong hot_time = m->graal_invocation_time(); |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
437 reset_counter_for_invocation_event(m); |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
438 |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
439 if (is_compilation_enabled() && can_be_compiled(m)) { |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
440 nmethod* nm = m->code(); |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
441 if (nm == NULL) { |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
442 if (hot_count > 1) { |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
443 jlong current_time = os::javaTimeNanos(); |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
444 int time_per_call = (int) ((current_time - hot_time) / hot_count); |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
445 m->set_graal_invocation_time(current_time); |
5263
d87155082c4d
guard all compilation queue accesses, to avoid deadlocks
Lukas Stadler <lukas.stadler@jku.at>
parents:
5176
diff
changeset
|
446 if (UseNewCode) { |
d87155082c4d
guard all compilation queue accesses, to avoid deadlocks
Lukas Stadler <lukas.stadler@jku.at>
parents:
5176
diff
changeset
|
447 if (m->queued_for_compilation()) { |
d87155082c4d
guard all compilation queue accesses, to avoid deadlocks
Lukas Stadler <lukas.stadler@jku.at>
parents:
5176
diff
changeset
|
448 if (time_per_call < (m->graal_priority() / 5)) { |
d87155082c4d
guard all compilation queue accesses, to avoid deadlocks
Lukas Stadler <lukas.stadler@jku.at>
parents:
5176
diff
changeset
|
449 m->set_graal_priority(time_per_call); |
d87155082c4d
guard all compilation queue accesses, to avoid deadlocks
Lukas Stadler <lukas.stadler@jku.at>
parents:
5176
diff
changeset
|
450 m->clear_queued_for_compilation(); |
d87155082c4d
guard all compilation queue accesses, to avoid deadlocks
Lukas Stadler <lukas.stadler@jku.at>
parents:
5176
diff
changeset
|
451 } |
d87155082c4d
guard all compilation queue accesses, to avoid deadlocks
Lukas Stadler <lukas.stadler@jku.at>
parents:
5176
diff
changeset
|
452 } else { |
d87155082c4d
guard all compilation queue accesses, to avoid deadlocks
Lukas Stadler <lukas.stadler@jku.at>
parents:
5176
diff
changeset
|
453 if (time_per_call < m->graal_priority()) { |
d87155082c4d
guard all compilation queue accesses, to avoid deadlocks
Lukas Stadler <lukas.stadler@jku.at>
parents:
5176
diff
changeset
|
454 m->set_graal_priority(time_per_call); |
d87155082c4d
guard all compilation queue accesses, to avoid deadlocks
Lukas Stadler <lukas.stadler@jku.at>
parents:
5176
diff
changeset
|
455 } |
5176
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
456 } |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
457 } else { |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
458 if (time_per_call < m->graal_priority()) { |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
459 m->set_graal_priority(time_per_call); |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
460 } |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
461 } |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
462 } |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
463 if (!m->queued_for_compilation()) { |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
464 CompileBroker::compile_method(m, InvocationEntryBci, CompLevel_highest_tier, m, hot_count, "count", thread); |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
465 } |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
466 } |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
467 } |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
468 } |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
469 |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
470 void GraalCompPolicy::method_back_branch_event(methodHandle m, int bci, JavaThread* thread) { |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
471 int hot_count = m->backedge_count(); |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
472 const char* comment = "backedge_count"; |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
473 |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
474 if (is_compilation_enabled() && !m->is_not_osr_compilable() && can_be_compiled(m)) { |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
475 CompileBroker::compile_method(m, bci, CompLevel_highest_tier, |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
476 m, hot_count, comment, thread); |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
477 NOT_PRODUCT(trace_osr_completion(m->lookup_osr_nmethod_for(bci, CompLevel_highest_tier, true));) |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
478 } |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
479 } |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
480 |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
481 #endif // GRAAL |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
482 |
af59b4dfc9e4
compilation queue changes:
Lukas Stadler <lukas.stadler@jku.at>
parents:
4977
diff
changeset
|
483 |
0 | 484 // StackWalkCompPolicy - walk up stack to find a suitable method to compile |
485 | |
4977
532be189cf09
Reducing diff to hsx24
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
4970
diff
changeset
|
486 #if defined(COMPILER2) || defined(GRAAL) |
0 | 487 const char* StackWalkCompPolicy::_msg = NULL; |
488 | |
489 | |
490 // Consider m for compilation | |
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
491 void StackWalkCompPolicy::method_invocation_event(methodHandle m, JavaThread* thread) { |
0 | 492 int hot_count = m->invocation_count(); |
493 reset_counter_for_invocation_event(m); | |
494 const char* comment = "count"; | |
495 | |
1783 | 496 if (is_compilation_enabled() && m->code() == NULL && can_be_compiled(m)) { |
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
497 ResourceMark rm(thread); |
0 | 498 frame fr = thread->last_frame(); |
499 assert(fr.is_interpreted_frame(), "must be interpreted"); | |
500 assert(fr.interpreter_frame_method() == m(), "bad method"); | |
501 | |
502 if (TraceCompilationPolicy) { | |
503 tty->print("method invocation trigger: "); | |
504 m->print_short_name(tty); | |
505 tty->print(" ( interpreted " INTPTR_FORMAT ", size=%d ) ", (address)m(), m->code_size()); | |
506 } | |
507 RegisterMap reg_map(thread, false); | |
508 javaVFrame* triggerVF = thread->last_java_vframe(®_map); | |
509 // triggerVF is the frame that triggered its counter | |
510 RFrame* first = new InterpretedRFrame(triggerVF->fr(), thread, m); | |
511 | |
512 if (first->top_method()->code() != NULL) { | |
513 // called obsolete method/nmethod -- no need to recompile | |
514 if (TraceCompilationPolicy) tty->print_cr(" --> " INTPTR_FORMAT, first->top_method()->code()); | |
515 } else { | |
516 if (TimeCompilationPolicy) accumulated_time()->start(); | |
517 GrowableArray<RFrame*>* stack = new GrowableArray<RFrame*>(50); | |
518 stack->push(first); | |
519 RFrame* top = findTopInlinableFrame(stack); | |
520 if (TimeCompilationPolicy) accumulated_time()->stop(); | |
521 assert(top != NULL, "findTopInlinableFrame returned null"); | |
522 if (TraceCompilationPolicy) top->print(); | |
1783 | 523 CompileBroker::compile_method(top->top_method(), InvocationEntryBci, CompLevel_highest_tier, |
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
524 m, hot_count, comment, thread); |
0 | 525 } |
526 } | |
527 } | |
528 | |
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
529 void StackWalkCompPolicy::method_back_branch_event(methodHandle m, int bci, JavaThread* thread) { |
0 | 530 int hot_count = m->backedge_count(); |
531 const char* comment = "backedge_count"; | |
532 | |
1783 | 533 if (is_compilation_enabled() && !m->is_not_osr_compilable() && can_be_compiled(m)) { |
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
3791
diff
changeset
|
534 CompileBroker::compile_method(m, bci, CompLevel_highest_tier, m, hot_count, comment, thread); |
0 | 535 |
1783 | 536 NOT_PRODUCT(trace_osr_completion(m->lookup_osr_nmethod_for(bci, CompLevel_highest_tier, true));) |
0 | 537 } |
538 } | |
539 | |
540 RFrame* StackWalkCompPolicy::findTopInlinableFrame(GrowableArray<RFrame*>* stack) { | |
541 // go up the stack until finding a frame that (probably) won't be inlined | |
542 // into its caller | |
543 RFrame* current = stack->at(0); // current choice for stopping | |
544 assert( current && !current->is_compiled(), "" ); | |
545 const char* msg = NULL; | |
546 | |
547 while (1) { | |
548 | |
549 // before going up the stack further, check if doing so would get us into | |
550 // compiled code | |
551 RFrame* next = senderOf(current, stack); | |
552 if( !next ) // No next frame up the stack? | |
553 break; // Then compile with current frame | |
554 | |
555 methodHandle m = current->top_method(); | |
556 methodHandle next_m = next->top_method(); | |
557 | |
558 if (TraceCompilationPolicy && Verbose) { | |
559 tty->print("[caller: "); | |
560 next_m->print_short_name(tty); | |
561 tty->print("] "); | |
562 } | |
563 | |
564 if( !Inline ) { // Inlining turned off | |
565 msg = "Inlining turned off"; | |
566 break; | |
567 } | |
568 if (next_m->is_not_compilable()) { // Did fail to compile this before/ | |
569 msg = "caller not compilable"; | |
570 break; | |
571 } | |
572 if (next->num() > MaxRecompilationSearchLength) { | |
573 // don't go up too high when searching for recompilees | |
574 msg = "don't go up any further: > MaxRecompilationSearchLength"; | |
575 break; | |
576 } | |
577 if (next->distance() > MaxInterpretedSearchLength) { | |
578 // don't go up too high when searching for recompilees | |
579 msg = "don't go up any further: next > MaxInterpretedSearchLength"; | |
580 break; | |
581 } | |
582 // Compiled frame above already decided not to inline; | |
583 // do not recompile him. | |
584 if (next->is_compiled()) { | |
585 msg = "not going up into optimized code"; | |
586 break; | |
587 } | |
588 | |
589 // Interpreted frame above us was already compiled. Do not force | |
590 // a recompile, although if the frame above us runs long enough an | |
591 // OSR might still happen. | |
592 if( current->is_interpreted() && next_m->has_compiled_code() ) { | |
593 msg = "not going up -- already compiled caller"; | |
594 break; | |
595 } | |
596 | |
597 // Compute how frequent this call site is. We have current method 'm'. | |
598 // We know next method 'next_m' is interpreted. Find the call site and | |
599 // check the various invocation counts. | |
600 int invcnt = 0; // Caller counts | |
601 if (ProfileInterpreter) { | |
602 invcnt = next_m->interpreter_invocation_count(); | |
603 } | |
604 int cnt = 0; // Call site counts | |
605 if (ProfileInterpreter && next_m->method_data() != NULL) { | |
606 ResourceMark rm; | |
607 int bci = next->top_vframe()->bci(); | |
608 ProfileData* data = next_m->method_data()->bci_to_data(bci); | |
609 if (data != NULL && data->is_CounterData()) | |
610 cnt = data->as_CounterData()->count(); | |
611 } | |
612 | |
613 // Caller counts / call-site counts; i.e. is this call site | |
614 // a hot call site for method next_m? | |
615 int freq = (invcnt) ? cnt/invcnt : cnt; | |
616 | |
617 // Check size and frequency limits | |
618 if ((msg = shouldInline(m, freq, cnt)) != NULL) { | |
619 break; | |
620 } | |
621 // Check inlining negative tests | |
622 if ((msg = shouldNotInline(m)) != NULL) { | |
623 break; | |
624 } | |
625 | |
626 | |
627 // If the caller method is too big or something then we do not want to | |
628 // compile it just to inline a method | |
1783 | 629 if (!can_be_compiled(next_m)) { |
0 | 630 msg = "caller cannot be compiled"; |
631 break; | |
632 } | |
633 | |
634 if( next_m->name() == vmSymbols::class_initializer_name() ) { | |
635 msg = "do not compile class initializer (OSR ok)"; | |
636 break; | |
637 } | |
638 | |
639 if (TraceCompilationPolicy && Verbose) { | |
640 tty->print("\n\t check caller: "); | |
641 next_m->print_short_name(tty); | |
642 tty->print(" ( interpreted " INTPTR_FORMAT ", size=%d ) ", (address)next_m(), next_m->code_size()); | |
643 } | |
644 | |
645 current = next; | |
646 } | |
647 | |
648 assert( !current || !current->is_compiled(), "" ); | |
649 | |
650 if (TraceCompilationPolicy && msg) tty->print("(%s)\n", msg); | |
651 | |
652 return current; | |
653 } | |
654 | |
655 RFrame* StackWalkCompPolicy::senderOf(RFrame* rf, GrowableArray<RFrame*>* stack) { | |
656 RFrame* sender = rf->caller(); | |
657 if (sender && sender->num() == stack->length()) stack->push(sender); | |
658 return sender; | |
659 } | |
660 | |
661 | |
662 const char* StackWalkCompPolicy::shouldInline(methodHandle m, float freq, int cnt) { | |
663 // Allows targeted inlining | |
664 // positive filter: should send be inlined? returns NULL (--> yes) | |
665 // or rejection msg | |
666 int max_size = MaxInlineSize; | |
667 int cost = m->code_size(); | |
668 | |
669 // Check for too many throws (and not too huge) | |
670 if (m->interpreter_throwout_count() > InlineThrowCount && cost < InlineThrowMaxSize ) { | |
671 return NULL; | |
672 } | |
673 | |
674 // bump the max size if the call is frequent | |
675 if ((freq >= InlineFrequencyRatio) || (cnt >= InlineFrequencyCount)) { | |
676 if (TraceFrequencyInlining) { | |
677 tty->print("(Inlined frequent method)\n"); | |
678 m->print(); | |
679 } | |
680 max_size = FreqInlineSize; | |
681 } | |
682 if (cost > max_size) { | |
683 return (_msg = "too big"); | |
684 } | |
685 return NULL; | |
686 } | |
687 | |
688 | |
689 const char* StackWalkCompPolicy::shouldNotInline(methodHandle m) { | |
690 // negative filter: should send NOT be inlined? returns NULL (--> inline) or rejection msg | |
691 if (m->is_abstract()) return (_msg = "abstract method"); | |
692 // note: we allow ik->is_abstract() | |
693 if (!instanceKlass::cast(m->method_holder())->is_initialized()) return (_msg = "method holder not initialized"); | |
694 if (m->is_native()) return (_msg = "native method"); | |
695 nmethod* m_code = m->code(); | |
1748 | 696 if (m_code != NULL && m_code->code_size() > InlineSmallCode) |
0 | 697 return (_msg = "already compiled into a big method"); |
698 | |
699 // use frequency-based objections only for non-trivial methods | |
700 if (m->code_size() <= MaxTrivialSize) return NULL; | |
701 if (UseInterpreter) { // don't use counts with -Xcomp | |
702 if ((m->code() == NULL) && m->was_never_executed()) return (_msg = "never executed"); | |
703 if (!m->was_executed_more_than(MIN2(MinInliningThreshold, CompileThreshold >> 1))) return (_msg = "executed < MinInliningThreshold times"); | |
704 } | |
705 if (methodOopDesc::has_unloaded_classes_in_signature(m, JavaThread::current())) return (_msg = "unloaded signature classes"); | |
706 | |
707 return NULL; | |
708 } | |
709 | |
710 | |
711 | |
4977
532be189cf09
Reducing diff to hsx24
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
4970
diff
changeset
|
712 #endif // COMPILER2 || GRAAL |