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