annotate src/share/vm/opto/callGenerator.cpp @ 1080:7c57aead6d3e

6892658: C2 should optimize some stringbuilder patterns Reviewed-by: kvn, twisti
author never
date Thu, 12 Nov 2009 09:24:21 -0800
parents 9ee9cf798b59
children 97125851f396
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1 /*
337
9ee9cf798b59 6754988: Update copyright year
xdono
parents: 293
diff changeset
2 * Copyright 2000-2008 Sun Microsystems, Inc. All Rights Reserved.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
a61af66fc99e Initial load
duke
parents:
diff changeset
4 *
a61af66fc99e Initial load
duke
parents:
diff changeset
5 * This code is free software; you can redistribute it and/or modify it
a61af66fc99e Initial load
duke
parents:
diff changeset
6 * under the terms of the GNU General Public License version 2 only, as
a61af66fc99e Initial load
duke
parents:
diff changeset
7 * published by the Free Software Foundation.
a61af66fc99e Initial load
duke
parents:
diff changeset
8 *
a61af66fc99e Initial load
duke
parents:
diff changeset
9 * This code is distributed in the hope that it will be useful, but WITHOUT
a61af66fc99e Initial load
duke
parents:
diff changeset
10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
a61af66fc99e Initial load
duke
parents:
diff changeset
11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
a61af66fc99e Initial load
duke
parents:
diff changeset
12 * version 2 for more details (a copy is included in the LICENSE file that
a61af66fc99e Initial load
duke
parents:
diff changeset
13 * accompanied this code).
a61af66fc99e Initial load
duke
parents:
diff changeset
14 *
a61af66fc99e Initial load
duke
parents:
diff changeset
15 * You should have received a copy of the GNU General Public License version
a61af66fc99e Initial load
duke
parents:
diff changeset
16 * 2 along with this work; if not, write to the Free Software Foundation,
a61af66fc99e Initial load
duke
parents:
diff changeset
17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
a61af66fc99e Initial load
duke
parents:
diff changeset
18 *
a61af66fc99e Initial load
duke
parents:
diff changeset
19 * Please contact Sun Microsystems, Inc., 4150 Network Circle, Santa Clara,
a61af66fc99e Initial load
duke
parents:
diff changeset
20 * CA 95054 USA or visit www.sun.com if you need additional information or
a61af66fc99e Initial load
duke
parents:
diff changeset
21 * have any questions.
a61af66fc99e Initial load
duke
parents:
diff changeset
22 *
a61af66fc99e Initial load
duke
parents:
diff changeset
23 */
a61af66fc99e Initial load
duke
parents:
diff changeset
24
a61af66fc99e Initial load
duke
parents:
diff changeset
25 #include "incls/_precompiled.incl"
a61af66fc99e Initial load
duke
parents:
diff changeset
26 #include "incls/_callGenerator.cpp.incl"
a61af66fc99e Initial load
duke
parents:
diff changeset
27
a61af66fc99e Initial load
duke
parents:
diff changeset
28 CallGenerator::CallGenerator(ciMethod* method) {
a61af66fc99e Initial load
duke
parents:
diff changeset
29 _method = method;
a61af66fc99e Initial load
duke
parents:
diff changeset
30 }
a61af66fc99e Initial load
duke
parents:
diff changeset
31
a61af66fc99e Initial load
duke
parents:
diff changeset
32 // Utility function.
a61af66fc99e Initial load
duke
parents:
diff changeset
33 const TypeFunc* CallGenerator::tf() const {
a61af66fc99e Initial load
duke
parents:
diff changeset
34 return TypeFunc::make(method());
a61af66fc99e Initial load
duke
parents:
diff changeset
35 }
a61af66fc99e Initial load
duke
parents:
diff changeset
36
a61af66fc99e Initial load
duke
parents:
diff changeset
37 //-----------------------------ParseGenerator---------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
38 // Internal class which handles all direct bytecode traversal.
a61af66fc99e Initial load
duke
parents:
diff changeset
39 class ParseGenerator : public InlineCallGenerator {
a61af66fc99e Initial load
duke
parents:
diff changeset
40 private:
a61af66fc99e Initial load
duke
parents:
diff changeset
41 bool _is_osr;
a61af66fc99e Initial load
duke
parents:
diff changeset
42 float _expected_uses;
a61af66fc99e Initial load
duke
parents:
diff changeset
43
a61af66fc99e Initial load
duke
parents:
diff changeset
44 public:
a61af66fc99e Initial load
duke
parents:
diff changeset
45 ParseGenerator(ciMethod* method, float expected_uses, bool is_osr = false)
a61af66fc99e Initial load
duke
parents:
diff changeset
46 : InlineCallGenerator(method)
a61af66fc99e Initial load
duke
parents:
diff changeset
47 {
a61af66fc99e Initial load
duke
parents:
diff changeset
48 _is_osr = is_osr;
a61af66fc99e Initial load
duke
parents:
diff changeset
49 _expected_uses = expected_uses;
a61af66fc99e Initial load
duke
parents:
diff changeset
50 assert(can_parse(method, is_osr), "parse must be possible");
a61af66fc99e Initial load
duke
parents:
diff changeset
51 }
a61af66fc99e Initial load
duke
parents:
diff changeset
52
a61af66fc99e Initial load
duke
parents:
diff changeset
53 // Can we build either an OSR or a regular parser for this method?
a61af66fc99e Initial load
duke
parents:
diff changeset
54 static bool can_parse(ciMethod* method, int is_osr = false);
a61af66fc99e Initial load
duke
parents:
diff changeset
55
a61af66fc99e Initial load
duke
parents:
diff changeset
56 virtual bool is_parse() const { return true; }
a61af66fc99e Initial load
duke
parents:
diff changeset
57 virtual JVMState* generate(JVMState* jvms);
a61af66fc99e Initial load
duke
parents:
diff changeset
58 int is_osr() { return _is_osr; }
a61af66fc99e Initial load
duke
parents:
diff changeset
59
a61af66fc99e Initial load
duke
parents:
diff changeset
60 };
a61af66fc99e Initial load
duke
parents:
diff changeset
61
a61af66fc99e Initial load
duke
parents:
diff changeset
62 JVMState* ParseGenerator::generate(JVMState* jvms) {
a61af66fc99e Initial load
duke
parents:
diff changeset
63 Compile* C = Compile::current();
a61af66fc99e Initial load
duke
parents:
diff changeset
64
a61af66fc99e Initial load
duke
parents:
diff changeset
65 if (is_osr()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
66 // The JVMS for a OSR has a single argument (see its TypeFunc).
a61af66fc99e Initial load
duke
parents:
diff changeset
67 assert(jvms->depth() == 1, "no inline OSR");
a61af66fc99e Initial load
duke
parents:
diff changeset
68 }
a61af66fc99e Initial load
duke
parents:
diff changeset
69
a61af66fc99e Initial load
duke
parents:
diff changeset
70 if (C->failing()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
71 return NULL; // bailing out of the compile; do not try to parse
a61af66fc99e Initial load
duke
parents:
diff changeset
72 }
a61af66fc99e Initial load
duke
parents:
diff changeset
73
a61af66fc99e Initial load
duke
parents:
diff changeset
74 Parse parser(jvms, method(), _expected_uses);
a61af66fc99e Initial load
duke
parents:
diff changeset
75 // Grab signature for matching/allocation
a61af66fc99e Initial load
duke
parents:
diff changeset
76 #ifdef ASSERT
a61af66fc99e Initial load
duke
parents:
diff changeset
77 if (parser.tf() != (parser.depth() == 1 ? C->tf() : tf())) {
a61af66fc99e Initial load
duke
parents:
diff changeset
78 MutexLockerEx ml(Compile_lock, Mutex::_no_safepoint_check_flag);
a61af66fc99e Initial load
duke
parents:
diff changeset
79 assert(C->env()->system_dictionary_modification_counter_changed(),
a61af66fc99e Initial load
duke
parents:
diff changeset
80 "Must invalidate if TypeFuncs differ");
a61af66fc99e Initial load
duke
parents:
diff changeset
81 }
a61af66fc99e Initial load
duke
parents:
diff changeset
82 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
83
a61af66fc99e Initial load
duke
parents:
diff changeset
84 GraphKit& exits = parser.exits();
a61af66fc99e Initial load
duke
parents:
diff changeset
85
a61af66fc99e Initial load
duke
parents:
diff changeset
86 if (C->failing()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
87 while (exits.pop_exception_state() != NULL) ;
a61af66fc99e Initial load
duke
parents:
diff changeset
88 return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
89 }
a61af66fc99e Initial load
duke
parents:
diff changeset
90
a61af66fc99e Initial load
duke
parents:
diff changeset
91 assert(exits.jvms()->same_calls_as(jvms), "sanity");
a61af66fc99e Initial load
duke
parents:
diff changeset
92
a61af66fc99e Initial load
duke
parents:
diff changeset
93 // Simply return the exit state of the parser,
a61af66fc99e Initial load
duke
parents:
diff changeset
94 // augmented by any exceptional states.
a61af66fc99e Initial load
duke
parents:
diff changeset
95 return exits.transfer_exceptions_into_jvms();
a61af66fc99e Initial load
duke
parents:
diff changeset
96 }
a61af66fc99e Initial load
duke
parents:
diff changeset
97
a61af66fc99e Initial load
duke
parents:
diff changeset
98 //---------------------------DirectCallGenerator------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
99 // Internal class which handles all out-of-line calls w/o receiver type checks.
a61af66fc99e Initial load
duke
parents:
diff changeset
100 class DirectCallGenerator : public CallGenerator {
1080
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
101 private:
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
102 CallStaticJavaNode* _call_node;
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
103 // Force separate memory and I/O projections for the exceptional
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
104 // paths to facilitate late inlinig.
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
105 bool _separate_io_proj;
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
106
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
107 public:
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
108 DirectCallGenerator(ciMethod* method, bool separate_io_proj)
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
109 : CallGenerator(method),
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
110 _separate_io_proj(separate_io_proj)
0
a61af66fc99e Initial load
duke
parents:
diff changeset
111 {
a61af66fc99e Initial load
duke
parents:
diff changeset
112 }
a61af66fc99e Initial load
duke
parents:
diff changeset
113 virtual JVMState* generate(JVMState* jvms);
1080
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
114
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
115 CallStaticJavaNode* call_node() const { return _call_node; }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
116 };
a61af66fc99e Initial load
duke
parents:
diff changeset
117
a61af66fc99e Initial load
duke
parents:
diff changeset
118 JVMState* DirectCallGenerator::generate(JVMState* jvms) {
a61af66fc99e Initial load
duke
parents:
diff changeset
119 GraphKit kit(jvms);
a61af66fc99e Initial load
duke
parents:
diff changeset
120 bool is_static = method()->is_static();
a61af66fc99e Initial load
duke
parents:
diff changeset
121 address target = is_static ? SharedRuntime::get_resolve_static_call_stub()
a61af66fc99e Initial load
duke
parents:
diff changeset
122 : SharedRuntime::get_resolve_opt_virtual_call_stub();
a61af66fc99e Initial load
duke
parents:
diff changeset
123
a61af66fc99e Initial load
duke
parents:
diff changeset
124 if (kit.C->log() != NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
125 kit.C->log()->elem("direct_call bci='%d'", jvms->bci());
a61af66fc99e Initial load
duke
parents:
diff changeset
126 }
a61af66fc99e Initial load
duke
parents:
diff changeset
127
a61af66fc99e Initial load
duke
parents:
diff changeset
128 CallStaticJavaNode *call = new (kit.C, tf()->domain()->cnt()) CallStaticJavaNode(tf(), target, method(), kit.bci());
a61af66fc99e Initial load
duke
parents:
diff changeset
129 if (!is_static) {
a61af66fc99e Initial load
duke
parents:
diff changeset
130 // Make an explicit receiver null_check as part of this call.
a61af66fc99e Initial load
duke
parents:
diff changeset
131 // Since we share a map with the caller, his JVMS gets adjusted.
a61af66fc99e Initial load
duke
parents:
diff changeset
132 kit.null_check_receiver(method());
a61af66fc99e Initial load
duke
parents:
diff changeset
133 if (kit.stopped()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
134 // And dump it back to the caller, decorated with any exceptions:
a61af66fc99e Initial load
duke
parents:
diff changeset
135 return kit.transfer_exceptions_into_jvms();
a61af66fc99e Initial load
duke
parents:
diff changeset
136 }
a61af66fc99e Initial load
duke
parents:
diff changeset
137 // Mark the call node as virtual, sort of:
a61af66fc99e Initial load
duke
parents:
diff changeset
138 call->set_optimized_virtual(true);
a61af66fc99e Initial load
duke
parents:
diff changeset
139 }
a61af66fc99e Initial load
duke
parents:
diff changeset
140 kit.set_arguments_for_java_call(call);
1080
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
141 kit.set_edges_for_java_call(call, false, _separate_io_proj);
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
142 Node* ret = kit.set_results_for_java_call(call, _separate_io_proj);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
143 kit.push_node(method()->return_type()->basic_type(), ret);
1080
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
144 _call_node = call; // Save the call node in case we need it later
0
a61af66fc99e Initial load
duke
parents:
diff changeset
145 return kit.transfer_exceptions_into_jvms();
a61af66fc99e Initial load
duke
parents:
diff changeset
146 }
a61af66fc99e Initial load
duke
parents:
diff changeset
147
a61af66fc99e Initial load
duke
parents:
diff changeset
148 class VirtualCallGenerator : public CallGenerator {
a61af66fc99e Initial load
duke
parents:
diff changeset
149 private:
a61af66fc99e Initial load
duke
parents:
diff changeset
150 int _vtable_index;
a61af66fc99e Initial load
duke
parents:
diff changeset
151 public:
a61af66fc99e Initial load
duke
parents:
diff changeset
152 VirtualCallGenerator(ciMethod* method, int vtable_index)
a61af66fc99e Initial load
duke
parents:
diff changeset
153 : CallGenerator(method), _vtable_index(vtable_index)
a61af66fc99e Initial load
duke
parents:
diff changeset
154 {
a61af66fc99e Initial load
duke
parents:
diff changeset
155 assert(vtable_index == methodOopDesc::invalid_vtable_index ||
a61af66fc99e Initial load
duke
parents:
diff changeset
156 vtable_index >= 0, "either invalid or usable");
a61af66fc99e Initial load
duke
parents:
diff changeset
157 }
a61af66fc99e Initial load
duke
parents:
diff changeset
158 virtual bool is_virtual() const { return true; }
a61af66fc99e Initial load
duke
parents:
diff changeset
159 virtual JVMState* generate(JVMState* jvms);
a61af66fc99e Initial load
duke
parents:
diff changeset
160 };
a61af66fc99e Initial load
duke
parents:
diff changeset
161
a61af66fc99e Initial load
duke
parents:
diff changeset
162 //--------------------------VirtualCallGenerator------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
163 // Internal class which handles all out-of-line calls checking receiver type.
a61af66fc99e Initial load
duke
parents:
diff changeset
164 JVMState* VirtualCallGenerator::generate(JVMState* jvms) {
a61af66fc99e Initial load
duke
parents:
diff changeset
165 GraphKit kit(jvms);
a61af66fc99e Initial load
duke
parents:
diff changeset
166 Node* receiver = kit.argument(0);
a61af66fc99e Initial load
duke
parents:
diff changeset
167
a61af66fc99e Initial load
duke
parents:
diff changeset
168 if (kit.C->log() != NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
169 kit.C->log()->elem("virtual_call bci='%d'", jvms->bci());
a61af66fc99e Initial load
duke
parents:
diff changeset
170 }
a61af66fc99e Initial load
duke
parents:
diff changeset
171
a61af66fc99e Initial load
duke
parents:
diff changeset
172 // If the receiver is a constant null, do not torture the system
a61af66fc99e Initial load
duke
parents:
diff changeset
173 // by attempting to call through it. The compile will proceed
a61af66fc99e Initial load
duke
parents:
diff changeset
174 // correctly, but may bail out in final_graph_reshaping, because
a61af66fc99e Initial load
duke
parents:
diff changeset
175 // the call instruction will have a seemingly deficient out-count.
a61af66fc99e Initial load
duke
parents:
diff changeset
176 // (The bailout says something misleading about an "infinite loop".)
a61af66fc99e Initial load
duke
parents:
diff changeset
177 if (kit.gvn().type(receiver)->higher_equal(TypePtr::NULL_PTR)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
178 kit.inc_sp(method()->arg_size()); // restore arguments
a61af66fc99e Initial load
duke
parents:
diff changeset
179 kit.uncommon_trap(Deoptimization::Reason_null_check,
a61af66fc99e Initial load
duke
parents:
diff changeset
180 Deoptimization::Action_none,
a61af66fc99e Initial load
duke
parents:
diff changeset
181 NULL, "null receiver");
a61af66fc99e Initial load
duke
parents:
diff changeset
182 return kit.transfer_exceptions_into_jvms();
a61af66fc99e Initial load
duke
parents:
diff changeset
183 }
a61af66fc99e Initial load
duke
parents:
diff changeset
184
a61af66fc99e Initial load
duke
parents:
diff changeset
185 // Ideally we would unconditionally do a null check here and let it
a61af66fc99e Initial load
duke
parents:
diff changeset
186 // be converted to an implicit check based on profile information.
a61af66fc99e Initial load
duke
parents:
diff changeset
187 // However currently the conversion to implicit null checks in
a61af66fc99e Initial load
duke
parents:
diff changeset
188 // Block::implicit_null_check() only looks for loads and stores, not calls.
a61af66fc99e Initial load
duke
parents:
diff changeset
189 ciMethod *caller = kit.method();
a61af66fc99e Initial load
duke
parents:
diff changeset
190 ciMethodData *caller_md = (caller == NULL) ? NULL : caller->method_data();
a61af66fc99e Initial load
duke
parents:
diff changeset
191 if (!UseInlineCaches || !ImplicitNullChecks ||
a61af66fc99e Initial load
duke
parents:
diff changeset
192 ((ImplicitNullCheckThreshold > 0) && caller_md &&
a61af66fc99e Initial load
duke
parents:
diff changeset
193 (caller_md->trap_count(Deoptimization::Reason_null_check)
a61af66fc99e Initial load
duke
parents:
diff changeset
194 >= (uint)ImplicitNullCheckThreshold))) {
a61af66fc99e Initial load
duke
parents:
diff changeset
195 // Make an explicit receiver null_check as part of this call.
a61af66fc99e Initial load
duke
parents:
diff changeset
196 // Since we share a map with the caller, his JVMS gets adjusted.
a61af66fc99e Initial load
duke
parents:
diff changeset
197 receiver = kit.null_check_receiver(method());
a61af66fc99e Initial load
duke
parents:
diff changeset
198 if (kit.stopped()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
199 // And dump it back to the caller, decorated with any exceptions:
a61af66fc99e Initial load
duke
parents:
diff changeset
200 return kit.transfer_exceptions_into_jvms();
a61af66fc99e Initial load
duke
parents:
diff changeset
201 }
a61af66fc99e Initial load
duke
parents:
diff changeset
202 }
a61af66fc99e Initial load
duke
parents:
diff changeset
203
a61af66fc99e Initial load
duke
parents:
diff changeset
204 assert(!method()->is_static(), "virtual call must not be to static");
a61af66fc99e Initial load
duke
parents:
diff changeset
205 assert(!method()->is_final(), "virtual call should not be to final");
a61af66fc99e Initial load
duke
parents:
diff changeset
206 assert(!method()->is_private(), "virtual call should not be to private");
a61af66fc99e Initial load
duke
parents:
diff changeset
207 assert(_vtable_index == methodOopDesc::invalid_vtable_index || !UseInlineCaches,
a61af66fc99e Initial load
duke
parents:
diff changeset
208 "no vtable calls if +UseInlineCaches ");
a61af66fc99e Initial load
duke
parents:
diff changeset
209 address target = SharedRuntime::get_resolve_virtual_call_stub();
a61af66fc99e Initial load
duke
parents:
diff changeset
210 // Normal inline cache used for call
a61af66fc99e Initial load
duke
parents:
diff changeset
211 CallDynamicJavaNode *call = new (kit.C, tf()->domain()->cnt()) CallDynamicJavaNode(tf(), target, method(), _vtable_index, kit.bci());
a61af66fc99e Initial load
duke
parents:
diff changeset
212 kit.set_arguments_for_java_call(call);
a61af66fc99e Initial load
duke
parents:
diff changeset
213 kit.set_edges_for_java_call(call);
a61af66fc99e Initial load
duke
parents:
diff changeset
214 Node* ret = kit.set_results_for_java_call(call);
a61af66fc99e Initial load
duke
parents:
diff changeset
215 kit.push_node(method()->return_type()->basic_type(), ret);
a61af66fc99e Initial load
duke
parents:
diff changeset
216
a61af66fc99e Initial load
duke
parents:
diff changeset
217 // Represent the effect of an implicit receiver null_check
a61af66fc99e Initial load
duke
parents:
diff changeset
218 // as part of this call. Since we share a map with the caller,
a61af66fc99e Initial load
duke
parents:
diff changeset
219 // his JVMS gets adjusted.
a61af66fc99e Initial load
duke
parents:
diff changeset
220 kit.cast_not_null(receiver);
a61af66fc99e Initial load
duke
parents:
diff changeset
221 return kit.transfer_exceptions_into_jvms();
a61af66fc99e Initial load
duke
parents:
diff changeset
222 }
a61af66fc99e Initial load
duke
parents:
diff changeset
223
a61af66fc99e Initial load
duke
parents:
diff changeset
224 bool ParseGenerator::can_parse(ciMethod* m, int entry_bci) {
a61af66fc99e Initial load
duke
parents:
diff changeset
225 // Certain methods cannot be parsed at all:
a61af66fc99e Initial load
duke
parents:
diff changeset
226 if (!m->can_be_compiled()) return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
227 if (!m->has_balanced_monitors()) return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
228 if (m->get_flow_analysis()->failing()) return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
229
a61af66fc99e Initial load
duke
parents:
diff changeset
230 // (Methods may bail out for other reasons, after the parser is run.
a61af66fc99e Initial load
duke
parents:
diff changeset
231 // We try to avoid this, but if forced, we must return (Node*)NULL.
a61af66fc99e Initial load
duke
parents:
diff changeset
232 // The user of the CallGenerator must check for this condition.)
a61af66fc99e Initial load
duke
parents:
diff changeset
233 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
234 }
a61af66fc99e Initial load
duke
parents:
diff changeset
235
a61af66fc99e Initial load
duke
parents:
diff changeset
236 CallGenerator* CallGenerator::for_inline(ciMethod* m, float expected_uses) {
a61af66fc99e Initial load
duke
parents:
diff changeset
237 if (!ParseGenerator::can_parse(m)) return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
238 return new ParseGenerator(m, expected_uses);
a61af66fc99e Initial load
duke
parents:
diff changeset
239 }
a61af66fc99e Initial load
duke
parents:
diff changeset
240
a61af66fc99e Initial load
duke
parents:
diff changeset
241 // As a special case, the JVMS passed to this CallGenerator is
a61af66fc99e Initial load
duke
parents:
diff changeset
242 // for the method execution already in progress, not just the JVMS
a61af66fc99e Initial load
duke
parents:
diff changeset
243 // of the caller. Thus, this CallGenerator cannot be mixed with others!
a61af66fc99e Initial load
duke
parents:
diff changeset
244 CallGenerator* CallGenerator::for_osr(ciMethod* m, int osr_bci) {
a61af66fc99e Initial load
duke
parents:
diff changeset
245 if (!ParseGenerator::can_parse(m, true)) return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
246 float past_uses = m->interpreter_invocation_count();
a61af66fc99e Initial load
duke
parents:
diff changeset
247 float expected_uses = past_uses;
a61af66fc99e Initial load
duke
parents:
diff changeset
248 return new ParseGenerator(m, expected_uses, true);
a61af66fc99e Initial load
duke
parents:
diff changeset
249 }
a61af66fc99e Initial load
duke
parents:
diff changeset
250
1080
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
251 CallGenerator* CallGenerator::for_direct_call(ciMethod* m, bool separate_io_proj) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
252 assert(!m->is_abstract(), "for_direct_call mismatch");
1080
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
253 return new DirectCallGenerator(m, separate_io_proj);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
254 }
a61af66fc99e Initial load
duke
parents:
diff changeset
255
a61af66fc99e Initial load
duke
parents:
diff changeset
256 CallGenerator* CallGenerator::for_virtual_call(ciMethod* m, int vtable_index) {
a61af66fc99e Initial load
duke
parents:
diff changeset
257 assert(!m->is_static(), "for_virtual_call mismatch");
a61af66fc99e Initial load
duke
parents:
diff changeset
258 return new VirtualCallGenerator(m, vtable_index);
a61af66fc99e Initial load
duke
parents:
diff changeset
259 }
a61af66fc99e Initial load
duke
parents:
diff changeset
260
1080
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
261 // Allow inlining decisions to be delayed
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
262 class LateInlineCallGenerator : public DirectCallGenerator {
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
263 CallGenerator* _inline_cg;
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
264
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
265 public:
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
266 LateInlineCallGenerator(ciMethod* method, CallGenerator* inline_cg) :
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
267 DirectCallGenerator(method, true), _inline_cg(inline_cg) {}
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
268
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
269 virtual bool is_late_inline() const { return true; }
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
270
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
271 // Convert the CallStaticJava into an inline
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
272 virtual void do_late_inline();
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
273
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
274 JVMState* generate(JVMState* jvms) {
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
275 // Record that this call site should be revisited once the main
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
276 // parse is finished.
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
277 Compile::current()->add_late_inline(this);
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
278
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
279 // Emit the CallStaticJava and request separate projections so
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
280 // that the late inlining logic can distinguish between fall
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
281 // through and exceptional uses of the memory and io projections
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
282 // as is done for allocations and macro expansion.
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
283 return DirectCallGenerator::generate(jvms);
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
284 }
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
285
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
286 };
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
287
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
288
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
289 void LateInlineCallGenerator::do_late_inline() {
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
290 // Can't inline it
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
291 if (call_node() == NULL || call_node()->outcnt() == 0 ||
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
292 call_node()->in(0) == NULL || call_node()->in(0)->is_top())
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
293 return;
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
294
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
295 CallStaticJavaNode* call = call_node();
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
296
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
297 // Make a clone of the JVMState that appropriate to use for driving a parse
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
298 Compile* C = Compile::current();
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
299 JVMState* jvms = call->jvms()->clone_shallow(C);
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
300 uint size = call->req();
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
301 SafePointNode* map = new (C, size) SafePointNode(size, jvms);
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
302 for (uint i1 = 0; i1 < size; i1++) {
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
303 map->init_req(i1, call->in(i1));
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
304 }
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
305
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
306 // Make sure the state is a MergeMem for parsing.
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
307 if (!map->in(TypeFunc::Memory)->is_MergeMem()) {
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
308 map->set_req(TypeFunc::Memory, MergeMemNode::make(C, map->in(TypeFunc::Memory)));
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
309 }
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
310
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
311 // Make enough space for the expression stack and transfer the incoming arguments
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
312 int nargs = method()->arg_size();
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
313 jvms->set_map(map);
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
314 map->ensure_stack(jvms, jvms->method()->max_stack());
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
315 if (nargs > 0) {
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
316 for (int i1 = 0; i1 < nargs; i1++) {
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
317 map->set_req(i1 + jvms->argoff(), call->in(TypeFunc::Parms + i1));
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
318 }
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
319 }
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
320
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
321 CompileLog* log = C->log();
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
322 if (log != NULL) {
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
323 log->head("late_inline method='%d'", log->identify(method()));
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
324 JVMState* p = jvms;
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
325 while (p != NULL) {
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
326 log->elem("jvms bci='%d' method='%d'", p->bci(), log->identify(p->method()));
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
327 p = p->caller();
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
328 }
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
329 log->tail("late_inline");
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
330 }
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
331
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
332 // Setup default node notes to be picked up by the inlining
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
333 Node_Notes* old_nn = C->default_node_notes();
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
334 if (old_nn != NULL) {
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
335 Node_Notes* entry_nn = old_nn->clone(C);
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
336 entry_nn->set_jvms(jvms);
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
337 C->set_default_node_notes(entry_nn);
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
338 }
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
339
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
340 // Now perform the inling using the synthesized JVMState
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
341 JVMState* new_jvms = _inline_cg->generate(jvms);
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
342 if (new_jvms == NULL) return; // no change
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
343 if (C->failing()) return;
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
344
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
345 // Capture any exceptional control flow
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
346 GraphKit kit(new_jvms);
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
347
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
348 // Find the result object
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
349 Node* result = C->top();
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
350 int result_size = method()->return_type()->size();
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
351 if (result_size != 0 && !kit.stopped()) {
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
352 result = (result_size == 1) ? kit.pop() : kit.pop_pair();
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
353 }
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
354
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
355 kit.replace_call(call, result);
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
356 }
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
357
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
358
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
359 CallGenerator* CallGenerator::for_late_inline(ciMethod* method, CallGenerator* inline_cg) {
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
360 return new LateInlineCallGenerator(method, inline_cg);
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
361 }
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
362
0
a61af66fc99e Initial load
duke
parents:
diff changeset
363
a61af66fc99e Initial load
duke
parents:
diff changeset
364 //---------------------------WarmCallGenerator--------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
365 // Internal class which handles initial deferral of inlining decisions.
a61af66fc99e Initial load
duke
parents:
diff changeset
366 class WarmCallGenerator : public CallGenerator {
a61af66fc99e Initial load
duke
parents:
diff changeset
367 WarmCallInfo* _call_info;
a61af66fc99e Initial load
duke
parents:
diff changeset
368 CallGenerator* _if_cold;
a61af66fc99e Initial load
duke
parents:
diff changeset
369 CallGenerator* _if_hot;
a61af66fc99e Initial load
duke
parents:
diff changeset
370 bool _is_virtual; // caches virtuality of if_cold
a61af66fc99e Initial load
duke
parents:
diff changeset
371 bool _is_inline; // caches inline-ness of if_hot
a61af66fc99e Initial load
duke
parents:
diff changeset
372
a61af66fc99e Initial load
duke
parents:
diff changeset
373 public:
a61af66fc99e Initial load
duke
parents:
diff changeset
374 WarmCallGenerator(WarmCallInfo* ci,
a61af66fc99e Initial load
duke
parents:
diff changeset
375 CallGenerator* if_cold,
a61af66fc99e Initial load
duke
parents:
diff changeset
376 CallGenerator* if_hot)
a61af66fc99e Initial load
duke
parents:
diff changeset
377 : CallGenerator(if_cold->method())
a61af66fc99e Initial load
duke
parents:
diff changeset
378 {
a61af66fc99e Initial load
duke
parents:
diff changeset
379 assert(method() == if_hot->method(), "consistent choices");
a61af66fc99e Initial load
duke
parents:
diff changeset
380 _call_info = ci;
a61af66fc99e Initial load
duke
parents:
diff changeset
381 _if_cold = if_cold;
a61af66fc99e Initial load
duke
parents:
diff changeset
382 _if_hot = if_hot;
a61af66fc99e Initial load
duke
parents:
diff changeset
383 _is_virtual = if_cold->is_virtual();
a61af66fc99e Initial load
duke
parents:
diff changeset
384 _is_inline = if_hot->is_inline();
a61af66fc99e Initial load
duke
parents:
diff changeset
385 }
a61af66fc99e Initial load
duke
parents:
diff changeset
386
a61af66fc99e Initial load
duke
parents:
diff changeset
387 virtual bool is_inline() const { return _is_inline; }
a61af66fc99e Initial load
duke
parents:
diff changeset
388 virtual bool is_virtual() const { return _is_virtual; }
a61af66fc99e Initial load
duke
parents:
diff changeset
389 virtual bool is_deferred() const { return true; }
a61af66fc99e Initial load
duke
parents:
diff changeset
390
a61af66fc99e Initial load
duke
parents:
diff changeset
391 virtual JVMState* generate(JVMState* jvms);
a61af66fc99e Initial load
duke
parents:
diff changeset
392 };
a61af66fc99e Initial load
duke
parents:
diff changeset
393
a61af66fc99e Initial load
duke
parents:
diff changeset
394
a61af66fc99e Initial load
duke
parents:
diff changeset
395 CallGenerator* CallGenerator::for_warm_call(WarmCallInfo* ci,
a61af66fc99e Initial load
duke
parents:
diff changeset
396 CallGenerator* if_cold,
a61af66fc99e Initial load
duke
parents:
diff changeset
397 CallGenerator* if_hot) {
a61af66fc99e Initial load
duke
parents:
diff changeset
398 return new WarmCallGenerator(ci, if_cold, if_hot);
a61af66fc99e Initial load
duke
parents:
diff changeset
399 }
a61af66fc99e Initial load
duke
parents:
diff changeset
400
a61af66fc99e Initial load
duke
parents:
diff changeset
401 JVMState* WarmCallGenerator::generate(JVMState* jvms) {
a61af66fc99e Initial load
duke
parents:
diff changeset
402 Compile* C = Compile::current();
a61af66fc99e Initial load
duke
parents:
diff changeset
403 if (C->log() != NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
404 C->log()->elem("warm_call bci='%d'", jvms->bci());
a61af66fc99e Initial load
duke
parents:
diff changeset
405 }
a61af66fc99e Initial load
duke
parents:
diff changeset
406 jvms = _if_cold->generate(jvms);
a61af66fc99e Initial load
duke
parents:
diff changeset
407 if (jvms != NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
408 Node* m = jvms->map()->control();
a61af66fc99e Initial load
duke
parents:
diff changeset
409 if (m->is_CatchProj()) m = m->in(0); else m = C->top();
a61af66fc99e Initial load
duke
parents:
diff changeset
410 if (m->is_Catch()) m = m->in(0); else m = C->top();
a61af66fc99e Initial load
duke
parents:
diff changeset
411 if (m->is_Proj()) m = m->in(0); else m = C->top();
a61af66fc99e Initial load
duke
parents:
diff changeset
412 if (m->is_CallJava()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
413 _call_info->set_call(m->as_Call());
a61af66fc99e Initial load
duke
parents:
diff changeset
414 _call_info->set_hot_cg(_if_hot);
a61af66fc99e Initial load
duke
parents:
diff changeset
415 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
416 if (PrintOpto || PrintOptoInlining) {
a61af66fc99e Initial load
duke
parents:
diff changeset
417 tty->print_cr("Queueing for warm inlining at bci %d:", jvms->bci());
a61af66fc99e Initial load
duke
parents:
diff changeset
418 tty->print("WCI: ");
a61af66fc99e Initial load
duke
parents:
diff changeset
419 _call_info->print();
a61af66fc99e Initial load
duke
parents:
diff changeset
420 }
a61af66fc99e Initial load
duke
parents:
diff changeset
421 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
422 _call_info->set_heat(_call_info->compute_heat());
a61af66fc99e Initial load
duke
parents:
diff changeset
423 C->set_warm_calls(_call_info->insert_into(C->warm_calls()));
a61af66fc99e Initial load
duke
parents:
diff changeset
424 }
a61af66fc99e Initial load
duke
parents:
diff changeset
425 }
a61af66fc99e Initial load
duke
parents:
diff changeset
426 return jvms;
a61af66fc99e Initial load
duke
parents:
diff changeset
427 }
a61af66fc99e Initial load
duke
parents:
diff changeset
428
a61af66fc99e Initial load
duke
parents:
diff changeset
429 void WarmCallInfo::make_hot() {
1080
7c57aead6d3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 337
diff changeset
430 Unimplemented();
0
a61af66fc99e Initial load
duke
parents:
diff changeset
431 }
a61af66fc99e Initial load
duke
parents:
diff changeset
432
a61af66fc99e Initial load
duke
parents:
diff changeset
433 void WarmCallInfo::make_cold() {
a61af66fc99e Initial load
duke
parents:
diff changeset
434 // No action: Just dequeue.
a61af66fc99e Initial load
duke
parents:
diff changeset
435 }
a61af66fc99e Initial load
duke
parents:
diff changeset
436
a61af66fc99e Initial load
duke
parents:
diff changeset
437
a61af66fc99e Initial load
duke
parents:
diff changeset
438 //------------------------PredictedCallGenerator------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
439 // Internal class which handles all out-of-line calls checking receiver type.
a61af66fc99e Initial load
duke
parents:
diff changeset
440 class PredictedCallGenerator : public CallGenerator {
a61af66fc99e Initial load
duke
parents:
diff changeset
441 ciKlass* _predicted_receiver;
a61af66fc99e Initial load
duke
parents:
diff changeset
442 CallGenerator* _if_missed;
a61af66fc99e Initial load
duke
parents:
diff changeset
443 CallGenerator* _if_hit;
a61af66fc99e Initial load
duke
parents:
diff changeset
444 float _hit_prob;
a61af66fc99e Initial load
duke
parents:
diff changeset
445
a61af66fc99e Initial load
duke
parents:
diff changeset
446 public:
a61af66fc99e Initial load
duke
parents:
diff changeset
447 PredictedCallGenerator(ciKlass* predicted_receiver,
a61af66fc99e Initial load
duke
parents:
diff changeset
448 CallGenerator* if_missed,
a61af66fc99e Initial load
duke
parents:
diff changeset
449 CallGenerator* if_hit, float hit_prob)
a61af66fc99e Initial load
duke
parents:
diff changeset
450 : CallGenerator(if_missed->method())
a61af66fc99e Initial load
duke
parents:
diff changeset
451 {
a61af66fc99e Initial load
duke
parents:
diff changeset
452 // The call profile data may predict the hit_prob as extreme as 0 or 1.
a61af66fc99e Initial load
duke
parents:
diff changeset
453 // Remove the extremes values from the range.
a61af66fc99e Initial load
duke
parents:
diff changeset
454 if (hit_prob > PROB_MAX) hit_prob = PROB_MAX;
a61af66fc99e Initial load
duke
parents:
diff changeset
455 if (hit_prob < PROB_MIN) hit_prob = PROB_MIN;
a61af66fc99e Initial load
duke
parents:
diff changeset
456
a61af66fc99e Initial load
duke
parents:
diff changeset
457 _predicted_receiver = predicted_receiver;
a61af66fc99e Initial load
duke
parents:
diff changeset
458 _if_missed = if_missed;
a61af66fc99e Initial load
duke
parents:
diff changeset
459 _if_hit = if_hit;
a61af66fc99e Initial load
duke
parents:
diff changeset
460 _hit_prob = hit_prob;
a61af66fc99e Initial load
duke
parents:
diff changeset
461 }
a61af66fc99e Initial load
duke
parents:
diff changeset
462
a61af66fc99e Initial load
duke
parents:
diff changeset
463 virtual bool is_virtual() const { return true; }
a61af66fc99e Initial load
duke
parents:
diff changeset
464 virtual bool is_inline() const { return _if_hit->is_inline(); }
a61af66fc99e Initial load
duke
parents:
diff changeset
465 virtual bool is_deferred() const { return _if_hit->is_deferred(); }
a61af66fc99e Initial load
duke
parents:
diff changeset
466
a61af66fc99e Initial load
duke
parents:
diff changeset
467 virtual JVMState* generate(JVMState* jvms);
a61af66fc99e Initial load
duke
parents:
diff changeset
468 };
a61af66fc99e Initial load
duke
parents:
diff changeset
469
a61af66fc99e Initial load
duke
parents:
diff changeset
470
a61af66fc99e Initial load
duke
parents:
diff changeset
471 CallGenerator* CallGenerator::for_predicted_call(ciKlass* predicted_receiver,
a61af66fc99e Initial load
duke
parents:
diff changeset
472 CallGenerator* if_missed,
a61af66fc99e Initial load
duke
parents:
diff changeset
473 CallGenerator* if_hit,
a61af66fc99e Initial load
duke
parents:
diff changeset
474 float hit_prob) {
a61af66fc99e Initial load
duke
parents:
diff changeset
475 return new PredictedCallGenerator(predicted_receiver, if_missed, if_hit, hit_prob);
a61af66fc99e Initial load
duke
parents:
diff changeset
476 }
a61af66fc99e Initial load
duke
parents:
diff changeset
477
a61af66fc99e Initial load
duke
parents:
diff changeset
478
a61af66fc99e Initial load
duke
parents:
diff changeset
479 JVMState* PredictedCallGenerator::generate(JVMState* jvms) {
a61af66fc99e Initial load
duke
parents:
diff changeset
480 GraphKit kit(jvms);
a61af66fc99e Initial load
duke
parents:
diff changeset
481 PhaseGVN& gvn = kit.gvn();
a61af66fc99e Initial load
duke
parents:
diff changeset
482 // We need an explicit receiver null_check before checking its type.
a61af66fc99e Initial load
duke
parents:
diff changeset
483 // We share a map with the caller, so his JVMS gets adjusted.
a61af66fc99e Initial load
duke
parents:
diff changeset
484 Node* receiver = kit.argument(0);
a61af66fc99e Initial load
duke
parents:
diff changeset
485
a61af66fc99e Initial load
duke
parents:
diff changeset
486 CompileLog* log = kit.C->log();
a61af66fc99e Initial load
duke
parents:
diff changeset
487 if (log != NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
488 log->elem("predicted_call bci='%d' klass='%d'",
a61af66fc99e Initial load
duke
parents:
diff changeset
489 jvms->bci(), log->identify(_predicted_receiver));
a61af66fc99e Initial load
duke
parents:
diff changeset
490 }
a61af66fc99e Initial load
duke
parents:
diff changeset
491
a61af66fc99e Initial load
duke
parents:
diff changeset
492 receiver = kit.null_check_receiver(method());
a61af66fc99e Initial load
duke
parents:
diff changeset
493 if (kit.stopped()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
494 return kit.transfer_exceptions_into_jvms();
a61af66fc99e Initial load
duke
parents:
diff changeset
495 }
a61af66fc99e Initial load
duke
parents:
diff changeset
496
a61af66fc99e Initial load
duke
parents:
diff changeset
497 Node* exact_receiver = receiver; // will get updated in place...
a61af66fc99e Initial load
duke
parents:
diff changeset
498 Node* slow_ctl = kit.type_check_receiver(receiver,
a61af66fc99e Initial load
duke
parents:
diff changeset
499 _predicted_receiver, _hit_prob,
a61af66fc99e Initial load
duke
parents:
diff changeset
500 &exact_receiver);
a61af66fc99e Initial load
duke
parents:
diff changeset
501
a61af66fc99e Initial load
duke
parents:
diff changeset
502 SafePointNode* slow_map = NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
503 JVMState* slow_jvms;
a61af66fc99e Initial load
duke
parents:
diff changeset
504 { PreserveJVMState pjvms(&kit);
a61af66fc99e Initial load
duke
parents:
diff changeset
505 kit.set_control(slow_ctl);
a61af66fc99e Initial load
duke
parents:
diff changeset
506 if (!kit.stopped()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
507 slow_jvms = _if_missed->generate(kit.sync_jvms());
a61af66fc99e Initial load
duke
parents:
diff changeset
508 assert(slow_jvms != NULL, "miss path must not fail to generate");
a61af66fc99e Initial load
duke
parents:
diff changeset
509 kit.add_exception_states_from(slow_jvms);
a61af66fc99e Initial load
duke
parents:
diff changeset
510 kit.set_map(slow_jvms->map());
a61af66fc99e Initial load
duke
parents:
diff changeset
511 if (!kit.stopped())
a61af66fc99e Initial load
duke
parents:
diff changeset
512 slow_map = kit.stop();
a61af66fc99e Initial load
duke
parents:
diff changeset
513 }
a61af66fc99e Initial load
duke
parents:
diff changeset
514 }
a61af66fc99e Initial load
duke
parents:
diff changeset
515
293
c3e045194476 6731641: assert(m->adr_type() == mach->adr_type(),"matcher should not change adr type")
kvn
parents: 0
diff changeset
516 if (kit.stopped()) {
c3e045194476 6731641: assert(m->adr_type() == mach->adr_type(),"matcher should not change adr type")
kvn
parents: 0
diff changeset
517 // Instance exactly does not matches the desired type.
c3e045194476 6731641: assert(m->adr_type() == mach->adr_type(),"matcher should not change adr type")
kvn
parents: 0
diff changeset
518 kit.set_jvms(slow_jvms);
c3e045194476 6731641: assert(m->adr_type() == mach->adr_type(),"matcher should not change adr type")
kvn
parents: 0
diff changeset
519 return kit.transfer_exceptions_into_jvms();
c3e045194476 6731641: assert(m->adr_type() == mach->adr_type(),"matcher should not change adr type")
kvn
parents: 0
diff changeset
520 }
c3e045194476 6731641: assert(m->adr_type() == mach->adr_type(),"matcher should not change adr type")
kvn
parents: 0
diff changeset
521
0
a61af66fc99e Initial load
duke
parents:
diff changeset
522 // fall through if the instance exactly matches the desired type
a61af66fc99e Initial load
duke
parents:
diff changeset
523 kit.replace_in_map(receiver, exact_receiver);
a61af66fc99e Initial load
duke
parents:
diff changeset
524
a61af66fc99e Initial load
duke
parents:
diff changeset
525 // Make the hot call:
a61af66fc99e Initial load
duke
parents:
diff changeset
526 JVMState* new_jvms = _if_hit->generate(kit.sync_jvms());
a61af66fc99e Initial load
duke
parents:
diff changeset
527 if (new_jvms == NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
528 // Inline failed, so make a direct call.
a61af66fc99e Initial load
duke
parents:
diff changeset
529 assert(_if_hit->is_inline(), "must have been a failed inline");
a61af66fc99e Initial load
duke
parents:
diff changeset
530 CallGenerator* cg = CallGenerator::for_direct_call(_if_hit->method());
a61af66fc99e Initial load
duke
parents:
diff changeset
531 new_jvms = cg->generate(kit.sync_jvms());
a61af66fc99e Initial load
duke
parents:
diff changeset
532 }
a61af66fc99e Initial load
duke
parents:
diff changeset
533 kit.add_exception_states_from(new_jvms);
a61af66fc99e Initial load
duke
parents:
diff changeset
534 kit.set_jvms(new_jvms);
a61af66fc99e Initial load
duke
parents:
diff changeset
535
a61af66fc99e Initial load
duke
parents:
diff changeset
536 // Need to merge slow and fast?
a61af66fc99e Initial load
duke
parents:
diff changeset
537 if (slow_map == NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
538 // The fast path is the only path remaining.
a61af66fc99e Initial load
duke
parents:
diff changeset
539 return kit.transfer_exceptions_into_jvms();
a61af66fc99e Initial load
duke
parents:
diff changeset
540 }
a61af66fc99e Initial load
duke
parents:
diff changeset
541
a61af66fc99e Initial load
duke
parents:
diff changeset
542 if (kit.stopped()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
543 // Inlined method threw an exception, so it's just the slow path after all.
a61af66fc99e Initial load
duke
parents:
diff changeset
544 kit.set_jvms(slow_jvms);
a61af66fc99e Initial load
duke
parents:
diff changeset
545 return kit.transfer_exceptions_into_jvms();
a61af66fc99e Initial load
duke
parents:
diff changeset
546 }
a61af66fc99e Initial load
duke
parents:
diff changeset
547
a61af66fc99e Initial load
duke
parents:
diff changeset
548 // Finish the diamond.
a61af66fc99e Initial load
duke
parents:
diff changeset
549 kit.C->set_has_split_ifs(true); // Has chance for split-if optimization
a61af66fc99e Initial load
duke
parents:
diff changeset
550 RegionNode* region = new (kit.C, 3) RegionNode(3);
a61af66fc99e Initial load
duke
parents:
diff changeset
551 region->init_req(1, kit.control());
a61af66fc99e Initial load
duke
parents:
diff changeset
552 region->init_req(2, slow_map->control());
a61af66fc99e Initial load
duke
parents:
diff changeset
553 kit.set_control(gvn.transform(region));
a61af66fc99e Initial load
duke
parents:
diff changeset
554 Node* iophi = PhiNode::make(region, kit.i_o(), Type::ABIO);
a61af66fc99e Initial load
duke
parents:
diff changeset
555 iophi->set_req(2, slow_map->i_o());
a61af66fc99e Initial load
duke
parents:
diff changeset
556 kit.set_i_o(gvn.transform(iophi));
a61af66fc99e Initial load
duke
parents:
diff changeset
557 kit.merge_memory(slow_map->merged_memory(), region, 2);
a61af66fc99e Initial load
duke
parents:
diff changeset
558 uint tos = kit.jvms()->stkoff() + kit.sp();
a61af66fc99e Initial load
duke
parents:
diff changeset
559 uint limit = slow_map->req();
a61af66fc99e Initial load
duke
parents:
diff changeset
560 for (uint i = TypeFunc::Parms; i < limit; i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
561 // Skip unused stack slots; fast forward to monoff();
a61af66fc99e Initial load
duke
parents:
diff changeset
562 if (i == tos) {
a61af66fc99e Initial load
duke
parents:
diff changeset
563 i = kit.jvms()->monoff();
a61af66fc99e Initial load
duke
parents:
diff changeset
564 if( i >= limit ) break;
a61af66fc99e Initial load
duke
parents:
diff changeset
565 }
a61af66fc99e Initial load
duke
parents:
diff changeset
566 Node* m = kit.map()->in(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
567 Node* n = slow_map->in(i);
a61af66fc99e Initial load
duke
parents:
diff changeset
568 if (m != n) {
a61af66fc99e Initial load
duke
parents:
diff changeset
569 const Type* t = gvn.type(m)->meet(gvn.type(n));
a61af66fc99e Initial load
duke
parents:
diff changeset
570 Node* phi = PhiNode::make(region, m, t);
a61af66fc99e Initial load
duke
parents:
diff changeset
571 phi->set_req(2, n);
a61af66fc99e Initial load
duke
parents:
diff changeset
572 kit.map()->set_req(i, gvn.transform(phi));
a61af66fc99e Initial load
duke
parents:
diff changeset
573 }
a61af66fc99e Initial load
duke
parents:
diff changeset
574 }
a61af66fc99e Initial load
duke
parents:
diff changeset
575 return kit.transfer_exceptions_into_jvms();
a61af66fc99e Initial load
duke
parents:
diff changeset
576 }
a61af66fc99e Initial load
duke
parents:
diff changeset
577
a61af66fc99e Initial load
duke
parents:
diff changeset
578
a61af66fc99e Initial load
duke
parents:
diff changeset
579 //-------------------------UncommonTrapCallGenerator-----------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
580 // Internal class which handles all out-of-line calls checking receiver type.
a61af66fc99e Initial load
duke
parents:
diff changeset
581 class UncommonTrapCallGenerator : public CallGenerator {
a61af66fc99e Initial load
duke
parents:
diff changeset
582 Deoptimization::DeoptReason _reason;
a61af66fc99e Initial load
duke
parents:
diff changeset
583 Deoptimization::DeoptAction _action;
a61af66fc99e Initial load
duke
parents:
diff changeset
584
a61af66fc99e Initial load
duke
parents:
diff changeset
585 public:
a61af66fc99e Initial load
duke
parents:
diff changeset
586 UncommonTrapCallGenerator(ciMethod* m,
a61af66fc99e Initial load
duke
parents:
diff changeset
587 Deoptimization::DeoptReason reason,
a61af66fc99e Initial load
duke
parents:
diff changeset
588 Deoptimization::DeoptAction action)
a61af66fc99e Initial load
duke
parents:
diff changeset
589 : CallGenerator(m)
a61af66fc99e Initial load
duke
parents:
diff changeset
590 {
a61af66fc99e Initial load
duke
parents:
diff changeset
591 _reason = reason;
a61af66fc99e Initial load
duke
parents:
diff changeset
592 _action = action;
a61af66fc99e Initial load
duke
parents:
diff changeset
593 }
a61af66fc99e Initial load
duke
parents:
diff changeset
594
a61af66fc99e Initial load
duke
parents:
diff changeset
595 virtual bool is_virtual() const { ShouldNotReachHere(); return false; }
a61af66fc99e Initial load
duke
parents:
diff changeset
596 virtual bool is_trap() const { return true; }
a61af66fc99e Initial load
duke
parents:
diff changeset
597
a61af66fc99e Initial load
duke
parents:
diff changeset
598 virtual JVMState* generate(JVMState* jvms);
a61af66fc99e Initial load
duke
parents:
diff changeset
599 };
a61af66fc99e Initial load
duke
parents:
diff changeset
600
a61af66fc99e Initial load
duke
parents:
diff changeset
601
a61af66fc99e Initial load
duke
parents:
diff changeset
602 CallGenerator*
a61af66fc99e Initial load
duke
parents:
diff changeset
603 CallGenerator::for_uncommon_trap(ciMethod* m,
a61af66fc99e Initial load
duke
parents:
diff changeset
604 Deoptimization::DeoptReason reason,
a61af66fc99e Initial load
duke
parents:
diff changeset
605 Deoptimization::DeoptAction action) {
a61af66fc99e Initial load
duke
parents:
diff changeset
606 return new UncommonTrapCallGenerator(m, reason, action);
a61af66fc99e Initial load
duke
parents:
diff changeset
607 }
a61af66fc99e Initial load
duke
parents:
diff changeset
608
a61af66fc99e Initial load
duke
parents:
diff changeset
609
a61af66fc99e Initial load
duke
parents:
diff changeset
610 JVMState* UncommonTrapCallGenerator::generate(JVMState* jvms) {
a61af66fc99e Initial load
duke
parents:
diff changeset
611 GraphKit kit(jvms);
a61af66fc99e Initial load
duke
parents:
diff changeset
612 // Take the trap with arguments pushed on the stack. (Cf. null_check_receiver).
a61af66fc99e Initial load
duke
parents:
diff changeset
613 int nargs = method()->arg_size();
a61af66fc99e Initial load
duke
parents:
diff changeset
614 kit.inc_sp(nargs);
a61af66fc99e Initial load
duke
parents:
diff changeset
615 assert(nargs <= kit.sp() && kit.sp() <= jvms->stk_size(), "sane sp w/ args pushed");
a61af66fc99e Initial load
duke
parents:
diff changeset
616 if (_reason == Deoptimization::Reason_class_check &&
a61af66fc99e Initial load
duke
parents:
diff changeset
617 _action == Deoptimization::Action_maybe_recompile) {
a61af66fc99e Initial load
duke
parents:
diff changeset
618 // Temp fix for 6529811
a61af66fc99e Initial load
duke
parents:
diff changeset
619 // Don't allow uncommon_trap to override our decision to recompile in the event
a61af66fc99e Initial load
duke
parents:
diff changeset
620 // of a class cast failure for a monomorphic call as it will never let us convert
a61af66fc99e Initial load
duke
parents:
diff changeset
621 // the call to either bi-morphic or megamorphic and can lead to unc-trap loops
a61af66fc99e Initial load
duke
parents:
diff changeset
622 bool keep_exact_action = true;
a61af66fc99e Initial load
duke
parents:
diff changeset
623 kit.uncommon_trap(_reason, _action, NULL, "monomorphic vcall checkcast", false, keep_exact_action);
a61af66fc99e Initial load
duke
parents:
diff changeset
624 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
625 kit.uncommon_trap(_reason, _action);
a61af66fc99e Initial load
duke
parents:
diff changeset
626 }
a61af66fc99e Initial load
duke
parents:
diff changeset
627 return kit.transfer_exceptions_into_jvms();
a61af66fc99e Initial load
duke
parents:
diff changeset
628 }
a61af66fc99e Initial load
duke
parents:
diff changeset
629
a61af66fc99e Initial load
duke
parents:
diff changeset
630 // (Note: Moved hook_up_call to GraphKit::set_edges_for_java_call.)
a61af66fc99e Initial load
duke
parents:
diff changeset
631
a61af66fc99e Initial load
duke
parents:
diff changeset
632 // (Node: Merged hook_up_exits into ParseGenerator::generate.)
a61af66fc99e Initial load
duke
parents:
diff changeset
633
a61af66fc99e Initial load
duke
parents:
diff changeset
634 #define NODES_OVERHEAD_PER_METHOD (30.0)
a61af66fc99e Initial load
duke
parents:
diff changeset
635 #define NODES_PER_BYTECODE (9.5)
a61af66fc99e Initial load
duke
parents:
diff changeset
636
a61af66fc99e Initial load
duke
parents:
diff changeset
637 void WarmCallInfo::init(JVMState* call_site, ciMethod* call_method, ciCallProfile& profile, float prof_factor) {
a61af66fc99e Initial load
duke
parents:
diff changeset
638 int call_count = profile.count();
a61af66fc99e Initial load
duke
parents:
diff changeset
639 int code_size = call_method->code_size();
a61af66fc99e Initial load
duke
parents:
diff changeset
640
a61af66fc99e Initial load
duke
parents:
diff changeset
641 // Expected execution count is based on the historical count:
a61af66fc99e Initial load
duke
parents:
diff changeset
642 _count = call_count < 0 ? 1 : call_site->method()->scale_count(call_count, prof_factor);
a61af66fc99e Initial load
duke
parents:
diff changeset
643
a61af66fc99e Initial load
duke
parents:
diff changeset
644 // Expected profit from inlining, in units of simple call-overheads.
a61af66fc99e Initial load
duke
parents:
diff changeset
645 _profit = 1.0;
a61af66fc99e Initial load
duke
parents:
diff changeset
646
a61af66fc99e Initial load
duke
parents:
diff changeset
647 // Expected work performed by the call in units of call-overheads.
a61af66fc99e Initial load
duke
parents:
diff changeset
648 // %%% need an empirical curve fit for "work" (time in call)
a61af66fc99e Initial load
duke
parents:
diff changeset
649 float bytecodes_per_call = 3;
a61af66fc99e Initial load
duke
parents:
diff changeset
650 _work = 1.0 + code_size / bytecodes_per_call;
a61af66fc99e Initial load
duke
parents:
diff changeset
651
a61af66fc99e Initial load
duke
parents:
diff changeset
652 // Expected size of compilation graph:
a61af66fc99e Initial load
duke
parents:
diff changeset
653 // -XX:+PrintParseStatistics once reported:
a61af66fc99e Initial load
duke
parents:
diff changeset
654 // Methods seen: 9184 Methods parsed: 9184 Nodes created: 1582391
a61af66fc99e Initial load
duke
parents:
diff changeset
655 // Histogram of 144298 parsed bytecodes:
a61af66fc99e Initial load
duke
parents:
diff changeset
656 // %%% Need an better predictor for graph size.
a61af66fc99e Initial load
duke
parents:
diff changeset
657 _size = NODES_OVERHEAD_PER_METHOD + (NODES_PER_BYTECODE * code_size);
a61af66fc99e Initial load
duke
parents:
diff changeset
658 }
a61af66fc99e Initial load
duke
parents:
diff changeset
659
a61af66fc99e Initial load
duke
parents:
diff changeset
660 // is_cold: Return true if the node should never be inlined.
a61af66fc99e Initial load
duke
parents:
diff changeset
661 // This is true if any of the key metrics are extreme.
a61af66fc99e Initial load
duke
parents:
diff changeset
662 bool WarmCallInfo::is_cold() const {
a61af66fc99e Initial load
duke
parents:
diff changeset
663 if (count() < WarmCallMinCount) return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
664 if (profit() < WarmCallMinProfit) return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
665 if (work() > WarmCallMaxWork) return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
666 if (size() > WarmCallMaxSize) return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
667 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
668 }
a61af66fc99e Initial load
duke
parents:
diff changeset
669
a61af66fc99e Initial load
duke
parents:
diff changeset
670 // is_hot: Return true if the node should be inlined immediately.
a61af66fc99e Initial load
duke
parents:
diff changeset
671 // This is true if any of the key metrics are extreme.
a61af66fc99e Initial load
duke
parents:
diff changeset
672 bool WarmCallInfo::is_hot() const {
a61af66fc99e Initial load
duke
parents:
diff changeset
673 assert(!is_cold(), "eliminate is_cold cases before testing is_hot");
a61af66fc99e Initial load
duke
parents:
diff changeset
674 if (count() >= HotCallCountThreshold) return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
675 if (profit() >= HotCallProfitThreshold) return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
676 if (work() <= HotCallTrivialWork) return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
677 if (size() <= HotCallTrivialSize) return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
678 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
679 }
a61af66fc99e Initial load
duke
parents:
diff changeset
680
a61af66fc99e Initial load
duke
parents:
diff changeset
681 // compute_heat:
a61af66fc99e Initial load
duke
parents:
diff changeset
682 float WarmCallInfo::compute_heat() const {
a61af66fc99e Initial load
duke
parents:
diff changeset
683 assert(!is_cold(), "compute heat only on warm nodes");
a61af66fc99e Initial load
duke
parents:
diff changeset
684 assert(!is_hot(), "compute heat only on warm nodes");
a61af66fc99e Initial load
duke
parents:
diff changeset
685 int min_size = MAX2(0, (int)HotCallTrivialSize);
a61af66fc99e Initial load
duke
parents:
diff changeset
686 int max_size = MIN2(500, (int)WarmCallMaxSize);
a61af66fc99e Initial load
duke
parents:
diff changeset
687 float method_size = (size() - min_size) / MAX2(1, max_size - min_size);
a61af66fc99e Initial load
duke
parents:
diff changeset
688 float size_factor;
a61af66fc99e Initial load
duke
parents:
diff changeset
689 if (method_size < 0.05) size_factor = 4; // 2 sigmas better than avg.
a61af66fc99e Initial load
duke
parents:
diff changeset
690 else if (method_size < 0.15) size_factor = 2; // 1 sigma better than avg.
a61af66fc99e Initial load
duke
parents:
diff changeset
691 else if (method_size < 0.5) size_factor = 1; // better than avg.
a61af66fc99e Initial load
duke
parents:
diff changeset
692 else size_factor = 0.5; // worse than avg.
a61af66fc99e Initial load
duke
parents:
diff changeset
693 return (count() * profit() * size_factor);
a61af66fc99e Initial load
duke
parents:
diff changeset
694 }
a61af66fc99e Initial load
duke
parents:
diff changeset
695
a61af66fc99e Initial load
duke
parents:
diff changeset
696 bool WarmCallInfo::warmer_than(WarmCallInfo* that) {
a61af66fc99e Initial load
duke
parents:
diff changeset
697 assert(this != that, "compare only different WCIs");
a61af66fc99e Initial load
duke
parents:
diff changeset
698 assert(this->heat() != 0 && that->heat() != 0, "call compute_heat 1st");
a61af66fc99e Initial load
duke
parents:
diff changeset
699 if (this->heat() > that->heat()) return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
700 if (this->heat() < that->heat()) return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
701 assert(this->heat() == that->heat(), "no NaN heat allowed");
a61af66fc99e Initial load
duke
parents:
diff changeset
702 // Equal heat. Break the tie some other way.
a61af66fc99e Initial load
duke
parents:
diff changeset
703 if (!this->call() || !that->call()) return (address)this > (address)that;
a61af66fc99e Initial load
duke
parents:
diff changeset
704 return this->call()->_idx > that->call()->_idx;
a61af66fc99e Initial load
duke
parents:
diff changeset
705 }
a61af66fc99e Initial load
duke
parents:
diff changeset
706
a61af66fc99e Initial load
duke
parents:
diff changeset
707 //#define UNINIT_NEXT ((WarmCallInfo*)badAddress)
a61af66fc99e Initial load
duke
parents:
diff changeset
708 #define UNINIT_NEXT ((WarmCallInfo*)NULL)
a61af66fc99e Initial load
duke
parents:
diff changeset
709
a61af66fc99e Initial load
duke
parents:
diff changeset
710 WarmCallInfo* WarmCallInfo::insert_into(WarmCallInfo* head) {
a61af66fc99e Initial load
duke
parents:
diff changeset
711 assert(next() == UNINIT_NEXT, "not yet on any list");
a61af66fc99e Initial load
duke
parents:
diff changeset
712 WarmCallInfo* prev_p = NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
713 WarmCallInfo* next_p = head;
a61af66fc99e Initial load
duke
parents:
diff changeset
714 while (next_p != NULL && next_p->warmer_than(this)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
715 prev_p = next_p;
a61af66fc99e Initial load
duke
parents:
diff changeset
716 next_p = prev_p->next();
a61af66fc99e Initial load
duke
parents:
diff changeset
717 }
a61af66fc99e Initial load
duke
parents:
diff changeset
718 // Install this between prev_p and next_p.
a61af66fc99e Initial load
duke
parents:
diff changeset
719 this->set_next(next_p);
a61af66fc99e Initial load
duke
parents:
diff changeset
720 if (prev_p == NULL)
a61af66fc99e Initial load
duke
parents:
diff changeset
721 head = this;
a61af66fc99e Initial load
duke
parents:
diff changeset
722 else
a61af66fc99e Initial load
duke
parents:
diff changeset
723 prev_p->set_next(this);
a61af66fc99e Initial load
duke
parents:
diff changeset
724 return head;
a61af66fc99e Initial load
duke
parents:
diff changeset
725 }
a61af66fc99e Initial load
duke
parents:
diff changeset
726
a61af66fc99e Initial load
duke
parents:
diff changeset
727 WarmCallInfo* WarmCallInfo::remove_from(WarmCallInfo* head) {
a61af66fc99e Initial load
duke
parents:
diff changeset
728 WarmCallInfo* prev_p = NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
729 WarmCallInfo* next_p = head;
a61af66fc99e Initial load
duke
parents:
diff changeset
730 while (next_p != this) {
a61af66fc99e Initial load
duke
parents:
diff changeset
731 assert(next_p != NULL, "this must be in the list somewhere");
a61af66fc99e Initial load
duke
parents:
diff changeset
732 prev_p = next_p;
a61af66fc99e Initial load
duke
parents:
diff changeset
733 next_p = prev_p->next();
a61af66fc99e Initial load
duke
parents:
diff changeset
734 }
a61af66fc99e Initial load
duke
parents:
diff changeset
735 next_p = this->next();
a61af66fc99e Initial load
duke
parents:
diff changeset
736 debug_only(this->set_next(UNINIT_NEXT));
a61af66fc99e Initial load
duke
parents:
diff changeset
737 // Remove this from between prev_p and next_p.
a61af66fc99e Initial load
duke
parents:
diff changeset
738 if (prev_p == NULL)
a61af66fc99e Initial load
duke
parents:
diff changeset
739 head = next_p;
a61af66fc99e Initial load
duke
parents:
diff changeset
740 else
a61af66fc99e Initial load
duke
parents:
diff changeset
741 prev_p->set_next(next_p);
a61af66fc99e Initial load
duke
parents:
diff changeset
742 return head;
a61af66fc99e Initial load
duke
parents:
diff changeset
743 }
a61af66fc99e Initial load
duke
parents:
diff changeset
744
a61af66fc99e Initial load
duke
parents:
diff changeset
745 WarmCallInfo* WarmCallInfo::_always_hot = NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
746 WarmCallInfo* WarmCallInfo::_always_cold = NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
747
a61af66fc99e Initial load
duke
parents:
diff changeset
748 WarmCallInfo* WarmCallInfo::always_hot() {
a61af66fc99e Initial load
duke
parents:
diff changeset
749 if (_always_hot == NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
750 static double bits[sizeof(WarmCallInfo) / sizeof(double) + 1] = {0};
a61af66fc99e Initial load
duke
parents:
diff changeset
751 WarmCallInfo* ci = (WarmCallInfo*) bits;
a61af66fc99e Initial load
duke
parents:
diff changeset
752 ci->_profit = ci->_count = MAX_VALUE();
a61af66fc99e Initial load
duke
parents:
diff changeset
753 ci->_work = ci->_size = MIN_VALUE();
a61af66fc99e Initial load
duke
parents:
diff changeset
754 _always_hot = ci;
a61af66fc99e Initial load
duke
parents:
diff changeset
755 }
a61af66fc99e Initial load
duke
parents:
diff changeset
756 assert(_always_hot->is_hot(), "must always be hot");
a61af66fc99e Initial load
duke
parents:
diff changeset
757 return _always_hot;
a61af66fc99e Initial load
duke
parents:
diff changeset
758 }
a61af66fc99e Initial load
duke
parents:
diff changeset
759
a61af66fc99e Initial load
duke
parents:
diff changeset
760 WarmCallInfo* WarmCallInfo::always_cold() {
a61af66fc99e Initial load
duke
parents:
diff changeset
761 if (_always_cold == NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
762 static double bits[sizeof(WarmCallInfo) / sizeof(double) + 1] = {0};
a61af66fc99e Initial load
duke
parents:
diff changeset
763 WarmCallInfo* ci = (WarmCallInfo*) bits;
a61af66fc99e Initial load
duke
parents:
diff changeset
764 ci->_profit = ci->_count = MIN_VALUE();
a61af66fc99e Initial load
duke
parents:
diff changeset
765 ci->_work = ci->_size = MAX_VALUE();
a61af66fc99e Initial load
duke
parents:
diff changeset
766 _always_cold = ci;
a61af66fc99e Initial load
duke
parents:
diff changeset
767 }
a61af66fc99e Initial load
duke
parents:
diff changeset
768 assert(_always_cold->is_cold(), "must always be cold");
a61af66fc99e Initial load
duke
parents:
diff changeset
769 return _always_cold;
a61af66fc99e Initial load
duke
parents:
diff changeset
770 }
a61af66fc99e Initial load
duke
parents:
diff changeset
771
a61af66fc99e Initial load
duke
parents:
diff changeset
772
a61af66fc99e Initial load
duke
parents:
diff changeset
773 #ifndef PRODUCT
a61af66fc99e Initial load
duke
parents:
diff changeset
774
a61af66fc99e Initial load
duke
parents:
diff changeset
775 void WarmCallInfo::print() const {
a61af66fc99e Initial load
duke
parents:
diff changeset
776 tty->print("%s : C=%6.1f P=%6.1f W=%6.1f S=%6.1f H=%6.1f -> %p",
a61af66fc99e Initial load
duke
parents:
diff changeset
777 is_cold() ? "cold" : is_hot() ? "hot " : "warm",
a61af66fc99e Initial load
duke
parents:
diff changeset
778 count(), profit(), work(), size(), compute_heat(), next());
a61af66fc99e Initial load
duke
parents:
diff changeset
779 tty->cr();
a61af66fc99e Initial load
duke
parents:
diff changeset
780 if (call() != NULL) call()->dump();
a61af66fc99e Initial load
duke
parents:
diff changeset
781 }
a61af66fc99e Initial load
duke
parents:
diff changeset
782
a61af66fc99e Initial load
duke
parents:
diff changeset
783 void print_wci(WarmCallInfo* ci) {
a61af66fc99e Initial load
duke
parents:
diff changeset
784 ci->print();
a61af66fc99e Initial load
duke
parents:
diff changeset
785 }
a61af66fc99e Initial load
duke
parents:
diff changeset
786
a61af66fc99e Initial load
duke
parents:
diff changeset
787 void WarmCallInfo::print_all() const {
a61af66fc99e Initial load
duke
parents:
diff changeset
788 for (const WarmCallInfo* p = this; p != NULL; p = p->next())
a61af66fc99e Initial load
duke
parents:
diff changeset
789 p->print();
a61af66fc99e Initial load
duke
parents:
diff changeset
790 }
a61af66fc99e Initial load
duke
parents:
diff changeset
791
a61af66fc99e Initial load
duke
parents:
diff changeset
792 int WarmCallInfo::count_all() const {
a61af66fc99e Initial load
duke
parents:
diff changeset
793 int cnt = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
794 for (const WarmCallInfo* p = this; p != NULL; p = p->next())
a61af66fc99e Initial load
duke
parents:
diff changeset
795 cnt++;
a61af66fc99e Initial load
duke
parents:
diff changeset
796 return cnt;
a61af66fc99e Initial load
duke
parents:
diff changeset
797 }
a61af66fc99e Initial load
duke
parents:
diff changeset
798
a61af66fc99e Initial load
duke
parents:
diff changeset
799 #endif //PRODUCT