Mon, 25 May 2020 14:24:27 +0800
8244407: JVM crashes after transformation in C2 IdealLoopTree::split_fall_in
Reviewed-by: thartmann, kvn, andrew
Contributed-by: zhouyong44@huawei.com
duke@435 | 1 | /* |
coleenp@8739 | 2 | * Copyright (c) 1998, 2016, Oracle and/or its affiliates. All rights reserved. |
duke@435 | 3 | * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
duke@435 | 4 | * |
duke@435 | 5 | * This code is free software; you can redistribute it and/or modify it |
duke@435 | 6 | * under the terms of the GNU General Public License version 2 only, as |
duke@435 | 7 | * published by the Free Software Foundation. |
duke@435 | 8 | * |
duke@435 | 9 | * This code is distributed in the hope that it will be useful, but WITHOUT |
duke@435 | 10 | * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
duke@435 | 11 | * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
duke@435 | 12 | * version 2 for more details (a copy is included in the LICENSE file that |
duke@435 | 13 | * accompanied this code). |
duke@435 | 14 | * |
duke@435 | 15 | * You should have received a copy of the GNU General Public License version |
duke@435 | 16 | * 2 along with this work; if not, write to the Free Software Foundation, |
duke@435 | 17 | * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
duke@435 | 18 | * |
trims@1907 | 19 | * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
trims@1907 | 20 | * or visit www.oracle.com if you need additional information or have any |
trims@1907 | 21 | * questions. |
duke@435 | 22 | * |
duke@435 | 23 | */ |
duke@435 | 24 | |
stefank@2314 | 25 | #include "precompiled.hpp" |
stefank@2314 | 26 | #include "ci/ciCallSite.hpp" |
stefank@2314 | 27 | #include "ci/ciMethodHandle.hpp" |
stefank@2314 | 28 | #include "classfile/vmSymbols.hpp" |
twisti@2687 | 29 | #include "compiler/compileBroker.hpp" |
stefank@2314 | 30 | #include "compiler/compileLog.hpp" |
stefank@2314 | 31 | #include "interpreter/linkResolver.hpp" |
stefank@2314 | 32 | #include "opto/addnode.hpp" |
stefank@2314 | 33 | #include "opto/callGenerator.hpp" |
stefank@2314 | 34 | #include "opto/cfgnode.hpp" |
stefank@2314 | 35 | #include "opto/mulnode.hpp" |
stefank@2314 | 36 | #include "opto/parse.hpp" |
stefank@2314 | 37 | #include "opto/rootnode.hpp" |
stefank@2314 | 38 | #include "opto/runtime.hpp" |
stefank@2314 | 39 | #include "opto/subnode.hpp" |
stefank@2314 | 40 | #include "prims/nativeLookup.hpp" |
stefank@2314 | 41 | #include "runtime/sharedRuntime.hpp" |
duke@435 | 42 | |
roland@4357 | 43 | void trace_type_profile(Compile* C, ciMethod *method, int depth, int bci, ciMethod *prof_method, ciKlass *prof_klass, int site_count, int receiver_count) { |
kvn@5763 | 44 | if (TraceTypeProfile || C->print_inlining()) { |
roland@4357 | 45 | outputStream* out = tty; |
kvn@5763 | 46 | if (!C->print_inlining()) { |
twisti@4104 | 47 | if (NOT_PRODUCT(!PrintOpto &&) !PrintCompilation) { |
twisti@2687 | 48 | method->print_short_name(); |
twisti@2687 | 49 | tty->cr(); |
twisti@2687 | 50 | } |
twisti@2687 | 51 | CompileTask::print_inlining(prof_method, depth, bci); |
roland@4357 | 52 | } else { |
roland@4357 | 53 | out = C->print_inlining_stream(); |
duke@435 | 54 | } |
roland@4357 | 55 | CompileTask::print_inline_indent(depth, out); |
roland@4357 | 56 | out->print(" \\-> TypeProfile (%d/%d counts) = ", receiver_count, site_count); |
roland@4357 | 57 | stringStream ss; |
roland@4357 | 58 | prof_klass->name()->print_symbol_on(&ss); |
drchase@6680 | 59 | out->print("%s", ss.as_string()); |
roland@4357 | 60 | out->cr(); |
duke@435 | 61 | } |
duke@435 | 62 | } |
duke@435 | 63 | |
twisti@4414 | 64 | CallGenerator* Compile::call_generator(ciMethod* callee, int vtable_index, bool call_does_dispatch, |
jrose@1592 | 65 | JVMState* jvms, bool allow_inline, |
roland@5991 | 66 | float prof_factor, ciKlass* speculative_receiver_type, |
roland@5991 | 67 | bool allow_intrinsics, bool delayed_forbidden) { |
twisti@2903 | 68 | ciMethod* caller = jvms->method(); |
twisti@2903 | 69 | int bci = jvms->bci(); |
twisti@2903 | 70 | Bytecodes::Code bytecode = caller->java_code_at_bci(bci); |
twisti@3969 | 71 | guarantee(callee != NULL, "failed method resolution"); |
duke@435 | 72 | |
duke@435 | 73 | // Dtrace currently doesn't work unless all calls are vanilla |
kvn@1215 | 74 | if (env()->dtrace_method_probes()) { |
duke@435 | 75 | allow_inline = false; |
duke@435 | 76 | } |
duke@435 | 77 | |
duke@435 | 78 | // Note: When we get profiling during stage-1 compiles, we want to pull |
duke@435 | 79 | // from more specific profile data which pertains to this inlining. |
duke@435 | 80 | // Right now, ignore the information in jvms->caller(), and do method[bci]. |
twisti@2903 | 81 | ciCallProfile profile = caller->call_profile_at_bci(bci); |
duke@435 | 82 | |
duke@435 | 83 | // See how many times this site has been invoked. |
duke@435 | 84 | int site_count = profile.count(); |
duke@435 | 85 | int receiver_count = -1; |
twisti@4414 | 86 | if (call_does_dispatch && UseTypeProfile && profile.has_receiver(0)) { |
duke@435 | 87 | // Receivers in the profile structure are ordered by call counts |
duke@435 | 88 | // so that the most called (major) receiver is profile.receiver(0). |
duke@435 | 89 | receiver_count = profile.receiver_count(0); |
duke@435 | 90 | } |
duke@435 | 91 | |
duke@435 | 92 | CompileLog* log = this->log(); |
duke@435 | 93 | if (log != NULL) { |
duke@435 | 94 | int rid = (receiver_count >= 0)? log->identify(profile.receiver(0)): -1; |
kvn@1686 | 95 | int r2id = (rid != -1 && profile.has_receiver(1))? log->identify(profile.receiver(1)):-1; |
duke@435 | 96 | log->begin_elem("call method='%d' count='%d' prof_factor='%g'", |
twisti@3969 | 97 | log->identify(callee), site_count, prof_factor); |
twisti@4414 | 98 | if (call_does_dispatch) log->print(" virtual='1'"); |
duke@435 | 99 | if (allow_inline) log->print(" inline='1'"); |
duke@435 | 100 | if (receiver_count >= 0) { |
duke@435 | 101 | log->print(" receiver='%d' receiver_count='%d'", rid, receiver_count); |
duke@435 | 102 | if (profile.has_receiver(1)) { |
duke@435 | 103 | log->print(" receiver2='%d' receiver2_count='%d'", r2id, profile.receiver_count(1)); |
duke@435 | 104 | } |
duke@435 | 105 | } |
duke@435 | 106 | log->end_elem(); |
duke@435 | 107 | } |
duke@435 | 108 | |
duke@435 | 109 | // Special case the handling of certain common, profitable library |
duke@435 | 110 | // methods. If these methods are replaced with specialized code, |
duke@435 | 111 | // then we return it as the inlined version of the call. |
duke@435 | 112 | // We do this before the strict f.p. check below because the |
duke@435 | 113 | // intrinsics handle strict f.p. correctly. |
shade@5798 | 114 | CallGenerator* cg_intrinsic = NULL; |
never@3631 | 115 | if (allow_inline && allow_intrinsics) { |
twisti@4414 | 116 | CallGenerator* cg = find_intrinsic(callee, call_does_dispatch); |
kvn@4205 | 117 | if (cg != NULL) { |
kvn@7026 | 118 | if (cg->is_predicated()) { |
kvn@4205 | 119 | // Code without intrinsic but, hopefully, inlined. |
kvn@4205 | 120 | CallGenerator* inline_cg = this->call_generator(callee, |
roland@5991 | 121 | vtable_index, call_does_dispatch, jvms, allow_inline, prof_factor, speculative_receiver_type, false); |
kvn@4205 | 122 | if (inline_cg != NULL) { |
kvn@7026 | 123 | cg = CallGenerator::for_predicated_intrinsic(cg, inline_cg); |
kvn@4205 | 124 | } |
kvn@4205 | 125 | } |
shade@5798 | 126 | |
shade@5798 | 127 | // If intrinsic does the virtual dispatch, we try to use the type profile |
shade@5798 | 128 | // first, and hopefully inline it as the regular virtual call below. |
shade@5798 | 129 | // We will retry the intrinsic if nothing had claimed it afterwards. |
shade@5798 | 130 | if (cg->does_virtual_dispatch()) { |
shade@5798 | 131 | cg_intrinsic = cg; |
shade@5798 | 132 | cg = NULL; |
shade@5798 | 133 | } else { |
shade@5798 | 134 | return cg; |
shade@5798 | 135 | } |
kvn@4205 | 136 | } |
duke@435 | 137 | } |
duke@435 | 138 | |
twisti@3050 | 139 | // Do method handle calls. |
twisti@2178 | 140 | // NOTE: This must happen before normal inlining logic below since |
twisti@2178 | 141 | // MethodHandle.invoke* are native methods which obviously don't |
twisti@2178 | 142 | // have bytecodes and so normal inlining fails. |
twisti@3969 | 143 | if (callee->is_method_handle_intrinsic()) { |
roland@4409 | 144 | CallGenerator* cg = CallGenerator::for_method_handle_call(jvms, caller, callee, delayed_forbidden); |
twisti@4414 | 145 | assert(cg == NULL || !delayed_forbidden || !cg->is_late_inline() || cg->is_mh_late_inline(), "unexpected CallGenerator"); |
roland@4409 | 146 | return cg; |
twisti@2178 | 147 | } |
twisti@2178 | 148 | |
duke@435 | 149 | // Do not inline strict fp into non-strict code, or the reverse |
twisti@3969 | 150 | if (caller->is_strict() ^ callee->is_strict()) { |
duke@435 | 151 | allow_inline = false; |
duke@435 | 152 | } |
duke@435 | 153 | |
duke@435 | 154 | // Attempt to inline... |
duke@435 | 155 | if (allow_inline) { |
duke@435 | 156 | // The profile data is only partly attributable to this caller, |
duke@435 | 157 | // scale back the call site information. |
duke@435 | 158 | float past_uses = jvms->method()->scale_count(site_count, prof_factor); |
duke@435 | 159 | // This is the number of times we expect the call code to be used. |
duke@435 | 160 | float expected_uses = past_uses; |
duke@435 | 161 | |
duke@435 | 162 | // Try inlining a bytecoded method: |
twisti@4414 | 163 | if (!call_does_dispatch) { |
shade@6314 | 164 | InlineTree* ilt = InlineTree::find_subtree_from_root(this->ilt(), jvms->caller(), jvms->method()); |
duke@435 | 165 | WarmCallInfo scratch_ci; |
roland@4409 | 166 | bool should_delay = false; |
roland@4409 | 167 | WarmCallInfo* ci = ilt->ok_to_inline(callee, jvms, profile, &scratch_ci, should_delay); |
duke@435 | 168 | assert(ci != &scratch_ci, "do not let this pointer escape"); |
duke@435 | 169 | bool allow_inline = (ci != NULL && !ci->is_cold()); |
duke@435 | 170 | bool require_inline = (allow_inline && ci->is_hot()); |
duke@435 | 171 | |
duke@435 | 172 | if (allow_inline) { |
twisti@3969 | 173 | CallGenerator* cg = CallGenerator::for_inline(callee, expected_uses); |
roland@4409 | 174 | |
roland@4409 | 175 | if (require_inline && cg != NULL) { |
never@1515 | 176 | // Delay the inlining of this method to give us the |
never@1515 | 177 | // opportunity to perform some high level optimizations |
never@1515 | 178 | // first. |
kvn@5110 | 179 | if (should_delay_string_inlining(callee, jvms)) { |
roland@4409 | 180 | assert(!delayed_forbidden, "strange"); |
roland@4409 | 181 | return CallGenerator::for_string_late_inline(callee, cg); |
kvn@5110 | 182 | } else if (should_delay_boxing_inlining(callee, jvms)) { |
kvn@5110 | 183 | assert(!delayed_forbidden, "strange"); |
kvn@5110 | 184 | return CallGenerator::for_boxing_late_inline(callee, cg); |
roland@4409 | 185 | } else if ((should_delay || AlwaysIncrementalInline) && !delayed_forbidden) { |
roland@4409 | 186 | return CallGenerator::for_late_inline(callee, cg); |
roland@4409 | 187 | } |
never@1515 | 188 | } |
roland@4409 | 189 | if (cg == NULL || should_delay) { |
duke@435 | 190 | // Fall through. |
duke@435 | 191 | } else if (require_inline || !InlineWarmCalls) { |
duke@435 | 192 | return cg; |
duke@435 | 193 | } else { |
twisti@4414 | 194 | CallGenerator* cold_cg = call_generator(callee, vtable_index, call_does_dispatch, jvms, false, prof_factor); |
duke@435 | 195 | return CallGenerator::for_warm_call(ci, cold_cg, cg); |
duke@435 | 196 | } |
duke@435 | 197 | } |
duke@435 | 198 | } |
duke@435 | 199 | |
duke@435 | 200 | // Try using the type profile. |
twisti@4414 | 201 | if (call_does_dispatch && site_count > 0 && receiver_count > 0) { |
duke@435 | 202 | // The major receiver's count >= TypeProfileMajorReceiverPercent of site_count. |
duke@435 | 203 | bool have_major_receiver = (100.*profile.receiver_prob(0) >= (float)TypeProfileMajorReceiverPercent); |
duke@435 | 204 | ciMethod* receiver_method = NULL; |
roland@5991 | 205 | |
roland@5991 | 206 | int morphism = profile.morphism(); |
roland@5991 | 207 | if (speculative_receiver_type != NULL) { |
roland@8610 | 208 | if (!too_many_traps(caller, bci, Deoptimization::Reason_speculate_class_check)) { |
roland@8610 | 209 | // We have a speculative type, we should be able to resolve |
roland@8610 | 210 | // the call. We do that before looking at the profiling at |
roland@8610 | 211 | // this invoke because it may lead to bimorphic inlining which |
roland@8610 | 212 | // a speculative type should help us avoid. |
roland@8610 | 213 | receiver_method = callee->resolve_invoke(jvms->method()->holder(), |
roland@8610 | 214 | speculative_receiver_type); |
roland@8610 | 215 | if (receiver_method == NULL) { |
roland@8610 | 216 | speculative_receiver_type = NULL; |
roland@8610 | 217 | } else { |
roland@8610 | 218 | morphism = 1; |
roland@8610 | 219 | } |
roland@8610 | 220 | } else { |
roland@8610 | 221 | // speculation failed before. Use profiling at the call |
roland@8610 | 222 | // (could allow bimorphic inlining for instance). |
roland@5991 | 223 | speculative_receiver_type = NULL; |
roland@5991 | 224 | } |
roland@5991 | 225 | } |
roland@5991 | 226 | if (receiver_method == NULL && |
roland@5991 | 227 | (have_major_receiver || morphism == 1 || |
roland@5991 | 228 | (morphism == 2 && UseBimorphicInlining))) { |
duke@435 | 229 | // receiver_method = profile.method(); |
duke@435 | 230 | // Profiles do not suggest methods now. Look it up in the major receiver. |
twisti@3969 | 231 | receiver_method = callee->resolve_invoke(jvms->method()->holder(), |
duke@435 | 232 | profile.receiver(0)); |
duke@435 | 233 | } |
duke@435 | 234 | if (receiver_method != NULL) { |
duke@435 | 235 | // The single majority receiver sufficiently outweighs the minority. |
duke@435 | 236 | CallGenerator* hit_cg = this->call_generator(receiver_method, |
twisti@4414 | 237 | vtable_index, !call_does_dispatch, jvms, allow_inline, prof_factor); |
duke@435 | 238 | if (hit_cg != NULL) { |
duke@435 | 239 | // Look up second receiver. |
duke@435 | 240 | CallGenerator* next_hit_cg = NULL; |
duke@435 | 241 | ciMethod* next_receiver_method = NULL; |
roland@5991 | 242 | if (morphism == 2 && UseBimorphicInlining) { |
twisti@3969 | 243 | next_receiver_method = callee->resolve_invoke(jvms->method()->holder(), |
duke@435 | 244 | profile.receiver(1)); |
duke@435 | 245 | if (next_receiver_method != NULL) { |
duke@435 | 246 | next_hit_cg = this->call_generator(next_receiver_method, |
twisti@4414 | 247 | vtable_index, !call_does_dispatch, jvms, |
duke@435 | 248 | allow_inline, prof_factor); |
duke@435 | 249 | if (next_hit_cg != NULL && !next_hit_cg->is_inline() && |
duke@435 | 250 | have_major_receiver && UseOnlyInlinedBimorphic) { |
duke@435 | 251 | // Skip if we can't inline second receiver's method |
duke@435 | 252 | next_hit_cg = NULL; |
duke@435 | 253 | } |
duke@435 | 254 | } |
duke@435 | 255 | } |
duke@435 | 256 | CallGenerator* miss_cg; |
roland@5991 | 257 | Deoptimization::DeoptReason reason = morphism == 2 ? |
kvn@1641 | 258 | Deoptimization::Reason_bimorphic : |
roland@6377 | 259 | (speculative_receiver_type == NULL ? Deoptimization::Reason_class_check : Deoptimization::Reason_speculate_class_check); |
roland@5991 | 260 | if ((morphism == 1 || (morphism == 2 && next_hit_cg != NULL)) && |
roland@8610 | 261 | !too_many_traps(caller, bci, reason) |
duke@435 | 262 | ) { |
duke@435 | 263 | // Generate uncommon trap for class check failure path |
duke@435 | 264 | // in case of monomorphic or bimorphic virtual call site. |
twisti@3969 | 265 | miss_cg = CallGenerator::for_uncommon_trap(callee, reason, |
duke@435 | 266 | Deoptimization::Action_maybe_recompile); |
duke@435 | 267 | } else { |
duke@435 | 268 | // Generate virtual call for class check failure path |
duke@435 | 269 | // in case of polymorphic virtual call site. |
twisti@3969 | 270 | miss_cg = CallGenerator::for_virtual_call(callee, vtable_index); |
duke@435 | 271 | } |
duke@435 | 272 | if (miss_cg != NULL) { |
duke@435 | 273 | if (next_hit_cg != NULL) { |
roland@5991 | 274 | assert(speculative_receiver_type == NULL, "shouldn't end up here if we used speculation"); |
roland@4357 | 275 | trace_type_profile(C, jvms->method(), jvms->depth() - 1, jvms->bci(), next_receiver_method, profile.receiver(1), site_count, profile.receiver_count(1)); |
duke@435 | 276 | // We don't need to record dependency on a receiver here and below. |
duke@435 | 277 | // Whenever we inline, the dependency is added by Parse::Parse(). |
duke@435 | 278 | miss_cg = CallGenerator::for_predicted_call(profile.receiver(1), miss_cg, next_hit_cg, PROB_MAX); |
duke@435 | 279 | } |
duke@435 | 280 | if (miss_cg != NULL) { |
roland@4357 | 281 | trace_type_profile(C, jvms->method(), jvms->depth() - 1, jvms->bci(), receiver_method, profile.receiver(0), site_count, receiver_count); |
roland@5991 | 282 | ciKlass* k = speculative_receiver_type != NULL ? speculative_receiver_type : profile.receiver(0); |
roland@5991 | 283 | float hit_prob = speculative_receiver_type != NULL ? 1.0 : profile.receiver_prob(0); |
roland@5991 | 284 | CallGenerator* cg = CallGenerator::for_predicted_call(k, miss_cg, hit_cg, hit_prob); |
duke@435 | 285 | if (cg != NULL) return cg; |
duke@435 | 286 | } |
duke@435 | 287 | } |
duke@435 | 288 | } |
duke@435 | 289 | } |
duke@435 | 290 | } |
duke@435 | 291 | } |
duke@435 | 292 | |
shade@5798 | 293 | // Nothing claimed the intrinsic, we go with straight-forward inlining |
shade@5798 | 294 | // for already discovered intrinsic. |
shade@5798 | 295 | if (allow_inline && allow_intrinsics && cg_intrinsic != NULL) { |
shade@5798 | 296 | assert(cg_intrinsic->does_virtual_dispatch(), "sanity"); |
shade@5798 | 297 | return cg_intrinsic; |
shade@5798 | 298 | } |
shade@5798 | 299 | |
duke@435 | 300 | // There was no special inlining tactic, or it bailed out. |
duke@435 | 301 | // Use a more generic tactic, like a simple call. |
twisti@4414 | 302 | if (call_does_dispatch) { |
twisti@3969 | 303 | return CallGenerator::for_virtual_call(callee, vtable_index); |
duke@435 | 304 | } else { |
duke@435 | 305 | // Class Hierarchy Analysis or Type Profile reveals a unique target, |
duke@435 | 306 | // or it is a static or special call. |
twisti@3969 | 307 | return CallGenerator::for_direct_call(callee, should_delay_inlining(callee, jvms)); |
duke@435 | 308 | } |
duke@435 | 309 | } |
duke@435 | 310 | |
never@1515 | 311 | // Return true for methods that shouldn't be inlined early so that |
never@1515 | 312 | // they are easier to analyze and optimize as intrinsics. |
kvn@5110 | 313 | bool Compile::should_delay_string_inlining(ciMethod* call_method, JVMState* jvms) { |
never@1515 | 314 | if (has_stringbuilder()) { |
never@1515 | 315 | |
never@1515 | 316 | if ((call_method->holder() == C->env()->StringBuilder_klass() || |
never@1515 | 317 | call_method->holder() == C->env()->StringBuffer_klass()) && |
never@1515 | 318 | (jvms->method()->holder() == C->env()->StringBuilder_klass() || |
never@1515 | 319 | jvms->method()->holder() == C->env()->StringBuffer_klass())) { |
never@1515 | 320 | // Delay SB calls only when called from non-SB code |
never@1515 | 321 | return false; |
never@1515 | 322 | } |
never@1515 | 323 | |
never@1515 | 324 | switch (call_method->intrinsic_id()) { |
never@1515 | 325 | case vmIntrinsics::_StringBuilder_void: |
never@1515 | 326 | case vmIntrinsics::_StringBuilder_int: |
never@1515 | 327 | case vmIntrinsics::_StringBuilder_String: |
never@1515 | 328 | case vmIntrinsics::_StringBuilder_append_char: |
never@1515 | 329 | case vmIntrinsics::_StringBuilder_append_int: |
never@1515 | 330 | case vmIntrinsics::_StringBuilder_append_String: |
never@1515 | 331 | case vmIntrinsics::_StringBuilder_toString: |
never@1515 | 332 | case vmIntrinsics::_StringBuffer_void: |
never@1515 | 333 | case vmIntrinsics::_StringBuffer_int: |
never@1515 | 334 | case vmIntrinsics::_StringBuffer_String: |
never@1515 | 335 | case vmIntrinsics::_StringBuffer_append_char: |
never@1515 | 336 | case vmIntrinsics::_StringBuffer_append_int: |
never@1515 | 337 | case vmIntrinsics::_StringBuffer_append_String: |
never@1515 | 338 | case vmIntrinsics::_StringBuffer_toString: |
never@1515 | 339 | case vmIntrinsics::_Integer_toString: |
never@1515 | 340 | return true; |
never@1515 | 341 | |
never@1515 | 342 | case vmIntrinsics::_String_String: |
never@1515 | 343 | { |
never@1515 | 344 | Node* receiver = jvms->map()->in(jvms->argoff() + 1); |
never@1515 | 345 | if (receiver->is_Proj() && receiver->in(0)->is_CallStaticJava()) { |
never@1515 | 346 | CallStaticJavaNode* csj = receiver->in(0)->as_CallStaticJava(); |
never@1515 | 347 | ciMethod* m = csj->method(); |
never@1515 | 348 | if (m != NULL && |
never@1515 | 349 | (m->intrinsic_id() == vmIntrinsics::_StringBuffer_toString || |
never@1515 | 350 | m->intrinsic_id() == vmIntrinsics::_StringBuilder_toString)) |
never@1515 | 351 | // Delay String.<init>(new SB()) |
never@1515 | 352 | return true; |
never@1515 | 353 | } |
never@1515 | 354 | return false; |
never@1515 | 355 | } |
never@1515 | 356 | |
never@1515 | 357 | default: |
never@1515 | 358 | return false; |
never@1515 | 359 | } |
never@1515 | 360 | } |
never@1515 | 361 | return false; |
never@1515 | 362 | } |
never@1515 | 363 | |
kvn@5110 | 364 | bool Compile::should_delay_boxing_inlining(ciMethod* call_method, JVMState* jvms) { |
kvn@5110 | 365 | if (eliminate_boxing() && call_method->is_boxing_method()) { |
kvn@5110 | 366 | set_has_boxed_value(true); |
kvn@6679 | 367 | return aggressive_unboxing(); |
kvn@5110 | 368 | } |
kvn@5110 | 369 | return false; |
kvn@5110 | 370 | } |
duke@435 | 371 | |
duke@435 | 372 | // uncommon-trap call-sites where callee is unloaded, uninitialized or will not link |
duke@435 | 373 | bool Parse::can_not_compile_call_site(ciMethod *dest_method, ciInstanceKlass* klass) { |
duke@435 | 374 | // Additional inputs to consider... |
duke@435 | 375 | // bc = bc() |
duke@435 | 376 | // caller = method() |
duke@435 | 377 | // iter().get_method_holder_index() |
duke@435 | 378 | assert( dest_method->is_loaded(), "ciTypeFlow should not let us get here" ); |
duke@435 | 379 | // Interface classes can be loaded & linked and never get around to |
duke@435 | 380 | // being initialized. Uncommon-trap for not-initialized static or |
duke@435 | 381 | // v-calls. Let interface calls happen. |
twisti@1572 | 382 | ciInstanceKlass* holder_klass = dest_method->holder(); |
never@2000 | 383 | if (!holder_klass->is_being_initialized() && |
never@2000 | 384 | !holder_klass->is_initialized() && |
duke@435 | 385 | !holder_klass->is_interface()) { |
duke@435 | 386 | uncommon_trap(Deoptimization::Reason_uninitialized, |
duke@435 | 387 | Deoptimization::Action_reinterpret, |
duke@435 | 388 | holder_klass); |
duke@435 | 389 | return true; |
duke@435 | 390 | } |
duke@435 | 391 | |
bharadwaj@4268 | 392 | assert(dest_method->is_loaded(), "dest_method: typeflow responsibility"); |
duke@435 | 393 | return false; |
duke@435 | 394 | } |
duke@435 | 395 | |
duke@435 | 396 | |
duke@435 | 397 | //------------------------------do_call---------------------------------------- |
duke@435 | 398 | // Handle your basic call. Inline if we can & want to, else just setup call. |
duke@435 | 399 | void Parse::do_call() { |
duke@435 | 400 | // It's likely we are going to add debug info soon. |
duke@435 | 401 | // Also, if we inline a guy who eventually needs debug info for this JVMS, |
duke@435 | 402 | // our contribution to it is cleaned up right here. |
duke@435 | 403 | kill_dead_locals(); |
duke@435 | 404 | |
duke@435 | 405 | // Set frequently used booleans |
twisti@4021 | 406 | const bool is_virtual = bc() == Bytecodes::_invokevirtual; |
twisti@4021 | 407 | const bool is_virtual_or_interface = is_virtual || bc() == Bytecodes::_invokeinterface; |
twisti@4313 | 408 | const bool has_receiver = Bytecodes::has_receiver(bc()); |
duke@435 | 409 | |
duke@435 | 410 | // Find target being called |
duke@435 | 411 | bool will_link; |
twisti@4021 | 412 | ciSignature* declared_signature = NULL; |
twisti@4021 | 413 | ciMethod* orig_callee = iter().get_method(will_link, &declared_signature); // callee in the bytecode |
twisti@4021 | 414 | ciInstanceKlass* holder_klass = orig_callee->holder(); |
twisti@4021 | 415 | ciKlass* holder = iter().get_declared_method_holder(); |
duke@435 | 416 | ciInstanceKlass* klass = ciEnv::get_instance_klass_for_declared_method_holder(holder); |
twisti@4021 | 417 | assert(declared_signature != NULL, "cannot be null"); |
duke@435 | 418 | |
vlivanov@7385 | 419 | // Bump max node limit for JSR292 users |
vlivanov@7385 | 420 | if (bc() == Bytecodes::_invokedynamic || orig_callee->is_method_handle_intrinsic()) { |
vlivanov@7385 | 421 | C->set_max_node_limit(3*MaxNodeLimit); |
vlivanov@7385 | 422 | } |
vlivanov@7385 | 423 | |
duke@435 | 424 | // uncommon-trap when callee is unloaded, uninitialized or will not link |
duke@435 | 425 | // bailout when too many arguments for register representation |
twisti@4021 | 426 | if (!will_link || can_not_compile_call_site(orig_callee, klass)) { |
duke@435 | 427 | #ifndef PRODUCT |
duke@435 | 428 | if (PrintOpto && (Verbose || WizardMode)) { |
duke@435 | 429 | method()->print_name(); tty->print_cr(" can not compile call at bci %d to:", bci()); |
twisti@4021 | 430 | orig_callee->print_name(); tty->cr(); |
duke@435 | 431 | } |
duke@435 | 432 | #endif |
duke@435 | 433 | return; |
duke@435 | 434 | } |
duke@435 | 435 | assert(holder_klass->is_loaded(), ""); |
twisti@3969 | 436 | //assert((bc_callee->is_static() || is_invokedynamic) == !has_receiver , "must match bc"); // XXX invokehandle (cur_bc_raw) |
duke@435 | 437 | // Note: this takes into account invokeinterface of methods declared in java/lang/Object, |
duke@435 | 438 | // which should be invokevirtuals but according to the VM spec may be invokeinterfaces |
duke@435 | 439 | assert(holder_klass->is_interface() || holder_klass->super() == NULL || (bc() != Bytecodes::_invokeinterface), "must match bc"); |
duke@435 | 440 | // Note: In the absence of miranda methods, an abstract class K can perform |
duke@435 | 441 | // an invokevirtual directly on an interface method I.m if K implements I. |
duke@435 | 442 | |
twisti@4313 | 443 | // orig_callee is the resolved callee which's signature includes the |
twisti@4313 | 444 | // appendix argument. |
twisti@4021 | 445 | const int nargs = orig_callee->arg_size(); |
twisti@4414 | 446 | const bool is_signature_polymorphic = MethodHandles::is_signature_polymorphic(orig_callee->intrinsic_id()); |
twisti@3969 | 447 | |
twisti@3969 | 448 | // Push appendix argument (MethodType, CallSite, etc.), if one. |
twisti@3969 | 449 | if (iter().has_appendix()) { |
twisti@3969 | 450 | ciObject* appendix_arg = iter().get_appendix(); |
twisti@3969 | 451 | const TypeOopPtr* appendix_arg_type = TypeOopPtr::make_from_constant(appendix_arg); |
twisti@3969 | 452 | Node* appendix_arg_node = _gvn.makecon(appendix_arg_type); |
twisti@3969 | 453 | push(appendix_arg_node); |
twisti@3969 | 454 | } |
twisti@3969 | 455 | |
duke@435 | 456 | // --------------------- |
duke@435 | 457 | // Does Class Hierarchy Analysis reveal only a single target of a v-call? |
duke@435 | 458 | // Then we may inline or make a static call, but become dependent on there being only 1 target. |
duke@435 | 459 | // Does the call-site type profile reveal only one receiver? |
duke@435 | 460 | // Then we may introduce a run-time check and inline on the path where it succeeds. |
duke@435 | 461 | // The other path may uncommon_trap, check for another receiver, or do a v-call. |
duke@435 | 462 | |
twisti@4414 | 463 | // Try to get the most accurate receiver type |
twisti@4414 | 464 | ciMethod* callee = orig_callee; |
twisti@4414 | 465 | int vtable_index = Method::invalid_vtable_index; |
twisti@4414 | 466 | bool call_does_dispatch = false; |
duke@435 | 467 | |
roland@5991 | 468 | // Speculative type of the receiver if any |
roland@5991 | 469 | ciKlass* speculative_receiver_type = NULL; |
duke@435 | 470 | if (is_virtual_or_interface) { |
roland@5991 | 471 | Node* receiver_node = stack(sp() - nargs); |
duke@435 | 472 | const TypeOopPtr* receiver_type = _gvn.type(receiver_node)->isa_oopptr(); |
twisti@4414 | 473 | // call_does_dispatch and vtable_index are out-parameters. They might be changed. |
roland@6746 | 474 | // For arrays, klass below is Object. When vtable calls are used, |
roland@6746 | 475 | // resolving the call with Object would allow an illegal call to |
roland@6746 | 476 | // finalize() on an array. We use holder instead: illegal calls to |
roland@6746 | 477 | // finalize() won't be compiled as vtable calls (IC call |
roland@6746 | 478 | // resolution will catch the illegal call) and the few legal calls |
roland@6746 | 479 | // on array types won't be either. |
roland@6746 | 480 | callee = C->optimize_virtual_call(method(), bci(), klass, holder, orig_callee, |
roland@6746 | 481 | receiver_type, is_virtual, |
twisti@4414 | 482 | call_does_dispatch, vtable_index); // out-parameters |
roland@5991 | 483 | speculative_receiver_type = receiver_type != NULL ? receiver_type->speculative_type() : NULL; |
duke@435 | 484 | } |
duke@435 | 485 | |
coleenp@8739 | 486 | // invoke-super-special |
coleenp@8739 | 487 | if (iter().cur_bc_raw() == Bytecodes::_invokespecial && !orig_callee->is_object_initializer()) { |
coleenp@8739 | 488 | ciInstanceKlass* calling_klass = method()->holder(); |
coleenp@8739 | 489 | ciInstanceKlass* sender_klass = |
coleenp@8739 | 490 | calling_klass->is_anonymous() ? calling_klass->host_klass() : |
coleenp@8739 | 491 | calling_klass; |
coleenp@8739 | 492 | if (sender_klass->is_interface()) { |
coleenp@8739 | 493 | Node* receiver_node = stack(sp() - nargs); |
coleenp@8739 | 494 | Node* cls_node = makecon(TypeKlassPtr::make(sender_klass)); |
coleenp@8739 | 495 | Node* bad_type_ctrl = NULL; |
coleenp@8739 | 496 | Node* casted_receiver = gen_checkcast(receiver_node, cls_node, &bad_type_ctrl); |
coleenp@8739 | 497 | if (bad_type_ctrl != NULL) { |
coleenp@8739 | 498 | PreserveJVMState pjvms(this); |
coleenp@8739 | 499 | set_control(bad_type_ctrl); |
coleenp@8739 | 500 | uncommon_trap(Deoptimization::Reason_class_check, |
coleenp@8739 | 501 | Deoptimization::Action_none); |
coleenp@8739 | 502 | } |
coleenp@8739 | 503 | if (stopped()) { |
coleenp@8739 | 504 | return; // MUST uncommon-trap? |
coleenp@8739 | 505 | } |
coleenp@8739 | 506 | set_stack(sp() - nargs, casted_receiver); |
coleenp@8739 | 507 | } |
coleenp@8739 | 508 | } |
coleenp@8739 | 509 | |
duke@435 | 510 | // Note: It's OK to try to inline a virtual call. |
duke@435 | 511 | // The call generator will not attempt to inline a polymorphic call |
duke@435 | 512 | // unless it knows how to optimize the receiver dispatch. |
duke@435 | 513 | bool try_inline = (C->do_inlining() || InlineAccessors); |
duke@435 | 514 | |
duke@435 | 515 | // --------------------- |
twisti@3969 | 516 | dec_sp(nargs); // Temporarily pop args for JVM state of call |
duke@435 | 517 | JVMState* jvms = sync_jvms(); |
duke@435 | 518 | |
duke@435 | 519 | // --------------------- |
duke@435 | 520 | // Decide call tactic. |
duke@435 | 521 | // This call checks with CHA, the interpreter profile, intrinsics table, etc. |
duke@435 | 522 | // It decides whether inlining is desirable or not. |
roland@5991 | 523 | CallGenerator* cg = C->call_generator(callee, vtable_index, call_does_dispatch, jvms, try_inline, prof_factor(), speculative_receiver_type); |
twisti@3969 | 524 | |
twisti@4021 | 525 | // NOTE: Don't use orig_callee and callee after this point! Use cg->method() instead. |
twisti@4021 | 526 | orig_callee = callee = NULL; |
duke@435 | 527 | |
duke@435 | 528 | // --------------------- |
duke@435 | 529 | // Round double arguments before call |
twisti@3969 | 530 | round_double_arguments(cg->method()); |
duke@435 | 531 | |
roland@5991 | 532 | // Feed profiling data for arguments to the type system so it can |
roland@5991 | 533 | // propagate it as speculative types |
roland@5991 | 534 | record_profiled_arguments_for_speculation(cg->method(), bc()); |
roland@5991 | 535 | |
duke@435 | 536 | #ifndef PRODUCT |
duke@435 | 537 | // bump global counters for calls |
twisti@3969 | 538 | count_compiled_calls(/*at_method_entry*/ false, cg->is_inline()); |
duke@435 | 539 | |
duke@435 | 540 | // Record first part of parsing work for this call |
duke@435 | 541 | parse_histogram()->record_change(); |
duke@435 | 542 | #endif // not PRODUCT |
duke@435 | 543 | |
duke@435 | 544 | assert(jvms == this->jvms(), "still operating on the right JVMS"); |
duke@435 | 545 | assert(jvms_in_sync(), "jvms must carry full info into CG"); |
duke@435 | 546 | |
duke@435 | 547 | // save across call, for a subsequent cast_not_null. |
duke@435 | 548 | Node* receiver = has_receiver ? argument(0) : NULL; |
duke@435 | 549 | |
roland@5991 | 550 | // The extra CheckCastPP for speculative types mess with PhaseStringOpts |
roland@5991 | 551 | if (receiver != NULL && !call_does_dispatch && !cg->is_string_late_inline()) { |
roland@5991 | 552 | // Feed profiling data for a single receiver to the type system so |
roland@5991 | 553 | // it can propagate it as a speculative type |
roland@5991 | 554 | receiver = record_profiled_receiver_for_speculation(receiver); |
roland@5991 | 555 | } |
roland@5991 | 556 | |
duke@435 | 557 | // Bump method data counters (We profile *before* the call is made |
duke@435 | 558 | // because exceptions don't return to the call site.) |
duke@435 | 559 | profile_call(receiver); |
duke@435 | 560 | |
roland@7041 | 561 | JVMState* new_jvms = cg->generate(jvms); |
twisti@3969 | 562 | if (new_jvms == NULL) { |
duke@435 | 563 | // When inlining attempt fails (e.g., too many arguments), |
duke@435 | 564 | // it may contaminate the current compile state, making it |
duke@435 | 565 | // impossible to pull back and try again. Once we call |
duke@435 | 566 | // cg->generate(), we are committed. If it fails, the whole |
duke@435 | 567 | // compilation task is compromised. |
duke@435 | 568 | if (failing()) return; |
never@3631 | 569 | |
duke@435 | 570 | // This can happen if a library intrinsic is available, but refuses |
duke@435 | 571 | // the call site, perhaps because it did not match a pattern the |
never@3631 | 572 | // intrinsic was expecting to optimize. Should always be possible to |
never@3631 | 573 | // get a normal java call that may inline in that case |
roland@5991 | 574 | cg = C->call_generator(cg->method(), vtable_index, call_does_dispatch, jvms, try_inline, prof_factor(), speculative_receiver_type, /* allow_intrinsics= */ false); |
roland@7041 | 575 | if ((new_jvms = cg->generate(jvms)) == NULL) { |
duke@435 | 576 | guarantee(failing(), "call failed to generate: calls should work"); |
duke@435 | 577 | return; |
duke@435 | 578 | } |
duke@435 | 579 | } |
duke@435 | 580 | |
duke@435 | 581 | if (cg->is_inline()) { |
never@502 | 582 | // Accumulate has_loops estimate |
twisti@3969 | 583 | C->set_has_loops(C->has_loops() || cg->method()->has_loops()); |
twisti@3969 | 584 | C->env()->notice_inlined_method(cg->method()); |
duke@435 | 585 | } |
duke@435 | 586 | |
duke@435 | 587 | // Reset parser state from [new_]jvms, which now carries results of the call. |
duke@435 | 588 | // Return value (if any) is already pushed on the stack by the cg. |
duke@435 | 589 | add_exception_states_from(new_jvms); |
duke@435 | 590 | if (new_jvms->map()->control() == top()) { |
duke@435 | 591 | stop_and_kill_map(); |
duke@435 | 592 | } else { |
duke@435 | 593 | assert(new_jvms->same_calls_as(jvms), "method/bci left unchanged"); |
duke@435 | 594 | set_jvms(new_jvms); |
duke@435 | 595 | } |
duke@435 | 596 | |
duke@435 | 597 | if (!stopped()) { |
duke@435 | 598 | // This was some sort of virtual call, which did a null check for us. |
duke@435 | 599 | // Now we can assert receiver-not-null, on the normal return path. |
duke@435 | 600 | if (receiver != NULL && cg->is_virtual()) { |
duke@435 | 601 | Node* cast = cast_not_null(receiver); |
duke@435 | 602 | // %%% assert(receiver == cast, "should already have cast the receiver"); |
duke@435 | 603 | } |
duke@435 | 604 | |
duke@435 | 605 | // Round double result after a call from strict to non-strict code |
twisti@3969 | 606 | round_double_result(cg->method()); |
twisti@3969 | 607 | |
twisti@3969 | 608 | ciType* rtype = cg->method()->return_type(); |
twisti@4414 | 609 | ciType* ctype = declared_signature->return_type(); |
twisti@4414 | 610 | |
twisti@4414 | 611 | if (Bytecodes::has_optional_appendix(iter().cur_bc_raw()) || is_signature_polymorphic) { |
twisti@3969 | 612 | // Be careful here with return types. |
twisti@3969 | 613 | if (ctype != rtype) { |
twisti@3969 | 614 | BasicType rt = rtype->basic_type(); |
twisti@3969 | 615 | BasicType ct = ctype->basic_type(); |
twisti@3969 | 616 | if (ct == T_VOID) { |
twisti@3969 | 617 | // It's OK for a method to return a value that is discarded. |
twisti@3969 | 618 | // The discarding does not require any special action from the caller. |
twisti@3969 | 619 | // The Java code knows this, at VerifyType.isNullConversion. |
twisti@3969 | 620 | pop_node(rt); // whatever it was, pop it |
twisti@3969 | 621 | } else if (rt == T_INT || is_subword_type(rt)) { |
twisti@4414 | 622 | // Nothing. These cases are handled in lambda form bytecode. |
twisti@4414 | 623 | assert(ct == T_INT || is_subword_type(ct), err_msg_res("must match: rt=%s, ct=%s", type2name(rt), type2name(ct))); |
twisti@3973 | 624 | } else if (rt == T_OBJECT || rt == T_ARRAY) { |
twisti@3973 | 625 | assert(ct == T_OBJECT || ct == T_ARRAY, err_msg_res("rt=%s, ct=%s", type2name(rt), type2name(ct))); |
twisti@3969 | 626 | if (ctype->is_loaded()) { |
twisti@4021 | 627 | const TypeOopPtr* arg_type = TypeOopPtr::make_from_klass(rtype->as_klass()); |
twisti@4021 | 628 | const Type* sig_type = TypeOopPtr::make_from_klass(ctype->as_klass()); |
twisti@4021 | 629 | if (arg_type != NULL && !arg_type->higher_equal(sig_type)) { |
twisti@4414 | 630 | Node* retnode = pop(); |
kvn@4115 | 631 | Node* cast_obj = _gvn.transform(new (C) CheckCastPPNode(control(), retnode, sig_type)); |
twisti@4021 | 632 | push(cast_obj); |
twisti@3969 | 633 | } |
twisti@3969 | 634 | } |
twisti@3969 | 635 | } else { |
twisti@4414 | 636 | assert(rt == ct, err_msg_res("unexpected mismatch: rt=%s, ct=%s", type2name(rt), type2name(ct))); |
twisti@3969 | 637 | // push a zero; it's better than getting an oop/int mismatch |
twisti@4414 | 638 | pop_node(rt); |
twisti@4414 | 639 | Node* retnode = zerocon(ct); |
twisti@3969 | 640 | push_node(ct, retnode); |
twisti@3969 | 641 | } |
twisti@3969 | 642 | // Now that the value is well-behaved, continue with the call-site type. |
twisti@3969 | 643 | rtype = ctype; |
twisti@3969 | 644 | } |
twisti@4414 | 645 | } else { |
twisti@4447 | 646 | // Symbolic resolution enforces the types to be the same. |
twisti@4447 | 647 | // NOTE: We must relax the assert for unloaded types because two |
twisti@4447 | 648 | // different ciType instances of the same unloaded class type |
twisti@4447 | 649 | // can appear to be "loaded" by different loaders (depending on |
twisti@4447 | 650 | // the accessing class). |
twisti@4447 | 651 | assert(!rtype->is_loaded() || !ctype->is_loaded() || rtype == ctype, |
twisti@4447 | 652 | err_msg_res("mismatched return types: rtype=%s, ctype=%s", rtype->name(), ctype->name())); |
twisti@3969 | 653 | } |
duke@435 | 654 | |
duke@435 | 655 | // If the return type of the method is not loaded, assert that the |
duke@435 | 656 | // value we got is a null. Otherwise, we need to recompile. |
twisti@3969 | 657 | if (!rtype->is_loaded()) { |
duke@435 | 658 | #ifndef PRODUCT |
duke@435 | 659 | if (PrintOpto && (Verbose || WizardMode)) { |
duke@435 | 660 | method()->print_name(); tty->print_cr(" asserting nullness of result at bci: %d", bci()); |
twisti@3969 | 661 | cg->method()->print_name(); tty->cr(); |
duke@435 | 662 | } |
duke@435 | 663 | #endif |
duke@435 | 664 | if (C->log() != NULL) { |
duke@435 | 665 | C->log()->elem("assert_null reason='return' klass='%d'", |
twisti@3969 | 666 | C->log()->identify(rtype)); |
duke@435 | 667 | } |
duke@435 | 668 | // If there is going to be a trap, put it at the next bytecode: |
duke@435 | 669 | set_bci(iter().next_bci()); |
twisti@4313 | 670 | null_assert(peek()); |
duke@435 | 671 | set_bci(iter().cur_bci()); // put it back |
duke@435 | 672 | } |
roland@5991 | 673 | BasicType ct = ctype->basic_type(); |
roland@5991 | 674 | if (ct == T_OBJECT || ct == T_ARRAY) { |
roland@5991 | 675 | ciKlass* better_type = method()->return_profiled_type(bci()); |
roland@5991 | 676 | if (UseTypeSpeculation && better_type != NULL) { |
roland@5991 | 677 | // If profiling reports a single type for the return value, |
roland@5991 | 678 | // feed it to the type system so it can propagate it as a |
roland@5991 | 679 | // speculative type |
roland@5991 | 680 | record_profile_for_speculation(stack(sp()-1), better_type); |
roland@5991 | 681 | } |
roland@5991 | 682 | } |
duke@435 | 683 | } |
duke@435 | 684 | |
duke@435 | 685 | // Restart record of parsing work after possible inlining of call |
duke@435 | 686 | #ifndef PRODUCT |
duke@435 | 687 | parse_histogram()->set_initial_state(bc()); |
duke@435 | 688 | #endif |
duke@435 | 689 | } |
duke@435 | 690 | |
duke@435 | 691 | //---------------------------catch_call_exceptions----------------------------- |
duke@435 | 692 | // Put a Catch and CatchProj nodes behind a just-created call. |
duke@435 | 693 | // Send their caught exceptions to the proper handler. |
duke@435 | 694 | // This may be used after a call to the rethrow VM stub, |
duke@435 | 695 | // when it is needed to process unloaded exception classes. |
duke@435 | 696 | void Parse::catch_call_exceptions(ciExceptionHandlerStream& handlers) { |
duke@435 | 697 | // Exceptions are delivered through this channel: |
duke@435 | 698 | Node* i_o = this->i_o(); |
duke@435 | 699 | |
duke@435 | 700 | // Add a CatchNode. |
duke@435 | 701 | GrowableArray<int>* bcis = new (C->node_arena()) GrowableArray<int>(C->node_arena(), 8, 0, -1); |
duke@435 | 702 | GrowableArray<const Type*>* extypes = new (C->node_arena()) GrowableArray<const Type*>(C->node_arena(), 8, 0, NULL); |
duke@435 | 703 | GrowableArray<int>* saw_unloaded = new (C->node_arena()) GrowableArray<int>(C->node_arena(), 8, 0, 0); |
duke@435 | 704 | |
duke@435 | 705 | for (; !handlers.is_done(); handlers.next()) { |
duke@435 | 706 | ciExceptionHandler* h = handlers.handler(); |
duke@435 | 707 | int h_bci = h->handler_bci(); |
duke@435 | 708 | ciInstanceKlass* h_klass = h->is_catch_all() ? env()->Throwable_klass() : h->catch_klass(); |
duke@435 | 709 | // Do not introduce unloaded exception types into the graph: |
duke@435 | 710 | if (!h_klass->is_loaded()) { |
duke@435 | 711 | if (saw_unloaded->contains(h_bci)) { |
duke@435 | 712 | /* We've already seen an unloaded exception with h_bci, |
duke@435 | 713 | so don't duplicate. Duplication will cause the CatchNode to be |
duke@435 | 714 | unnecessarily large. See 4713716. */ |
duke@435 | 715 | continue; |
duke@435 | 716 | } else { |
duke@435 | 717 | saw_unloaded->append(h_bci); |
duke@435 | 718 | } |
duke@435 | 719 | } |
duke@435 | 720 | const Type* h_extype = TypeOopPtr::make_from_klass(h_klass); |
duke@435 | 721 | // (We use make_from_klass because it respects UseUniqueSubclasses.) |
duke@435 | 722 | h_extype = h_extype->join(TypeInstPtr::NOTNULL); |
duke@435 | 723 | assert(!h_extype->empty(), "sanity"); |
duke@435 | 724 | // Note: It's OK if the BCIs repeat themselves. |
duke@435 | 725 | bcis->append(h_bci); |
duke@435 | 726 | extypes->append(h_extype); |
duke@435 | 727 | } |
duke@435 | 728 | |
duke@435 | 729 | int len = bcis->length(); |
kvn@4115 | 730 | CatchNode *cn = new (C) CatchNode(control(), i_o, len+1); |
duke@435 | 731 | Node *catch_ = _gvn.transform(cn); |
duke@435 | 732 | |
duke@435 | 733 | // now branch with the exception state to each of the (potential) |
duke@435 | 734 | // handlers |
duke@435 | 735 | for(int i=0; i < len; i++) { |
duke@435 | 736 | // Setup JVM state to enter the handler. |
duke@435 | 737 | PreserveJVMState pjvms(this); |
duke@435 | 738 | // Locals are just copied from before the call. |
duke@435 | 739 | // Get control from the CatchNode. |
duke@435 | 740 | int handler_bci = bcis->at(i); |
kvn@4115 | 741 | Node* ctrl = _gvn.transform( new (C) CatchProjNode(catch_, i+1,handler_bci)); |
duke@435 | 742 | // This handler cannot happen? |
duke@435 | 743 | if (ctrl == top()) continue; |
duke@435 | 744 | set_control(ctrl); |
duke@435 | 745 | |
duke@435 | 746 | // Create exception oop |
duke@435 | 747 | const TypeInstPtr* extype = extypes->at(i)->is_instptr(); |
kvn@4115 | 748 | Node *ex_oop = _gvn.transform(new (C) CreateExNode(extypes->at(i), ctrl, i_o)); |
duke@435 | 749 | |
duke@435 | 750 | // Handle unloaded exception classes. |
duke@435 | 751 | if (saw_unloaded->contains(handler_bci)) { |
duke@435 | 752 | // An unloaded exception type is coming here. Do an uncommon trap. |
duke@435 | 753 | #ifndef PRODUCT |
duke@435 | 754 | // We do not expect the same handler bci to take both cold unloaded |
duke@435 | 755 | // and hot loaded exceptions. But, watch for it. |
twisti@3970 | 756 | if ((Verbose || WizardMode) && extype->is_loaded()) { |
twisti@3970 | 757 | tty->print("Warning: Handler @%d takes mixed loaded/unloaded exceptions in ", bci()); |
duke@435 | 758 | method()->print_name(); tty->cr(); |
duke@435 | 759 | } else if (PrintOpto && (Verbose || WizardMode)) { |
duke@435 | 760 | tty->print("Bailing out on unloaded exception type "); |
duke@435 | 761 | extype->klass()->print_name(); |
duke@435 | 762 | tty->print(" at bci:%d in ", bci()); |
duke@435 | 763 | method()->print_name(); tty->cr(); |
duke@435 | 764 | } |
duke@435 | 765 | #endif |
duke@435 | 766 | // Emit an uncommon trap instead of processing the block. |
duke@435 | 767 | set_bci(handler_bci); |
duke@435 | 768 | push_ex_oop(ex_oop); |
duke@435 | 769 | uncommon_trap(Deoptimization::Reason_unloaded, |
duke@435 | 770 | Deoptimization::Action_reinterpret, |
duke@435 | 771 | extype->klass(), "!loaded exception"); |
duke@435 | 772 | set_bci(iter().cur_bci()); // put it back |
duke@435 | 773 | continue; |
duke@435 | 774 | } |
duke@435 | 775 | |
duke@435 | 776 | // go to the exception handler |
duke@435 | 777 | if (handler_bci < 0) { // merge with corresponding rethrow node |
duke@435 | 778 | throw_to_exit(make_exception_state(ex_oop)); |
duke@435 | 779 | } else { // Else jump to corresponding handle |
duke@435 | 780 | push_ex_oop(ex_oop); // Clear stack and push just the oop. |
duke@435 | 781 | merge_exception(handler_bci); |
duke@435 | 782 | } |
duke@435 | 783 | } |
duke@435 | 784 | |
duke@435 | 785 | // The first CatchProj is for the normal return. |
duke@435 | 786 | // (Note: If this is a call to rethrow_Java, this node goes dead.) |
kvn@4115 | 787 | set_control(_gvn.transform( new (C) CatchProjNode(catch_, CatchProjNode::fall_through_index, CatchProjNode::no_handler_bci))); |
duke@435 | 788 | } |
duke@435 | 789 | |
duke@435 | 790 | |
duke@435 | 791 | //----------------------------catch_inline_exceptions-------------------------- |
duke@435 | 792 | // Handle all exceptions thrown by an inlined method or individual bytecode. |
duke@435 | 793 | // Common case 1: we have no handler, so all exceptions merge right into |
duke@435 | 794 | // the rethrow case. |
duke@435 | 795 | // Case 2: we have some handlers, with loaded exception klasses that have |
duke@435 | 796 | // no subklasses. We do a Deutsch-Shiffman style type-check on the incoming |
duke@435 | 797 | // exception oop and branch to the handler directly. |
duke@435 | 798 | // Case 3: We have some handlers with subklasses or are not loaded at |
duke@435 | 799 | // compile-time. We have to call the runtime to resolve the exception. |
duke@435 | 800 | // So we insert a RethrowCall and all the logic that goes with it. |
duke@435 | 801 | void Parse::catch_inline_exceptions(SafePointNode* ex_map) { |
duke@435 | 802 | // Caller is responsible for saving away the map for normal control flow! |
duke@435 | 803 | assert(stopped(), "call set_map(NULL) first"); |
duke@435 | 804 | assert(method()->has_exception_handlers(), "don't come here w/o work to do"); |
duke@435 | 805 | |
duke@435 | 806 | Node* ex_node = saved_ex_oop(ex_map); |
duke@435 | 807 | if (ex_node == top()) { |
duke@435 | 808 | // No action needed. |
duke@435 | 809 | return; |
duke@435 | 810 | } |
duke@435 | 811 | const TypeInstPtr* ex_type = _gvn.type(ex_node)->isa_instptr(); |
duke@435 | 812 | NOT_PRODUCT(if (ex_type==NULL) tty->print_cr("*** Exception not InstPtr")); |
duke@435 | 813 | if (ex_type == NULL) |
duke@435 | 814 | ex_type = TypeOopPtr::make_from_klass(env()->Throwable_klass())->is_instptr(); |
duke@435 | 815 | |
duke@435 | 816 | // determine potential exception handlers |
duke@435 | 817 | ciExceptionHandlerStream handlers(method(), bci(), |
duke@435 | 818 | ex_type->klass()->as_instance_klass(), |
duke@435 | 819 | ex_type->klass_is_exact()); |
duke@435 | 820 | |
duke@435 | 821 | // Start executing from the given throw state. (Keep its stack, for now.) |
duke@435 | 822 | // Get the exception oop as known at compile time. |
duke@435 | 823 | ex_node = use_exception_state(ex_map); |
duke@435 | 824 | |
duke@435 | 825 | // Get the exception oop klass from its header |
duke@435 | 826 | Node* ex_klass_node = NULL; |
duke@435 | 827 | if (has_ex_handler() && !ex_type->klass_is_exact()) { |
duke@435 | 828 | Node* p = basic_plus_adr( ex_node, ex_node, oopDesc::klass_offset_in_bytes()); |
zmajo@7341 | 829 | ex_klass_node = _gvn.transform(LoadKlassNode::make(_gvn, NULL, immutable_memory(), p, TypeInstPtr::KLASS, TypeKlassPtr::OBJECT)); |
duke@435 | 830 | |
duke@435 | 831 | // Compute the exception klass a little more cleverly. |
duke@435 | 832 | // Obvious solution is to simple do a LoadKlass from the 'ex_node'. |
duke@435 | 833 | // However, if the ex_node is a PhiNode, I'm going to do a LoadKlass for |
duke@435 | 834 | // each arm of the Phi. If I know something clever about the exceptions |
duke@435 | 835 | // I'm loading the class from, I can replace the LoadKlass with the |
duke@435 | 836 | // klass constant for the exception oop. |
kvn@7266 | 837 | if (ex_node->is_Phi()) { |
kvn@7266 | 838 | ex_klass_node = new (C) PhiNode(ex_node->in(0), TypeKlassPtr::OBJECT); |
kvn@7266 | 839 | for (uint i = 1; i < ex_node->req(); i++) { |
kvn@7266 | 840 | Node* ex_in = ex_node->in(i); |
kvn@7266 | 841 | if (ex_in == top() || ex_in == NULL) { |
kvn@7266 | 842 | // This path was not taken. |
kvn@7266 | 843 | ex_klass_node->init_req(i, top()); |
kvn@7266 | 844 | continue; |
kvn@7266 | 845 | } |
kvn@7266 | 846 | Node* p = basic_plus_adr(ex_in, ex_in, oopDesc::klass_offset_in_bytes()); |
zmajo@7341 | 847 | Node* k = _gvn.transform(LoadKlassNode::make(_gvn, NULL, immutable_memory(), p, TypeInstPtr::KLASS, TypeKlassPtr::OBJECT)); |
duke@435 | 848 | ex_klass_node->init_req( i, k ); |
duke@435 | 849 | } |
duke@435 | 850 | _gvn.set_type(ex_klass_node, TypeKlassPtr::OBJECT); |
duke@435 | 851 | |
duke@435 | 852 | } |
duke@435 | 853 | } |
duke@435 | 854 | |
duke@435 | 855 | // Scan the exception table for applicable handlers. |
duke@435 | 856 | // If none, we can call rethrow() and be done! |
duke@435 | 857 | // If precise (loaded with no subklasses), insert a D.S. style |
duke@435 | 858 | // pointer compare to the correct handler and loop back. |
duke@435 | 859 | // If imprecise, switch to the Rethrow VM-call style handling. |
duke@435 | 860 | |
duke@435 | 861 | int remaining = handlers.count_remaining(); |
duke@435 | 862 | |
duke@435 | 863 | // iterate through all entries sequentially |
duke@435 | 864 | for (;!handlers.is_done(); handlers.next()) { |
duke@435 | 865 | ciExceptionHandler* handler = handlers.handler(); |
duke@435 | 866 | |
duke@435 | 867 | if (handler->is_rethrow()) { |
duke@435 | 868 | // If we fell off the end of the table without finding an imprecise |
duke@435 | 869 | // exception klass (and without finding a generic handler) then we |
duke@435 | 870 | // know this exception is not handled in this method. We just rethrow |
duke@435 | 871 | // the exception into the caller. |
duke@435 | 872 | throw_to_exit(make_exception_state(ex_node)); |
duke@435 | 873 | return; |
duke@435 | 874 | } |
duke@435 | 875 | |
duke@435 | 876 | // exception handler bci range covers throw_bci => investigate further |
duke@435 | 877 | int handler_bci = handler->handler_bci(); |
duke@435 | 878 | |
duke@435 | 879 | if (remaining == 1) { |
duke@435 | 880 | push_ex_oop(ex_node); // Push exception oop for handler |
duke@435 | 881 | #ifndef PRODUCT |
duke@435 | 882 | if (PrintOpto && WizardMode) { |
duke@435 | 883 | tty->print_cr(" Catching every inline exception bci:%d -> handler_bci:%d", bci(), handler_bci); |
duke@435 | 884 | } |
duke@435 | 885 | #endif |
duke@435 | 886 | merge_exception(handler_bci); // jump to handler |
duke@435 | 887 | return; // No more handling to be done here! |
duke@435 | 888 | } |
duke@435 | 889 | |
never@1779 | 890 | // Get the handler's klass |
never@1779 | 891 | ciInstanceKlass* klass = handler->catch_klass(); |
duke@435 | 892 | |
never@1779 | 893 | if (!klass->is_loaded()) { // klass is not loaded? |
never@1779 | 894 | // fall through into catch_call_exceptions which will emit a |
never@1779 | 895 | // handler with an uncommon trap. |
never@1779 | 896 | break; |
duke@435 | 897 | } |
duke@435 | 898 | |
duke@435 | 899 | if (klass->is_interface()) // should not happen, but... |
duke@435 | 900 | break; // bail out |
duke@435 | 901 | |
never@1779 | 902 | // Check the type of the exception against the catch type |
duke@435 | 903 | const TypeKlassPtr *tk = TypeKlassPtr::make(klass); |
duke@435 | 904 | Node* con = _gvn.makecon(tk); |
never@1779 | 905 | Node* not_subtype_ctrl = gen_subtype_check(ex_klass_node, con); |
never@1779 | 906 | if (!stopped()) { |
never@1779 | 907 | PreserveJVMState pjvms(this); |
never@1779 | 908 | const TypeInstPtr* tinst = TypeOopPtr::make_from_klass_unique(klass)->cast_to_ptr_type(TypePtr::NotNull)->is_instptr(); |
never@1779 | 909 | assert(klass->has_subklass() || tinst->klass_is_exact(), "lost exactness"); |
kvn@4115 | 910 | Node* ex_oop = _gvn.transform(new (C) CheckCastPPNode(control(), ex_node, tinst)); |
duke@435 | 911 | push_ex_oop(ex_oop); // Push exception oop for handler |
duke@435 | 912 | #ifndef PRODUCT |
duke@435 | 913 | if (PrintOpto && WizardMode) { |
duke@435 | 914 | tty->print(" Catching inline exception bci:%d -> handler_bci:%d -- ", bci(), handler_bci); |
duke@435 | 915 | klass->print_name(); |
duke@435 | 916 | tty->cr(); |
duke@435 | 917 | } |
duke@435 | 918 | #endif |
duke@435 | 919 | merge_exception(handler_bci); |
duke@435 | 920 | } |
never@1779 | 921 | set_control(not_subtype_ctrl); |
duke@435 | 922 | |
duke@435 | 923 | // Come here if exception does not match handler. |
duke@435 | 924 | // Carry on with more handler checks. |
duke@435 | 925 | --remaining; |
duke@435 | 926 | } |
duke@435 | 927 | |
duke@435 | 928 | assert(!stopped(), "you should return if you finish the chain"); |
duke@435 | 929 | |
duke@435 | 930 | // Oops, need to call into the VM to resolve the klasses at runtime. |
duke@435 | 931 | // Note: This call must not deoptimize, since it is not a real at this bci! |
duke@435 | 932 | kill_dead_locals(); |
duke@435 | 933 | |
duke@435 | 934 | make_runtime_call(RC_NO_LEAF | RC_MUST_THROW, |
duke@435 | 935 | OptoRuntime::rethrow_Type(), |
duke@435 | 936 | OptoRuntime::rethrow_stub(), |
duke@435 | 937 | NULL, NULL, |
duke@435 | 938 | ex_node); |
duke@435 | 939 | |
duke@435 | 940 | // Rethrow is a pure call, no side effects, only a result. |
duke@435 | 941 | // The result cannot be allocated, so we use I_O |
duke@435 | 942 | |
duke@435 | 943 | // Catch exceptions from the rethrow |
duke@435 | 944 | catch_call_exceptions(handlers); |
duke@435 | 945 | } |
duke@435 | 946 | |
duke@435 | 947 | |
duke@435 | 948 | // (Note: Moved add_debug_info into GraphKit::add_safepoint_edges.) |
duke@435 | 949 | |
duke@435 | 950 | |
duke@435 | 951 | #ifndef PRODUCT |
duke@435 | 952 | void Parse::count_compiled_calls(bool at_method_entry, bool is_inline) { |
duke@435 | 953 | if( CountCompiledCalls ) { |
duke@435 | 954 | if( at_method_entry ) { |
duke@435 | 955 | // bump invocation counter if top method (for statistics) |
duke@435 | 956 | if (CountCompiledCalls && depth() == 1) { |
coleenp@4037 | 957 | const TypePtr* addr_type = TypeMetadataPtr::make(method()); |
duke@435 | 958 | Node* adr1 = makecon(addr_type); |
coleenp@4037 | 959 | Node* adr2 = basic_plus_adr(adr1, adr1, in_bytes(Method::compiled_invocation_counter_offset())); |
duke@435 | 960 | increment_counter(adr2); |
duke@435 | 961 | } |
duke@435 | 962 | } else if (is_inline) { |
duke@435 | 963 | switch (bc()) { |
duke@435 | 964 | case Bytecodes::_invokevirtual: increment_counter(SharedRuntime::nof_inlined_calls_addr()); break; |
duke@435 | 965 | case Bytecodes::_invokeinterface: increment_counter(SharedRuntime::nof_inlined_interface_calls_addr()); break; |
duke@435 | 966 | case Bytecodes::_invokestatic: |
jrose@1161 | 967 | case Bytecodes::_invokedynamic: |
duke@435 | 968 | case Bytecodes::_invokespecial: increment_counter(SharedRuntime::nof_inlined_static_calls_addr()); break; |
duke@435 | 969 | default: fatal("unexpected call bytecode"); |
duke@435 | 970 | } |
duke@435 | 971 | } else { |
duke@435 | 972 | switch (bc()) { |
duke@435 | 973 | case Bytecodes::_invokevirtual: increment_counter(SharedRuntime::nof_normal_calls_addr()); break; |
duke@435 | 974 | case Bytecodes::_invokeinterface: increment_counter(SharedRuntime::nof_interface_calls_addr()); break; |
duke@435 | 975 | case Bytecodes::_invokestatic: |
jrose@1161 | 976 | case Bytecodes::_invokedynamic: |
duke@435 | 977 | case Bytecodes::_invokespecial: increment_counter(SharedRuntime::nof_static_calls_addr()); break; |
duke@435 | 978 | default: fatal("unexpected call bytecode"); |
duke@435 | 979 | } |
duke@435 | 980 | } |
duke@435 | 981 | } |
duke@435 | 982 | } |
duke@435 | 983 | #endif //PRODUCT |
duke@435 | 984 | |
duke@435 | 985 | |
twisti@4414 | 986 | ciMethod* Compile::optimize_virtual_call(ciMethod* caller, int bci, ciInstanceKlass* klass, |
roland@6746 | 987 | ciKlass* holder, ciMethod* callee, |
roland@6746 | 988 | const TypeOopPtr* receiver_type, bool is_virtual, |
vlivanov@7792 | 989 | bool& call_does_dispatch, int& vtable_index, |
vlivanov@7792 | 990 | bool check_access) { |
twisti@4414 | 991 | // Set default values for out-parameters. |
twisti@4414 | 992 | call_does_dispatch = true; |
twisti@4414 | 993 | vtable_index = Method::invalid_vtable_index; |
twisti@4414 | 994 | |
twisti@4414 | 995 | // Choose call strategy. |
vlivanov@7792 | 996 | ciMethod* optimized_virtual_method = optimize_inlining(caller, bci, klass, callee, |
vlivanov@7792 | 997 | receiver_type, check_access); |
twisti@4414 | 998 | |
twisti@4414 | 999 | // Have the call been sufficiently improved such that it is no longer a virtual? |
twisti@4414 | 1000 | if (optimized_virtual_method != NULL) { |
twisti@4414 | 1001 | callee = optimized_virtual_method; |
twisti@4414 | 1002 | call_does_dispatch = false; |
twisti@4414 | 1003 | } else if (!UseInlineCaches && is_virtual && callee->is_loaded()) { |
twisti@4414 | 1004 | // We can make a vtable call at this site |
roland@6746 | 1005 | vtable_index = callee->resolve_vtable_index(caller->holder(), holder); |
twisti@4414 | 1006 | } |
twisti@4414 | 1007 | return callee; |
twisti@4414 | 1008 | } |
twisti@4414 | 1009 | |
duke@435 | 1010 | // Identify possible target method and inlining style |
twisti@4414 | 1011 | ciMethod* Compile::optimize_inlining(ciMethod* caller, int bci, ciInstanceKlass* klass, |
vlivanov@7792 | 1012 | ciMethod* callee, const TypeOopPtr* receiver_type, |
vlivanov@7792 | 1013 | bool check_access) { |
duke@435 | 1014 | // only use for virtual or interface calls |
duke@435 | 1015 | |
duke@435 | 1016 | // If it is obviously final, do not bother to call find_monomorphic_target, |
duke@435 | 1017 | // because the class hierarchy checks are not needed, and may fail due to |
duke@435 | 1018 | // incompletely loaded classes. Since we do our own class loading checks |
duke@435 | 1019 | // in this module, we may confidently bind to any method. |
twisti@4414 | 1020 | if (callee->can_be_statically_bound()) { |
twisti@4414 | 1021 | return callee; |
duke@435 | 1022 | } |
duke@435 | 1023 | |
duke@435 | 1024 | // Attempt to improve the receiver |
duke@435 | 1025 | bool actual_receiver_is_exact = false; |
duke@435 | 1026 | ciInstanceKlass* actual_receiver = klass; |
duke@435 | 1027 | if (receiver_type != NULL) { |
duke@435 | 1028 | // Array methods are all inherited from Object, and are monomorphic. |
roland@6746 | 1029 | // finalize() call on array is not allowed. |
duke@435 | 1030 | if (receiver_type->isa_aryptr() && |
roland@6746 | 1031 | callee->holder() == env()->Object_klass() && |
roland@6746 | 1032 | callee->name() != ciSymbol::finalize_method_name()) { |
twisti@4414 | 1033 | return callee; |
duke@435 | 1034 | } |
duke@435 | 1035 | |
duke@435 | 1036 | // All other interesting cases are instance klasses. |
duke@435 | 1037 | if (!receiver_type->isa_instptr()) { |
duke@435 | 1038 | return NULL; |
duke@435 | 1039 | } |
duke@435 | 1040 | |
duke@435 | 1041 | ciInstanceKlass *ikl = receiver_type->klass()->as_instance_klass(); |
duke@435 | 1042 | if (ikl->is_loaded() && ikl->is_initialized() && !ikl->is_interface() && |
never@802 | 1043 | (ikl == actual_receiver || ikl->is_subtype_of(actual_receiver))) { |
duke@435 | 1044 | // ikl is a same or better type than the original actual_receiver, |
duke@435 | 1045 | // e.g. static receiver from bytecodes. |
duke@435 | 1046 | actual_receiver = ikl; |
duke@435 | 1047 | // Is the actual_receiver exact? |
duke@435 | 1048 | actual_receiver_is_exact = receiver_type->klass_is_exact(); |
duke@435 | 1049 | } |
duke@435 | 1050 | } |
duke@435 | 1051 | |
duke@435 | 1052 | ciInstanceKlass* calling_klass = caller->holder(); |
vlivanov@7792 | 1053 | ciMethod* cha_monomorphic_target = callee->find_monomorphic_target(calling_klass, klass, actual_receiver, check_access); |
duke@435 | 1054 | if (cha_monomorphic_target != NULL) { |
duke@435 | 1055 | assert(!cha_monomorphic_target->is_abstract(), ""); |
duke@435 | 1056 | // Look at the method-receiver type. Does it add "too much information"? |
duke@435 | 1057 | ciKlass* mr_klass = cha_monomorphic_target->holder(); |
duke@435 | 1058 | const Type* mr_type = TypeInstPtr::make(TypePtr::BotPTR, mr_klass); |
duke@435 | 1059 | if (receiver_type == NULL || !receiver_type->higher_equal(mr_type)) { |
duke@435 | 1060 | // Calling this method would include an implicit cast to its holder. |
duke@435 | 1061 | // %%% Not yet implemented. Would throw minor asserts at present. |
duke@435 | 1062 | // %%% The most common wins are already gained by +UseUniqueSubclasses. |
duke@435 | 1063 | // To fix, put the higher_equal check at the call of this routine, |
duke@435 | 1064 | // and add a CheckCastPP to the receiver. |
duke@435 | 1065 | if (TraceDependencies) { |
duke@435 | 1066 | tty->print_cr("found unique CHA method, but could not cast up"); |
duke@435 | 1067 | tty->print(" method = "); |
duke@435 | 1068 | cha_monomorphic_target->print(); |
duke@435 | 1069 | tty->cr(); |
duke@435 | 1070 | } |
twisti@4414 | 1071 | if (log() != NULL) { |
twisti@4414 | 1072 | log()->elem("missed_CHA_opportunity klass='%d' method='%d'", |
twisti@4414 | 1073 | log()->identify(klass), |
twisti@4414 | 1074 | log()->identify(cha_monomorphic_target)); |
duke@435 | 1075 | } |
duke@435 | 1076 | cha_monomorphic_target = NULL; |
duke@435 | 1077 | } |
duke@435 | 1078 | } |
duke@435 | 1079 | if (cha_monomorphic_target != NULL) { |
duke@435 | 1080 | // Hardwiring a virtual. |
duke@435 | 1081 | // If we inlined because CHA revealed only a single target method, |
duke@435 | 1082 | // then we are dependent on that target method not getting overridden |
duke@435 | 1083 | // by dynamic class loading. Be sure to test the "static" receiver |
duke@435 | 1084 | // dest_method here, as opposed to the actual receiver, which may |
duke@435 | 1085 | // falsely lead us to believe that the receiver is final or private. |
twisti@4414 | 1086 | dependencies()->assert_unique_concrete_method(actual_receiver, cha_monomorphic_target); |
duke@435 | 1087 | return cha_monomorphic_target; |
duke@435 | 1088 | } |
duke@435 | 1089 | |
duke@435 | 1090 | // If the type is exact, we can still bind the method w/o a vcall. |
duke@435 | 1091 | // (This case comes after CHA so we can see how much extra work it does.) |
duke@435 | 1092 | if (actual_receiver_is_exact) { |
duke@435 | 1093 | // In case of evolution, there is a dependence on every inlined method, since each |
duke@435 | 1094 | // such method can be changed when its class is redefined. |
twisti@4414 | 1095 | ciMethod* exact_method = callee->resolve_invoke(calling_klass, actual_receiver); |
duke@435 | 1096 | if (exact_method != NULL) { |
duke@435 | 1097 | #ifndef PRODUCT |
duke@435 | 1098 | if (PrintOpto) { |
duke@435 | 1099 | tty->print(" Calling method via exact type @%d --- ", bci); |
duke@435 | 1100 | exact_method->print_name(); |
duke@435 | 1101 | tty->cr(); |
duke@435 | 1102 | } |
duke@435 | 1103 | #endif |
duke@435 | 1104 | return exact_method; |
duke@435 | 1105 | } |
duke@435 | 1106 | } |
duke@435 | 1107 | |
duke@435 | 1108 | return NULL; |
duke@435 | 1109 | } |