Mercurial > hg > truffle
annotate src/share/vm/runtime/advancedThresholdPolicy.cpp @ 17381:40dce17629ac
correct IfNode probabilities in If-Deopt situations
author | Lukas Stadler <lukas.stadler@oracle.com> |
---|---|
date | Wed, 08 Oct 2014 10:49:53 +0200 |
parents | c7db9dc73694 |
children | 52b4284cb496 |
rev | line source |
---|---|
2348 | 1 /* |
10105
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6831
diff
changeset
|
2 * Copyright (c) 2010, 2013, Oracle and/or its affiliates. All rights reserved. |
3358 | 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 * | |
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA | |
20 * or visit www.oracle.com if you need additional information or have any | |
21 * questions. | |
22 * | |
23 */ | |
2348 | 24 |
25 #include "precompiled.hpp" | |
26 #include "runtime/advancedThresholdPolicy.hpp" | |
27 #include "runtime/simpleThresholdPolicy.inline.hpp" | |
28 | |
29 #ifdef TIERED | |
30 // Print an event. | |
31 void AdvancedThresholdPolicy::print_specific(EventType type, methodHandle mh, methodHandle imh, | |
32 int bci, CompLevel level) { | |
6800
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6725
diff
changeset
|
33 tty->print(" rate="); |
2348 | 34 if (mh->prev_time() == 0) tty->print("n/a"); |
35 else tty->print("%f", mh->rate()); | |
36 | |
6800
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6725
diff
changeset
|
37 tty->print(" k=%.2lf,%.2lf", threshold_scale(CompLevel_full_profile, Tier3LoadFeedback), |
9191895df19d
7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents:
6725
diff
changeset
|
38 threshold_scale(CompLevel_full_optimization, Tier4LoadFeedback)); |
2348 | 39 |
40 } | |
41 | |
42 void AdvancedThresholdPolicy::initialize() { | |
43 // Turn on ergonomic compiler count selection | |
44 if (FLAG_IS_DEFAULT(CICompilerCountPerCPU) && FLAG_IS_DEFAULT(CICompilerCount)) { | |
45 FLAG_SET_DEFAULT(CICompilerCountPerCPU, true); | |
46 } | |
47 int count = CICompilerCount; | |
48 if (CICompilerCountPerCPU) { | |
49 // Simple log n seems to grow too slowly for tiered, try something faster: log n * log log n | |
50 int log_cpu = log2_intptr(os::active_processor_count()); | |
51 int loglog_cpu = log2_intptr(MAX2(log_cpu, 1)); | |
52 count = MAX2(log_cpu * loglog_cpu, 1) * 3 / 2; | |
53 } | |
54 | |
55 set_c1_count(MAX2(count / 3, 1)); | |
56 set_c2_count(MAX2(count - count / 3, 1)); | |
57 | |
58 // Some inlining tuning | |
59 #ifdef X86 | |
60 if (FLAG_IS_DEFAULT(InlineSmallCode)) { | |
61 FLAG_SET_DEFAULT(InlineSmallCode, 2000); | |
62 } | |
63 #endif | |
64 | |
65 #ifdef SPARC | |
66 if (FLAG_IS_DEFAULT(InlineSmallCode)) { | |
67 FLAG_SET_DEFAULT(InlineSmallCode, 2500); | |
68 } | |
69 #endif | |
70 | |
10319
91eba9f82325
8012371: Adjust Tiered compile threshold according to available space in code cache
anoll
parents:
10105
diff
changeset
|
71 set_increase_threshold_at_ratio(); |
2348 | 72 set_start_time(os::javaTimeMillis()); |
73 } | |
74 | |
75 // update_rate() is called from select_task() while holding a compile queue lock. | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
76 void AdvancedThresholdPolicy::update_rate(jlong t, Method* m) { |
10105
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6831
diff
changeset
|
77 JavaThread* THREAD = JavaThread::current(); |
2348 | 78 if (is_old(m)) { |
79 // We don't remove old methods from the queue, | |
80 // so we can just zero the rate. | |
10105
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6831
diff
changeset
|
81 m->set_rate(0, THREAD); |
2348 | 82 return; |
83 } | |
84 | |
85 // We don't update the rate if we've just came out of a safepoint. | |
86 // delta_s is the time since last safepoint in milliseconds. | |
87 jlong delta_s = t - SafepointSynchronize::end_of_last_safepoint(); | |
88 jlong delta_t = t - (m->prev_time() != 0 ? m->prev_time() : start_time()); // milliseconds since the last measurement | |
89 // How many events were there since the last time? | |
90 int event_count = m->invocation_count() + m->backedge_count(); | |
91 int delta_e = event_count - m->prev_event_count(); | |
92 | |
93 // We should be running for at least 1ms. | |
94 if (delta_s >= TieredRateUpdateMinTime) { | |
95 // And we must've taken the previous point at least 1ms before. | |
96 if (delta_t >= TieredRateUpdateMinTime && delta_e > 0) { | |
10105
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6831
diff
changeset
|
97 m->set_prev_time(t, THREAD); |
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6831
diff
changeset
|
98 m->set_prev_event_count(event_count, THREAD); |
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6831
diff
changeset
|
99 m->set_rate((float)delta_e / (float)delta_t, THREAD); // Rate is events per millisecond |
2348 | 100 } else |
101 if (delta_t > TieredRateUpdateMaxTime && delta_e == 0) { | |
102 // If nothing happened for 25ms, zero the rate. Don't modify prev values. | |
10105
aeaca88565e6
8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents:
6831
diff
changeset
|
103 m->set_rate(0, THREAD); |
2348 | 104 } |
105 } | |
106 } | |
107 | |
108 // Check if this method has been stale from a given number of milliseconds. | |
109 // See select_task(). | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
110 bool AdvancedThresholdPolicy::is_stale(jlong t, jlong timeout, Method* m) { |
2348 | 111 jlong delta_s = t - SafepointSynchronize::end_of_last_safepoint(); |
112 jlong delta_t = t - m->prev_time(); | |
113 if (delta_t > timeout && delta_s > timeout) { | |
114 int event_count = m->invocation_count() + m->backedge_count(); | |
115 int delta_e = event_count - m->prev_event_count(); | |
116 // Return true if there were no events. | |
117 return delta_e == 0; | |
118 } | |
119 return false; | |
120 } | |
121 | |
122 // We don't remove old methods from the compile queue even if they have | |
123 // very low activity. See select_task(). | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
124 bool AdvancedThresholdPolicy::is_old(Method* method) { |
2348 | 125 return method->invocation_count() > 50000 || method->backedge_count() > 500000; |
126 } | |
127 | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
128 double AdvancedThresholdPolicy::weight(Method* method) { |
2348 | 129 return (method->rate() + 1) * ((method->invocation_count() + 1) * (method->backedge_count() + 1)); |
130 } | |
131 | |
132 // Apply heuristics and return true if x should be compiled before y | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
133 bool AdvancedThresholdPolicy::compare_methods(Method* x, Method* y) { |
2348 | 134 if (x->highest_comp_level() > y->highest_comp_level()) { |
135 // recompilation after deopt | |
136 return true; | |
137 } else | |
138 if (x->highest_comp_level() == y->highest_comp_level()) { | |
139 if (weight(x) > weight(y)) { | |
140 return true; | |
141 } | |
142 } | |
143 return false; | |
144 } | |
145 | |
146 // Is method profiled enough? | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
147 bool AdvancedThresholdPolicy::is_method_profiled(Method* method) { |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
148 MethodData* mdo = method->method_data(); |
2348 | 149 if (mdo != NULL) { |
150 int i = mdo->invocation_count_delta(); | |
151 int b = mdo->backedge_count_delta(); | |
152 return call_predicate_helper<CompLevel_full_profile>(i, b, 1); | |
153 } | |
154 return false; | |
155 } | |
156 | |
157 // Called with the queue locked and with at least one element | |
158 CompileTask* AdvancedThresholdPolicy::select_task(CompileQueue* compile_queue) { | |
159 CompileTask *max_task = NULL; | |
6831
d8ce2825b193
8000213: NPG: Should have renamed arrayKlass and typeArrayKlass
coleenp
parents:
6800
diff
changeset
|
160 Method* max_method = NULL; |
2348 | 161 jlong t = os::javaTimeMillis(); |
162 // Iterate through the queue and find a method with a maximum rate. | |
163 for (CompileTask* task = compile_queue->first(); task != NULL;) { | |
164 CompileTask* next_task = task->next(); | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
165 Method* method = task->method(); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
166 MethodData* mdo = method->method_data(); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
167 update_rate(t, method); |
2348 | 168 if (max_task == NULL) { |
169 max_task = task; | |
170 max_method = method; | |
171 } else { | |
172 // If a method has been stale for some time, remove it from the queue. | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
173 if (is_stale(t, TieredCompileTaskTimeout, method) && !is_old(method)) { |
2348 | 174 if (PrintTieredEvents) { |
3791
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
175 print_event(REMOVE_FROM_QUEUE, method, method, task->osr_bci(), (CompLevel)task->comp_level()); |
2348 | 176 } |
177 CompileTaskWrapper ctw(task); // Frees the task | |
178 compile_queue->remove(task); | |
179 method->clear_queued_for_compilation(); | |
180 task = next_task; | |
181 continue; | |
182 } | |
183 | |
184 // Select a method with a higher rate | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
185 if (compare_methods(method, max_method)) { |
2348 | 186 max_task = task; |
187 max_method = method; | |
188 } | |
189 } | |
190 task = next_task; | |
191 } | |
192 | |
3837
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
193 if (max_task->comp_level() == CompLevel_full_profile && TieredStopAtLevel > CompLevel_full_profile |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
194 && is_method_profiled(max_method)) { |
2348 | 195 max_task->set_comp_level(CompLevel_limited_profile); |
196 if (PrintTieredEvents) { | |
3791
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
197 print_event(UPDATE_IN_QUEUE, max_method, max_method, max_task->osr_bci(), (CompLevel)max_task->comp_level()); |
2348 | 198 } |
199 } | |
200 | |
201 return max_task; | |
202 } | |
203 | |
204 double AdvancedThresholdPolicy::threshold_scale(CompLevel level, int feedback_k) { | |
205 double queue_size = CompileBroker::queue_size(level); | |
206 int comp_count = compiler_count(level); | |
207 double k = queue_size / (feedback_k * comp_count) + 1; | |
10319
91eba9f82325
8012371: Adjust Tiered compile threshold according to available space in code cache
anoll
parents:
10105
diff
changeset
|
208 |
91eba9f82325
8012371: Adjust Tiered compile threshold according to available space in code cache
anoll
parents:
10105
diff
changeset
|
209 // Increase C1 compile threshold when the code cache is filled more |
91eba9f82325
8012371: Adjust Tiered compile threshold according to available space in code cache
anoll
parents:
10105
diff
changeset
|
210 // than specified by IncreaseFirstTierCompileThresholdAt percentage. |
91eba9f82325
8012371: Adjust Tiered compile threshold according to available space in code cache
anoll
parents:
10105
diff
changeset
|
211 // The main intention is to keep enough free space for C2 compiled code |
91eba9f82325
8012371: Adjust Tiered compile threshold according to available space in code cache
anoll
parents:
10105
diff
changeset
|
212 // to achieve peak performance if the code cache is under stress. |
91eba9f82325
8012371: Adjust Tiered compile threshold according to available space in code cache
anoll
parents:
10105
diff
changeset
|
213 if ((TieredStopAtLevel == CompLevel_full_optimization) && (level != CompLevel_full_optimization)) { |
91eba9f82325
8012371: Adjust Tiered compile threshold according to available space in code cache
anoll
parents:
10105
diff
changeset
|
214 double current_reverse_free_ratio = CodeCache::reverse_free_ratio(); |
91eba9f82325
8012371: Adjust Tiered compile threshold according to available space in code cache
anoll
parents:
10105
diff
changeset
|
215 if (current_reverse_free_ratio > _increase_threshold_at_ratio) { |
91eba9f82325
8012371: Adjust Tiered compile threshold according to available space in code cache
anoll
parents:
10105
diff
changeset
|
216 k *= exp(current_reverse_free_ratio - _increase_threshold_at_ratio); |
91eba9f82325
8012371: Adjust Tiered compile threshold according to available space in code cache
anoll
parents:
10105
diff
changeset
|
217 } |
91eba9f82325
8012371: Adjust Tiered compile threshold according to available space in code cache
anoll
parents:
10105
diff
changeset
|
218 } |
2348 | 219 return k; |
220 } | |
221 | |
222 // Call and loop predicates determine whether a transition to a higher | |
223 // compilation level should be performed (pointers to predicate functions | |
224 // are passed to common()). | |
225 // Tier?LoadFeedback is basically a coefficient that determines of | |
226 // how many methods per compiler thread can be in the queue before | |
227 // the threshold values double. | |
228 bool AdvancedThresholdPolicy::loop_predicate(int i, int b, CompLevel cur_level) { | |
229 switch(cur_level) { | |
230 case CompLevel_none: | |
231 case CompLevel_limited_profile: { | |
232 double k = threshold_scale(CompLevel_full_profile, Tier3LoadFeedback); | |
233 return loop_predicate_helper<CompLevel_none>(i, b, k); | |
234 } | |
235 case CompLevel_full_profile: { | |
236 double k = threshold_scale(CompLevel_full_optimization, Tier4LoadFeedback); | |
237 return loop_predicate_helper<CompLevel_full_profile>(i, b, k); | |
238 } | |
239 default: | |
240 return true; | |
241 } | |
242 } | |
243 | |
244 bool AdvancedThresholdPolicy::call_predicate(int i, int b, CompLevel cur_level) { | |
245 switch(cur_level) { | |
246 case CompLevel_none: | |
247 case CompLevel_limited_profile: { | |
248 double k = threshold_scale(CompLevel_full_profile, Tier3LoadFeedback); | |
249 return call_predicate_helper<CompLevel_none>(i, b, k); | |
250 } | |
251 case CompLevel_full_profile: { | |
252 double k = threshold_scale(CompLevel_full_optimization, Tier4LoadFeedback); | |
253 return call_predicate_helper<CompLevel_full_profile>(i, b, k); | |
254 } | |
255 default: | |
256 return true; | |
257 } | |
258 } | |
259 | |
260 // If a method is old enough and is still in the interpreter we would want to | |
261 // start profiling without waiting for the compiled method to arrive. | |
262 // We also take the load on compilers into the account. | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
263 bool AdvancedThresholdPolicy::should_create_mdo(Method* method, CompLevel cur_level) { |
2348 | 264 if (cur_level == CompLevel_none && |
265 CompileBroker::queue_size(CompLevel_full_optimization) <= | |
266 Tier3DelayOn * compiler_count(CompLevel_full_optimization)) { | |
267 int i = method->invocation_count(); | |
268 int b = method->backedge_count(); | |
269 double k = Tier0ProfilingStartPercentage / 100.0; | |
270 return call_predicate_helper<CompLevel_none>(i, b, k) || loop_predicate_helper<CompLevel_none>(i, b, k); | |
271 } | |
272 return false; | |
273 } | |
274 | |
3791
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
275 // Inlining control: if we're compiling a profiled method with C1 and the callee |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
276 // is known to have OSRed in a C2 version, don't inline it. |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
277 bool AdvancedThresholdPolicy::should_not_inline(ciEnv* env, ciMethod* callee) { |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
278 CompLevel comp_level = (CompLevel)env->comp_level(); |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
279 if (comp_level == CompLevel_full_profile || |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
280 comp_level == CompLevel_limited_profile) { |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
281 return callee->highest_osr_comp_level() == CompLevel_full_optimization; |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
282 } |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
283 return false; |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
284 } |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
285 |
2348 | 286 // Create MDO if necessary. |
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
4756
diff
changeset
|
287 void AdvancedThresholdPolicy::create_mdo(methodHandle mh, JavaThread* THREAD) { |
2348 | 288 if (mh->is_native() || mh->is_abstract() || mh->is_accessor()) return; |
289 if (mh->method_data() == NULL) { | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
290 Method::build_interpreter_method_data(mh, CHECK_AND_CLEAR); |
2348 | 291 } |
292 } | |
293 | |
294 | |
295 /* | |
296 * Method states: | |
297 * 0 - interpreter (CompLevel_none) | |
298 * 1 - pure C1 (CompLevel_simple) | |
299 * 2 - C1 with invocation and backedge counting (CompLevel_limited_profile) | |
300 * 3 - C1 with full profiling (CompLevel_full_profile) | |
301 * 4 - C2 (CompLevel_full_optimization) | |
302 * | |
303 * Common state transition patterns: | |
304 * a. 0 -> 3 -> 4. | |
305 * The most common path. But note that even in this straightforward case | |
306 * profiling can start at level 0 and finish at level 3. | |
307 * | |
308 * b. 0 -> 2 -> 3 -> 4. | |
14909 | 309 * This case occures when the load on C2 is deemed too high. So, instead of transitioning |
2348 | 310 * into state 3 directly and over-profiling while a method is in the C2 queue we transition to |
311 * level 2 and wait until the load on C2 decreases. This path is disabled for OSRs. | |
312 * | |
313 * c. 0 -> (3->2) -> 4. | |
314 * In this case we enqueue a method for compilation at level 3, but the C1 queue is long enough | |
315 * to enable the profiling to fully occur at level 0. In this case we change the compilation level | |
316 * of the method to 2, because it'll allow it to run much faster without full profiling while c2 | |
317 * is compiling. | |
318 * | |
319 * d. 0 -> 3 -> 1 or 0 -> 2 -> 1. | |
320 * After a method was once compiled with C1 it can be identified as trivial and be compiled to | |
321 * level 1. These transition can also occur if a method can't be compiled with C2 but can with C1. | |
322 * | |
323 * e. 0 -> 4. | |
324 * This can happen if a method fails C1 compilation (it will still be profiled in the interpreter) | |
325 * or because of a deopt that didn't require reprofiling (compilation won't happen in this case because | |
326 * the compiled version already exists). | |
327 * | |
328 * Note that since state 0 can be reached from any other state via deoptimization different loops | |
329 * are possible. | |
330 * | |
331 */ | |
332 | |
333 // Common transition function. Given a predicate determines if a method should transition to another level. | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
334 CompLevel AdvancedThresholdPolicy::common(Predicate p, Method* method, CompLevel cur_level, bool disable_feedback) { |
2348 | 335 CompLevel next_level = cur_level; |
336 int i = method->invocation_count(); | |
337 int b = method->backedge_count(); | |
338 | |
3837
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
339 if (is_trivial(method)) { |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
340 next_level = CompLevel_simple; |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
341 } else { |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
342 switch(cur_level) { |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
343 case CompLevel_none: |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
344 // If we were at full profile level, would we switch to full opt? |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
345 if (common(p, method, CompLevel_full_profile, disable_feedback) == CompLevel_full_optimization) { |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
346 next_level = CompLevel_full_optimization; |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
347 } else if ((this->*p)(i, b, cur_level)) { |
16107
c7db9dc73694
disabled dialing C1 compilation back to partial profiling when Graal is the top tier compiler
Doug Simon <doug.simon@oracle.com>
parents:
14909
diff
changeset
|
348 #ifdef COMPILERGRAAL |
c7db9dc73694
disabled dialing C1 compilation back to partial profiling when Graal is the top tier compiler
Doug Simon <doug.simon@oracle.com>
parents:
14909
diff
changeset
|
349 // Since Graal takes a while to warm up, its queue inevitably backs up during |
c7db9dc73694
disabled dialing C1 compilation back to partial profiling when Graal is the top tier compiler
Doug Simon <doug.simon@oracle.com>
parents:
14909
diff
changeset
|
350 // early VM execution. As of 2014-06-13, Graal's inliner assumes that the root |
c7db9dc73694
disabled dialing C1 compilation back to partial profiling when Graal is the top tier compiler
Doug Simon <doug.simon@oracle.com>
parents:
14909
diff
changeset
|
351 // compilation method and all potential inlinees have mature profiles (which |
c7db9dc73694
disabled dialing C1 compilation back to partial profiling when Graal is the top tier compiler
Doug Simon <doug.simon@oracle.com>
parents:
14909
diff
changeset
|
352 // includes type profiling). If it sees immature profiles, Graal's inliner |
c7db9dc73694
disabled dialing C1 compilation back to partial profiling when Graal is the top tier compiler
Doug Simon <doug.simon@oracle.com>
parents:
14909
diff
changeset
|
353 // can perform pathologically bad (e.g., causing OutOfMemoryErrors due to |
c7db9dc73694
disabled dialing C1 compilation back to partial profiling when Graal is the top tier compiler
Doug Simon <doug.simon@oracle.com>
parents:
14909
diff
changeset
|
354 // exploring/inlining too many graphs). Since a rewrite of the inliner is |
c7db9dc73694
disabled dialing C1 compilation back to partial profiling when Graal is the top tier compiler
Doug Simon <doug.simon@oracle.com>
parents:
14909
diff
changeset
|
355 // in progress, we simply disable the dialing back heuristic for now and will |
c7db9dc73694
disabled dialing C1 compilation back to partial profiling when Graal is the top tier compiler
Doug Simon <doug.simon@oracle.com>
parents:
14909
diff
changeset
|
356 // revisit this decision once the new inliner is completed. |
c7db9dc73694
disabled dialing C1 compilation back to partial profiling when Graal is the top tier compiler
Doug Simon <doug.simon@oracle.com>
parents:
14909
diff
changeset
|
357 next_level = CompLevel_full_profile; |
c7db9dc73694
disabled dialing C1 compilation back to partial profiling when Graal is the top tier compiler
Doug Simon <doug.simon@oracle.com>
parents:
14909
diff
changeset
|
358 #else |
3837
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
359 // C1-generated fully profiled code is about 30% slower than the limited profile |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
360 // code that has only invocation and backedge counters. The observation is that |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
361 // if C2 queue is large enough we can spend too much time in the fully profiled code |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
362 // while waiting for C2 to pick the method from the queue. To alleviate this problem |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
363 // we introduce a feedback on the C2 queue size. If the C2 queue is sufficiently long |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
364 // we choose to compile a limited profiled version and then recompile with full profiling |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
365 // when the load on C2 goes down. |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
366 if (!disable_feedback && CompileBroker::queue_size(CompLevel_full_optimization) > |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
367 Tier3DelayOn * compiler_count(CompLevel_full_optimization)) { |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
368 next_level = CompLevel_limited_profile; |
2348 | 369 } else { |
3837
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
370 next_level = CompLevel_full_profile; |
2348 | 371 } |
16107
c7db9dc73694
disabled dialing C1 compilation back to partial profiling when Graal is the top tier compiler
Doug Simon <doug.simon@oracle.com>
parents:
14909
diff
changeset
|
372 #endif |
2348 | 373 } |
3837
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
374 break; |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
375 case CompLevel_limited_profile: |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
376 if (is_method_profiled(method)) { |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
377 // Special case: we got here because this method was fully profiled in the interpreter. |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
378 next_level = CompLevel_full_optimization; |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
379 } else { |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
380 MethodData* mdo = method->method_data(); |
3837
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
381 if (mdo != NULL) { |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
382 if (mdo->would_profile()) { |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
383 if (disable_feedback || (CompileBroker::queue_size(CompLevel_full_optimization) <= |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
384 Tier3DelayOff * compiler_count(CompLevel_full_optimization) && |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
385 (this->*p)(i, b, cur_level))) { |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
386 next_level = CompLevel_full_profile; |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
387 } |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
388 } else { |
2348 | 389 next_level = CompLevel_full_optimization; |
390 } | |
391 } | |
392 } | |
3837
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
393 break; |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
394 case CompLevel_full_profile: |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
395 { |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
396 MethodData* mdo = method->method_data(); |
3837
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
397 if (mdo != NULL) { |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
398 if (mdo->would_profile()) { |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
399 int mdo_i = mdo->invocation_count_delta(); |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
400 int mdo_b = mdo->backedge_count_delta(); |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
401 if ((this->*p)(mdo_i, mdo_b, cur_level)) { |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
402 next_level = CompLevel_full_optimization; |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
403 } |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
404 } else { |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
405 next_level = CompLevel_full_optimization; |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
406 } |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
407 } |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
408 } |
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
409 break; |
2348 | 410 } |
411 } | |
3837
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
412 return MIN2(next_level, (CompLevel)TieredStopAtLevel); |
2348 | 413 } |
414 | |
415 // Determine if a method should be compiled with a normal entry point at a different level. | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
416 CompLevel AdvancedThresholdPolicy::call_event(Method* method, CompLevel cur_level) { |
3790
6f6e91603a45
7058689: Tiered: Reprofiling doesn't happen in presence of level 4 OSR methods
iveresov
parents:
3358
diff
changeset
|
417 CompLevel osr_level = MIN2((CompLevel) method->highest_osr_comp_level(), |
3837
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
418 common(&AdvancedThresholdPolicy::loop_predicate, method, cur_level, true)); |
2348 | 419 CompLevel next_level = common(&AdvancedThresholdPolicy::call_predicate, method, cur_level); |
420 | |
421 // If OSR method level is greater than the regular method level, the levels should be | |
422 // equalized by raising the regular method level in order to avoid OSRs during each | |
423 // invocation of the method. | |
424 if (osr_level == CompLevel_full_optimization && cur_level == CompLevel_full_profile) { | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
425 MethodData* mdo = method->method_data(); |
2348 | 426 guarantee(mdo != NULL, "MDO should not be NULL"); |
427 if (mdo->invocation_count() >= 1) { | |
428 next_level = CompLevel_full_optimization; | |
429 } | |
430 } else { | |
431 next_level = MAX2(osr_level, next_level); | |
432 } | |
433 return next_level; | |
434 } | |
435 | |
436 // Determine if we should do an OSR compilation of a given method. | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
4825
diff
changeset
|
437 CompLevel AdvancedThresholdPolicy::loop_event(Method* method, CompLevel cur_level) { |
3837
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
438 CompLevel next_level = common(&AdvancedThresholdPolicy::loop_predicate, method, cur_level, true); |
2348 | 439 if (cur_level == CompLevel_none) { |
440 // If there is a live OSR method that means that we deopted to the interpreter | |
441 // for the transition. | |
3790
6f6e91603a45
7058689: Tiered: Reprofiling doesn't happen in presence of level 4 OSR methods
iveresov
parents:
3358
diff
changeset
|
442 CompLevel osr_level = MIN2((CompLevel)method->highest_osr_comp_level(), next_level); |
2348 | 443 if (osr_level > CompLevel_none) { |
444 return osr_level; | |
445 } | |
446 } | |
3790
6f6e91603a45
7058689: Tiered: Reprofiling doesn't happen in presence of level 4 OSR methods
iveresov
parents:
3358
diff
changeset
|
447 return next_level; |
2348 | 448 } |
449 | |
450 // Update the rate and submit compile | |
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
4756
diff
changeset
|
451 void AdvancedThresholdPolicy::submit_compile(methodHandle mh, int bci, CompLevel level, JavaThread* thread) { |
2348 | 452 int hot_count = (bci == InvocationEntryBci) ? mh->invocation_count() : mh->backedge_count(); |
453 update_rate(os::javaTimeMillis(), mh()); | |
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
4756
diff
changeset
|
454 CompileBroker::compile_method(mh, bci, level, mh, hot_count, "tiered", thread); |
2348 | 455 } |
456 | |
457 // Handle the invocation event. | |
458 void AdvancedThresholdPolicy::method_invocation_event(methodHandle mh, methodHandle imh, | |
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
4756
diff
changeset
|
459 CompLevel level, nmethod* nm, JavaThread* thread) { |
2348 | 460 if (should_create_mdo(mh(), level)) { |
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
4756
diff
changeset
|
461 create_mdo(mh, thread); |
2348 | 462 } |
463 if (is_compilation_enabled() && !CompileBroker::compilation_is_in_queue(mh, InvocationEntryBci)) { | |
464 CompLevel next_level = call_event(mh(), level); | |
465 if (next_level != level) { | |
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
4756
diff
changeset
|
466 compile(mh, InvocationEntryBci, next_level, thread); |
2348 | 467 } |
468 } | |
469 } | |
470 | |
471 // Handle the back branch event. Notice that we can compile the method | |
472 // with a regular entry from here. | |
473 void AdvancedThresholdPolicy::method_back_branch_event(methodHandle mh, methodHandle imh, | |
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
4756
diff
changeset
|
474 int bci, CompLevel level, nmethod* nm, JavaThread* thread) { |
2348 | 475 if (should_create_mdo(mh(), level)) { |
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
4756
diff
changeset
|
476 create_mdo(mh, thread); |
2348 | 477 } |
3791
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
478 // Check if MDO should be created for the inlined method |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
479 if (should_create_mdo(imh(), level)) { |
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
4756
diff
changeset
|
480 create_mdo(imh, thread); |
3791
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
481 } |
2348 | 482 |
3791
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
483 if (is_compilation_enabled()) { |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
484 CompLevel next_osr_level = loop_event(imh(), level); |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
485 CompLevel max_osr_level = (CompLevel)imh->highest_osr_comp_level(); |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
486 // At the very least compile the OSR version |
3837
43f9d800f276
7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents:
3791
diff
changeset
|
487 if (!CompileBroker::compilation_is_in_queue(imh, bci) && next_osr_level != level) { |
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
4756
diff
changeset
|
488 compile(imh, bci, next_osr_level, thread); |
2348 | 489 } |
490 | |
3791
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
491 // Use loop event as an opportunity to also check if there's been |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
492 // enough calls. |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
493 CompLevel cur_level, next_level; |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
494 if (mh() != imh()) { // If there is an enclosing method |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
495 guarantee(nm != NULL, "Should have nmethod here"); |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
496 cur_level = comp_level(mh()); |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
497 next_level = call_event(mh(), cur_level); |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
498 |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
499 if (max_osr_level == CompLevel_full_optimization) { |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
500 // The inlinee OSRed to full opt, we need to modify the enclosing method to avoid deopts |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
501 bool make_not_entrant = false; |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
502 if (nm->is_osr_method()) { |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
503 // This is an osr method, just make it not entrant and recompile later if needed |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
504 make_not_entrant = true; |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
505 } else { |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
506 if (next_level != CompLevel_full_optimization) { |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
507 // next_level is not full opt, so we need to recompile the |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
508 // enclosing method without the inlinee |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
509 cur_level = CompLevel_none; |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
510 make_not_entrant = true; |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
511 } |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
512 } |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
513 if (make_not_entrant) { |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
514 if (PrintTieredEvents) { |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
515 int osr_bci = nm->is_osr_method() ? nm->osr_entry_bci() : InvocationEntryBci; |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
516 print_event(MAKE_NOT_ENTRANT, mh(), mh(), osr_bci, level); |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
517 } |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
518 nm->make_not_entrant(); |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
519 } |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
520 } |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
521 if (!CompileBroker::compilation_is_in_queue(mh, InvocationEntryBci)) { |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
522 // Fix up next_level if necessary to avoid deopts |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
523 if (next_level == CompLevel_limited_profile && max_osr_level == CompLevel_full_profile) { |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
524 next_level = CompLevel_full_profile; |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
525 } |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
526 if (cur_level != next_level) { |
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
4756
diff
changeset
|
527 compile(mh, InvocationEntryBci, next_level, thread); |
3791
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
528 } |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
529 } |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
530 } else { |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
531 cur_level = comp_level(imh()); |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
532 next_level = call_event(imh(), cur_level); |
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
533 if (!CompileBroker::compilation_is_in_queue(imh, bci) && next_level != cur_level) { |
4825
20334ed5ed3c
7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents:
4756
diff
changeset
|
534 compile(imh, InvocationEntryBci, next_level, thread); |
3791
2c359f27615c
7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents:
3790
diff
changeset
|
535 } |
2348 | 536 } |
537 } | |
538 } | |
539 | |
540 #endif // TIERED |