annotate src/share/vm/opto/doCall.cpp @ 11738:e9f9f6ffc351

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