hotspot/src/share/vm/opto/callGenerator.cpp
author kvn
Tue, 26 Jun 2012 09:06:16 -0700
changeset 13111 e8a578a9b20a
parent 13107 fe2475f9b38a
child 13391 30245956af37
permissions -rw-r--r--
7179138: Incorrect result with String concatenation optimization Summary: check for and skip diamond shaped NULL check code for the result of toString() Reviewed-by: twisti, roland
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     1
/*
8676
9098d4e927e1 7012648: move JSR 292 to package java.lang.invoke and adjust names
jrose
parents: 7397
diff changeset
     2
 * Copyright (c) 2000, 2011, 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"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    28
#include "ci/ciCPCache.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    29
#include "ci/ciMethodHandle.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    30
#include "classfile/javaClasses.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    31
#include "compiler/compileLog.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    32
#include "opto/addnode.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    33
#include "opto/callGenerator.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    34
#include "opto/callnode.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    35
#include "opto/cfgnode.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    36
#include "opto/connode.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    37
#include "opto/parse.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    38
#include "opto/rootnode.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    39
#include "opto/runtime.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    40
#include "opto/subnode.hpp"
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    41
489c9b5090e2 Initial load
duke
parents:
diff changeset
    42
CallGenerator::CallGenerator(ciMethod* method) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    43
  _method = method;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    44
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    45
489c9b5090e2 Initial load
duke
parents:
diff changeset
    46
// Utility function.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    47
const TypeFunc* CallGenerator::tf() const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    48
  return TypeFunc::make(method());
489c9b5090e2 Initial load
duke
parents:
diff changeset
    49
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    50
489c9b5090e2 Initial load
duke
parents:
diff changeset
    51
//-----------------------------ParseGenerator---------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
    52
// Internal class which handles all direct bytecode traversal.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    53
class ParseGenerator : public InlineCallGenerator {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    54
private:
489c9b5090e2 Initial load
duke
parents:
diff changeset
    55
  bool  _is_osr;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    56
  float _expected_uses;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    57
489c9b5090e2 Initial load
duke
parents:
diff changeset
    58
public:
489c9b5090e2 Initial load
duke
parents:
diff changeset
    59
  ParseGenerator(ciMethod* method, float expected_uses, bool is_osr = false)
489c9b5090e2 Initial load
duke
parents:
diff changeset
    60
    : InlineCallGenerator(method)
489c9b5090e2 Initial load
duke
parents:
diff changeset
    61
  {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    62
    _is_osr        = is_osr;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    63
    _expected_uses = expected_uses;
10509
43d670e5701e 7079673: JSR 292: C1 should inline bytecoded method handle adapters
twisti
parents: 10503
diff changeset
    64
    assert(InlineTree::check_can_parse(method) == NULL, "parse must be possible");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    65
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    66
489c9b5090e2 Initial load
duke
parents:
diff changeset
    67
  virtual bool      is_parse() const           { return true; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    68
  virtual JVMState* generate(JVMState* jvms);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    69
  int is_osr() { return _is_osr; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    70
489c9b5090e2 Initial load
duke
parents:
diff changeset
    71
};
489c9b5090e2 Initial load
duke
parents:
diff changeset
    72
489c9b5090e2 Initial load
duke
parents:
diff changeset
    73
JVMState* ParseGenerator::generate(JVMState* jvms) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    74
  Compile* C = Compile::current();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    75
489c9b5090e2 Initial load
duke
parents:
diff changeset
    76
  if (is_osr()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    77
    // The JVMS for a OSR has a single argument (see its TypeFunc).
489c9b5090e2 Initial load
duke
parents:
diff changeset
    78
    assert(jvms->depth() == 1, "no inline OSR");
489c9b5090e2 Initial load
duke
parents:
diff changeset
    79
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    80
489c9b5090e2 Initial load
duke
parents:
diff changeset
    81
  if (C->failing()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    82
    return NULL;  // bailing out of the compile; do not try to parse
489c9b5090e2 Initial load
duke
parents:
diff changeset
    83
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    84
489c9b5090e2 Initial load
duke
parents:
diff changeset
    85
  Parse parser(jvms, method(), _expected_uses);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    86
  // Grab signature for matching/allocation
489c9b5090e2 Initial load
duke
parents:
diff changeset
    87
#ifdef ASSERT
489c9b5090e2 Initial load
duke
parents:
diff changeset
    88
  if (parser.tf() != (parser.depth() == 1 ? C->tf() : tf())) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    89
    MutexLockerEx ml(Compile_lock, Mutex::_no_safepoint_check_flag);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    90
    assert(C->env()->system_dictionary_modification_counter_changed(),
489c9b5090e2 Initial load
duke
parents:
diff changeset
    91
           "Must invalidate if TypeFuncs differ");
489c9b5090e2 Initial load
duke
parents:
diff changeset
    92
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    93
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
    94
489c9b5090e2 Initial load
duke
parents:
diff changeset
    95
  GraphKit& exits = parser.exits();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    96
489c9b5090e2 Initial load
duke
parents:
diff changeset
    97
  if (C->failing()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    98
    while (exits.pop_exception_state() != NULL) ;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    99
    return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   100
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   101
489c9b5090e2 Initial load
duke
parents:
diff changeset
   102
  assert(exits.jvms()->same_calls_as(jvms), "sanity");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   103
489c9b5090e2 Initial load
duke
parents:
diff changeset
   104
  // Simply return the exit state of the parser,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   105
  // augmented by any exceptional states.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   106
  return exits.transfer_exceptions_into_jvms();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   107
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   108
489c9b5090e2 Initial load
duke
parents:
diff changeset
   109
//---------------------------DirectCallGenerator------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   110
// Internal class which handles all out-of-line calls w/o receiver type checks.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   111
class DirectCallGenerator : public CallGenerator {
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   112
 private:
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   113
  CallStaticJavaNode* _call_node;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   114
  // Force separate memory and I/O projections for the exceptional
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   115
  // paths to facilitate late inlinig.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   116
  bool                _separate_io_proj;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   117
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   118
 public:
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   119
  DirectCallGenerator(ciMethod* method, bool separate_io_proj)
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   120
    : CallGenerator(method),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   121
      _separate_io_proj(separate_io_proj)
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   122
  {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   123
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   124
  virtual JVMState* generate(JVMState* jvms);
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   125
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   126
  CallStaticJavaNode* call_node() const { return _call_node; }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   127
};
489c9b5090e2 Initial load
duke
parents:
diff changeset
   128
489c9b5090e2 Initial load
duke
parents:
diff changeset
   129
JVMState* DirectCallGenerator::generate(JVMState* jvms) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   130
  GraphKit kit(jvms);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   131
  bool is_static = method()->is_static();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   132
  address target = is_static ? SharedRuntime::get_resolve_static_call_stub()
489c9b5090e2 Initial load
duke
parents:
diff changeset
   133
                             : SharedRuntime::get_resolve_opt_virtual_call_stub();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   134
489c9b5090e2 Initial load
duke
parents:
diff changeset
   135
  if (kit.C->log() != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   136
    kit.C->log()->elem("direct_call bci='%d'", jvms->bci());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   137
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   138
489c9b5090e2 Initial load
duke
parents:
diff changeset
   139
  CallStaticJavaNode *call = new (kit.C, tf()->domain()->cnt()) CallStaticJavaNode(tf(), target, method(), kit.bci());
12591
064b5c1aa88a 7162094: LateInlineCallGenerator::do_late_inline crashed on uninitialized _call_node
never
parents: 11193
diff changeset
   140
  _call_node = call;  // Save the call node in case we need it later
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   141
  if (!is_static) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   142
    // Make an explicit receiver null_check as part of this call.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   143
    // Since we share a map with the caller, his JVMS gets adjusted.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   144
    kit.null_check_receiver(method());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   145
    if (kit.stopped()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   146
      // And dump it back to the caller, decorated with any exceptions:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   147
      return kit.transfer_exceptions_into_jvms();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   148
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   149
    // Mark the call node as virtual, sort of:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   150
    call->set_optimized_virtual(true);
4906
90640ad2d401 6926782: CodeBuffer size too small after 6921352
twisti
parents: 4567
diff changeset
   151
    if (method()->is_method_handle_invoke()) {
4566
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   152
      call->set_method_handle_invoke(true);
4906
90640ad2d401 6926782: CodeBuffer size too small after 6921352
twisti
parents: 4567
diff changeset
   153
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   154
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   155
  kit.set_arguments_for_java_call(call);
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   156
  kit.set_edges_for_java_call(call, false, _separate_io_proj);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   157
  Node* ret = kit.set_results_for_java_call(call, _separate_io_proj);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   158
  kit.push_node(method()->return_type()->basic_type(), ret);
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
4566
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   162
//---------------------------DynamicCallGenerator-----------------------------
4567
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   163
// Internal class which handles all out-of-line invokedynamic calls.
4566
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   164
class DynamicCallGenerator : public CallGenerator {
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   165
public:
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   166
  DynamicCallGenerator(ciMethod* method)
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   167
    : CallGenerator(method)
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   168
  {
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   169
  }
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   170
  virtual JVMState* generate(JVMState* jvms);
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   171
};
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   172
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   173
JVMState* DynamicCallGenerator::generate(JVMState* jvms) {
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   174
  GraphKit kit(jvms);
13107
fe2475f9b38a 7157365: jruby/bench.bench_timeout crashes with JVM internal error
twisti
parents: 12591
diff changeset
   175
  Compile* C = kit.C;
fe2475f9b38a 7157365: jruby/bench.bench_timeout crashes with JVM internal error
twisti
parents: 12591
diff changeset
   176
  PhaseGVN& gvn = kit.gvn();
4566
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   177
13107
fe2475f9b38a 7157365: jruby/bench.bench_timeout crashes with JVM internal error
twisti
parents: 12591
diff changeset
   178
  if (C->log() != NULL) {
fe2475f9b38a 7157365: jruby/bench.bench_timeout crashes with JVM internal error
twisti
parents: 12591
diff changeset
   179
    C->log()->elem("dynamic_call bci='%d'", jvms->bci());
4566
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   180
  }
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   181
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   182
  // Get the constant pool cache from the caller class.
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   183
  ciMethod* caller_method = jvms->method();
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   184
  ciBytecodeStream str(caller_method);
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   185
  str.force_bci(jvms->bci());  // Set the stream to the invokedynamic bci.
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   186
  assert(str.cur_bc() == Bytecodes::_invokedynamic, "wrong place to issue a dynamic call!");
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   187
  ciCPCache* cpcache = str.get_cpcache();
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   188
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   189
  // Get the offset of the CallSite from the constant pool cache
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   190
  // pointer.
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   191
  int index = str.get_method_index();
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   192
  size_t call_site_offset = cpcache->get_f1_offset(index);
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   193
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   194
  // Load the CallSite object from the constant pool cache.
13107
fe2475f9b38a 7157365: jruby/bench.bench_timeout crashes with JVM internal error
twisti
parents: 12591
diff changeset
   195
  const TypeOopPtr* cpcache_type   = TypeOopPtr::make_from_constant(cpcache);  // returns TypeAryPtr of type T_OBJECT
fe2475f9b38a 7157365: jruby/bench.bench_timeout crashes with JVM internal error
twisti
parents: 12591
diff changeset
   196
  const TypeOopPtr* call_site_type = TypeOopPtr::make_from_klass(C->env()->CallSite_klass());
fe2475f9b38a 7157365: jruby/bench.bench_timeout crashes with JVM internal error
twisti
parents: 12591
diff changeset
   197
  Node* cpcache_adr   = kit.makecon(cpcache_type);
fe2475f9b38a 7157365: jruby/bench.bench_timeout crashes with JVM internal error
twisti
parents: 12591
diff changeset
   198
  Node* call_site_adr = kit.basic_plus_adr(cpcache_adr, call_site_offset);
fe2475f9b38a 7157365: jruby/bench.bench_timeout crashes with JVM internal error
twisti
parents: 12591
diff changeset
   199
  // The oops in the constant pool cache are not compressed; load then as raw pointers.
fe2475f9b38a 7157365: jruby/bench.bench_timeout crashes with JVM internal error
twisti
parents: 12591
diff changeset
   200
  Node* call_site     = kit.make_load(kit.control(), call_site_adr, call_site_type, T_ADDRESS, Compile::AliasIdxRaw);
4566
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   201
4567
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   202
  // Load the target MethodHandle from the CallSite object.
13107
fe2475f9b38a 7157365: jruby/bench.bench_timeout crashes with JVM internal error
twisti
parents: 12591
diff changeset
   203
  const TypeOopPtr* target_type = TypeOopPtr::make_from_klass(C->env()->MethodHandle_klass());
fe2475f9b38a 7157365: jruby/bench.bench_timeout crashes with JVM internal error
twisti
parents: 12591
diff changeset
   204
  Node* target_mh_adr = kit.basic_plus_adr(call_site, java_lang_invoke_CallSite::target_offset_in_bytes());
fe2475f9b38a 7157365: jruby/bench.bench_timeout crashes with JVM internal error
twisti
parents: 12591
diff changeset
   205
  Node* target_mh     = kit.make_load(kit.control(), target_mh_adr, target_type, T_OBJECT);
4566
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   206
4567
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   207
  address resolve_stub = SharedRuntime::get_resolve_opt_virtual_call_stub();
4566
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   208
13107
fe2475f9b38a 7157365: jruby/bench.bench_timeout crashes with JVM internal error
twisti
parents: 12591
diff changeset
   209
  CallStaticJavaNode* call = new (C, tf()->domain()->cnt()) CallStaticJavaNode(tf(), resolve_stub, method(), kit.bci());
4566
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   210
  // invokedynamic is treated as an optimized invokevirtual.
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   211
  call->set_optimized_virtual(true);
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   212
  // Take extra care (in the presence of argument motion) not to trash the SP:
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   213
  call->set_method_handle_invoke(true);
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   214
4567
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   215
  // Pass the target MethodHandle as first argument and shift the
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   216
  // other arguments.
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   217
  call->init_req(0 + TypeFunc::Parms, target_mh);
4566
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   218
  uint nargs = call->method()->arg_size();
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   219
  for (uint i = 1; i < nargs; i++) {
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   220
    Node* arg = kit.argument(i - 1);
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   221
    call->init_req(i + TypeFunc::Parms, arg);
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   222
  }
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   223
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   224
  kit.set_edges_for_java_call(call);
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   225
  Node* ret = kit.set_results_for_java_call(call);
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   226
  kit.push_node(method()->return_type()->basic_type(), ret);
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   227
  return kit.transfer_exceptions_into_jvms();
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   228
}
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   229
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   230
//--------------------------VirtualCallGenerator------------------------------
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   231
// Internal class which handles all out-of-line calls checking receiver type.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   232
class VirtualCallGenerator : public CallGenerator {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   233
private:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   234
  int _vtable_index;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   235
public:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   236
  VirtualCallGenerator(ciMethod* method, int vtable_index)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   237
    : CallGenerator(method), _vtable_index(vtable_index)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   238
  {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   239
    assert(vtable_index == methodOopDesc::invalid_vtable_index ||
489c9b5090e2 Initial load
duke
parents:
diff changeset
   240
           vtable_index >= 0, "either invalid or usable");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   241
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   242
  virtual bool      is_virtual() const          { return true; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   243
  virtual JVMState* generate(JVMState* jvms);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   244
};
489c9b5090e2 Initial load
duke
parents:
diff changeset
   245
489c9b5090e2 Initial load
duke
parents:
diff changeset
   246
JVMState* VirtualCallGenerator::generate(JVMState* jvms) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   247
  GraphKit kit(jvms);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   248
  Node* receiver = kit.argument(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   249
489c9b5090e2 Initial load
duke
parents:
diff changeset
   250
  if (kit.C->log() != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   251
    kit.C->log()->elem("virtual_call bci='%d'", jvms->bci());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   252
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   253
489c9b5090e2 Initial load
duke
parents:
diff changeset
   254
  // If the receiver is a constant null, do not torture the system
489c9b5090e2 Initial load
duke
parents:
diff changeset
   255
  // by attempting to call through it.  The compile will proceed
489c9b5090e2 Initial load
duke
parents:
diff changeset
   256
  // correctly, but may bail out in final_graph_reshaping, because
489c9b5090e2 Initial load
duke
parents:
diff changeset
   257
  // the call instruction will have a seemingly deficient out-count.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   258
  // (The bailout says something misleading about an "infinite loop".)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   259
  if (kit.gvn().type(receiver)->higher_equal(TypePtr::NULL_PTR)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   260
    kit.inc_sp(method()->arg_size());  // restore arguments
489c9b5090e2 Initial load
duke
parents:
diff changeset
   261
    kit.uncommon_trap(Deoptimization::Reason_null_check,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   262
                      Deoptimization::Action_none,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   263
                      NULL, "null receiver");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   264
    return kit.transfer_exceptions_into_jvms();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   265
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   266
489c9b5090e2 Initial load
duke
parents:
diff changeset
   267
  // Ideally we would unconditionally do a null check here and let it
489c9b5090e2 Initial load
duke
parents:
diff changeset
   268
  // be converted to an implicit check based on profile information.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   269
  // However currently the conversion to implicit null checks in
489c9b5090e2 Initial load
duke
parents:
diff changeset
   270
  // Block::implicit_null_check() only looks for loads and stores, not calls.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   271
  ciMethod *caller = kit.method();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   272
  ciMethodData *caller_md = (caller == NULL) ? NULL : caller->method_data();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   273
  if (!UseInlineCaches || !ImplicitNullChecks ||
489c9b5090e2 Initial load
duke
parents:
diff changeset
   274
       ((ImplicitNullCheckThreshold > 0) && caller_md &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
   275
       (caller_md->trap_count(Deoptimization::Reason_null_check)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   276
       >= (uint)ImplicitNullCheckThreshold))) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   277
    // Make an explicit receiver null_check as part of this call.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   278
    // Since we share a map with the caller, his JVMS gets adjusted.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   279
    receiver = kit.null_check_receiver(method());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   280
    if (kit.stopped()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   281
      // And dump it back to the caller, decorated with any exceptions:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   282
      return kit.transfer_exceptions_into_jvms();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   283
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   284
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   285
489c9b5090e2 Initial load
duke
parents:
diff changeset
   286
  assert(!method()->is_static(), "virtual call must not be to static");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   287
  assert(!method()->is_final(), "virtual call should not be to final");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   288
  assert(!method()->is_private(), "virtual call should not be to private");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   289
  assert(_vtable_index == methodOopDesc::invalid_vtable_index || !UseInlineCaches,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   290
         "no vtable calls if +UseInlineCaches ");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   291
  address target = SharedRuntime::get_resolve_virtual_call_stub();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   292
  // Normal inline cache used for call
489c9b5090e2 Initial load
duke
parents:
diff changeset
   293
  CallDynamicJavaNode *call = new (kit.C, tf()->domain()->cnt()) CallDynamicJavaNode(tf(), target, method(), _vtable_index, kit.bci());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   294
  kit.set_arguments_for_java_call(call);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   295
  kit.set_edges_for_java_call(call);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   296
  Node* ret = kit.set_results_for_java_call(call);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   297
  kit.push_node(method()->return_type()->basic_type(), ret);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   298
489c9b5090e2 Initial load
duke
parents:
diff changeset
   299
  // Represent the effect of an implicit receiver null_check
489c9b5090e2 Initial load
duke
parents:
diff changeset
   300
  // as part of this call.  Since we share a map with the caller,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   301
  // his JVMS gets adjusted.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   302
  kit.cast_not_null(receiver);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   303
  return kit.transfer_exceptions_into_jvms();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   304
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   305
489c9b5090e2 Initial load
duke
parents:
diff changeset
   306
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
   307
  if (InlineTree::check_can_parse(m) != NULL)  return NULL;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   308
  return new ParseGenerator(m, expected_uses);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   309
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   310
489c9b5090e2 Initial load
duke
parents:
diff changeset
   311
// As a special case, the JVMS passed to this CallGenerator is
489c9b5090e2 Initial load
duke
parents:
diff changeset
   312
// for the method execution already in progress, not just the JVMS
489c9b5090e2 Initial load
duke
parents:
diff changeset
   313
// of the caller.  Thus, this CallGenerator cannot be mixed with others!
489c9b5090e2 Initial load
duke
parents:
diff changeset
   314
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
   315
  if (InlineTree::check_can_parse(m) != NULL)  return NULL;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   316
  float past_uses = m->interpreter_invocation_count();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   317
  float expected_uses = past_uses;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   318
  return new ParseGenerator(m, expected_uses, true);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   319
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   320
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   321
CallGenerator* CallGenerator::for_direct_call(ciMethod* m, bool separate_io_proj) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   322
  assert(!m->is_abstract(), "for_direct_call mismatch");
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   323
  return new DirectCallGenerator(m, separate_io_proj);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   324
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   325
489c9b5090e2 Initial load
duke
parents:
diff changeset
   326
CallGenerator* CallGenerator::for_virtual_call(ciMethod* m, int vtable_index) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   327
  assert(!m->is_static(), "for_virtual_call mismatch");
4566
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4450
diff changeset
   328
  assert(!m->is_method_handle_invoke(), "should be a direct call");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   329
  return new VirtualCallGenerator(m, vtable_index);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   330
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   331
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
   332
CallGenerator* CallGenerator::for_dynamic_call(ciMethod* m) {
d8de495d05e0 7108383: JSR 292: JRuby bench_define_method_methods.rb: assert(slow_jvms != NULL) failed: miss path must not
twisti
parents: 10986
diff changeset
   333
  assert(m->is_method_handle_invoke() || m->is_method_handle_adapter(), "for_dynamic_call mismatch");
d8de495d05e0 7108383: JSR 292: JRuby bench_define_method_methods.rb: assert(slow_jvms != NULL) failed: miss path must not
twisti
parents: 10986
diff changeset
   334
  return new DynamicCallGenerator(m);
d8de495d05e0 7108383: JSR 292: JRuby bench_define_method_methods.rb: assert(slow_jvms != NULL) failed: miss path must not
twisti
parents: 10986
diff changeset
   335
}
d8de495d05e0 7108383: JSR 292: JRuby bench_define_method_methods.rb: assert(slow_jvms != NULL) failed: miss path must not
twisti
parents: 10986
diff changeset
   336
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   337
// Allow inlining decisions to be delayed
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   338
class LateInlineCallGenerator : public DirectCallGenerator {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   339
  CallGenerator* _inline_cg;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   340
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   341
 public:
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   342
  LateInlineCallGenerator(ciMethod* method, CallGenerator* inline_cg) :
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   343
    DirectCallGenerator(method, true), _inline_cg(inline_cg) {}
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   344
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   345
  virtual bool      is_late_inline() const { return true; }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   346
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   347
  // Convert the CallStaticJava into an inline
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   348
  virtual void do_late_inline();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   349
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   350
  JVMState* generate(JVMState* jvms) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   351
    // Record that this call site should be revisited once the main
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   352
    // parse is finished.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   353
    Compile::current()->add_late_inline(this);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   354
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   355
    // Emit the CallStaticJava and request separate projections so
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   356
    // that the late inlining logic can distinguish between fall
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   357
    // through and exceptional uses of the memory and io projections
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   358
    // as is done for allocations and macro expansion.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   359
    return DirectCallGenerator::generate(jvms);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   360
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   361
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   362
};
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   363
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   364
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   365
void LateInlineCallGenerator::do_late_inline() {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   366
  // Can't inline it
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   367
  if (call_node() == NULL || call_node()->outcnt() == 0 ||
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   368
      call_node()->in(0) == NULL || call_node()->in(0)->is_top())
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   369
    return;
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
  CallStaticJavaNode* call = call_node();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   372
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   373
  // Make a clone of the JVMState that appropriate to use for driving a parse
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   374
  Compile* C = Compile::current();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   375
  JVMState* jvms     = call->jvms()->clone_shallow(C);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   376
  uint size = call->req();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   377
  SafePointNode* map = new (C, size) SafePointNode(size, jvms);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   378
  for (uint i1 = 0; i1 < size; i1++) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   379
    map->init_req(i1, call->in(i1));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   380
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   381
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   382
  // Make sure the state is a MergeMem for parsing.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   383
  if (!map->in(TypeFunc::Memory)->is_MergeMem()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   384
    map->set_req(TypeFunc::Memory, MergeMemNode::make(C, map->in(TypeFunc::Memory)));
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
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   387
  // Make enough space for the expression stack and transfer the incoming arguments
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   388
  int nargs    = method()->arg_size();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   389
  jvms->set_map(map);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   390
  map->ensure_stack(jvms, jvms->method()->max_stack());
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   391
  if (nargs > 0) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   392
    for (int i1 = 0; i1 < nargs; i1++) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   393
      map->set_req(i1 + jvms->argoff(), call->in(TypeFunc::Parms + i1));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   394
    }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   395
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   396
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   397
  CompileLog* log = C->log();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   398
  if (log != NULL) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   399
    log->head("late_inline method='%d'", log->identify(method()));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   400
    JVMState* p = jvms;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   401
    while (p != NULL) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   402
      log->elem("jvms bci='%d' method='%d'", p->bci(), log->identify(p->method()));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   403
      p = p->caller();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   404
    }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   405
    log->tail("late_inline");
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   406
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   407
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   408
  // Setup default node notes to be picked up by the inlining
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   409
  Node_Notes* old_nn = C->default_node_notes();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   410
  if (old_nn != NULL) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   411
    Node_Notes* entry_nn = old_nn->clone(C);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   412
    entry_nn->set_jvms(jvms);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   413
    C->set_default_node_notes(entry_nn);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   414
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   415
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   416
  // Now perform the inling using the synthesized JVMState
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   417
  JVMState* new_jvms = _inline_cg->generate(jvms);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   418
  if (new_jvms == NULL)  return;  // no change
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   419
  if (C->failing())      return;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   420
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   421
  // Capture any exceptional control flow
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   422
  GraphKit kit(new_jvms);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   423
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   424
  // Find the result object
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   425
  Node* result = C->top();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   426
  int   result_size = method()->return_type()->size();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   427
  if (result_size != 0 && !kit.stopped()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   428
    result = (result_size == 1) ? kit.pop() : kit.pop_pair();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   429
  }
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
  kit.replace_call(call, result);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   432
}
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   433
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   434
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   435
CallGenerator* CallGenerator::for_late_inline(ciMethod* method, CallGenerator* inline_cg) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   436
  return new LateInlineCallGenerator(method, inline_cg);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   437
}
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   438
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   439
489c9b5090e2 Initial load
duke
parents:
diff changeset
   440
//---------------------------WarmCallGenerator--------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   441
// Internal class which handles initial deferral of inlining decisions.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   442
class WarmCallGenerator : public CallGenerator {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   443
  WarmCallInfo*   _call_info;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   444
  CallGenerator*  _if_cold;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   445
  CallGenerator*  _if_hot;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   446
  bool            _is_virtual;   // caches virtuality of if_cold
489c9b5090e2 Initial load
duke
parents:
diff changeset
   447
  bool            _is_inline;    // caches inline-ness of if_hot
489c9b5090e2 Initial load
duke
parents:
diff changeset
   448
489c9b5090e2 Initial load
duke
parents:
diff changeset
   449
public:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   450
  WarmCallGenerator(WarmCallInfo* ci,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   451
                    CallGenerator* if_cold,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   452
                    CallGenerator* if_hot)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   453
    : CallGenerator(if_cold->method())
489c9b5090e2 Initial load
duke
parents:
diff changeset
   454
  {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   455
    assert(method() == if_hot->method(), "consistent choices");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   456
    _call_info  = ci;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   457
    _if_cold    = if_cold;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   458
    _if_hot     = if_hot;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   459
    _is_virtual = if_cold->is_virtual();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   460
    _is_inline  = if_hot->is_inline();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   461
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   462
489c9b5090e2 Initial load
duke
parents:
diff changeset
   463
  virtual bool      is_inline() const           { return _is_inline; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   464
  virtual bool      is_virtual() const          { return _is_virtual; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   465
  virtual bool      is_deferred() const         { return true; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   466
489c9b5090e2 Initial load
duke
parents:
diff changeset
   467
  virtual JVMState* generate(JVMState* jvms);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   468
};
489c9b5090e2 Initial load
duke
parents:
diff changeset
   469
489c9b5090e2 Initial load
duke
parents:
diff changeset
   470
489c9b5090e2 Initial load
duke
parents:
diff changeset
   471
CallGenerator* CallGenerator::for_warm_call(WarmCallInfo* ci,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   472
                                            CallGenerator* if_cold,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   473
                                            CallGenerator* if_hot) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   474
  return new WarmCallGenerator(ci, if_cold, if_hot);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   475
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   476
489c9b5090e2 Initial load
duke
parents:
diff changeset
   477
JVMState* WarmCallGenerator::generate(JVMState* jvms) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   478
  Compile* C = Compile::current();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   479
  if (C->log() != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   480
    C->log()->elem("warm_call bci='%d'", jvms->bci());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   481
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   482
  jvms = _if_cold->generate(jvms);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   483
  if (jvms != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   484
    Node* m = jvms->map()->control();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   485
    if (m->is_CatchProj()) m = m->in(0);  else m = C->top();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   486
    if (m->is_Catch())     m = m->in(0);  else m = C->top();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   487
    if (m->is_Proj())      m = m->in(0);  else m = C->top();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   488
    if (m->is_CallJava()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   489
      _call_info->set_call(m->as_Call());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   490
      _call_info->set_hot_cg(_if_hot);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   491
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   492
      if (PrintOpto || PrintOptoInlining) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   493
        tty->print_cr("Queueing for warm inlining at bci %d:", jvms->bci());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   494
        tty->print("WCI: ");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   495
        _call_info->print();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   496
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   497
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   498
      _call_info->set_heat(_call_info->compute_heat());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   499
      C->set_warm_calls(_call_info->insert_into(C->warm_calls()));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   500
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   501
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   502
  return jvms;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   503
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   504
489c9b5090e2 Initial load
duke
parents:
diff changeset
   505
void WarmCallInfo::make_hot() {
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents: 1217
diff changeset
   506
  Unimplemented();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   507
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   508
489c9b5090e2 Initial load
duke
parents:
diff changeset
   509
void WarmCallInfo::make_cold() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   510
  // No action:  Just dequeue.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   511
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   512
489c9b5090e2 Initial load
duke
parents:
diff changeset
   513
489c9b5090e2 Initial load
duke
parents:
diff changeset
   514
//------------------------PredictedCallGenerator------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   515
// Internal class which handles all out-of-line calls checking receiver type.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   516
class PredictedCallGenerator : public CallGenerator {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   517
  ciKlass*       _predicted_receiver;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   518
  CallGenerator* _if_missed;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   519
  CallGenerator* _if_hit;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   520
  float          _hit_prob;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   521
489c9b5090e2 Initial load
duke
parents:
diff changeset
   522
public:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   523
  PredictedCallGenerator(ciKlass* predicted_receiver,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   524
                         CallGenerator* if_missed,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   525
                         CallGenerator* if_hit, float hit_prob)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   526
    : CallGenerator(if_missed->method())
489c9b5090e2 Initial load
duke
parents:
diff changeset
   527
  {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   528
    // The call profile data may predict the hit_prob as extreme as 0 or 1.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   529
    // Remove the extremes values from the range.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   530
    if (hit_prob > PROB_MAX)   hit_prob = PROB_MAX;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   531
    if (hit_prob < PROB_MIN)   hit_prob = PROB_MIN;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   532
489c9b5090e2 Initial load
duke
parents:
diff changeset
   533
    _predicted_receiver = predicted_receiver;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   534
    _if_missed          = if_missed;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   535
    _if_hit             = if_hit;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   536
    _hit_prob           = hit_prob;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   537
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   538
489c9b5090e2 Initial load
duke
parents:
diff changeset
   539
  virtual bool      is_virtual()   const    { return true; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   540
  virtual bool      is_inline()    const    { return _if_hit->is_inline(); }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   541
  virtual bool      is_deferred()  const    { return _if_hit->is_deferred(); }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   542
489c9b5090e2 Initial load
duke
parents:
diff changeset
   543
  virtual JVMState* generate(JVMState* jvms);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   544
};
489c9b5090e2 Initial load
duke
parents:
diff changeset
   545
489c9b5090e2 Initial load
duke
parents:
diff changeset
   546
489c9b5090e2 Initial load
duke
parents:
diff changeset
   547
CallGenerator* CallGenerator::for_predicted_call(ciKlass* predicted_receiver,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   548
                                                 CallGenerator* if_missed,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   549
                                                 CallGenerator* if_hit,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   550
                                                 float hit_prob) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   551
  return new PredictedCallGenerator(predicted_receiver, if_missed, if_hit, hit_prob);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   552
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   553
489c9b5090e2 Initial load
duke
parents:
diff changeset
   554
489c9b5090e2 Initial load
duke
parents:
diff changeset
   555
JVMState* PredictedCallGenerator::generate(JVMState* jvms) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   556
  GraphKit kit(jvms);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   557
  PhaseGVN& gvn = kit.gvn();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   558
  // We need an explicit receiver null_check before checking its type.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   559
  // We share a map with the caller, so his JVMS gets adjusted.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   560
  Node* receiver = kit.argument(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   561
489c9b5090e2 Initial load
duke
parents:
diff changeset
   562
  CompileLog* log = kit.C->log();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   563
  if (log != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   564
    log->elem("predicted_call bci='%d' klass='%d'",
489c9b5090e2 Initial load
duke
parents:
diff changeset
   565
              jvms->bci(), log->identify(_predicted_receiver));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   566
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   567
489c9b5090e2 Initial load
duke
parents:
diff changeset
   568
  receiver = kit.null_check_receiver(method());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   569
  if (kit.stopped()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   570
    return kit.transfer_exceptions_into_jvms();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   571
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   572
489c9b5090e2 Initial load
duke
parents:
diff changeset
   573
  Node* exact_receiver = receiver;  // will get updated in place...
489c9b5090e2 Initial load
duke
parents:
diff changeset
   574
  Node* slow_ctl = kit.type_check_receiver(receiver,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   575
                                           _predicted_receiver, _hit_prob,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   576
                                           &exact_receiver);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   577
489c9b5090e2 Initial load
duke
parents:
diff changeset
   578
  SafePointNode* slow_map = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   579
  JVMState* slow_jvms;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   580
  { PreserveJVMState pjvms(&kit);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   581
    kit.set_control(slow_ctl);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   582
    if (!kit.stopped()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   583
      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
   584
      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
   585
        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
   586
      assert(slow_jvms != NULL, "must be");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   587
      kit.add_exception_states_from(slow_jvms);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   588
      kit.set_map(slow_jvms->map());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   589
      if (!kit.stopped())
489c9b5090e2 Initial load
duke
parents:
diff changeset
   590
        slow_map = kit.stop();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   591
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   592
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   593
1055
f4fb9fb08038 6731641: assert(m->adr_type() == mach->adr_type(),"matcher should not change adr type")
kvn
parents: 1
diff changeset
   594
  if (kit.stopped()) {
f4fb9fb08038 6731641: assert(m->adr_type() == mach->adr_type(),"matcher should not change adr type")
kvn
parents: 1
diff changeset
   595
    // 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
   596
    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
   597
    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
   598
  }
f4fb9fb08038 6731641: assert(m->adr_type() == mach->adr_type(),"matcher should not change adr type")
kvn
parents: 1
diff changeset
   599
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   600
  // fall through if the instance exactly matches the desired type
489c9b5090e2 Initial load
duke
parents:
diff changeset
   601
  kit.replace_in_map(receiver, exact_receiver);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   602
489c9b5090e2 Initial load
duke
parents:
diff changeset
   603
  // Make the hot call:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   604
  JVMState* new_jvms = _if_hit->generate(kit.sync_jvms());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   605
  if (new_jvms == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   606
    // Inline failed, so make a direct call.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   607
    assert(_if_hit->is_inline(), "must have been a failed inline");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   608
    CallGenerator* cg = CallGenerator::for_direct_call(_if_hit->method());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   609
    new_jvms = cg->generate(kit.sync_jvms());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   610
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   611
  kit.add_exception_states_from(new_jvms);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   612
  kit.set_jvms(new_jvms);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   613
489c9b5090e2 Initial load
duke
parents:
diff changeset
   614
  // Need to merge slow and fast?
489c9b5090e2 Initial load
duke
parents:
diff changeset
   615
  if (slow_map == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   616
    // The fast path is the only path remaining.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   617
    return kit.transfer_exceptions_into_jvms();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   618
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   619
489c9b5090e2 Initial load
duke
parents:
diff changeset
   620
  if (kit.stopped()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   621
    // Inlined method threw an exception, so it's just the slow path after all.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   622
    kit.set_jvms(slow_jvms);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   623
    return kit.transfer_exceptions_into_jvms();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   624
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   625
489c9b5090e2 Initial load
duke
parents:
diff changeset
   626
  // Finish the diamond.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   627
  kit.C->set_has_split_ifs(true); // Has chance for split-if optimization
489c9b5090e2 Initial load
duke
parents:
diff changeset
   628
  RegionNode* region = new (kit.C, 3) RegionNode(3);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   629
  region->init_req(1, kit.control());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   630
  region->init_req(2, slow_map->control());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   631
  kit.set_control(gvn.transform(region));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   632
  Node* iophi = PhiNode::make(region, kit.i_o(), Type::ABIO);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   633
  iophi->set_req(2, slow_map->i_o());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   634
  kit.set_i_o(gvn.transform(iophi));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   635
  kit.merge_memory(slow_map->merged_memory(), region, 2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   636
  uint tos = kit.jvms()->stkoff() + kit.sp();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   637
  uint limit = slow_map->req();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   638
  for (uint i = TypeFunc::Parms; i < limit; i++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   639
    // Skip unused stack slots; fast forward to monoff();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   640
    if (i == tos) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   641
      i = kit.jvms()->monoff();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   642
      if( i >= limit ) break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   643
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   644
    Node* m = kit.map()->in(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   645
    Node* n = slow_map->in(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   646
    if (m != n) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   647
      const Type* t = gvn.type(m)->meet(gvn.type(n));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   648
      Node* phi = PhiNode::make(region, m, t);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   649
      phi->set_req(2, n);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   650
      kit.map()->set_req(i, gvn.transform(phi));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   651
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   652
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   653
  return kit.transfer_exceptions_into_jvms();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   654
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   655
489c9b5090e2 Initial load
duke
parents:
diff changeset
   656
4567
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   657
//------------------------PredictedDynamicCallGenerator-----------------------
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   658
// Internal class which handles all out-of-line calls checking receiver type.
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   659
class PredictedDynamicCallGenerator : public CallGenerator {
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   660
  ciMethodHandle* _predicted_method_handle;
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   661
  CallGenerator*  _if_missed;
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   662
  CallGenerator*  _if_hit;
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   663
  float           _hit_prob;
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   664
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   665
public:
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   666
  PredictedDynamicCallGenerator(ciMethodHandle* predicted_method_handle,
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   667
                                CallGenerator* if_missed,
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   668
                                CallGenerator* if_hit,
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   669
                                float hit_prob)
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   670
    : CallGenerator(if_missed->method()),
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   671
      _predicted_method_handle(predicted_method_handle),
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   672
      _if_missed(if_missed),
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   673
      _if_hit(if_hit),
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   674
      _hit_prob(hit_prob)
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   675
  {}
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   676
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   677
  virtual bool is_inline()   const { return _if_hit->is_inline(); }
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   678
  virtual bool is_deferred() const { return _if_hit->is_deferred(); }
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   679
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   680
  virtual JVMState* generate(JVMState* jvms);
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   681
};
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   682
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   683
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   684
CallGenerator* CallGenerator::for_predicted_dynamic_call(ciMethodHandle* predicted_method_handle,
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   685
                                                         CallGenerator* if_missed,
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   686
                                                         CallGenerator* if_hit,
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   687
                                                         float hit_prob) {
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   688
  return new PredictedDynamicCallGenerator(predicted_method_handle, if_missed, if_hit, hit_prob);
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   689
}
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   690
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   691
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
   692
CallGenerator* CallGenerator::for_method_handle_call(Node* method_handle, JVMState* jvms,
d8de495d05e0 7108383: JSR 292: JRuby bench_define_method_methods.rb: assert(slow_jvms != NULL) failed: miss path must not
twisti
parents: 10986
diff changeset
   693
                                                     ciMethod* caller, ciMethod* callee, ciCallProfile profile) {
d8de495d05e0 7108383: JSR 292: JRuby bench_define_method_methods.rb: assert(slow_jvms != NULL) failed: miss path must not
twisti
parents: 10986
diff changeset
   694
  assert(callee->is_method_handle_invoke() || callee->is_method_handle_adapter(), "for_method_handle_call mismatch");
d8de495d05e0 7108383: JSR 292: JRuby bench_define_method_methods.rb: assert(slow_jvms != NULL) failed: miss path must not
twisti
parents: 10986
diff changeset
   695
  CallGenerator* cg = CallGenerator::for_method_handle_inline(method_handle, jvms, caller, callee, profile);
d8de495d05e0 7108383: JSR 292: JRuby bench_define_method_methods.rb: assert(slow_jvms != NULL) failed: miss path must not
twisti
parents: 10986
diff changeset
   696
  if (cg != NULL)
d8de495d05e0 7108383: JSR 292: JRuby bench_define_method_methods.rb: assert(slow_jvms != NULL) failed: miss path must not
twisti
parents: 10986
diff changeset
   697
    return cg;
d8de495d05e0 7108383: JSR 292: JRuby bench_define_method_methods.rb: assert(slow_jvms != NULL) failed: miss path must not
twisti
parents: 10986
diff changeset
   698
  return CallGenerator::for_direct_call(callee);
d8de495d05e0 7108383: JSR 292: JRuby bench_define_method_methods.rb: assert(slow_jvms != NULL) failed: miss path must not
twisti
parents: 10986
diff changeset
   699
}
d8de495d05e0 7108383: JSR 292: JRuby bench_define_method_methods.rb: assert(slow_jvms != NULL) failed: miss path must not
twisti
parents: 10986
diff changeset
   700
9975
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   701
CallGenerator* CallGenerator::for_method_handle_inline(Node* method_handle, JVMState* jvms,
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   702
                                                       ciMethod* caller, ciMethod* callee, ciCallProfile profile) {
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   703
  if (method_handle->Opcode() == Op_ConP) {
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   704
    const TypeOopPtr* oop_ptr = method_handle->bottom_type()->is_oopptr();
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   705
    ciObject* const_oop = oop_ptr->const_oop();
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   706
    ciMethodHandle* method_handle = const_oop->as_method_handle();
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   707
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   708
    // Set the callee to have access to the class and signature in
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   709
    // the MethodHandleCompiler.
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   710
    method_handle->set_callee(callee);
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   711
    method_handle->set_caller(caller);
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   712
    method_handle->set_call_profile(profile);
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   713
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   714
    // Get an adapter for the MethodHandle.
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   715
    ciMethod* target_method = method_handle->get_method_handle_adapter();
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   716
    if (target_method != NULL) {
10503
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
   717
      CallGenerator* cg = Compile::current()->call_generator(target_method, -1, false, jvms, true, PROB_ALWAYS);
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
   718
      if (cg != NULL && cg->is_inline())
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
   719
        return cg;
9975
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   720
    }
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   721
  } else if (method_handle->Opcode() == Op_Phi && method_handle->req() == 3 &&
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   722
             method_handle->in(1)->Opcode() == Op_ConP && method_handle->in(2)->Opcode() == Op_ConP) {
10514
e229a19078cf 7071307: MethodHandle bimorphic inlining should consider the frequency
never
parents: 10510
diff changeset
   723
    float prob = PROB_FAIR;
e229a19078cf 7071307: MethodHandle bimorphic inlining should consider the frequency
never
parents: 10510
diff changeset
   724
    Node* meth_region = method_handle->in(0);
e229a19078cf 7071307: MethodHandle bimorphic inlining should consider the frequency
never
parents: 10510
diff changeset
   725
    if (meth_region->is_Region() &&
e229a19078cf 7071307: MethodHandle bimorphic inlining should consider the frequency
never
parents: 10510
diff changeset
   726
        meth_region->in(1)->is_Proj() && meth_region->in(2)->is_Proj() &&
e229a19078cf 7071307: MethodHandle bimorphic inlining should consider the frequency
never
parents: 10510
diff changeset
   727
        meth_region->in(1)->in(0) == meth_region->in(2)->in(0) &&
e229a19078cf 7071307: MethodHandle bimorphic inlining should consider the frequency
never
parents: 10510
diff changeset
   728
        meth_region->in(1)->in(0)->is_If()) {
e229a19078cf 7071307: MethodHandle bimorphic inlining should consider the frequency
never
parents: 10510
diff changeset
   729
      // If diamond, so grab the probability of the test to drive the inlining below
e229a19078cf 7071307: MethodHandle bimorphic inlining should consider the frequency
never
parents: 10510
diff changeset
   730
      prob = meth_region->in(1)->in(0)->as_If()->_prob;
e229a19078cf 7071307: MethodHandle bimorphic inlining should consider the frequency
never
parents: 10510
diff changeset
   731
      if (meth_region->in(1)->is_IfTrue()) {
e229a19078cf 7071307: MethodHandle bimorphic inlining should consider the frequency
never
parents: 10510
diff changeset
   732
        prob = 1 - prob;
e229a19078cf 7071307: MethodHandle bimorphic inlining should consider the frequency
never
parents: 10510
diff changeset
   733
      }
e229a19078cf 7071307: MethodHandle bimorphic inlining should consider the frequency
never
parents: 10510
diff changeset
   734
    }
e229a19078cf 7071307: MethodHandle bimorphic inlining should consider the frequency
never
parents: 10510
diff changeset
   735
9975
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   736
    // selectAlternative idiom merging two constant MethodHandles.
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   737
    // Generate a guard so that each can be inlined.  We might want to
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   738
    // do more inputs at later point but this gets the most common
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   739
    // case.
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
   740
    CallGenerator* cg1 = for_method_handle_call(method_handle->in(1), jvms, caller, callee, profile.rescale(1.0 - prob));
d8de495d05e0 7108383: JSR 292: JRuby bench_define_method_methods.rb: assert(slow_jvms != NULL) failed: miss path must not
twisti
parents: 10986
diff changeset
   741
    CallGenerator* cg2 = for_method_handle_call(method_handle->in(2), jvms, caller, callee, profile.rescale(prob));
9975
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   742
    if (cg1 != NULL && cg2 != NULL) {
10514
e229a19078cf 7071307: MethodHandle bimorphic inlining should consider the frequency
never
parents: 10510
diff changeset
   743
      const TypeOopPtr* oop_ptr = method_handle->in(1)->bottom_type()->is_oopptr();
e229a19078cf 7071307: MethodHandle bimorphic inlining should consider the frequency
never
parents: 10510
diff changeset
   744
      ciObject* const_oop = oop_ptr->const_oop();
e229a19078cf 7071307: MethodHandle bimorphic inlining should consider the frequency
never
parents: 10510
diff changeset
   745
      ciMethodHandle* mh = const_oop->as_method_handle();
e229a19078cf 7071307: MethodHandle bimorphic inlining should consider the frequency
never
parents: 10510
diff changeset
   746
      return new PredictedDynamicCallGenerator(mh, cg2, cg1, prob);
9975
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   747
    }
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   748
  }
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   749
  return NULL;
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   750
}
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   751
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
   752
CallGenerator* CallGenerator::for_invokedynamic_call(JVMState* jvms, ciMethod* caller, ciMethod* callee, ciCallProfile profile) {
d8de495d05e0 7108383: JSR 292: JRuby bench_define_method_methods.rb: assert(slow_jvms != NULL) failed: miss path must not
twisti
parents: 10986
diff changeset
   753
  assert(callee->is_method_handle_invoke() || callee->is_method_handle_adapter(), "for_invokedynamic_call mismatch");
d8de495d05e0 7108383: JSR 292: JRuby bench_define_method_methods.rb: assert(slow_jvms != NULL) failed: miss path must not
twisti
parents: 10986
diff changeset
   754
  // Get the CallSite object.
d8de495d05e0 7108383: JSR 292: JRuby bench_define_method_methods.rb: assert(slow_jvms != NULL) failed: miss path must not
twisti
parents: 10986
diff changeset
   755
  ciBytecodeStream str(caller);
d8de495d05e0 7108383: JSR 292: JRuby bench_define_method_methods.rb: assert(slow_jvms != NULL) failed: miss path must not
twisti
parents: 10986
diff changeset
   756
  str.force_bci(jvms->bci());  // Set the stream to the invokedynamic bci.
d8de495d05e0 7108383: JSR 292: JRuby bench_define_method_methods.rb: assert(slow_jvms != NULL) failed: miss path must not
twisti
parents: 10986
diff changeset
   757
  ciCallSite* call_site = str.get_call_site();
d8de495d05e0 7108383: JSR 292: JRuby bench_define_method_methods.rb: assert(slow_jvms != NULL) failed: miss path must not
twisti
parents: 10986
diff changeset
   758
  CallGenerator* cg = CallGenerator::for_invokedynamic_inline(call_site, jvms, caller, callee, profile);
d8de495d05e0 7108383: JSR 292: JRuby bench_define_method_methods.rb: assert(slow_jvms != NULL) failed: miss path must not
twisti
parents: 10986
diff changeset
   759
  if (cg != NULL)
d8de495d05e0 7108383: JSR 292: JRuby bench_define_method_methods.rb: assert(slow_jvms != NULL) failed: miss path must not
twisti
parents: 10986
diff changeset
   760
    return cg;
d8de495d05e0 7108383: JSR 292: JRuby bench_define_method_methods.rb: assert(slow_jvms != NULL) failed: miss path must not
twisti
parents: 10986
diff changeset
   761
  return CallGenerator::for_dynamic_call(callee);
d8de495d05e0 7108383: JSR 292: JRuby bench_define_method_methods.rb: assert(slow_jvms != NULL) failed: miss path must not
twisti
parents: 10986
diff changeset
   762
}
9975
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   763
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 9975
diff changeset
   764
CallGenerator* CallGenerator::for_invokedynamic_inline(ciCallSite* call_site, JVMState* jvms,
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 9975
diff changeset
   765
                                                       ciMethod* caller, ciMethod* callee, ciCallProfile profile) {
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 9975
diff changeset
   766
  ciMethodHandle* method_handle = call_site->get_target();
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 9975
diff changeset
   767
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 9975
diff changeset
   768
  // Set the callee to have access to the class and signature in the
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 9975
diff changeset
   769
  // MethodHandleCompiler.
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 9975
diff changeset
   770
  method_handle->set_callee(callee);
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 9975
diff changeset
   771
  method_handle->set_caller(caller);
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 9975
diff changeset
   772
  method_handle->set_call_profile(profile);
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 9975
diff changeset
   773
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 9975
diff changeset
   774
  // Get an adapter for the MethodHandle.
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 9975
diff changeset
   775
  ciMethod* target_method = method_handle->get_invokedynamic_adapter();
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 9975
diff changeset
   776
  if (target_method != NULL) {
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 9975
diff changeset
   777
    Compile *C = Compile::current();
10503
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
   778
    CallGenerator* cg = C->call_generator(target_method, -1, false, jvms, true, PROB_ALWAYS);
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
   779
    if (cg != NULL && cg->is_inline()) {
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 9975
diff changeset
   780
      // Add a dependence for invalidation of the optimization.
10510
ab626d1bdf53 7085404: JSR 292: VolatileCallSites should have push notification too
twisti
parents: 10509
diff changeset
   781
      if (!call_site->is_constant_call_site()) {
10503
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
   782
        C->dependencies()->assert_call_site_target_value(call_site, method_handle);
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 9975
diff changeset
   783
      }
10503
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
   784
      return cg;
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 9975
diff changeset
   785
    }
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 9975
diff changeset
   786
  }
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 9975
diff changeset
   787
  return NULL;
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 9975
diff changeset
   788
}
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 9975
diff changeset
   789
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 9975
diff changeset
   790
4567
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   791
JVMState* PredictedDynamicCallGenerator::generate(JVMState* jvms) {
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   792
  GraphKit kit(jvms);
13107
fe2475f9b38a 7157365: jruby/bench.bench_timeout crashes with JVM internal error
twisti
parents: 12591
diff changeset
   793
  Compile* C = kit.C;
4567
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   794
  PhaseGVN& gvn = kit.gvn();
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   795
13107
fe2475f9b38a 7157365: jruby/bench.bench_timeout crashes with JVM internal error
twisti
parents: 12591
diff changeset
   796
  CompileLog* log = C->log();
4567
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   797
  if (log != NULL) {
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   798
    log->elem("predicted_dynamic_call bci='%d'", jvms->bci());
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   799
  }
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   800
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   801
  const TypeOopPtr* predicted_mh_ptr = TypeOopPtr::make_from_constant(_predicted_method_handle, true);
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   802
  Node* predicted_mh = kit.makecon(predicted_mh_ptr);
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   803
9975
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   804
  Node* bol = NULL;
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   805
  int bc = jvms->method()->java_code_at_bci(jvms->bci());
10986
1952e39a6364 7087727: JSR 292: C2 crash if ScavengeRootsInCode=2 when "static final" MethodHandle constants are in use
twisti
parents: 10514
diff changeset
   806
  if (bc != Bytecodes::_invokedynamic) {
1952e39a6364 7087727: JSR 292: C2 crash if ScavengeRootsInCode=2 when "static final" MethodHandle constants are in use
twisti
parents: 10514
diff changeset
   807
    // This is the selectAlternative idiom for guardWithTest or
1952e39a6364 7087727: JSR 292: C2 crash if ScavengeRootsInCode=2 when "static final" MethodHandle constants are in use
twisti
parents: 10514
diff changeset
   808
    // similar idioms.
9975
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   809
    Node* receiver = kit.argument(0);
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   810
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   811
    // Check if the MethodHandle is the expected one
13107
fe2475f9b38a 7157365: jruby/bench.bench_timeout crashes with JVM internal error
twisti
parents: 12591
diff changeset
   812
    Node* cmp = gvn.transform(new (C, 3) CmpPNode(receiver, predicted_mh));
fe2475f9b38a 7157365: jruby/bench.bench_timeout crashes with JVM internal error
twisti
parents: 12591
diff changeset
   813
    bol = gvn.transform(new (C, 2) BoolNode(cmp, BoolTest::eq) );
9975
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   814
  } else {
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   815
    // Get the constant pool cache from the caller class.
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   816
    ciMethod* caller_method = jvms->method();
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   817
    ciBytecodeStream str(caller_method);
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   818
    str.force_bci(jvms->bci());  // Set the stream to the invokedynamic bci.
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   819
    ciCPCache* cpcache = str.get_cpcache();
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   820
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   821
    // Get the offset of the CallSite from the constant pool cache
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   822
    // pointer.
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   823
    int index = str.get_method_index();
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   824
    size_t call_site_offset = cpcache->get_f1_offset(index);
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   825
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   826
    // Load the CallSite object from the constant pool cache.
13107
fe2475f9b38a 7157365: jruby/bench.bench_timeout crashes with JVM internal error
twisti
parents: 12591
diff changeset
   827
    const TypeOopPtr* cpcache_type   = TypeOopPtr::make_from_constant(cpcache);  // returns TypeAryPtr of type T_OBJECT
fe2475f9b38a 7157365: jruby/bench.bench_timeout crashes with JVM internal error
twisti
parents: 12591
diff changeset
   828
    const TypeOopPtr* call_site_type = TypeOopPtr::make_from_klass(C->env()->CallSite_klass());
fe2475f9b38a 7157365: jruby/bench.bench_timeout crashes with JVM internal error
twisti
parents: 12591
diff changeset
   829
    Node* cpcache_adr   = kit.makecon(cpcache_type);
fe2475f9b38a 7157365: jruby/bench.bench_timeout crashes with JVM internal error
twisti
parents: 12591
diff changeset
   830
    Node* call_site_adr = kit.basic_plus_adr(cpcache_adr, call_site_offset);
fe2475f9b38a 7157365: jruby/bench.bench_timeout crashes with JVM internal error
twisti
parents: 12591
diff changeset
   831
    // The oops in the constant pool cache are not compressed; load then as raw pointers.
fe2475f9b38a 7157365: jruby/bench.bench_timeout crashes with JVM internal error
twisti
parents: 12591
diff changeset
   832
    Node* call_site     = kit.make_load(kit.control(), call_site_adr, call_site_type, T_ADDRESS, Compile::AliasIdxRaw);
9975
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   833
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   834
    // Load the target MethodHandle from the CallSite object.
13107
fe2475f9b38a 7157365: jruby/bench.bench_timeout crashes with JVM internal error
twisti
parents: 12591
diff changeset
   835
    const TypeOopPtr* target_type = TypeOopPtr::make_from_klass(C->env()->MethodHandle_klass());
9975
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   836
    Node* target_adr = kit.basic_plus_adr(call_site, call_site, java_lang_invoke_CallSite::target_offset_in_bytes());
13107
fe2475f9b38a 7157365: jruby/bench.bench_timeout crashes with JVM internal error
twisti
parents: 12591
diff changeset
   837
    Node* target_mh  = kit.make_load(kit.control(), target_adr, target_type, T_OBJECT);
9975
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   838
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   839
    // Check if the MethodHandle is still the same.
13107
fe2475f9b38a 7157365: jruby/bench.bench_timeout crashes with JVM internal error
twisti
parents: 12591
diff changeset
   840
    Node* cmp = gvn.transform(new (C, 3) CmpPNode(target_mh, predicted_mh));
fe2475f9b38a 7157365: jruby/bench.bench_timeout crashes with JVM internal error
twisti
parents: 12591
diff changeset
   841
    bol = gvn.transform(new (C, 2) BoolNode(cmp, BoolTest::eq) );
9975
82190b49ce14 7050554: JSR 292 - need optimization for selectAlternative
never
parents: 9099
diff changeset
   842
  }
4567
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   843
  IfNode* iff = kit.create_and_xform_if(kit.control(), bol, _hit_prob, COUNT_UNKNOWN);
13107
fe2475f9b38a 7157365: jruby/bench.bench_timeout crashes with JVM internal error
twisti
parents: 12591
diff changeset
   844
  kit.set_control( gvn.transform(new (C, 1) IfTrueNode (iff)));
fe2475f9b38a 7157365: jruby/bench.bench_timeout crashes with JVM internal error
twisti
parents: 12591
diff changeset
   845
  Node* slow_ctl = gvn.transform(new (C, 1) IfFalseNode(iff));
4567
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   846
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   847
  SafePointNode* slow_map = NULL;
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   848
  JVMState* slow_jvms;
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   849
  { PreserveJVMState pjvms(&kit);
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   850
    kit.set_control(slow_ctl);
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   851
    if (!kit.stopped()) {
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   852
      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
   853
      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
   854
        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
   855
      assert(slow_jvms != NULL, "must be");
4567
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   856
      kit.add_exception_states_from(slow_jvms);
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   857
      kit.set_map(slow_jvms->map());
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   858
      if (!kit.stopped())
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   859
        slow_map = kit.stop();
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   860
    }
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   861
  }
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   862
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   863
  if (kit.stopped()) {
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   864
    // Instance exactly does not matches the desired type.
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   865
    kit.set_jvms(slow_jvms);
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   866
    return kit.transfer_exceptions_into_jvms();
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   867
  }
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   868
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   869
  // Make the hot call:
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   870
  JVMState* new_jvms = _if_hit->generate(kit.sync_jvms());
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   871
  if (new_jvms == NULL) {
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   872
    // Inline failed, so make a direct call.
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   873
    assert(_if_hit->is_inline(), "must have been a failed inline");
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   874
    CallGenerator* cg = CallGenerator::for_direct_call(_if_hit->method());
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   875
    new_jvms = cg->generate(kit.sync_jvms());
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   876
  }
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   877
  kit.add_exception_states_from(new_jvms);
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   878
  kit.set_jvms(new_jvms);
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   879
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   880
  // Need to merge slow and fast?
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   881
  if (slow_map == NULL) {
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   882
    // The fast path is the only path remaining.
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   883
    return kit.transfer_exceptions_into_jvms();
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   884
  }
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   885
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   886
  if (kit.stopped()) {
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   887
    // Inlined method threw an exception, so it's just the slow path after all.
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   888
    kit.set_jvms(slow_jvms);
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   889
    return kit.transfer_exceptions_into_jvms();
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   890
  }
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   891
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   892
  // Finish the diamond.
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   893
  kit.C->set_has_split_ifs(true); // Has chance for split-if optimization
13107
fe2475f9b38a 7157365: jruby/bench.bench_timeout crashes with JVM internal error
twisti
parents: 12591
diff changeset
   894
  RegionNode* region = new (C, 3) RegionNode(3);
4567
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   895
  region->init_req(1, kit.control());
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   896
  region->init_req(2, slow_map->control());
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   897
  kit.set_control(gvn.transform(region));
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   898
  Node* iophi = PhiNode::make(region, kit.i_o(), Type::ABIO);
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   899
  iophi->set_req(2, slow_map->i_o());
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   900
  kit.set_i_o(gvn.transform(iophi));
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   901
  kit.merge_memory(slow_map->merged_memory(), region, 2);
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   902
  uint tos = kit.jvms()->stkoff() + kit.sp();
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   903
  uint limit = slow_map->req();
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   904
  for (uint i = TypeFunc::Parms; i < limit; i++) {
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   905
    // Skip unused stack slots; fast forward to monoff();
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   906
    if (i == tos) {
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   907
      i = kit.jvms()->monoff();
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   908
      if( i >= limit ) break;
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   909
    }
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   910
    Node* m = kit.map()->in(i);
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   911
    Node* n = slow_map->in(i);
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   912
    if (m != n) {
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   913
      const Type* t = gvn.type(m)->meet(gvn.type(n));
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   914
      Node* phi = PhiNode::make(region, m, t);
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   915
      phi->set_req(2, n);
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   916
      kit.map()->set_req(i, gvn.transform(phi));
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   917
    }
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   918
  }
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   919
  return kit.transfer_exceptions_into_jvms();
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   920
}
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   921
7fc02fbe5c7a 6893268: additional dynamic language related optimizations in C2
twisti
parents: 4566
diff changeset
   922
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   923
//-------------------------UncommonTrapCallGenerator-----------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   924
// Internal class which handles all out-of-line calls checking receiver type.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   925
class UncommonTrapCallGenerator : public CallGenerator {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   926
  Deoptimization::DeoptReason _reason;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   927
  Deoptimization::DeoptAction _action;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   928
489c9b5090e2 Initial load
duke
parents:
diff changeset
   929
public:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   930
  UncommonTrapCallGenerator(ciMethod* m,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   931
                            Deoptimization::DeoptReason reason,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   932
                            Deoptimization::DeoptAction action)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   933
    : CallGenerator(m)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   934
  {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   935
    _reason = reason;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   936
    _action = action;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   937
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   938
489c9b5090e2 Initial load
duke
parents:
diff changeset
   939
  virtual bool      is_virtual() const          { ShouldNotReachHere(); return false; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   940
  virtual bool      is_trap() const             { return true; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   941
489c9b5090e2 Initial load
duke
parents:
diff changeset
   942
  virtual JVMState* generate(JVMState* jvms);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   943
};
489c9b5090e2 Initial load
duke
parents:
diff changeset
   944
489c9b5090e2 Initial load
duke
parents:
diff changeset
   945
489c9b5090e2 Initial load
duke
parents:
diff changeset
   946
CallGenerator*
489c9b5090e2 Initial load
duke
parents:
diff changeset
   947
CallGenerator::for_uncommon_trap(ciMethod* m,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   948
                                 Deoptimization::DeoptReason reason,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   949
                                 Deoptimization::DeoptAction action) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   950
  return new UncommonTrapCallGenerator(m, reason, action);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   951
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   952
489c9b5090e2 Initial load
duke
parents:
diff changeset
   953
489c9b5090e2 Initial load
duke
parents:
diff changeset
   954
JVMState* UncommonTrapCallGenerator::generate(JVMState* jvms) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   955
  GraphKit kit(jvms);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   956
  // Take the trap with arguments pushed on the stack.  (Cf. null_check_receiver).
489c9b5090e2 Initial load
duke
parents:
diff changeset
   957
  int nargs = method()->arg_size();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   958
  kit.inc_sp(nargs);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   959
  assert(nargs <= kit.sp() && kit.sp() <= jvms->stk_size(), "sane sp w/ args pushed");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   960
  if (_reason == Deoptimization::Reason_class_check &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
   961
      _action == Deoptimization::Action_maybe_recompile) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   962
    // Temp fix for 6529811
489c9b5090e2 Initial load
duke
parents:
diff changeset
   963
    // Don't allow uncommon_trap to override our decision to recompile in the event
489c9b5090e2 Initial load
duke
parents:
diff changeset
   964
    // of a class cast failure for a monomorphic call as it will never let us convert
489c9b5090e2 Initial load
duke
parents:
diff changeset
   965
    // the call to either bi-morphic or megamorphic and can lead to unc-trap loops
489c9b5090e2 Initial load
duke
parents:
diff changeset
   966
    bool keep_exact_action = true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   967
    kit.uncommon_trap(_reason, _action, NULL, "monomorphic vcall checkcast", false, keep_exact_action);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   968
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   969
    kit.uncommon_trap(_reason, _action);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   970
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   971
  return kit.transfer_exceptions_into_jvms();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   972
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   973
489c9b5090e2 Initial load
duke
parents:
diff changeset
   974
// (Note:  Moved hook_up_call to GraphKit::set_edges_for_java_call.)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   975
489c9b5090e2 Initial load
duke
parents:
diff changeset
   976
// (Node:  Merged hook_up_exits into ParseGenerator::generate.)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   977
489c9b5090e2 Initial load
duke
parents:
diff changeset
   978
#define NODES_OVERHEAD_PER_METHOD (30.0)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   979
#define NODES_PER_BYTECODE (9.5)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   980
489c9b5090e2 Initial load
duke
parents:
diff changeset
   981
void WarmCallInfo::init(JVMState* call_site, ciMethod* call_method, ciCallProfile& profile, float prof_factor) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   982
  int call_count = profile.count();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   983
  int code_size = call_method->code_size();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   984
489c9b5090e2 Initial load
duke
parents:
diff changeset
   985
  // Expected execution count is based on the historical count:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   986
  _count = call_count < 0 ? 1 : call_site->method()->scale_count(call_count, prof_factor);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   987
489c9b5090e2 Initial load
duke
parents:
diff changeset
   988
  // Expected profit from inlining, in units of simple call-overheads.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   989
  _profit = 1.0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   990
489c9b5090e2 Initial load
duke
parents:
diff changeset
   991
  // Expected work performed by the call in units of call-overheads.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   992
  // %%% need an empirical curve fit for "work" (time in call)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   993
  float bytecodes_per_call = 3;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   994
  _work = 1.0 + code_size / bytecodes_per_call;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   995
489c9b5090e2 Initial load
duke
parents:
diff changeset
   996
  // Expected size of compilation graph:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   997
  // -XX:+PrintParseStatistics once reported:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   998
  //  Methods seen: 9184  Methods parsed: 9184  Nodes created: 1582391
489c9b5090e2 Initial load
duke
parents:
diff changeset
   999
  //  Histogram of 144298 parsed bytecodes:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1000
  // %%% Need an better predictor for graph size.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1001
  _size = NODES_OVERHEAD_PER_METHOD + (NODES_PER_BYTECODE * code_size);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1002
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1003
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1004
// is_cold:  Return true if the node should never be inlined.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1005
// This is true if any of the key metrics are extreme.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1006
bool WarmCallInfo::is_cold() const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1007
  if (count()  <  WarmCallMinCount)        return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1008
  if (profit() <  WarmCallMinProfit)       return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1009
  if (work()   >  WarmCallMaxWork)         return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1010
  if (size()   >  WarmCallMaxSize)         return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1011
  return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1012
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1013
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1014
// is_hot:  Return true if the node should be inlined immediately.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1015
// This is true if any of the key metrics are extreme.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1016
bool WarmCallInfo::is_hot() const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1017
  assert(!is_cold(), "eliminate is_cold cases before testing is_hot");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1018
  if (count()  >= HotCallCountThreshold)   return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1019
  if (profit() >= HotCallProfitThreshold)  return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1020
  if (work()   <= HotCallTrivialWork)      return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1021
  if (size()   <= HotCallTrivialSize)      return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1022
  return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1023
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1024
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1025
// compute_heat:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1026
float WarmCallInfo::compute_heat() const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1027
  assert(!is_cold(), "compute heat only on warm nodes");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1028
  assert(!is_hot(),  "compute heat only on warm nodes");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1029
  int min_size = MAX2(0,   (int)HotCallTrivialSize);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1030
  int max_size = MIN2(500, (int)WarmCallMaxSize);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1031
  float method_size = (size() - min_size) / MAX2(1, max_size - min_size);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1032
  float size_factor;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1033
  if      (method_size < 0.05)  size_factor = 4;   // 2 sigmas better than avg.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1034
  else if (method_size < 0.15)  size_factor = 2;   // 1 sigma better than avg.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1035
  else if (method_size < 0.5)   size_factor = 1;   // better than avg.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1036
  else                          size_factor = 0.5; // worse than avg.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1037
  return (count() * profit() * size_factor);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1038
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1039
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1040
bool WarmCallInfo::warmer_than(WarmCallInfo* that) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1041
  assert(this != that, "compare only different WCIs");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1042
  assert(this->heat() != 0 && that->heat() != 0, "call compute_heat 1st");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1043
  if (this->heat() > that->heat())   return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1044
  if (this->heat() < that->heat())   return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1045
  assert(this->heat() == that->heat(), "no NaN heat allowed");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1046
  // Equal heat.  Break the tie some other way.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1047
  if (!this->call() || !that->call())  return (address)this > (address)that;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1048
  return this->call()->_idx > that->call()->_idx;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1049
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1050
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1051
//#define UNINIT_NEXT ((WarmCallInfo*)badAddress)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1052
#define UNINIT_NEXT ((WarmCallInfo*)NULL)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1053
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1054
WarmCallInfo* WarmCallInfo::insert_into(WarmCallInfo* head) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1055
  assert(next() == UNINIT_NEXT, "not yet on any list");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1056
  WarmCallInfo* prev_p = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1057
  WarmCallInfo* next_p = head;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1058
  while (next_p != NULL && next_p->warmer_than(this)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1059
    prev_p = next_p;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1060
    next_p = prev_p->next();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1061
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1062
  // Install this between prev_p and next_p.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1063
  this->set_next(next_p);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1064
  if (prev_p == NULL)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1065
    head = this;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1066
  else
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1067
    prev_p->set_next(this);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1068
  return head;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1069
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1070
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1071
WarmCallInfo* WarmCallInfo::remove_from(WarmCallInfo* head) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1072
  WarmCallInfo* prev_p = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1073
  WarmCallInfo* next_p = head;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1074
  while (next_p != this) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1075
    assert(next_p != NULL, "this must be in the list somewhere");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1076
    prev_p = next_p;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1077
    next_p = prev_p->next();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1078
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1079
  next_p = this->next();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1080
  debug_only(this->set_next(UNINIT_NEXT));
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1081
  // Remove this from between prev_p and next_p.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1082
  if (prev_p == NULL)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1083
    head = next_p;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1084
  else
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1085
    prev_p->set_next(next_p);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1086
  return head;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1087
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1088
9099
bdeb610d3cb1 6909440: C2 fails with assertion (_always_cold->is_cold(),"must always be cold")
never
parents: 8676
diff changeset
  1089
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
  1090
                                       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
  1091
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
  1092
                                        WarmCallInfo::MAX_VALUE(), WarmCallInfo::MAX_VALUE());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1093
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1094
WarmCallInfo* WarmCallInfo::always_hot() {
9099
bdeb610d3cb1 6909440: C2 fails with assertion (_always_cold->is_cold(),"must always be cold")
never
parents: 8676
diff changeset
  1095
  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
  1096
  return &_always_hot;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1097
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1098
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1099
WarmCallInfo* WarmCallInfo::always_cold() {
9099
bdeb610d3cb1 6909440: C2 fails with assertion (_always_cold->is_cold(),"must always be cold")
never
parents: 8676
diff changeset
  1100
  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
  1101
  return &_always_cold;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1102
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1103
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1104
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1105
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1106
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1107
void WarmCallInfo::print() const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1108
  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
  1109
             is_cold() ? "cold" : is_hot() ? "hot " : "warm",
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1110
             count(), profit(), work(), size(), compute_heat(), next());
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1111
  tty->cr();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1112
  if (call() != NULL)  call()->dump();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1113
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1114
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1115
void print_wci(WarmCallInfo* ci) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1116
  ci->print();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1117
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1118
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1119
void WarmCallInfo::print_all() const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1120
  for (const WarmCallInfo* p = this; p != NULL; p = p->next())
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1121
    p->print();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1122
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1123
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1124
int WarmCallInfo::count_all() const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1125
  int cnt = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1126
  for (const WarmCallInfo* p = this; p != NULL; p = p->next())
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1127
    cnt++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1128
  return cnt;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1129
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1130
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1131
#endif //PRODUCT