hotspot/src/share/vm/opto/callGenerator.cpp
author vlivanov
Fri, 10 Feb 2017 20:45:39 +0300
changeset 43947 a52ee13998f3
parent 43943 e2fdae30bbc1
permissions -rw-r--r--
8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe Reviewed-by: iveresov
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     1
/*
29081
c61eb4914428 8072911: Remove includes of oop.inline.hpp from .hpp files
stefank
parents: 26945
diff changeset
     2
 * Copyright (c) 2000, 2015, 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: 4906
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 4906
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: 4906
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: 5547
diff changeset
    25
#include "precompiled.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    26
#include "ci/bcEscapeAnalyzer.hpp"
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 9975
diff changeset
    27
#include "ci/ciCallSite.hpp"
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13487
diff changeset
    28
#include "ci/ciObjArray.hpp"
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
    29
#include "ci/ciMemberName.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    30
#include "ci/ciMethodHandle.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    31
#include "classfile/javaClasses.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    32
#include "compiler/compileLog.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    33
#include "opto/addnode.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    34
#include "opto/callGenerator.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    35
#include "opto/callnode.hpp"
23528
8f1a7f5e8066 8001532: C2 node files refactoring
morris
parents: 22873
diff changeset
    36
#include "opto/castnode.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    37
#include "opto/cfgnode.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    38
#include "opto/parse.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    39
#include "opto/rootnode.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    40
#include "opto/runtime.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    41
#include "opto/subnode.hpp"
29081
c61eb4914428 8072911: Remove includes of oop.inline.hpp from .hpp files
stefank
parents: 26945
diff changeset
    42
#include "runtime/sharedRuntime.hpp"
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    43
489c9b5090e2 Initial load
duke
parents:
diff changeset
    44
// Utility function.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    45
const TypeFunc* CallGenerator::tf() const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    46
  return TypeFunc::make(method());
489c9b5090e2 Initial load
duke
parents:
diff changeset
    47
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    48
43943
e2fdae30bbc1 8166110: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents: 40882
diff changeset
    49
bool CallGenerator::is_inlined_method_handle_intrinsic(JVMState* jvms, ciMethod* callee) {
35086
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 33589
diff changeset
    50
  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: 33589
diff changeset
    51
  return symbolic_info->is_method_handle_intrinsic() && !callee->is_method_handle_intrinsic();
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 33589
diff changeset
    52
}
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 33589
diff changeset
    53
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    54
//-----------------------------ParseGenerator---------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
    55
// Internal class which handles all direct bytecode traversal.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    56
class ParseGenerator : public InlineCallGenerator {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    57
private:
489c9b5090e2 Initial load
duke
parents:
diff changeset
    58
  bool  _is_osr;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    59
  float _expected_uses;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    60
489c9b5090e2 Initial load
duke
parents:
diff changeset
    61
public:
489c9b5090e2 Initial load
duke
parents:
diff changeset
    62
  ParseGenerator(ciMethod* method, float expected_uses, bool is_osr = false)
489c9b5090e2 Initial load
duke
parents:
diff changeset
    63
    : InlineCallGenerator(method)
489c9b5090e2 Initial load
duke
parents:
diff changeset
    64
  {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    65
    _is_osr        = is_osr;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    66
    _expected_uses = expected_uses;
10509
43d670e5701e 7079673: JSR 292: C1 should inline bytecoded method handle adapters
twisti
parents: 10503
diff changeset
    67
    assert(InlineTree::check_can_parse(method) == NULL, "parse must be possible");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    68
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    69
489c9b5090e2 Initial load
duke
parents:
diff changeset
    70
  virtual bool      is_parse() const           { return true; }
24946
24b68ccf3fc4 8026796: Make replace_in_map() on parent maps generic
roland
parents: 24923
diff changeset
    71
  virtual JVMState* generate(JVMState* jvms);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    72
  int is_osr() { return _is_osr; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    73
489c9b5090e2 Initial load
duke
parents:
diff changeset
    74
};
489c9b5090e2 Initial load
duke
parents:
diff changeset
    75
24946
24b68ccf3fc4 8026796: Make replace_in_map() on parent maps generic
roland
parents: 24923
diff changeset
    76
JVMState* ParseGenerator::generate(JVMState* jvms) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    77
  Compile* C = Compile::current();
23846
490ace57944d 8007988: PrintInlining output is inconsistent with incremental inlining
roland
parents: 23528
diff changeset
    78
  C->print_inlining_update(this);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    79
489c9b5090e2 Initial load
duke
parents:
diff changeset
    80
  if (is_osr()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    81
    // The JVMS for a OSR has a single argument (see its TypeFunc).
489c9b5090e2 Initial load
duke
parents:
diff changeset
    82
    assert(jvms->depth() == 1, "no inline OSR");
489c9b5090e2 Initial load
duke
parents:
diff changeset
    83
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    84
489c9b5090e2 Initial load
duke
parents:
diff changeset
    85
  if (C->failing()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    86
    return NULL;  // bailing out of the compile; do not try to parse
489c9b5090e2 Initial load
duke
parents:
diff changeset
    87
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    88
24946
24b68ccf3fc4 8026796: Make replace_in_map() on parent maps generic
roland
parents: 24923
diff changeset
    89
  Parse parser(jvms, method(), _expected_uses);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    90
  // Grab signature for matching/allocation
489c9b5090e2 Initial load
duke
parents:
diff changeset
    91
#ifdef ASSERT
489c9b5090e2 Initial load
duke
parents:
diff changeset
    92
  if (parser.tf() != (parser.depth() == 1 ? C->tf() : tf())) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    93
    MutexLockerEx ml(Compile_lock, Mutex::_no_safepoint_check_flag);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    94
    assert(C->env()->system_dictionary_modification_counter_changed(),
489c9b5090e2 Initial load
duke
parents:
diff changeset
    95
           "Must invalidate if TypeFuncs differ");
489c9b5090e2 Initial load
duke
parents:
diff changeset
    96
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    97
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
    98
489c9b5090e2 Initial load
duke
parents:
diff changeset
    99
  GraphKit& exits = parser.exits();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   100
489c9b5090e2 Initial load
duke
parents:
diff changeset
   101
  if (C->failing()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   102
    while (exits.pop_exception_state() != NULL) ;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   103
    return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   104
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   105
489c9b5090e2 Initial load
duke
parents:
diff changeset
   106
  assert(exits.jvms()->same_calls_as(jvms), "sanity");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   107
489c9b5090e2 Initial load
duke
parents:
diff changeset
   108
  // Simply return the exit state of the parser,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   109
  // augmented by any exceptional states.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   110
  return exits.transfer_exceptions_into_jvms();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   111
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   112
489c9b5090e2 Initial load
duke
parents:
diff changeset
   113
//---------------------------DirectCallGenerator------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   114
// Internal class which handles all out-of-line calls w/o receiver type checks.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   115
class DirectCallGenerator : public CallGenerator {
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   116
 private:
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   117
  CallStaticJavaNode* _call_node;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   118
  // Force separate memory and I/O projections for the exceptional
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   119
  // paths to facilitate late inlinig.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   120
  bool                _separate_io_proj;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   121
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   122
 public:
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   123
  DirectCallGenerator(ciMethod* method, bool separate_io_proj)
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   124
    : CallGenerator(method),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   125
      _separate_io_proj(separate_io_proj)
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   126
  {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   127
  }
24946
24b68ccf3fc4 8026796: Make replace_in_map() on parent maps generic
roland
parents: 24923
diff changeset
   128
  virtual JVMState* generate(JVMState* jvms);
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   129
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   130
  CallStaticJavaNode* call_node() const { return _call_node; }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   131
};
489c9b5090e2 Initial load
duke
parents:
diff changeset
   132
24946
24b68ccf3fc4 8026796: Make replace_in_map() on parent maps generic
roland
parents: 24923
diff changeset
   133
JVMState* DirectCallGenerator::generate(JVMState* jvms) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   134
  GraphKit kit(jvms);
23846
490ace57944d 8007988: PrintInlining output is inconsistent with incremental inlining
roland
parents: 23528
diff changeset
   135
  kit.C->print_inlining_update(this);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   136
  bool is_static = method()->is_static();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   137
  address target = is_static ? SharedRuntime::get_resolve_static_call_stub()
489c9b5090e2 Initial load
duke
parents:
diff changeset
   138
                             : SharedRuntime::get_resolve_opt_virtual_call_stub();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   139
489c9b5090e2 Initial load
duke
parents:
diff changeset
   140
  if (kit.C->log() != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   141
    kit.C->log()->elem("direct_call bci='%d'", jvms->bci());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   142
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   143
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
   144
  CallStaticJavaNode *call = new CallStaticJavaNode(kit.C, tf(), target, method(), kit.bci());
43943
e2fdae30bbc1 8166110: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents: 40882
diff changeset
   145
  if (is_inlined_method_handle_intrinsic(jvms, method())) {
35086
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 33589
diff changeset
   146
    // To be able to issue a direct call and skip a call to MH.linkTo*/invokeBasic adapter,
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 33589
diff changeset
   147
    // additional information about the method being invoked should be attached
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 33589
diff changeset
   148
    // to the call site to make resolution logic work
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 33589
diff changeset
   149
    // (see SharedRuntime::resolve_static_call_C).
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 33589
diff changeset
   150
    call->set_override_symbolic_info(true);
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 33589
diff changeset
   151
  }
12591
064b5c1aa88a 7162094: LateInlineCallGenerator::do_late_inline crashed on uninitialized _call_node
never
parents: 11193
diff changeset
   152
  _call_node = call;  // Save the call node in case we need it later
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   153
  if (!is_static) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   154
    // Make an explicit receiver null_check as part of this call.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   155
    // Since we share a map with the caller, his JVMS gets adjusted.
14621
fd9265ab0f67 7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents: 14132
diff changeset
   156
    kit.null_check_receiver_before_call(method());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   157
    if (kit.stopped()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   158
      // And dump it back to the caller, decorated with any exceptions:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   159
      return kit.transfer_exceptions_into_jvms();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   160
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   161
    // Mark the call node as virtual, sort of:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   162
    call->set_optimized_virtual(true);
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   163
    if (method()->is_method_handle_intrinsic() ||
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   164
        method()->is_compiled_lambda_form()) {
4566
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   165
      call->set_method_handle_invoke(true);
4906
90640ad2d401 6926782: CodeBuffer size too small after 6921352
twisti
parents: 4567
diff changeset
   166
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   167
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   168
  kit.set_arguments_for_java_call(call);
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   169
  kit.set_edges_for_java_call(call, false, _separate_io_proj);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   170
  Node* ret = kit.set_results_for_java_call(call, _separate_io_proj);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   171
  kit.push_node(method()->return_type()->basic_type(), ret);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   172
  return kit.transfer_exceptions_into_jvms();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   173
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   174
4566
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   175
//--------------------------VirtualCallGenerator------------------------------
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   176
// Internal class which handles all out-of-line calls checking receiver type.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   177
class VirtualCallGenerator : public CallGenerator {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   178
private:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   179
  int _vtable_index;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   180
public:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   181
  VirtualCallGenerator(ciMethod* method, int vtable_index)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   182
    : CallGenerator(method), _vtable_index(vtable_index)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   183
  {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13487
diff changeset
   184
    assert(vtable_index == Method::invalid_vtable_index ||
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   185
           vtable_index >= 0, "either invalid or usable");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   186
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   187
  virtual bool      is_virtual() const          { return true; }
24946
24b68ccf3fc4 8026796: Make replace_in_map() on parent maps generic
roland
parents: 24923
diff changeset
   188
  virtual JVMState* generate(JVMState* jvms);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   189
};
489c9b5090e2 Initial load
duke
parents:
diff changeset
   190
24946
24b68ccf3fc4 8026796: Make replace_in_map() on parent maps generic
roland
parents: 24923
diff changeset
   191
JVMState* VirtualCallGenerator::generate(JVMState* jvms) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   192
  GraphKit kit(jvms);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   193
  Node* receiver = kit.argument(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   194
23846
490ace57944d 8007988: PrintInlining output is inconsistent with incremental inlining
roland
parents: 23528
diff changeset
   195
  kit.C->print_inlining_update(this);
490ace57944d 8007988: PrintInlining output is inconsistent with incremental inlining
roland
parents: 23528
diff changeset
   196
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   197
  if (kit.C->log() != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   198
    kit.C->log()->elem("virtual_call bci='%d'", jvms->bci());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   199
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   200
489c9b5090e2 Initial load
duke
parents:
diff changeset
   201
  // If the receiver is a constant null, do not torture the system
489c9b5090e2 Initial load
duke
parents:
diff changeset
   202
  // by attempting to call through it.  The compile will proceed
489c9b5090e2 Initial load
duke
parents:
diff changeset
   203
  // correctly, but may bail out in final_graph_reshaping, because
489c9b5090e2 Initial load
duke
parents:
diff changeset
   204
  // the call instruction will have a seemingly deficient out-count.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   205
  // (The bailout says something misleading about an "infinite loop".)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   206
  if (kit.gvn().type(receiver)->higher_equal(TypePtr::NULL_PTR)) {
35086
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 33589
diff changeset
   207
    assert(Bytecodes::is_invoke(kit.java_bc()), "%d: %s", kit.java_bc(), Bytecodes::name(kit.java_bc()));
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 33589
diff changeset
   208
    ciMethod* declared_method = kit.method()->get_method_at_bci(kit.bci());
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 33589
diff changeset
   209
    int arg_size = declared_method->signature()->arg_size_for_bc(kit.java_bc());
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 33589
diff changeset
   210
    kit.inc_sp(arg_size);  // restore arguments
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   211
    kit.uncommon_trap(Deoptimization::Reason_null_check,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   212
                      Deoptimization::Action_none,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   213
                      NULL, "null receiver");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   214
    return kit.transfer_exceptions_into_jvms();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   215
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   216
489c9b5090e2 Initial load
duke
parents:
diff changeset
   217
  // Ideally we would unconditionally do a null check here and let it
489c9b5090e2 Initial load
duke
parents:
diff changeset
   218
  // be converted to an implicit check based on profile information.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   219
  // However currently the conversion to implicit null checks in
489c9b5090e2 Initial load
duke
parents:
diff changeset
   220
  // Block::implicit_null_check() only looks for loads and stores, not calls.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   221
  ciMethod *caller = kit.method();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   222
  ciMethodData *caller_md = (caller == NULL) ? NULL : caller->method_data();
22852
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 21582
diff changeset
   223
  if (!UseInlineCaches || !ImplicitNullChecks || !os::zero_page_read_protected() ||
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   224
       ((ImplicitNullCheckThreshold > 0) && caller_md &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
   225
       (caller_md->trap_count(Deoptimization::Reason_null_check)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   226
       >= (uint)ImplicitNullCheckThreshold))) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   227
    // Make an explicit receiver null_check as part of this call.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   228
    // Since we share a map with the caller, his JVMS gets adjusted.
14621
fd9265ab0f67 7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents: 14132
diff changeset
   229
    receiver = kit.null_check_receiver_before_call(method());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   230
    if (kit.stopped()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   231
      // And dump it back to the caller, decorated with any exceptions:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   232
      return kit.transfer_exceptions_into_jvms();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   233
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   234
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   235
489c9b5090e2 Initial load
duke
parents:
diff changeset
   236
  assert(!method()->is_static(), "virtual call must not be to static");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   237
  assert(!method()->is_final(), "virtual call should not be to final");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   238
  assert(!method()->is_private(), "virtual call should not be to private");
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13487
diff changeset
   239
  assert(_vtable_index == Method::invalid_vtable_index || !UseInlineCaches,
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   240
         "no vtable calls if +UseInlineCaches ");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   241
  address target = SharedRuntime::get_resolve_virtual_call_stub();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   242
  // Normal inline cache used for call
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
   243
  CallDynamicJavaNode *call = new CallDynamicJavaNode(tf(), target, method(), _vtable_index, kit.bci());
43943
e2fdae30bbc1 8166110: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents: 40882
diff changeset
   244
  if (is_inlined_method_handle_intrinsic(jvms, method())) {
35086
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 33589
diff changeset
   245
    // To be able to issue a direct call (optimized virtual or virtual)
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 33589
diff changeset
   246
    // and skip a call to MH.linkTo*/invokeBasic adapter, additional information
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 33589
diff changeset
   247
    // about the method being invoked should be attached to the call site to
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 33589
diff changeset
   248
    // make resolution logic work (see SharedRuntime::resolve_{virtual,opt_virtual}_call_C).
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 33589
diff changeset
   249
    call->set_override_symbolic_info(true);
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 33589
diff changeset
   250
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   251
  kit.set_arguments_for_java_call(call);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   252
  kit.set_edges_for_java_call(call);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   253
  Node* ret = kit.set_results_for_java_call(call);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   254
  kit.push_node(method()->return_type()->basic_type(), ret);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   255
489c9b5090e2 Initial load
duke
parents:
diff changeset
   256
  // Represent the effect of an implicit receiver null_check
489c9b5090e2 Initial load
duke
parents:
diff changeset
   257
  // as part of this call.  Since we share a map with the caller,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   258
  // his JVMS gets adjusted.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   259
  kit.cast_not_null(receiver);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   260
  return kit.transfer_exceptions_into_jvms();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   261
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   262
489c9b5090e2 Initial load
duke
parents:
diff changeset
   263
CallGenerator* CallGenerator::for_inline(ciMethod* m, float expected_uses) {
10509
43d670e5701e 7079673: JSR 292: C1 should inline bytecoded method handle adapters
twisti
parents: 10503
diff changeset
   264
  if (InlineTree::check_can_parse(m) != NULL)  return NULL;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   265
  return new ParseGenerator(m, expected_uses);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   266
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   267
489c9b5090e2 Initial load
duke
parents:
diff changeset
   268
// As a special case, the JVMS passed to this CallGenerator is
489c9b5090e2 Initial load
duke
parents:
diff changeset
   269
// for the method execution already in progress, not just the JVMS
489c9b5090e2 Initial load
duke
parents:
diff changeset
   270
// of the caller.  Thus, this CallGenerator cannot be mixed with others!
489c9b5090e2 Initial load
duke
parents:
diff changeset
   271
CallGenerator* CallGenerator::for_osr(ciMethod* m, int osr_bci) {
10509
43d670e5701e 7079673: JSR 292: C1 should inline bytecoded method handle adapters
twisti
parents: 10503
diff changeset
   272
  if (InlineTree::check_can_parse(m) != NULL)  return NULL;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   273
  float past_uses = m->interpreter_invocation_count();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   274
  float expected_uses = past_uses;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   275
  return new ParseGenerator(m, expected_uses, true);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   276
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   277
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   278
CallGenerator* CallGenerator::for_direct_call(ciMethod* m, bool separate_io_proj) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   279
  assert(!m->is_abstract(), "for_direct_call mismatch");
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   280
  return new DirectCallGenerator(m, separate_io_proj);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   281
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   282
489c9b5090e2 Initial load
duke
parents:
diff changeset
   283
CallGenerator* CallGenerator::for_virtual_call(ciMethod* m, int vtable_index) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   284
  assert(!m->is_static(), "for_virtual_call mismatch");
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   285
  assert(!m->is_method_handle_intrinsic(), "should be a direct call");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   286
  return new VirtualCallGenerator(m, vtable_index);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   287
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   288
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   289
// Allow inlining decisions to be delayed
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   290
class LateInlineCallGenerator : public DirectCallGenerator {
24002
4e6a72032a99 8005079: fix LogCompilation for incremental inlining
roland
parents: 23846
diff changeset
   291
 private:
4e6a72032a99 8005079: fix LogCompilation for incremental inlining
roland
parents: 23846
diff changeset
   292
  // unique id for log compilation
4e6a72032a99 8005079: fix LogCompilation for incremental inlining
roland
parents: 23846
diff changeset
   293
  jlong _unique_id;
4e6a72032a99 8005079: fix LogCompilation for incremental inlining
roland
parents: 23846
diff changeset
   294
15113
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   295
 protected:
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   296
  CallGenerator* _inline_cg;
15113
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   297
  virtual bool do_late_inline_check(JVMState* jvms) { return true; }
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   298
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   299
 public:
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   300
  LateInlineCallGenerator(ciMethod* method, CallGenerator* inline_cg) :
24002
4e6a72032a99 8005079: fix LogCompilation for incremental inlining
roland
parents: 23846
diff changeset
   301
    DirectCallGenerator(method, true), _inline_cg(inline_cg), _unique_id(0) {}
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   302
23846
490ace57944d 8007988: PrintInlining output is inconsistent with incremental inlining
roland
parents: 23528
diff changeset
   303
  virtual bool is_late_inline() const { return true; }
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   304
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   305
  // Convert the CallStaticJava into an inline
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   306
  virtual void do_late_inline();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   307
24946
24b68ccf3fc4 8026796: Make replace_in_map() on parent maps generic
roland
parents: 24923
diff changeset
   308
  virtual JVMState* generate(JVMState* jvms) {
14828
bb9dffedf46c 8005031: Some cleanup in c2 to prepare for incremental inlining support
roland
parents: 14621
diff changeset
   309
    Compile *C = Compile::current();
bb9dffedf46c 8005031: Some cleanup in c2 to prepare for incremental inlining support
roland
parents: 14621
diff changeset
   310
24002
4e6a72032a99 8005079: fix LogCompilation for incremental inlining
roland
parents: 23846
diff changeset
   311
    C->log_inline_id(this);
4e6a72032a99 8005079: fix LogCompilation for incremental inlining
roland
parents: 23846
diff changeset
   312
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   313
    // Record that this call site should be revisited once the main
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   314
    // parse is finished.
15113
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   315
    if (!is_mh_late_inline()) {
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   316
      C->add_late_inline(this);
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   317
    }
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   318
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   319
    // Emit the CallStaticJava and request separate projections so
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   320
    // that the late inlining logic can distinguish between fall
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   321
    // through and exceptional uses of the memory and io projections
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   322
    // as is done for allocations and macro expansion.
24946
24b68ccf3fc4 8026796: Make replace_in_map() on parent maps generic
roland
parents: 24923
diff changeset
   323
    return DirectCallGenerator::generate(jvms);
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   324
  }
15113
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   325
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   326
  virtual void print_inlining_late(const char* msg) {
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   327
    CallNode* call = call_node();
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   328
    Compile* C = Compile::current();
23846
490ace57944d 8007988: PrintInlining output is inconsistent with incremental inlining
roland
parents: 23528
diff changeset
   329
    C->print_inlining_assert_ready();
15113
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   330
    C->print_inlining(method(), call->jvms()->depth()-1, call->jvms()->bci(), msg);
23846
490ace57944d 8007988: PrintInlining output is inconsistent with incremental inlining
roland
parents: 23528
diff changeset
   331
    C->print_inlining_move_to(this);
490ace57944d 8007988: PrintInlining output is inconsistent with incremental inlining
roland
parents: 23528
diff changeset
   332
    C->print_inlining_update_delayed(this);
15113
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   333
  }
24002
4e6a72032a99 8005079: fix LogCompilation for incremental inlining
roland
parents: 23846
diff changeset
   334
4e6a72032a99 8005079: fix LogCompilation for incremental inlining
roland
parents: 23846
diff changeset
   335
  virtual void set_unique_id(jlong id) {
4e6a72032a99 8005079: fix LogCompilation for incremental inlining
roland
parents: 23846
diff changeset
   336
    _unique_id = id;
4e6a72032a99 8005079: fix LogCompilation for incremental inlining
roland
parents: 23846
diff changeset
   337
  }
4e6a72032a99 8005079: fix LogCompilation for incremental inlining
roland
parents: 23846
diff changeset
   338
4e6a72032a99 8005079: fix LogCompilation for incremental inlining
roland
parents: 23846
diff changeset
   339
  virtual jlong unique_id() const {
4e6a72032a99 8005079: fix LogCompilation for incremental inlining
roland
parents: 23846
diff changeset
   340
    return _unique_id;
4e6a72032a99 8005079: fix LogCompilation for incremental inlining
roland
parents: 23846
diff changeset
   341
  }
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   342
};
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   343
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   344
void LateInlineCallGenerator::do_late_inline() {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   345
  // Can't inline it
17383
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   346
  CallStaticJavaNode* call = call_node();
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   347
  if (call == NULL || call->outcnt() == 0 ||
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   348
      call->in(0) == NULL || call->in(0)->is_top()) {
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   349
    return;
15478
6e6f37256157 8007144: Incremental inlining mistakes some call sites for dead ones and doesn't inline them
roland
parents: 15118
diff changeset
   350
  }
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   351
17383
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   352
  const TypeTuple *r = call->tf()->domain();
15113
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   353
  for (int i1 = 0; i1 < method()->arg_size(); i1++) {
17383
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   354
    if (call->in(TypeFunc::Parms + i1)->is_top() && r->field_at(TypeFunc::Parms + i1) != Type::HALF) {
15113
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   355
      assert(Compile::current()->inlining_incrementally(), "shouldn't happen during parsing");
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   356
      return;
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   357
    }
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   358
  }
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   359
17383
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   360
  if (call->in(TypeFunc::Memory)->is_top()) {
15113
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   361
    assert(Compile::current()->inlining_incrementally(), "shouldn't happen during parsing");
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   362
    return;
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   363
  }
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   364
17383
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   365
  Compile* C = Compile::current();
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   366
  // Remove inlined methods from Compiler's lists.
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   367
  if (call->is_macro()) {
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   368
    C->remove_macro_node(call);
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   369
  }
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   370
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   371
  // Make a clone of the JVMState that appropriate to use for driving a parse
17383
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   372
  JVMState* old_jvms = call->jvms();
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   373
  JVMState* jvms = old_jvms->clone_shallow(C);
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   374
  uint size = call->req();
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
   375
  SafePointNode* map = new SafePointNode(size, jvms);
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   376
  for (uint i1 = 0; i1 < size; i1++) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   377
    map->init_req(i1, call->in(i1));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   378
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   379
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   380
  // Make sure the state is a MergeMem for parsing.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   381
  if (!map->in(TypeFunc::Memory)->is_MergeMem()) {
25930
eae8b7490d2c 8054033: Remove unused references to Compile*
thartmann
parents: 25494
diff changeset
   382
    Node* mem = MergeMemNode::make(map->in(TypeFunc::Memory));
14828
bb9dffedf46c 8005031: Some cleanup in c2 to prepare for incremental inlining support
roland
parents: 14621
diff changeset
   383
    C->initial_gvn()->set_type_bottom(mem);
bb9dffedf46c 8005031: Some cleanup in c2 to prepare for incremental inlining support
roland
parents: 14621
diff changeset
   384
    map->set_req(TypeFunc::Memory, mem);
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   385
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   386
17383
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   387
  uint nargs = method()->arg_size();
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   388
  // blow away old call arguments
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   389
  Node* top = C->top();
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   390
  for (uint i1 = 0; i1 < nargs; i1++) {
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   391
    map->set_req(TypeFunc::Parms + i1, top);
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   392
  }
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   393
  jvms->set_map(map);
17383
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   394
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   395
  // Make enough space in the expression stack to transfer
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   396
  // the incoming arguments and return value.
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   397
  map->ensure_stack(jvms, jvms->method()->max_stack());
17383
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   398
  for (uint i1 = 0; i1 < nargs; i1++) {
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   399
    map->set_argument(jvms, i1, call->in(TypeFunc::Parms + i1));
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   400
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   401
23846
490ace57944d 8007988: PrintInlining output is inconsistent with incremental inlining
roland
parents: 23528
diff changeset
   402
  C->print_inlining_assert_ready();
490ace57944d 8007988: PrintInlining output is inconsistent with incremental inlining
roland
parents: 23528
diff changeset
   403
490ace57944d 8007988: PrintInlining output is inconsistent with incremental inlining
roland
parents: 23528
diff changeset
   404
  C->print_inlining_move_to(this);
490ace57944d 8007988: PrintInlining output is inconsistent with incremental inlining
roland
parents: 23528
diff changeset
   405
24002
4e6a72032a99 8005079: fix LogCompilation for incremental inlining
roland
parents: 23846
diff changeset
   406
  C->log_late_inline(this);
4e6a72032a99 8005079: fix LogCompilation for incremental inlining
roland
parents: 23846
diff changeset
   407
17383
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   408
  // This check is done here because for_method_handle_inline() method
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   409
  // needs jvms for inlined state.
15113
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   410
  if (!do_late_inline_check(jvms)) {
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   411
    map->disconnect_inputs(NULL, C);
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   412
    return;
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   413
  }
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   414
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   415
  // Setup default node notes to be picked up by the inlining
24479
20eb4c752fb5 8042786: Proper fix for 8032566
kvn
parents: 24002
diff changeset
   416
  Node_Notes* old_nn = C->node_notes_at(call->_idx);
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   417
  if (old_nn != NULL) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   418
    Node_Notes* entry_nn = old_nn->clone(C);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   419
    entry_nn->set_jvms(jvms);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   420
    C->set_default_node_notes(entry_nn);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   421
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   422
23846
490ace57944d 8007988: PrintInlining output is inconsistent with incremental inlining
roland
parents: 23528
diff changeset
   423
  // Now perform the inlining using the synthesized JVMState
24946
24b68ccf3fc4 8026796: Make replace_in_map() on parent maps generic
roland
parents: 24923
diff changeset
   424
  JVMState* new_jvms = _inline_cg->generate(jvms);
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   425
  if (new_jvms == NULL)  return;  // no change
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   426
  if (C->failing())      return;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   427
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   428
  // Capture any exceptional control flow
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   429
  GraphKit kit(new_jvms);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   430
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   431
  // Find the result object
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   432
  Node* result = C->top();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   433
  int   result_size = method()->return_type()->size();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   434
  if (result_size != 0 && !kit.stopped()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   435
    result = (result_size == 1) ? kit.pop() : kit.pop_pair();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   436
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   437
15113
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   438
  C->set_has_loops(C->has_loops() || _inline_cg->method()->has_loops());
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   439
  C->env()->notice_inlined_method(_inline_cg->method());
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   440
  C->set_inlining_progress(true);
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   441
24946
24b68ccf3fc4 8026796: Make replace_in_map() on parent maps generic
roland
parents: 24923
diff changeset
   442
  kit.replace_call(call, result, true);
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   443
}
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   444
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   445
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   446
CallGenerator* CallGenerator::for_late_inline(ciMethod* method, CallGenerator* inline_cg) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   447
  return new LateInlineCallGenerator(method, inline_cg);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   448
}
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   449
15113
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   450
class LateInlineMHCallGenerator : public LateInlineCallGenerator {
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   451
  ciMethod* _caller;
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   452
  int _attempt;
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   453
  bool _input_not_const;
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   454
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   455
  virtual bool do_late_inline_check(JVMState* jvms);
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   456
  virtual bool already_attempted() const { return _attempt > 0; }
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   457
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   458
 public:
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   459
  LateInlineMHCallGenerator(ciMethod* caller, ciMethod* callee, bool input_not_const) :
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   460
    LateInlineCallGenerator(callee, NULL), _caller(caller), _attempt(0), _input_not_const(input_not_const) {}
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   461
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   462
  virtual bool is_mh_late_inline() const { return true; }
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   463
24946
24b68ccf3fc4 8026796: Make replace_in_map() on parent maps generic
roland
parents: 24923
diff changeset
   464
  virtual JVMState* generate(JVMState* jvms) {
24b68ccf3fc4 8026796: Make replace_in_map() on parent maps generic
roland
parents: 24923
diff changeset
   465
    JVMState* new_jvms = LateInlineCallGenerator::generate(jvms);
23846
490ace57944d 8007988: PrintInlining output is inconsistent with incremental inlining
roland
parents: 23528
diff changeset
   466
24002
4e6a72032a99 8005079: fix LogCompilation for incremental inlining
roland
parents: 23846
diff changeset
   467
    Compile* C = Compile::current();
15113
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   468
    if (_input_not_const) {
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   469
      // inlining won't be possible so no need to enqueue right now.
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   470
      call_node()->set_generator(this);
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   471
    } else {
24002
4e6a72032a99 8005079: fix LogCompilation for incremental inlining
roland
parents: 23846
diff changeset
   472
      C->add_late_inline(this);
15113
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   473
    }
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   474
    return new_jvms;
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   475
  }
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   476
};
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   477
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   478
bool LateInlineMHCallGenerator::do_late_inline_check(JVMState* jvms) {
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   479
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   480
  CallGenerator* cg = for_method_handle_inline(jvms, _caller, method(), _input_not_const);
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   481
23846
490ace57944d 8007988: PrintInlining output is inconsistent with incremental inlining
roland
parents: 23528
diff changeset
   482
  Compile::current()->print_inlining_update_delayed(this);
490ace57944d 8007988: PrintInlining output is inconsistent with incremental inlining
roland
parents: 23528
diff changeset
   483
15113
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   484
  if (!_input_not_const) {
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   485
    _attempt++;
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   486
  }
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   487
35086
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 33589
diff changeset
   488
  if (cg != NULL && cg->is_inline()) {
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 33589
diff changeset
   489
    assert(!cg->is_late_inline(), "we're doing late inlining");
15113
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   490
    _inline_cg = cg;
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   491
    Compile::current()->dec_number_of_mh_late_inlines();
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   492
    return true;
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   493
  }
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   494
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   495
  call_node()->set_generator(this);
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   496
  return false;
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   497
}
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   498
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   499
CallGenerator* CallGenerator::for_mh_late_inline(ciMethod* caller, ciMethod* callee, bool input_not_const) {
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   500
  Compile::current()->inc_number_of_mh_late_inlines();
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   501
  CallGenerator* cg = new LateInlineMHCallGenerator(caller, callee, input_not_const);
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   502
  return cg;
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   503
}
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   504
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   505
class LateInlineStringCallGenerator : public LateInlineCallGenerator {
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   506
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   507
 public:
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   508
  LateInlineStringCallGenerator(ciMethod* method, CallGenerator* inline_cg) :
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   509
    LateInlineCallGenerator(method, inline_cg) {}
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   510
24946
24b68ccf3fc4 8026796: Make replace_in_map() on parent maps generic
roland
parents: 24923
diff changeset
   511
  virtual JVMState* generate(JVMState* jvms) {
15113
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   512
    Compile *C = Compile::current();
24002
4e6a72032a99 8005079: fix LogCompilation for incremental inlining
roland
parents: 23846
diff changeset
   513
4e6a72032a99 8005079: fix LogCompilation for incremental inlining
roland
parents: 23846
diff changeset
   514
    C->log_inline_id(this);
4e6a72032a99 8005079: fix LogCompilation for incremental inlining
roland
parents: 23846
diff changeset
   515
15113
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   516
    C->add_string_late_inline(this);
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   517
24946
24b68ccf3fc4 8026796: Make replace_in_map() on parent maps generic
roland
parents: 24923
diff changeset
   518
    JVMState* new_jvms =  DirectCallGenerator::generate(jvms);
15113
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   519
    return new_jvms;
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   520
  }
21099
46e6bbecd9e5 8024070: C2 needs some form of type speculation
roland
parents: 21089
diff changeset
   521
46e6bbecd9e5 8024070: C2 needs some form of type speculation
roland
parents: 21089
diff changeset
   522
  virtual bool is_string_late_inline() const { return true; }
15113
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   523
};
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   524
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   525
CallGenerator* CallGenerator::for_string_late_inline(ciMethod* method, CallGenerator* inline_cg) {
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   526
  return new LateInlineStringCallGenerator(method, inline_cg);
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   527
}
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   528
17383
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   529
class LateInlineBoxingCallGenerator : public LateInlineCallGenerator {
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   530
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   531
 public:
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   532
  LateInlineBoxingCallGenerator(ciMethod* method, CallGenerator* inline_cg) :
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   533
    LateInlineCallGenerator(method, inline_cg) {}
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   534
24946
24b68ccf3fc4 8026796: Make replace_in_map() on parent maps generic
roland
parents: 24923
diff changeset
   535
  virtual JVMState* generate(JVMState* jvms) {
17383
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   536
    Compile *C = Compile::current();
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   537
24002
4e6a72032a99 8005079: fix LogCompilation for incremental inlining
roland
parents: 23846
diff changeset
   538
    C->log_inline_id(this);
4e6a72032a99 8005079: fix LogCompilation for incremental inlining
roland
parents: 23846
diff changeset
   539
17383
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   540
    C->add_boxing_late_inline(this);
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   541
24946
24b68ccf3fc4 8026796: Make replace_in_map() on parent maps generic
roland
parents: 24923
diff changeset
   542
    JVMState* new_jvms =  DirectCallGenerator::generate(jvms);
17383
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   543
    return new_jvms;
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   544
  }
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   545
};
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   546
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   547
CallGenerator* CallGenerator::for_boxing_late_inline(ciMethod* method, CallGenerator* inline_cg) {
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   548
  return new LateInlineBoxingCallGenerator(method, inline_cg);
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 15478
diff changeset
   549
}
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   550
489c9b5090e2 Initial load
duke
parents:
diff changeset
   551
//---------------------------WarmCallGenerator--------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   552
// Internal class which handles initial deferral of inlining decisions.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   553
class WarmCallGenerator : public CallGenerator {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   554
  WarmCallInfo*   _call_info;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   555
  CallGenerator*  _if_cold;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   556
  CallGenerator*  _if_hot;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   557
  bool            _is_virtual;   // caches virtuality of if_cold
489c9b5090e2 Initial load
duke
parents:
diff changeset
   558
  bool            _is_inline;    // caches inline-ness of if_hot
489c9b5090e2 Initial load
duke
parents:
diff changeset
   559
489c9b5090e2 Initial load
duke
parents:
diff changeset
   560
public:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   561
  WarmCallGenerator(WarmCallInfo* ci,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   562
                    CallGenerator* if_cold,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   563
                    CallGenerator* if_hot)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   564
    : CallGenerator(if_cold->method())
489c9b5090e2 Initial load
duke
parents:
diff changeset
   565
  {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   566
    assert(method() == if_hot->method(), "consistent choices");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   567
    _call_info  = ci;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   568
    _if_cold    = if_cold;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   569
    _if_hot     = if_hot;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   570
    _is_virtual = if_cold->is_virtual();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   571
    _is_inline  = if_hot->is_inline();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   572
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   573
489c9b5090e2 Initial load
duke
parents:
diff changeset
   574
  virtual bool      is_inline() const           { return _is_inline; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   575
  virtual bool      is_virtual() const          { return _is_virtual; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   576
  virtual bool      is_deferred() const         { return true; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   577
24946
24b68ccf3fc4 8026796: Make replace_in_map() on parent maps generic
roland
parents: 24923
diff changeset
   578
  virtual JVMState* generate(JVMState* jvms);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   579
};
489c9b5090e2 Initial load
duke
parents:
diff changeset
   580
489c9b5090e2 Initial load
duke
parents:
diff changeset
   581
489c9b5090e2 Initial load
duke
parents:
diff changeset
   582
CallGenerator* CallGenerator::for_warm_call(WarmCallInfo* ci,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   583
                                            CallGenerator* if_cold,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   584
                                            CallGenerator* if_hot) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   585
  return new WarmCallGenerator(ci, if_cold, if_hot);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   586
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   587
24946
24b68ccf3fc4 8026796: Make replace_in_map() on parent maps generic
roland
parents: 24923
diff changeset
   588
JVMState* WarmCallGenerator::generate(JVMState* jvms) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   589
  Compile* C = Compile::current();
23846
490ace57944d 8007988: PrintInlining output is inconsistent with incremental inlining
roland
parents: 23528
diff changeset
   590
  C->print_inlining_update(this);
490ace57944d 8007988: PrintInlining output is inconsistent with incremental inlining
roland
parents: 23528
diff changeset
   591
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   592
  if (C->log() != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   593
    C->log()->elem("warm_call bci='%d'", jvms->bci());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   594
  }
24946
24b68ccf3fc4 8026796: Make replace_in_map() on parent maps generic
roland
parents: 24923
diff changeset
   595
  jvms = _if_cold->generate(jvms);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   596
  if (jvms != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   597
    Node* m = jvms->map()->control();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   598
    if (m->is_CatchProj()) m = m->in(0);  else m = C->top();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   599
    if (m->is_Catch())     m = m->in(0);  else m = C->top();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   600
    if (m->is_Proj())      m = m->in(0);  else m = C->top();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   601
    if (m->is_CallJava()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   602
      _call_info->set_call(m->as_Call());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   603
      _call_info->set_hot_cg(_if_hot);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   604
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   605
      if (PrintOpto || PrintOptoInlining) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   606
        tty->print_cr("Queueing for warm inlining at bci %d:", jvms->bci());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   607
        tty->print("WCI: ");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   608
        _call_info->print();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   609
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   610
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   611
      _call_info->set_heat(_call_info->compute_heat());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   612
      C->set_warm_calls(_call_info->insert_into(C->warm_calls()));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   613
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   614
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   615
  return jvms;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   616
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   617
489c9b5090e2 Initial load
duke
parents:
diff changeset
   618
void WarmCallInfo::make_hot() {
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   619
  Unimplemented();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   620
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   621
489c9b5090e2 Initial load
duke
parents:
diff changeset
   622
void WarmCallInfo::make_cold() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   623
  // No action:  Just dequeue.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   624
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   625
489c9b5090e2 Initial load
duke
parents:
diff changeset
   626
489c9b5090e2 Initial load
duke
parents:
diff changeset
   627
//------------------------PredictedCallGenerator------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   628
// Internal class which handles all out-of-line calls checking receiver type.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   629
class PredictedCallGenerator : public CallGenerator {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   630
  ciKlass*       _predicted_receiver;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   631
  CallGenerator* _if_missed;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   632
  CallGenerator* _if_hit;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   633
  float          _hit_prob;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   634
489c9b5090e2 Initial load
duke
parents:
diff changeset
   635
public:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   636
  PredictedCallGenerator(ciKlass* predicted_receiver,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   637
                         CallGenerator* if_missed,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   638
                         CallGenerator* if_hit, float hit_prob)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   639
    : CallGenerator(if_missed->method())
489c9b5090e2 Initial load
duke
parents:
diff changeset
   640
  {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   641
    // The call profile data may predict the hit_prob as extreme as 0 or 1.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   642
    // Remove the extremes values from the range.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   643
    if (hit_prob > PROB_MAX)   hit_prob = PROB_MAX;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   644
    if (hit_prob < PROB_MIN)   hit_prob = PROB_MIN;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   645
489c9b5090e2 Initial load
duke
parents:
diff changeset
   646
    _predicted_receiver = predicted_receiver;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   647
    _if_missed          = if_missed;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   648
    _if_hit             = if_hit;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   649
    _hit_prob           = hit_prob;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   650
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   651
489c9b5090e2 Initial load
duke
parents:
diff changeset
   652
  virtual bool      is_virtual()   const    { return true; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   653
  virtual bool      is_inline()    const    { return _if_hit->is_inline(); }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   654
  virtual bool      is_deferred()  const    { return _if_hit->is_deferred(); }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   655
24946
24b68ccf3fc4 8026796: Make replace_in_map() on parent maps generic
roland
parents: 24923
diff changeset
   656
  virtual JVMState* generate(JVMState* jvms);
1
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
CallGenerator* CallGenerator::for_predicted_call(ciKlass* predicted_receiver,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   661
                                                 CallGenerator* if_missed,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   662
                                                 CallGenerator* if_hit,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   663
                                                 float hit_prob) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   664
  return new PredictedCallGenerator(predicted_receiver, if_missed, if_hit, hit_prob);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   665
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   666
489c9b5090e2 Initial load
duke
parents:
diff changeset
   667
24946
24b68ccf3fc4 8026796: Make replace_in_map() on parent maps generic
roland
parents: 24923
diff changeset
   668
JVMState* PredictedCallGenerator::generate(JVMState* jvms) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   669
  GraphKit kit(jvms);
23846
490ace57944d 8007988: PrintInlining output is inconsistent with incremental inlining
roland
parents: 23528
diff changeset
   670
  kit.C->print_inlining_update(this);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   671
  PhaseGVN& gvn = kit.gvn();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   672
  // We need an explicit receiver null_check before checking its type.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   673
  // We share a map with the caller, so his JVMS gets adjusted.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   674
  Node* receiver = kit.argument(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   675
  CompileLog* log = kit.C->log();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   676
  if (log != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   677
    log->elem("predicted_call bci='%d' klass='%d'",
489c9b5090e2 Initial load
duke
parents:
diff changeset
   678
              jvms->bci(), log->identify(_predicted_receiver));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   679
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   680
14621
fd9265ab0f67 7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents: 14132
diff changeset
   681
  receiver = kit.null_check_receiver_before_call(method());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   682
  if (kit.stopped()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   683
    return kit.transfer_exceptions_into_jvms();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   684
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   685
24946
24b68ccf3fc4 8026796: Make replace_in_map() on parent maps generic
roland
parents: 24923
diff changeset
   686
  // Make a copy of the replaced nodes in case we need to restore them
24b68ccf3fc4 8026796: Make replace_in_map() on parent maps generic
roland
parents: 24923
diff changeset
   687
  ReplacedNodes replaced_nodes = kit.map()->replaced_nodes();
24b68ccf3fc4 8026796: Make replace_in_map() on parent maps generic
roland
parents: 24923
diff changeset
   688
  replaced_nodes.clone();
24b68ccf3fc4 8026796: Make replace_in_map() on parent maps generic
roland
parents: 24923
diff changeset
   689
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   690
  Node* exact_receiver = receiver;  // will get updated in place...
489c9b5090e2 Initial load
duke
parents:
diff changeset
   691
  Node* slow_ctl = kit.type_check_receiver(receiver,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   692
                                           _predicted_receiver, _hit_prob,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   693
                                           &exact_receiver);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   694
489c9b5090e2 Initial load
duke
parents:
diff changeset
   695
  SafePointNode* slow_map = NULL;
33589
7cbd1b2c139b 8139040: Fix initializations before ShouldNotReachHere() etc. and enable -Wuninitialized on linux.
goetz
parents: 33105
diff changeset
   696
  JVMState* slow_jvms = NULL;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   697
  { PreserveJVMState pjvms(&kit);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   698
    kit.set_control(slow_ctl);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   699
    if (!kit.stopped()) {
24946
24b68ccf3fc4 8026796: Make replace_in_map() on parent maps generic
roland
parents: 24923
diff changeset
   700
      slow_jvms = _if_missed->generate(kit.sync_jvms());
11193
d8de495d05e0 7108383: JSR 292: JRuby bench_define_method_methods.rb: assert(slow_jvms != NULL) failed: miss path must not
twisti
parents: 10986
diff changeset
   701
      if (kit.failing())
d8de495d05e0 7108383: JSR 292: JRuby bench_define_method_methods.rb: assert(slow_jvms != NULL) failed: miss path must not
twisti
parents: 10986
diff changeset
   702
        return NULL;  // might happen because of NodeCountInliningCutoff
d8de495d05e0 7108383: JSR 292: JRuby bench_define_method_methods.rb: assert(slow_jvms != NULL) failed: miss path must not
twisti
parents: 10986
diff changeset
   703
      assert(slow_jvms != NULL, "must be");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   704
      kit.add_exception_states_from(slow_jvms);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   705
      kit.set_map(slow_jvms->map());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   706
      if (!kit.stopped())
489c9b5090e2 Initial load
duke
parents:
diff changeset
   707
        slow_map = kit.stop();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   708
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   709
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   710
1055
f4fb9fb08038 6731641: assert(m->adr_type() == mach->adr_type(),"matcher should not change adr type")
kvn
parents: 1
diff changeset
   711
  if (kit.stopped()) {
f4fb9fb08038 6731641: assert(m->adr_type() == mach->adr_type(),"matcher should not change adr type")
kvn
parents: 1
diff changeset
   712
    // Instance exactly does not matches the desired type.
f4fb9fb08038 6731641: assert(m->adr_type() == mach->adr_type(),"matcher should not change adr type")
kvn
parents: 1
diff changeset
   713
    kit.set_jvms(slow_jvms);
f4fb9fb08038 6731641: assert(m->adr_type() == mach->adr_type(),"matcher should not change adr type")
kvn
parents: 1
diff changeset
   714
    return kit.transfer_exceptions_into_jvms();
f4fb9fb08038 6731641: assert(m->adr_type() == mach->adr_type(),"matcher should not change adr type")
kvn
parents: 1
diff changeset
   715
  }
f4fb9fb08038 6731641: assert(m->adr_type() == mach->adr_type(),"matcher should not change adr type")
kvn
parents: 1
diff changeset
   716
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   717
  // fall through if the instance exactly matches the desired type
489c9b5090e2 Initial load
duke
parents:
diff changeset
   718
  kit.replace_in_map(receiver, exact_receiver);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   719
489c9b5090e2 Initial load
duke
parents:
diff changeset
   720
  // Make the hot call:
24946
24b68ccf3fc4 8026796: Make replace_in_map() on parent maps generic
roland
parents: 24923
diff changeset
   721
  JVMState* new_jvms = _if_hit->generate(kit.sync_jvms());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   722
  if (new_jvms == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   723
    // Inline failed, so make a direct call.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   724
    assert(_if_hit->is_inline(), "must have been a failed inline");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   725
    CallGenerator* cg = CallGenerator::for_direct_call(_if_hit->method());
24946
24b68ccf3fc4 8026796: Make replace_in_map() on parent maps generic
roland
parents: 24923
diff changeset
   726
    new_jvms = cg->generate(kit.sync_jvms());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   727
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   728
  kit.add_exception_states_from(new_jvms);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   729
  kit.set_jvms(new_jvms);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   730
489c9b5090e2 Initial load
duke
parents:
diff changeset
   731
  // Need to merge slow and fast?
489c9b5090e2 Initial load
duke
parents:
diff changeset
   732
  if (slow_map == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   733
    // The fast path is the only path remaining.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   734
    return kit.transfer_exceptions_into_jvms();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   735
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   736
489c9b5090e2 Initial load
duke
parents:
diff changeset
   737
  if (kit.stopped()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   738
    // Inlined method threw an exception, so it's just the slow path after all.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   739
    kit.set_jvms(slow_jvms);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   740
    return kit.transfer_exceptions_into_jvms();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   741
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   742
24946
24b68ccf3fc4 8026796: Make replace_in_map() on parent maps generic
roland
parents: 24923
diff changeset
   743
  // There are 2 branches and the replaced nodes are only valid on
24b68ccf3fc4 8026796: Make replace_in_map() on parent maps generic
roland
parents: 24923
diff changeset
   744
  // one: restore the replaced nodes to what they were before the
24b68ccf3fc4 8026796: Make replace_in_map() on parent maps generic
roland
parents: 24923
diff changeset
   745
  // branch.
24b68ccf3fc4 8026796: Make replace_in_map() on parent maps generic
roland
parents: 24923
diff changeset
   746
  kit.map()->set_replaced_nodes(replaced_nodes);
24b68ccf3fc4 8026796: Make replace_in_map() on parent maps generic
roland
parents: 24923
diff changeset
   747
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   748
  // Finish the diamond.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   749
  kit.C->set_has_split_ifs(true); // Has chance for split-if optimization
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
   750
  RegionNode* region = new RegionNode(3);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   751
  region->init_req(1, kit.control());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   752
  region->init_req(2, slow_map->control());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   753
  kit.set_control(gvn.transform(region));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   754
  Node* iophi = PhiNode::make(region, kit.i_o(), Type::ABIO);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   755
  iophi->set_req(2, slow_map->i_o());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   756
  kit.set_i_o(gvn.transform(iophi));
24948
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
   757
  // Merge memory
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   758
  kit.merge_memory(slow_map->merged_memory(), region, 2);
24948
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
   759
  // Transform new memory Phis.
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
   760
  for (MergeMemStream mms(kit.merged_memory()); mms.next_non_empty();) {
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
   761
    Node* phi = mms.memory();
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
   762
    if (phi->is_Phi() && phi->in(0) == region) {
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
   763
      mms.set_memory(gvn.transform(phi));
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
   764
    }
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
   765
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   766
  uint tos = kit.jvms()->stkoff() + kit.sp();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   767
  uint limit = slow_map->req();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   768
  for (uint i = TypeFunc::Parms; i < limit; i++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   769
    // Skip unused stack slots; fast forward to monoff();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   770
    if (i == tos) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   771
      i = kit.jvms()->monoff();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   772
      if( i >= limit ) break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   773
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   774
    Node* m = kit.map()->in(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   775
    Node* n = slow_map->in(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   776
    if (m != n) {
22799
83e58bac7980 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 22234
diff changeset
   777
      const Type* t = gvn.type(m)->meet_speculative(gvn.type(n));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   778
      Node* phi = PhiNode::make(region, m, t);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   779
      phi->set_req(2, n);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   780
      kit.map()->set_req(i, gvn.transform(phi));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   781
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   782
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   783
  return kit.transfer_exceptions_into_jvms();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   784
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   785
489c9b5090e2 Initial load
duke
parents:
diff changeset
   786
15113
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   787
CallGenerator* CallGenerator::for_method_handle_call(JVMState* jvms, ciMethod* caller, ciMethod* callee, bool delayed_forbidden) {
43943
e2fdae30bbc1 8166110: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents: 40882
diff changeset
   788
  assert(callee->is_method_handle_intrinsic(), "for_method_handle_call mismatch");
15113
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   789
  bool input_not_const;
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   790
  CallGenerator* cg = CallGenerator::for_method_handle_inline(jvms, caller, callee, input_not_const);
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   791
  Compile* C = Compile::current();
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   792
  if (cg != NULL) {
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   793
    if (!delayed_forbidden && AlwaysIncrementalInline) {
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   794
      return CallGenerator::for_late_inline(callee, cg);
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   795
    } else {
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   796
      return cg;
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   797
    }
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   798
  }
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   799
  int bci = jvms->bci();
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   800
  ciCallProfile profile = caller->call_profile_at_bci(bci);
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   801
  int call_site_count = caller->scale_count(profile.count());
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   802
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   803
  if (IncrementalInline && call_site_count > 0 &&
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   804
      (input_not_const || !C->inlining_incrementally() || C->over_inlining_cutoff())) {
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   805
    return CallGenerator::for_mh_late_inline(caller, callee, input_not_const);
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   806
  } else {
15118
1a1a6d1dfaab 8005418: JSR 292: virtual dispatch bug in 292 impl
twisti
parents: 15113
diff changeset
   807
    // Out-of-line call.
15113
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   808
    return CallGenerator::for_direct_call(callee);
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   809
  }
11193
d8de495d05e0 7108383: JSR 292: JRuby bench_define_method_methods.rb: assert(slow_jvms != NULL) failed: miss path must not
twisti
parents: 10986
diff changeset
   810
}
d8de495d05e0 7108383: JSR 292: JRuby bench_define_method_methods.rb: assert(slow_jvms != NULL) failed: miss path must not
twisti
parents: 10986
diff changeset
   811
15113
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   812
CallGenerator* CallGenerator::for_method_handle_inline(JVMState* jvms, ciMethod* caller, ciMethod* callee, bool& input_not_const) {
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   813
  GraphKit kit(jvms);
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   814
  PhaseGVN& gvn = kit.gvn();
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   815
  Compile* C = kit.C;
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   816
  vmIntrinsics::ID iid = callee->intrinsic_id();
15113
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   817
  input_not_const = true;
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   818
  switch (iid) {
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   819
  case vmIntrinsics::_invokeBasic:
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   820
    {
14621
fd9265ab0f67 7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents: 14132
diff changeset
   821
      // Get MethodHandle receiver:
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   822
      Node* receiver = kit.argument(0);
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   823
      if (receiver->Opcode() == Op_ConP) {
15113
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   824
        input_not_const = false;
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   825
        const TypeOopPtr* oop_ptr = receiver->bottom_type()->is_oopptr();
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   826
        ciMethod* target = oop_ptr->const_oop()->as_method_handle()->get_vmtarget();
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13487
diff changeset
   827
        const int vtable_index = Method::invalid_vtable_index;
43943
e2fdae30bbc1 8166110: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents: 40882
diff changeset
   828
43947
a52ee13998f3 8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents: 43943
diff changeset
   829
        if (!ciMethod::is_consistent_info(callee, target)) {
43943
e2fdae30bbc1 8166110: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents: 40882
diff changeset
   830
          print_inlining_failure(C, callee, jvms->depth() - 1, jvms->bci(),
e2fdae30bbc1 8166110: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents: 40882
diff changeset
   831
                                 "signatures mismatch");
e2fdae30bbc1 8166110: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents: 40882
diff changeset
   832
          return NULL;
e2fdae30bbc1 8166110: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents: 40882
diff changeset
   833
        }
e2fdae30bbc1 8166110: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents: 40882
diff changeset
   834
36075
e1685e30beca 8148994: Replacing MH::invokeBasic with a direct call breaks LF customization
vlivanov
parents: 35846
diff changeset
   835
        CallGenerator* cg = C->call_generator(target, vtable_index,
e1685e30beca 8148994: Replacing MH::invokeBasic with a direct call breaks LF customization
vlivanov
parents: 35846
diff changeset
   836
                                              false /* call_does_dispatch */,
e1685e30beca 8148994: Replacing MH::invokeBasic with a direct call breaks LF customization
vlivanov
parents: 35846
diff changeset
   837
                                              jvms,
e1685e30beca 8148994: Replacing MH::invokeBasic with a direct call breaks LF customization
vlivanov
parents: 35846
diff changeset
   838
                                              true /* allow_inline */,
e1685e30beca 8148994: Replacing MH::invokeBasic with a direct call breaks LF customization
vlivanov
parents: 35846
diff changeset
   839
                                              PROB_ALWAYS);
35086
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 33589
diff changeset
   840
        return cg;
23846
490ace57944d 8007988: PrintInlining output is inconsistent with incremental inlining
roland
parents: 23528
diff changeset
   841
      } else {
43943
e2fdae30bbc1 8166110: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents: 40882
diff changeset
   842
        print_inlining_failure(C, callee, jvms->depth() - 1, jvms->bci(),
e2fdae30bbc1 8166110: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents: 40882
diff changeset
   843
                               "receiver not constant");
10514
e229a19078cf 7071307: MethodHandle bimorphic inlining should consider the frequency
never
parents: 10510
diff changeset
   844
      }
e229a19078cf 7071307: MethodHandle bimorphic inlining should consider the frequency
never
parents: 10510
diff changeset
   845
    }
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   846
    break;
10514
e229a19078cf 7071307: MethodHandle bimorphic inlining should consider the frequency
never
parents: 10510
diff changeset
   847
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   848
  case vmIntrinsics::_linkToVirtual:
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   849
  case vmIntrinsics::_linkToStatic:
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   850
  case vmIntrinsics::_linkToSpecial:
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   851
  case vmIntrinsics::_linkToInterface:
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   852
    {
14621
fd9265ab0f67 7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents: 14132
diff changeset
   853
      // Get MemberName argument:
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   854
      Node* member_name = kit.argument(callee->arg_size() - 1);
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   855
      if (member_name->Opcode() == Op_ConP) {
15113
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   856
        input_not_const = false;
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   857
        const TypeOopPtr* oop_ptr = member_name->bottom_type()->is_oopptr();
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   858
        ciMethod* target = oop_ptr->const_oop()->as_member_name()->get_vmtarget();
9975
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   859
43947
a52ee13998f3 8174721: C1: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents: 43943
diff changeset
   860
        if (!ciMethod::is_consistent_info(callee, target)) {
43943
e2fdae30bbc1 8166110: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents: 40882
diff changeset
   861
          print_inlining_failure(C, callee, jvms->depth() - 1, jvms->bci(),
e2fdae30bbc1 8166110: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents: 40882
diff changeset
   862
                                 "signatures mismatch");
e2fdae30bbc1 8166110: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents: 40882
diff changeset
   863
          return NULL;
e2fdae30bbc1 8166110: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents: 40882
diff changeset
   864
        }
e2fdae30bbc1 8166110: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents: 40882
diff changeset
   865
35086
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 33589
diff changeset
   866
        // In lambda forms we erase signature types to avoid resolving issues
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   867
        // involving class loaders.  When we optimize a method handle invoke
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   868
        // to a direct call we must cast the receiver and arguments to its
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   869
        // actual types.
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   870
        ciSignature* signature = target->signature();
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   871
        const int receiver_skip = target->is_static() ? 0 : 1;
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   872
        // Cast receiver to its type.
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   873
        if (!target->is_static()) {
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   874
          Node* arg = kit.argument(0);
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   875
          const TypeOopPtr* arg_type = arg->bottom_type()->isa_oopptr();
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   876
          const Type*       sig_type = TypeOopPtr::make_from_klass(signature->accessing_klass());
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   877
          if (arg_type != NULL && !arg_type->higher_equal(sig_type)) {
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
   878
            Node* cast_obj = gvn.transform(new CheckCastPPNode(kit.control(), arg, sig_type));
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   879
            kit.set_argument(0, cast_obj);
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   880
          }
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   881
        }
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   882
        // Cast reference arguments to its type.
35846
f5a8b47778ef 8148752: Compiled StringBuilder code throws StringIndexOutOfBoundsException
thartmann
parents: 35086
diff changeset
   883
        for (int i = 0, j = 0; i < signature->count(); i++) {
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   884
          ciType* t = signature->type_at(i);
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   885
          if (t->is_klass()) {
35846
f5a8b47778ef 8148752: Compiled StringBuilder code throws StringIndexOutOfBoundsException
thartmann
parents: 35086
diff changeset
   886
            Node* arg = kit.argument(receiver_skip + j);
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   887
            const TypeOopPtr* arg_type = arg->bottom_type()->isa_oopptr();
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   888
            const Type*       sig_type = TypeOopPtr::make_from_klass(t->as_klass());
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   889
            if (arg_type != NULL && !arg_type->higher_equal(sig_type)) {
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
   890
              Node* cast_obj = gvn.transform(new CheckCastPPNode(kit.control(), arg, sig_type));
35846
f5a8b47778ef 8148752: Compiled StringBuilder code throws StringIndexOutOfBoundsException
thartmann
parents: 35086
diff changeset
   891
              kit.set_argument(receiver_skip + j, cast_obj);
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   892
            }
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   893
          }
35846
f5a8b47778ef 8148752: Compiled StringBuilder code throws StringIndexOutOfBoundsException
thartmann
parents: 35086
diff changeset
   894
          j += t->size();  // long and double take two slots
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   895
        }
15118
1a1a6d1dfaab 8005418: JSR 292: virtual dispatch bug in 292 impl
twisti
parents: 15113
diff changeset
   896
1a1a6d1dfaab 8005418: JSR 292: virtual dispatch bug in 292 impl
twisti
parents: 15113
diff changeset
   897
        // Try to get the most accurate receiver type
1a1a6d1dfaab 8005418: JSR 292: virtual dispatch bug in 292 impl
twisti
parents: 15113
diff changeset
   898
        const bool is_virtual              = (iid == vmIntrinsics::_linkToVirtual);
1a1a6d1dfaab 8005418: JSR 292: virtual dispatch bug in 292 impl
twisti
parents: 15113
diff changeset
   899
        const bool is_virtual_or_interface = (is_virtual || iid == vmIntrinsics::_linkToInterface);
1a1a6d1dfaab 8005418: JSR 292: virtual dispatch bug in 292 impl
twisti
parents: 15113
diff changeset
   900
        int  vtable_index       = Method::invalid_vtable_index;
1a1a6d1dfaab 8005418: JSR 292: virtual dispatch bug in 292 impl
twisti
parents: 15113
diff changeset
   901
        bool call_does_dispatch = false;
1a1a6d1dfaab 8005418: JSR 292: virtual dispatch bug in 292 impl
twisti
parents: 15113
diff changeset
   902
21099
46e6bbecd9e5 8024070: C2 needs some form of type speculation
roland
parents: 21089
diff changeset
   903
        ciKlass* speculative_receiver_type = NULL;
15118
1a1a6d1dfaab 8005418: JSR 292: virtual dispatch bug in 292 impl
twisti
parents: 15113
diff changeset
   904
        if (is_virtual_or_interface) {
1a1a6d1dfaab 8005418: JSR 292: virtual dispatch bug in 292 impl
twisti
parents: 15113
diff changeset
   905
          ciInstanceKlass* klass = target->holder();
1a1a6d1dfaab 8005418: JSR 292: virtual dispatch bug in 292 impl
twisti
parents: 15113
diff changeset
   906
          Node*             receiver_node = kit.argument(0);
1a1a6d1dfaab 8005418: JSR 292: virtual dispatch bug in 292 impl
twisti
parents: 15113
diff changeset
   907
          const TypeOopPtr* receiver_type = gvn.type(receiver_node)->isa_oopptr();
1a1a6d1dfaab 8005418: JSR 292: virtual dispatch bug in 292 impl
twisti
parents: 15113
diff changeset
   908
          // 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
   909
          // optimize_virtual_call() takes 2 different holder
c18718ecf276 8046542: [I.finalize() calls from methods compiled by C1 do not cause IllegalAccessError on Sparc
roland
parents: 24948
diff changeset
   910
          // arguments for a corner case that doesn't apply here (see
c18718ecf276 8046542: [I.finalize() calls from methods compiled by C1 do not cause IllegalAccessError on Sparc
roland
parents: 24948
diff changeset
   911
          // Parse::do_call())
c18718ecf276 8046542: [I.finalize() calls from methods compiled by C1 do not cause IllegalAccessError on Sparc
roland
parents: 24948
diff changeset
   912
          target = C->optimize_virtual_call(caller, jvms->bci(), klass, klass,
c18718ecf276 8046542: [I.finalize() calls from methods compiled by C1 do not cause IllegalAccessError on Sparc
roland
parents: 24948
diff changeset
   913
                                            target, receiver_type, is_virtual,
30223
82ab7b6b4927 8062280: C2: inlining failure due to access checks being too strict
vlivanov
parents: 29081
diff changeset
   914
                                            call_does_dispatch, vtable_index, // out-parameters
36075
e1685e30beca 8148994: Replacing MH::invokeBasic with a direct call breaks LF customization
vlivanov
parents: 35846
diff changeset
   915
                                            false /* check_access */);
21099
46e6bbecd9e5 8024070: C2 needs some form of type speculation
roland
parents: 21089
diff changeset
   916
          // We lack profiling at this call but type speculation may
46e6bbecd9e5 8024070: C2 needs some form of type speculation
roland
parents: 21089
diff changeset
   917
          // provide us with a type
26945
981df0f8be4e 8059556: C2: crash while inlining MethodHandle invocation w/ null receiver
vlivanov
parents: 25930
diff changeset
   918
          speculative_receiver_type = (receiver_type != NULL) ? receiver_type->speculative_type() : NULL;
15118
1a1a6d1dfaab 8005418: JSR 292: virtual dispatch bug in 292 impl
twisti
parents: 15113
diff changeset
   919
        }
36075
e1685e30beca 8148994: Replacing MH::invokeBasic with a direct call breaks LF customization
vlivanov
parents: 35846
diff changeset
   920
        CallGenerator* cg = C->call_generator(target, vtable_index, call_does_dispatch, jvms,
e1685e30beca 8148994: Replacing MH::invokeBasic with a direct call breaks LF customization
vlivanov
parents: 35846
diff changeset
   921
                                              true /* allow_inline */,
e1685e30beca 8148994: Replacing MH::invokeBasic with a direct call breaks LF customization
vlivanov
parents: 35846
diff changeset
   922
                                              PROB_ALWAYS,
e1685e30beca 8148994: Replacing MH::invokeBasic with a direct call breaks LF customization
vlivanov
parents: 35846
diff changeset
   923
                                              speculative_receiver_type);
35086
bbf32241d851 8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents: 33589
diff changeset
   924
        return cg;
23846
490ace57944d 8007988: PrintInlining output is inconsistent with incremental inlining
roland
parents: 23528
diff changeset
   925
      } else {
43943
e2fdae30bbc1 8166110: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents: 40882
diff changeset
   926
        print_inlining_failure(C, callee, jvms->depth() - 1, jvms->bci(),
e2fdae30bbc1 8166110: Inlining through MH invokers/linkers in unreachable code is unsafe
vlivanov
parents: 40882
diff changeset
   927
                               "member_name not constant");
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   928
      }
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   929
    }
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   930
    break;
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 9975
diff changeset
   931
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   932
  default:
33105
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 30223
diff changeset
   933
    fatal("unexpected intrinsic %d: %s", iid, vmIntrinsics::name_at(iid));
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 13107
diff changeset
   934
    break;
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 9975
diff changeset
   935
  }
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 9975
diff changeset
   936
  return NULL;
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 9975
diff changeset
   937
}
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 9975
diff changeset
   938
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 9975
diff changeset
   939
24948
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
   940
//------------------------PredicatedIntrinsicGenerator------------------------------
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
   941
// Internal class which handles all predicated Intrinsic calls.
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
   942
class PredicatedIntrinsicGenerator : public CallGenerator {
14132
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
   943
  CallGenerator* _intrinsic;
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
   944
  CallGenerator* _cg;
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
   945
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
   946
public:
24948
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
   947
  PredicatedIntrinsicGenerator(CallGenerator* intrinsic,
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
   948
                               CallGenerator* cg)
14132
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
   949
    : CallGenerator(cg->method())
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
   950
  {
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
   951
    _intrinsic = intrinsic;
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
   952
    _cg        = cg;
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
   953
  }
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
   954
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
   955
  virtual bool      is_virtual()   const    { return true; }
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
   956
  virtual bool      is_inlined()   const    { return true; }
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
   957
  virtual bool      is_intrinsic() const    { return true; }
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
   958
24946
24b68ccf3fc4 8026796: Make replace_in_map() on parent maps generic
roland
parents: 24923
diff changeset
   959
  virtual JVMState* generate(JVMState* jvms);
14132
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
   960
};
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
   961
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
   962
24948
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
   963
CallGenerator* CallGenerator::for_predicated_intrinsic(CallGenerator* intrinsic,
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
   964
                                                       CallGenerator* cg) {
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
   965
  return new PredicatedIntrinsicGenerator(intrinsic, cg);
14132
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
   966
}
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
   967
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
   968
24948
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
   969
JVMState* PredicatedIntrinsicGenerator::generate(JVMState* jvms) {
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
   970
  // The code we want to generate here is:
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
   971
  //    if (receiver == NULL)
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
   972
  //        uncommon_Trap
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
   973
  //    if (predicate(0))
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
   974
  //        do_intrinsic(0)
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
   975
  //    else
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
   976
  //    if (predicate(1))
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
   977
  //        do_intrinsic(1)
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
   978
  //    ...
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
   979
  //    else
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
   980
  //        do_java_comp
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
   981
14132
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
   982
  GraphKit kit(jvms);
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
   983
  PhaseGVN& gvn = kit.gvn();
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
   984
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
   985
  CompileLog* log = kit.C->log();
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
   986
  if (log != NULL) {
24948
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
   987
    log->elem("predicated_intrinsic bci='%d' method='%d'",
14132
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
   988
              jvms->bci(), log->identify(method()));
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
   989
  }
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
   990
24948
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
   991
  if (!method()->is_static()) {
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
   992
    // We need an explicit receiver null_check before checking its type in predicate.
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
   993
    // We share a map with the caller, so his JVMS gets adjusted.
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
   994
    Node* receiver = kit.null_check_receiver_before_call(method());
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
   995
    if (kit.stopped()) {
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
   996
      return kit.transfer_exceptions_into_jvms();
14132
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
   997
    }
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
   998
  }
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
   999
24948
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1000
  int n_predicates = _intrinsic->predicates_count();
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1001
  assert(n_predicates > 0, "sanity");
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1002
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1003
  JVMState** result_jvms = NEW_RESOURCE_ARRAY(JVMState*, (n_predicates+1));
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1004
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1005
  // Region for normal compilation code if intrinsic failed.
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1006
  Node* slow_region = new RegionNode(1);
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1007
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1008
  int results = 0;
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1009
  for (int predicate = 0; (predicate < n_predicates) && !kit.stopped(); predicate++) {
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1010
#ifdef ASSERT
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1011
    JVMState* old_jvms = kit.jvms();
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1012
    SafePointNode* old_map = kit.map();
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1013
    Node* old_io  = old_map->i_o();
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1014
    Node* old_mem = old_map->memory();
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1015
    Node* old_exc = old_map->next_exception();
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1016
#endif
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1017
    Node* else_ctrl = _intrinsic->generate_predicate(kit.sync_jvms(), predicate);
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1018
#ifdef ASSERT
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1019
    // Assert(no_new_memory && no_new_io && no_new_exceptions) after generate_predicate.
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1020
    assert(old_jvms == kit.jvms(), "generate_predicate should not change jvm state");
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1021
    SafePointNode* new_map = kit.map();
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1022
    assert(old_io  == new_map->i_o(), "generate_predicate should not change i_o");
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1023
    assert(old_mem == new_map->memory(), "generate_predicate should not change memory");
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1024
    assert(old_exc == new_map->next_exception(), "generate_predicate should not add exceptions");
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1025
#endif
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1026
    if (!kit.stopped()) {
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1027
      PreserveJVMState pjvms(&kit);
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1028
      // Generate intrinsic code:
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1029
      JVMState* new_jvms = _intrinsic->generate(kit.sync_jvms());
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1030
      if (new_jvms == NULL) {
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1031
        // Intrinsic failed, use normal compilation path for this predicate.
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1032
        slow_region->add_req(kit.control());
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1033
      } else {
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1034
        kit.add_exception_states_from(new_jvms);
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1035
        kit.set_jvms(new_jvms);
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1036
        if (!kit.stopped()) {
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1037
          result_jvms[results++] = kit.jvms();
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1038
        }
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1039
      }
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1040
    }
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1041
    if (else_ctrl == NULL) {
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1042
      else_ctrl = kit.C->top();
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1043
    }
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1044
    kit.set_control(else_ctrl);
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1045
  }
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1046
  if (!kit.stopped()) {
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1047
    // Final 'else' after predicates.
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1048
    slow_region->add_req(kit.control());
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1049
  }
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1050
  if (slow_region->req() > 1) {
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1051
    PreserveJVMState pjvms(&kit);
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1052
    // Generate normal compilation code:
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1053
    kit.set_control(gvn.transform(slow_region));
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1054
    JVMState* new_jvms = _cg->generate(kit.sync_jvms());
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1055
    if (kit.failing())
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1056
      return NULL;  // might happen because of NodeCountInliningCutoff
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1057
    assert(new_jvms != NULL, "must be");
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1058
    kit.add_exception_states_from(new_jvms);
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1059
    kit.set_jvms(new_jvms);
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1060
    if (!kit.stopped()) {
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1061
      result_jvms[results++] = kit.jvms();
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1062
    }
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1063
  }
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1064
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1065
  if (results == 0) {
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1066
    // All paths ended in uncommon traps.
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1067
    (void) kit.stop();
14132
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
  1068
    return kit.transfer_exceptions_into_jvms();
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
  1069
  }
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
  1070
24948
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1071
  if (results == 1) { // Only one path
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1072
    kit.set_jvms(result_jvms[0]);
14132
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
  1073
    return kit.transfer_exceptions_into_jvms();
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
  1074
  }
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
  1075
24948
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1076
  // Merge all paths.
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1077
  kit.C->set_has_split_ifs(true); // Has chance for split-if optimization
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1078
  RegionNode* region = new RegionNode(results + 1);
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1079
  Node* iophi = PhiNode::make(region, kit.i_o(), Type::ABIO);
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1080
  for (int i = 0; i < results; i++) {
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1081
    JVMState* jvms = result_jvms[i];
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1082
    int path = i + 1;
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1083
    SafePointNode* map = jvms->map();
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1084
    region->init_req(path, map->control());
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1085
    iophi->set_req(path, map->i_o());
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1086
    if (i == 0) {
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1087
      kit.set_jvms(jvms);
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1088
    } else {
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1089
      kit.merge_memory(map->merged_memory(), region, path);
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1090
    }
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1091
  }
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1092
  kit.set_control(gvn.transform(region));
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1093
  kit.set_i_o(gvn.transform(iophi));
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1094
  // Transform new memory Phis.
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1095
  for (MergeMemStream mms(kit.merged_memory()); mms.next_non_empty();) {
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1096
    Node* phi = mms.memory();
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1097
    if (phi->is_Phi() && phi->in(0) == region) {
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1098
      mms.set_memory(gvn.transform(phi));
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1099
    }
14132
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
  1100
  }
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
  1101
24948
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1102
  // Merge debug info.
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1103
  Node** ins = NEW_RESOURCE_ARRAY(Node*, results);
14132
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
  1104
  uint tos = kit.jvms()->stkoff() + kit.sp();
24948
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1105
  Node* map = kit.map();
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1106
  uint limit = map->req();
14132
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
  1107
  for (uint i = TypeFunc::Parms; i < limit; i++) {
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
  1108
    // Skip unused stack slots; fast forward to monoff();
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
  1109
    if (i == tos) {
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
  1110
      i = kit.jvms()->monoff();
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
  1111
      if( i >= limit ) break;
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
  1112
    }
24948
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1113
    Node* n = map->in(i);
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1114
    ins[0] = n;
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1115
    const Type* t = gvn.type(n);
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1116
    bool needs_phi = false;
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1117
    for (int j = 1; j < results; j++) {
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1118
      JVMState* jvms = result_jvms[j];
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1119
      Node* jmap = jvms->map();
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1120
      Node* m = NULL;
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1121
      if (jmap->req() > i) {
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1122
        m = jmap->in(i);
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1123
        if (m != n) {
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1124
          needs_phi = true;
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1125
          t = t->meet_speculative(gvn.type(m));
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1126
        }
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1127
      }
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1128
      ins[j] = m;
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1129
    }
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1130
    if (needs_phi) {
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1131
      Node* phi = PhiNode::make(region, n, t);
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1132
      for (int j = 1; j < results; j++) {
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1133
        phi->set_req(j + 1, ins[j]);
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1134
      }
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1135
      map->set_req(i, gvn.transform(phi));
14132
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
  1136
    }
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
  1137
  }
24948
bc074f400757 8035605: Expand functionality of PredictedIntrinsicGenerator
kvn
parents: 24946
diff changeset
  1138
14132
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
  1139
  return kit.transfer_exceptions_into_jvms();
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
  1140
}
3c1437abcefd 7184394: add intrinsics to use AES instructions
kvn
parents: 13895
diff changeset
  1141
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1142
//-------------------------UncommonTrapCallGenerator-----------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1143
// Internal class which handles all out-of-line calls checking receiver type.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1144
class UncommonTrapCallGenerator : public CallGenerator {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1145
  Deoptimization::DeoptReason _reason;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1146
  Deoptimization::DeoptAction _action;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1147
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1148
public:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1149
  UncommonTrapCallGenerator(ciMethod* m,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1150
                            Deoptimization::DeoptReason reason,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1151
                            Deoptimization::DeoptAction action)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1152
    : CallGenerator(m)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1153
  {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1154
    _reason = reason;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1155
    _action = action;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1156
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1157
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1158
  virtual bool      is_virtual() const          { ShouldNotReachHere(); return false; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1159
  virtual bool      is_trap() const             { return true; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1160
24946
24b68ccf3fc4 8026796: Make replace_in_map() on parent maps generic
roland
parents: 24923
diff changeset
  1161
  virtual JVMState* generate(JVMState* jvms);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1162
};
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1163
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1164
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1165
CallGenerator*
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1166
CallGenerator::for_uncommon_trap(ciMethod* m,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1167
                                 Deoptimization::DeoptReason reason,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1168
                                 Deoptimization::DeoptAction action) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1169
  return new UncommonTrapCallGenerator(m, reason, action);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1170
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1171
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1172
24946
24b68ccf3fc4 8026796: Make replace_in_map() on parent maps generic
roland
parents: 24923
diff changeset
  1173
JVMState* UncommonTrapCallGenerator::generate(JVMState* jvms) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1174
  GraphKit kit(jvms);
23846
490ace57944d 8007988: PrintInlining output is inconsistent with incremental inlining
roland
parents: 23528
diff changeset
  1175
  kit.C->print_inlining_update(this);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1176
  // Take the trap with arguments pushed on the stack.  (Cf. null_check_receiver).
40882
23f882a600ce 8158639: C2 compilation fails with SIGSEGV
jcm
parents: 36075
diff changeset
  1177
  // Callsite signature can be different from actual method being called (i.e _linkTo* sites).
23f882a600ce 8158639: C2 compilation fails with SIGSEGV
jcm
parents: 36075
diff changeset
  1178
  // Use callsite signature always.
23f882a600ce 8158639: C2 compilation fails with SIGSEGV
jcm
parents: 36075
diff changeset
  1179
  ciMethod* declared_method = kit.method()->get_method_at_bci(kit.bci());
23f882a600ce 8158639: C2 compilation fails with SIGSEGV
jcm
parents: 36075
diff changeset
  1180
  int nargs = declared_method->arg_size();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1181
  kit.inc_sp(nargs);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1182
  assert(nargs <= kit.sp() && kit.sp() <= jvms->stk_size(), "sane sp w/ args pushed");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1183
  if (_reason == Deoptimization::Reason_class_check &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1184
      _action == Deoptimization::Action_maybe_recompile) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1185
    // Temp fix for 6529811
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1186
    // Don't allow uncommon_trap to override our decision to recompile in the event
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1187
    // of a class cast failure for a monomorphic call as it will never let us convert
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1188
    // the call to either bi-morphic or megamorphic and can lead to unc-trap loops
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1189
    bool keep_exact_action = true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1190
    kit.uncommon_trap(_reason, _action, NULL, "monomorphic vcall checkcast", false, keep_exact_action);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1191
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1192
    kit.uncommon_trap(_reason, _action);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1193
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1194
  return kit.transfer_exceptions_into_jvms();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1195
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1196
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1197
// (Note:  Moved hook_up_call to GraphKit::set_edges_for_java_call.)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1198
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1199
// (Node:  Merged hook_up_exits into ParseGenerator::generate.)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1200
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1201
#define NODES_OVERHEAD_PER_METHOD (30.0)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1202
#define NODES_PER_BYTECODE (9.5)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1203
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1204
void WarmCallInfo::init(JVMState* call_site, ciMethod* call_method, ciCallProfile& profile, float prof_factor) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1205
  int call_count = profile.count();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1206
  int code_size = call_method->code_size();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1207
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1208
  // Expected execution count is based on the historical count:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1209
  _count = call_count < 0 ? 1 : call_site->method()->scale_count(call_count, prof_factor);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1210
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1211
  // Expected profit from inlining, in units of simple call-overheads.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1212
  _profit = 1.0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1213
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1214
  // Expected work performed by the call in units of call-overheads.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1215
  // %%% need an empirical curve fit for "work" (time in call)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1216
  float bytecodes_per_call = 3;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1217
  _work = 1.0 + code_size / bytecodes_per_call;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1218
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1219
  // Expected size of compilation graph:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1220
  // -XX:+PrintParseStatistics once reported:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1221
  //  Methods seen: 9184  Methods parsed: 9184  Nodes created: 1582391
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1222
  //  Histogram of 144298 parsed bytecodes:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1223
  // %%% Need an better predictor for graph size.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1224
  _size = NODES_OVERHEAD_PER_METHOD + (NODES_PER_BYTECODE * code_size);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1225
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1226
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1227
// is_cold:  Return true if the node should never be inlined.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1228
// This is true if any of the key metrics are extreme.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1229
bool WarmCallInfo::is_cold() const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1230
  if (count()  <  WarmCallMinCount)        return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1231
  if (profit() <  WarmCallMinProfit)       return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1232
  if (work()   >  WarmCallMaxWork)         return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1233
  if (size()   >  WarmCallMaxSize)         return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1234
  return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1235
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1236
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1237
// is_hot:  Return true if the node should be inlined immediately.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1238
// This is true if any of the key metrics are extreme.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1239
bool WarmCallInfo::is_hot() const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1240
  assert(!is_cold(), "eliminate is_cold cases before testing is_hot");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1241
  if (count()  >= HotCallCountThreshold)   return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1242
  if (profit() >= HotCallProfitThreshold)  return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1243
  if (work()   <= HotCallTrivialWork)      return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1244
  if (size()   <= HotCallTrivialSize)      return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1245
  return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1246
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1247
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1248
// compute_heat:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1249
float WarmCallInfo::compute_heat() const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1250
  assert(!is_cold(), "compute heat only on warm nodes");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1251
  assert(!is_hot(),  "compute heat only on warm nodes");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1252
  int min_size = MAX2(0,   (int)HotCallTrivialSize);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1253
  int max_size = MIN2(500, (int)WarmCallMaxSize);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1254
  float method_size = (size() - min_size) / MAX2(1, max_size - min_size);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1255
  float size_factor;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1256
  if      (method_size < 0.05)  size_factor = 4;   // 2 sigmas better than avg.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1257
  else if (method_size < 0.15)  size_factor = 2;   // 1 sigma better than avg.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1258
  else if (method_size < 0.5)   size_factor = 1;   // better than avg.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1259
  else                          size_factor = 0.5; // worse than avg.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1260
  return (count() * profit() * size_factor);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1261
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1262
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1263
bool WarmCallInfo::warmer_than(WarmCallInfo* that) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1264
  assert(this != that, "compare only different WCIs");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1265
  assert(this->heat() != 0 && that->heat() != 0, "call compute_heat 1st");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1266
  if (this->heat() > that->heat())   return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1267
  if (this->heat() < that->heat())   return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1268
  assert(this->heat() == that->heat(), "no NaN heat allowed");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1269
  // Equal heat.  Break the tie some other way.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1270
  if (!this->call() || !that->call())  return (address)this > (address)that;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1271
  return this->call()->_idx > that->call()->_idx;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1272
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1273
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1274
//#define UNINIT_NEXT ((WarmCallInfo*)badAddress)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1275
#define UNINIT_NEXT ((WarmCallInfo*)NULL)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1276
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1277
WarmCallInfo* WarmCallInfo::insert_into(WarmCallInfo* head) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1278
  assert(next() == UNINIT_NEXT, "not yet on any list");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1279
  WarmCallInfo* prev_p = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1280
  WarmCallInfo* next_p = head;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1281
  while (next_p != NULL && next_p->warmer_than(this)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1282
    prev_p = next_p;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1283
    next_p = prev_p->next();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1284
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1285
  // Install this between prev_p and next_p.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1286
  this->set_next(next_p);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1287
  if (prev_p == NULL)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1288
    head = this;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1289
  else
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1290
    prev_p->set_next(this);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1291
  return head;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1292
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1293
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1294
WarmCallInfo* WarmCallInfo::remove_from(WarmCallInfo* head) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1295
  WarmCallInfo* prev_p = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1296
  WarmCallInfo* next_p = head;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1297
  while (next_p != this) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1298
    assert(next_p != NULL, "this must be in the list somewhere");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1299
    prev_p = next_p;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1300
    next_p = prev_p->next();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1301
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1302
  next_p = this->next();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1303
  debug_only(this->set_next(UNINIT_NEXT));
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1304
  // Remove this from between prev_p and next_p.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1305
  if (prev_p == NULL)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1306
    head = next_p;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1307
  else
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1308
    prev_p->set_next(next_p);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1309
  return head;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1310
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1311
9099
bdeb610d3cb1 6909440: C2 fails with assertion (_always_cold->is_cold(),"must always be cold")
never
parents: 8676
diff changeset
  1312
WarmCallInfo WarmCallInfo::_always_hot(WarmCallInfo::MAX_VALUE(), WarmCallInfo::MAX_VALUE(),
bdeb610d3cb1 6909440: C2 fails with assertion (_always_cold->is_cold(),"must always be cold")
never
parents: 8676
diff changeset
  1313
                                       WarmCallInfo::MIN_VALUE(), WarmCallInfo::MIN_VALUE());
bdeb610d3cb1 6909440: C2 fails with assertion (_always_cold->is_cold(),"must always be cold")
never
parents: 8676
diff changeset
  1314
WarmCallInfo WarmCallInfo::_always_cold(WarmCallInfo::MIN_VALUE(), WarmCallInfo::MIN_VALUE(),
bdeb610d3cb1 6909440: C2 fails with assertion (_always_cold->is_cold(),"must always be cold")
never
parents: 8676
diff changeset
  1315
                                        WarmCallInfo::MAX_VALUE(), WarmCallInfo::MAX_VALUE());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1316
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1317
WarmCallInfo* WarmCallInfo::always_hot() {
9099
bdeb610d3cb1 6909440: C2 fails with assertion (_always_cold->is_cold(),"must always be cold")
never
parents: 8676
diff changeset
  1318
  assert(_always_hot.is_hot(), "must always be hot");
bdeb610d3cb1 6909440: C2 fails with assertion (_always_cold->is_cold(),"must always be cold")
never
parents: 8676
diff changeset
  1319
  return &_always_hot;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1320
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1321
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1322
WarmCallInfo* WarmCallInfo::always_cold() {
9099
bdeb610d3cb1 6909440: C2 fails with assertion (_always_cold->is_cold(),"must always be cold")
never
parents: 8676
diff changeset
  1323
  assert(_always_cold.is_cold(), "must always be cold");
bdeb610d3cb1 6909440: C2 fails with assertion (_always_cold->is_cold(),"must always be cold")
never
parents: 8676
diff changeset
  1324
  return &_always_cold;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1325
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1326
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1327
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1328
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1329
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1330
void WarmCallInfo::print() const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1331
  tty->print("%s : C=%6.1f P=%6.1f W=%6.1f S=%6.1f H=%6.1f -> %p",
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1332
             is_cold() ? "cold" : is_hot() ? "hot " : "warm",
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1333
             count(), profit(), work(), size(), compute_heat(), next());
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1334
  tty->cr();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1335
  if (call() != NULL)  call()->dump();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1336
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1337
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1338
void print_wci(WarmCallInfo* ci) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1339
  ci->print();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1340
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1341
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1342
void WarmCallInfo::print_all() const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1343
  for (const WarmCallInfo* p = this; p != NULL; p = p->next())
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1344
    p->print();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1345
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1346
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1347
int WarmCallInfo::count_all() const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1348
  int cnt = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1349
  for (const WarmCallInfo* p = this; p != NULL; p = p->next())
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1350
    cnt++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1351
  return cnt;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1352
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1353
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1354
#endif //PRODUCT