src/hotspot/share/runtime/compilationPolicy.cpp
author redestad
Fri, 10 Aug 2018 00:20:15 +0200
changeset 51369 f32e61253792
parent 50068 5201c9474ee7
child 51959 db0c3952de52
permissions -rw-r--r--
8209186: Rename SimpleThresholdPolicy to TieredThresholdPolicy Reviewed-by: thartmann, pliden, kvn
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     1
/*
48884
7e17b00dc245 8196923: [REDO] NMT: Report array class count in NMT summary
zgu
parents: 48874
diff changeset
     2
 * Copyright (c) 2000, 2018, 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: 4750
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 4750
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: 4750
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: 6747
diff changeset
    25
#include "precompiled.hpp"
48884
7e17b00dc245 8196923: [REDO] NMT: Report array class count in NMT summary
zgu
parents: 48874
diff changeset
    26
#include "classfile/classLoaderData.inline.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6747
diff changeset
    27
#include "code/compiledIC.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6747
diff changeset
    28
#include "code/nmethod.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6747
diff changeset
    29
#include "code/scopeDesc.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6747
diff changeset
    30
#include "interpreter/interpreter.hpp"
37248
11a660dbbb8e 8132524: Missing includes to resourceArea.hpp
jprovino
parents: 35547
diff changeset
    31
#include "memory/resourceArea.hpp"
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 11572
diff changeset
    32
#include "oops/methodData.hpp"
49340
4e82736053ae 8191102: Incorrect include file use in classLoader.hpp
hseigel
parents: 48884
diff changeset
    33
#include "oops/method.inline.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6747
diff changeset
    34
#include "oops/oop.inline.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6747
diff changeset
    35
#include "prims/nativeLookup.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6747
diff changeset
    36
#include "runtime/compilationPolicy.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6747
diff changeset
    37
#include "runtime/frame.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6747
diff changeset
    38
#include "runtime/handles.inline.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6747
diff changeset
    39
#include "runtime/rframe.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6747
diff changeset
    40
#include "runtime/stubRoutines.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6747
diff changeset
    41
#include "runtime/thread.hpp"
51369
f32e61253792 8209186: Rename SimpleThresholdPolicy to TieredThresholdPolicy
redestad
parents: 50068
diff changeset
    42
#include "runtime/tieredThresholdPolicy.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6747
diff changeset
    43
#include "runtime/timer.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6747
diff changeset
    44
#include "runtime/vframe.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6747
diff changeset
    45
#include "runtime/vm_operations.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6747
diff changeset
    46
#include "utilities/events.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6747
diff changeset
    47
#include "utilities/globalDefinitions.hpp"
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    48
489c9b5090e2 Initial load
duke
parents:
diff changeset
    49
CompilationPolicy* CompilationPolicy::_policy;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    50
elapsedTimer       CompilationPolicy::_accumulated_time;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    51
bool               CompilationPolicy::_in_vm_startup;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    52
489c9b5090e2 Initial load
duke
parents:
diff changeset
    53
// Determine compilation policy based on command line argument
489c9b5090e2 Initial load
duke
parents:
diff changeset
    54
void compilationPolicy_init() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    55
  CompilationPolicy::set_in_vm_startup(DelayCompilationDuringStartup);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    56
489c9b5090e2 Initial load
duke
parents:
diff changeset
    57
  switch(CompilationPolicyChoice) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    58
  case 0:
489c9b5090e2 Initial load
duke
parents:
diff changeset
    59
    CompilationPolicy::set_policy(new SimpleCompPolicy());
489c9b5090e2 Initial load
duke
parents:
diff changeset
    60
    break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    61
489c9b5090e2 Initial load
duke
parents:
diff changeset
    62
  case 1:
489c9b5090e2 Initial load
duke
parents:
diff changeset
    63
#ifdef COMPILER2
489c9b5090e2 Initial load
duke
parents:
diff changeset
    64
    CompilationPolicy::set_policy(new StackWalkCompPolicy());
489c9b5090e2 Initial load
duke
parents:
diff changeset
    65
#else
489c9b5090e2 Initial load
duke
parents:
diff changeset
    66
    Unimplemented();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    67
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
    68
    break;
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
    69
  case 2:
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
    70
#ifdef TIERED
51369
f32e61253792 8209186: Rename SimpleThresholdPolicy to TieredThresholdPolicy
redestad
parents: 50068
diff changeset
    71
    CompilationPolicy::set_policy(new TieredThresholdPolicy());
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
    72
#else
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
    73
    Unimplemented();
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
    74
#endif
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
    75
    break;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    76
  default:
50068
5201c9474ee7 8202711: Merge tiered compilation policies
redestad
parents: 49340
diff changeset
    77
    fatal("CompilationPolicyChoice must be in the range: [0-2]");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    78
  }
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
    79
  CompilationPolicy::policy()->initialize();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    80
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    81
489c9b5090e2 Initial load
duke
parents:
diff changeset
    82
void CompilationPolicy::completed_vm_startup() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    83
  if (TraceCompilationPolicy) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    84
    tty->print("CompilationPolicy: completed vm startup.\n");
489c9b5090e2 Initial load
duke
parents:
diff changeset
    85
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    86
  _in_vm_startup = false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    87
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    88
489c9b5090e2 Initial load
duke
parents:
diff changeset
    89
// Returns true if m must be compiled before executing it
489c9b5090e2 Initial load
duke
parents:
diff changeset
    90
// This is intended to force compiles for methods (usually for
489c9b5090e2 Initial load
duke
parents:
diff changeset
    91
// debugging) that would otherwise be interpreted for some reason.
46727
6e4a84748e2c 8183039: Re-examine methodHandle methods uninlined by 8144256
coleenp
parents: 43455
diff changeset
    92
bool CompilationPolicy::must_be_compiled(const methodHandle& m, int comp_level) {
14477
95e66ea71f71 6830717: replay of compilations would help with debugging
minqi
parents: 14391
diff changeset
    93
  // Don't allow Xcomp to cause compiles in replay mode
95e66ea71f71 6830717: replay of compilations would help with debugging
minqi
parents: 14391
diff changeset
    94
  if (ReplayCompiles) return false;
95e66ea71f71 6830717: replay of compilations would help with debugging
minqi
parents: 14391
diff changeset
    95
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    96
  if (m->has_compiled_code()) return false;       // already compiled
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
    97
  if (!can_be_compiled(m, comp_level)) return false;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    98
489c9b5090e2 Initial load
duke
parents:
diff changeset
    99
  return !UseInterpreter ||                                              // must compile all methods
4750
71fd601907dc 4360113: Evict nmethods when code cache gets full
kvn
parents: 4645
diff changeset
   100
         (UseCompiler && AlwaysCompileLoopMethods && m->has_loops() && CompileBroker::should_compile_new_jobs()); // eagerly compile loop methods
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   101
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   102
46727
6e4a84748e2c 8183039: Re-examine methodHandle methods uninlined by 8144256
coleenp
parents: 43455
diff changeset
   103
void CompilationPolicy::compile_if_required(const methodHandle& selected_method, TRAPS) {
38139
cf6f5c1b7205 8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents: 38133
diff changeset
   104
  if (must_be_compiled(selected_method)) {
cf6f5c1b7205 8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents: 38133
diff changeset
   105
    // This path is unusual, mostly used by the '-Xcomp' stress test mode.
cf6f5c1b7205 8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents: 38133
diff changeset
   106
cf6f5c1b7205 8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents: 38133
diff changeset
   107
    // Note: with several active threads, the must_be_compiled may be true
cf6f5c1b7205 8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents: 38133
diff changeset
   108
    //       while can_be_compiled is false; remove assert
cf6f5c1b7205 8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents: 38133
diff changeset
   109
    // assert(CompilationPolicy::can_be_compiled(selected_method), "cannot compile");
cf6f5c1b7205 8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents: 38133
diff changeset
   110
    if (!THREAD->can_call_java() || THREAD->is_Compiler_thread()) {
cf6f5c1b7205 8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents: 38133
diff changeset
   111
      // don't force compilation, resolve was on behalf of compiler
cf6f5c1b7205 8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents: 38133
diff changeset
   112
      return;
cf6f5c1b7205 8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents: 38133
diff changeset
   113
    }
cf6f5c1b7205 8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents: 38133
diff changeset
   114
    if (selected_method->method_holder()->is_not_initialized()) {
cf6f5c1b7205 8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents: 38133
diff changeset
   115
      // 'is_not_initialized' means not only '!is_initialized', but also that
cf6f5c1b7205 8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents: 38133
diff changeset
   116
      // initialization has not been started yet ('!being_initialized')
cf6f5c1b7205 8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents: 38133
diff changeset
   117
      // Do not force compilation of methods in uninitialized classes.
cf6f5c1b7205 8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents: 38133
diff changeset
   118
      // Note that doing this would throw an assert later,
cf6f5c1b7205 8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents: 38133
diff changeset
   119
      // in CompileBroker::compile_method.
cf6f5c1b7205 8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents: 38133
diff changeset
   120
      // We sometimes use the link resolver to do reflective lookups
cf6f5c1b7205 8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents: 38133
diff changeset
   121
      // even before classes are initialized.
cf6f5c1b7205 8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents: 38133
diff changeset
   122
      return;
cf6f5c1b7205 8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents: 38133
diff changeset
   123
    }
cf6f5c1b7205 8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents: 38133
diff changeset
   124
    CompileBroker::compile_method(selected_method, InvocationEntryBci,
cf6f5c1b7205 8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents: 38133
diff changeset
   125
        CompilationPolicy::policy()->initial_compile_level(),
38218
f5ba1dea04eb 8153013: BlockingCompilation test times out
neliasso
parents: 38139
diff changeset
   126
        methodHandle(), 0, CompileTask::Reason_MustBeCompiled, CHECK);
38139
cf6f5c1b7205 8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents: 38133
diff changeset
   127
  }
cf6f5c1b7205 8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents: 38133
diff changeset
   128
}
cf6f5c1b7205 8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents: 38133
diff changeset
   129
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   130
// Returns true if m is allowed to be compiled
46727
6e4a84748e2c 8183039: Re-examine methodHandle methods uninlined by 8144256
coleenp
parents: 43455
diff changeset
   131
bool CompilationPolicy::can_be_compiled(const methodHandle& m, int comp_level) {
17126
42a942feeea2 8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents: 17002
diff changeset
   132
  // allow any levels for WhiteBox
42a942feeea2 8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents: 17002
diff changeset
   133
  assert(WhiteBoxAPI || comp_level == CompLevel_all || is_compile(comp_level), "illegal compilation level");
42a942feeea2 8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents: 17002
diff changeset
   134
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   135
  if (m->is_abstract()) return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   136
  if (DontCompileHugeMethods && m->code_size() > HugeMethodLimit) return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   137
4645
0c5f5b94e93a 6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents: 1
diff changeset
   138
  // Math intrinsics should never be compiled as this can lead to
0c5f5b94e93a 6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents: 1
diff changeset
   139
  // monotonicity problems because the interpreter will prefer the
0c5f5b94e93a 6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents: 1
diff changeset
   140
  // compiled code to the intrinsic version.  This can't happen in
0c5f5b94e93a 6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents: 1
diff changeset
   141
  // production because the invocation counter can't be incremented
0c5f5b94e93a 6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents: 1
diff changeset
   142
  // but we shouldn't expose the system to this problem in testing
0c5f5b94e93a 6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents: 1
diff changeset
   143
  // modes.
0c5f5b94e93a 6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents: 1
diff changeset
   144
  if (!AbstractInterpreter::can_be_compiled(m)) {
0c5f5b94e93a 6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents: 1
diff changeset
   145
    return false;
0c5f5b94e93a 6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents: 1
diff changeset
   146
  }
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   147
  if (comp_level == CompLevel_all) {
17126
42a942feeea2 8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents: 17002
diff changeset
   148
    if (TieredCompilation) {
42a942feeea2 8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents: 17002
diff changeset
   149
      // enough to be compilable at any level for tiered
42a942feeea2 8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents: 17002
diff changeset
   150
      return !m->is_not_compilable(CompLevel_simple) || !m->is_not_compilable(CompLevel_full_optimization);
42a942feeea2 8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents: 17002
diff changeset
   151
    } else {
42a942feeea2 8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents: 17002
diff changeset
   152
      // must be compilable at available level for non-tiered
42a942feeea2 8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents: 17002
diff changeset
   153
      return !m->is_not_compilable(CompLevel_highest_tier);
42a942feeea2 8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents: 17002
diff changeset
   154
    }
16689
efce070b8d42 8007288: Additional WB API for compiler's testing
iignatyev
parents: 14477
diff changeset
   155
  } else if (is_compile(comp_level)) {
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   156
    return !m->is_not_compilable(comp_level);
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   157
  }
16689
efce070b8d42 8007288: Additional WB API for compiler's testing
iignatyev
parents: 14477
diff changeset
   158
  return false;
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   159
}
4645
0c5f5b94e93a 6849984: Value methods for platform dependent math functions constant fold incorrectly
never
parents: 1
diff changeset
   160
19332
ee4c8c2af356 8022832: Add WB APIs for OSR compilation
iignatyev
parents: 17126
diff changeset
   161
// Returns true if m is allowed to be osr compiled
46727
6e4a84748e2c 8183039: Re-examine methodHandle methods uninlined by 8144256
coleenp
parents: 43455
diff changeset
   162
bool CompilationPolicy::can_be_osr_compiled(const methodHandle& m, int comp_level) {
19332
ee4c8c2af356 8022832: Add WB APIs for OSR compilation
iignatyev
parents: 17126
diff changeset
   163
  bool result = false;
ee4c8c2af356 8022832: Add WB APIs for OSR compilation
iignatyev
parents: 17126
diff changeset
   164
  if (comp_level == CompLevel_all) {
ee4c8c2af356 8022832: Add WB APIs for OSR compilation
iignatyev
parents: 17126
diff changeset
   165
    if (TieredCompilation) {
ee4c8c2af356 8022832: Add WB APIs for OSR compilation
iignatyev
parents: 17126
diff changeset
   166
      // enough to be osr compilable at any level for tiered
ee4c8c2af356 8022832: Add WB APIs for OSR compilation
iignatyev
parents: 17126
diff changeset
   167
      result = !m->is_not_osr_compilable(CompLevel_simple) || !m->is_not_osr_compilable(CompLevel_full_optimization);
ee4c8c2af356 8022832: Add WB APIs for OSR compilation
iignatyev
parents: 17126
diff changeset
   168
    } else {
ee4c8c2af356 8022832: Add WB APIs for OSR compilation
iignatyev
parents: 17126
diff changeset
   169
      // must be osr compilable at available level for non-tiered
ee4c8c2af356 8022832: Add WB APIs for OSR compilation
iignatyev
parents: 17126
diff changeset
   170
      result = !m->is_not_osr_compilable(CompLevel_highest_tier);
ee4c8c2af356 8022832: Add WB APIs for OSR compilation
iignatyev
parents: 17126
diff changeset
   171
    }
ee4c8c2af356 8022832: Add WB APIs for OSR compilation
iignatyev
parents: 17126
diff changeset
   172
  } else if (is_compile(comp_level)) {
ee4c8c2af356 8022832: Add WB APIs for OSR compilation
iignatyev
parents: 17126
diff changeset
   173
    result = !m->is_not_osr_compilable(comp_level);
ee4c8c2af356 8022832: Add WB APIs for OSR compilation
iignatyev
parents: 17126
diff changeset
   174
  }
ee4c8c2af356 8022832: Add WB APIs for OSR compilation
iignatyev
parents: 17126
diff changeset
   175
  return (result && can_be_compiled(m, comp_level));
ee4c8c2af356 8022832: Add WB APIs for OSR compilation
iignatyev
parents: 17126
diff changeset
   176
}
ee4c8c2af356 8022832: Add WB APIs for OSR compilation
iignatyev
parents: 17126
diff changeset
   177
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   178
bool CompilationPolicy::is_compilation_enabled() {
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   179
  // NOTE: CompileBroker::should_compile_new_jobs() checks for UseCompiler
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   180
  return !delay_compilation_during_startup() && CompileBroker::should_compile_new_jobs();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   181
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   182
35547
0ee84aa8e705 8146705: Improve JVMCI support for blocking compilation
dnsimon
parents: 33612
diff changeset
   183
CompileTask* CompilationPolicy::select_task_helper(CompileQueue* compile_queue) {
0ee84aa8e705 8146705: Improve JVMCI support for blocking compilation
dnsimon
parents: 33612
diff changeset
   184
#if INCLUDE_JVMCI
0ee84aa8e705 8146705: Improve JVMCI support for blocking compilation
dnsimon
parents: 33612
diff changeset
   185
  if (UseJVMCICompiler && !BackgroundCompilation) {
0ee84aa8e705 8146705: Improve JVMCI support for blocking compilation
dnsimon
parents: 33612
diff changeset
   186
    /*
0ee84aa8e705 8146705: Improve JVMCI support for blocking compilation
dnsimon
parents: 33612
diff changeset
   187
     * In blocking compilation mode, the CompileBroker will make
0ee84aa8e705 8146705: Improve JVMCI support for blocking compilation
dnsimon
parents: 33612
diff changeset
   188
     * compilations submitted by a JVMCI compiler thread non-blocking. These
0ee84aa8e705 8146705: Improve JVMCI support for blocking compilation
dnsimon
parents: 33612
diff changeset
   189
     * compilations should be scheduled after all blocking compilations
0ee84aa8e705 8146705: Improve JVMCI support for blocking compilation
dnsimon
parents: 33612
diff changeset
   190
     * to service non-compiler related compilations sooner and reduce the
0ee84aa8e705 8146705: Improve JVMCI support for blocking compilation
dnsimon
parents: 33612
diff changeset
   191
     * chance of such compilations timing out.
0ee84aa8e705 8146705: Improve JVMCI support for blocking compilation
dnsimon
parents: 33612
diff changeset
   192
     */
0ee84aa8e705 8146705: Improve JVMCI support for blocking compilation
dnsimon
parents: 33612
diff changeset
   193
    for (CompileTask* task = compile_queue->first(); task != NULL; task = task->next()) {
0ee84aa8e705 8146705: Improve JVMCI support for blocking compilation
dnsimon
parents: 33612
diff changeset
   194
      if (task->is_blocking()) {
0ee84aa8e705 8146705: Improve JVMCI support for blocking compilation
dnsimon
parents: 33612
diff changeset
   195
        return task;
0ee84aa8e705 8146705: Improve JVMCI support for blocking compilation
dnsimon
parents: 33612
diff changeset
   196
      }
0ee84aa8e705 8146705: Improve JVMCI support for blocking compilation
dnsimon
parents: 33612
diff changeset
   197
    }
0ee84aa8e705 8146705: Improve JVMCI support for blocking compilation
dnsimon
parents: 33612
diff changeset
   198
  }
0ee84aa8e705 8146705: Improve JVMCI support for blocking compilation
dnsimon
parents: 33612
diff changeset
   199
#endif
0ee84aa8e705 8146705: Improve JVMCI support for blocking compilation
dnsimon
parents: 33612
diff changeset
   200
  return compile_queue->first();
0ee84aa8e705 8146705: Improve JVMCI support for blocking compilation
dnsimon
parents: 33612
diff changeset
   201
}
0ee84aa8e705 8146705: Improve JVMCI support for blocking compilation
dnsimon
parents: 33612
diff changeset
   202
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   203
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   204
void CompilationPolicy::print_time() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   205
  tty->print_cr ("Accumulated compilationPolicy times:");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   206
  tty->print_cr ("---------------------------");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   207
  tty->print_cr ("  Total: %3.3f sec.", _accumulated_time.seconds());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   208
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   209
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   210
void NonTieredCompPolicy::trace_osr_completion(nmethod* osr_nm) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   211
  if (TraceOnStackReplacement) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   212
    if (osr_nm == NULL) tty->print_cr("compilation failed");
24424
2658d7834c6e 8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents: 24013
diff changeset
   213
    else tty->print_cr("nmethod " INTPTR_FORMAT, p2i(osr_nm));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   214
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   215
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   216
#endif // !PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   217
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   218
void NonTieredCompPolicy::initialize() {
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   219
  // Setup the compiler thread numbers
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   220
  if (CICompilerCountPerCPU) {
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   221
    // Example: if CICompilerCountPerCPU is true, then we get
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   222
    // max(log2(8)-1,1) = 2 compiler threads on an 8-way machine.
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   223
    // May help big-app startup time.
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   224
    _compiler_count = MAX2(log2_intptr(os::active_processor_count())-1,1);
24013
1d16b0f1060d 8029436: CICompilerCount is not updated when the number of compiler threads is adjusted to the number of CPUs
anoll
parents: 22551
diff changeset
   225
    FLAG_SET_ERGO(intx, CICompilerCount, _compiler_count);
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   226
  } else {
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   227
    _compiler_count = CICompilerCount;
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   228
  }
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   229
}
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   230
6747
ab166511728e 6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents: 6453
diff changeset
   231
// Note: this policy is used ONLY if TieredCompilation is off.
ab166511728e 6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents: 6453
diff changeset
   232
// compiler_count() behaves the following way:
ab166511728e 6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents: 6453
diff changeset
   233
// - with TIERED build (with both COMPILER1 and COMPILER2 defined) it should return
43455
96560cffef4d 8166002: Emulate client build on platforms with reduced virtual address space
jcm
parents: 38218
diff changeset
   234
//   zero for the c1 compilation levels in server compilation mode runs
96560cffef4d 8166002: Emulate client build on platforms with reduced virtual address space
jcm
parents: 38218
diff changeset
   235
//   and c2 compilation levels in client compilation mode runs.
96560cffef4d 8166002: Emulate client build on platforms with reduced virtual address space
jcm
parents: 38218
diff changeset
   236
// - with COMPILER2 not defined it should return zero for c2 compilation levels.
96560cffef4d 8166002: Emulate client build on platforms with reduced virtual address space
jcm
parents: 38218
diff changeset
   237
// - with COMPILER1 not defined it should return zero for c1 compilation levels.
6747
ab166511728e 6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents: 6453
diff changeset
   238
// - if neither is defined - always return zero.
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   239
int NonTieredCompPolicy::compiler_count(CompLevel comp_level) {
6747
ab166511728e 6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents: 6453
diff changeset
   240
  assert(!TieredCompilation, "This policy should not be used with TieredCompilation");
43455
96560cffef4d 8166002: Emulate client build on platforms with reduced virtual address space
jcm
parents: 38218
diff changeset
   241
  if (COMPILER2_PRESENT(is_server_compilation_mode_vm() && is_c2_compile(comp_level) ||)
96560cffef4d 8166002: Emulate client build on platforms with reduced virtual address space
jcm
parents: 38218
diff changeset
   242
      is_client_compilation_mode_vm() && is_c1_compile(comp_level)) {
6747
ab166511728e 6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents: 6453
diff changeset
   243
    return _compiler_count;
ab166511728e 6987115: Non-tiered compilation policy creates unnecessary C1 threads
iveresov
parents: 6453
diff changeset
   244
  }
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   245
  return 0;
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   246
}
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   247
33593
60764a78fa5c 8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents: 32582
diff changeset
   248
void NonTieredCompPolicy::reset_counter_for_invocation_event(const methodHandle& m) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   249
  // Make sure invocation and backedge counter doesn't overflow again right away
489c9b5090e2 Initial load
duke
parents:
diff changeset
   250
  // as would be the case for native methods.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   251
489c9b5090e2 Initial load
duke
parents:
diff changeset
   252
  // BUT also make sure the method doesn't look like it was never executed.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   253
  // Set carry bit and reduce counter's value to min(count, CompileThreshold/2).
17000
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   254
  MethodCounters* mcs = m->method_counters();
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   255
  assert(mcs != NULL, "MethodCounters cannot be NULL for profiling");
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   256
  mcs->invocation_counter()->set_carry();
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   257
  mcs->backedge_counter()->set_carry();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   258
489c9b5090e2 Initial load
duke
parents:
diff changeset
   259
  assert(!m->was_never_executed(), "don't reset to 0 -- could be mistaken for never-executed");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   260
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   261
33593
60764a78fa5c 8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents: 32582
diff changeset
   262
void NonTieredCompPolicy::reset_counter_for_back_branch_event(const methodHandle& m) {
22551
9bf46d16dcc6 8025856: Fix typos in the GC code
jwilhelm
parents: 19332
diff changeset
   263
  // Delay next back-branch event but pump up invocation counter to trigger
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   264
  // whole method compilation.
17000
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   265
  MethodCounters* mcs = m->method_counters();
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   266
  assert(mcs != NULL, "MethodCounters cannot be NULL for profiling");
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   267
  InvocationCounter* i = mcs->invocation_counter();
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   268
  InvocationCounter* b = mcs->backedge_counter();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   269
489c9b5090e2 Initial load
duke
parents:
diff changeset
   270
  // Don't set invocation_counter's value too low otherwise the method will
489c9b5090e2 Initial load
duke
parents:
diff changeset
   271
  // look like immature (ic < ~5300) which prevents the inlining based on
489c9b5090e2 Initial load
duke
parents:
diff changeset
   272
  // the type profiling.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   273
  i->set(i->state(), CompileThreshold);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   274
  // Don't reset counter too low - it is used to check if OSR method is ready.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   275
  b->set(b->state(), CompileThreshold / 2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   276
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   277
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   278
//
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   279
// CounterDecay
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   280
//
22551
9bf46d16dcc6 8025856: Fix typos in the GC code
jwilhelm
parents: 19332
diff changeset
   281
// Iterates through invocation counters and decrements them. This
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   282
// is done at each safepoint.
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   283
//
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   284
class CounterDecay : public AllStatic {
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   285
  static jlong _last_timestamp;
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 11572
diff changeset
   286
  static void do_method(Method* m) {
17000
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   287
    MethodCounters* mcs = m->method_counters();
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   288
    if (mcs != NULL) {
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   289
      mcs->invocation_counter()->decay();
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   290
    }
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   291
  }
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   292
public:
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   293
  static void decay();
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   294
  static bool is_decay_needed() {
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   295
    return (os::javaTimeMillis() - _last_timestamp) > CounterDecayMinIntervalLength;
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   296
  }
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   297
};
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   298
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   299
jlong CounterDecay::_last_timestamp = 0;
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   300
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   301
void CounterDecay::decay() {
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   302
  _last_timestamp = os::javaTimeMillis();
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   303
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   304
  // This operation is going to be performed only at the end of a safepoint
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   305
  // and hence GC's will not be going on, all Java mutators are suspended
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   306
  // at this point and hence SystemDictionary_lock is also not needed.
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   307
  assert(SafepointSynchronize::is_at_safepoint(), "can only be executed at a safepoint");
48884
7e17b00dc245 8196923: [REDO] NMT: Report array class count in NMT summary
zgu
parents: 48874
diff changeset
   308
  size_t nclasses = ClassLoaderDataGraph::num_instance_classes();
7e17b00dc245 8196923: [REDO] NMT: Report array class count in NMT summary
zgu
parents: 48874
diff changeset
   309
  size_t classes_per_tick = nclasses * (CounterDecayMinIntervalLength * 1e-3 /
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   310
                                        CounterHalfLifeTime);
48884
7e17b00dc245 8196923: [REDO] NMT: Report array class count in NMT summary
zgu
parents: 48874
diff changeset
   311
  for (size_t i = 0; i < classes_per_tick; i++) {
46729
c62d2e8b2728 7133093: Improve system dictionary performance
coleenp
parents: 46727
diff changeset
   312
    InstanceKlass* k = ClassLoaderDataGraph::try_get_next_class();
c62d2e8b2728 7133093: Improve system dictionary performance
coleenp
parents: 46727
diff changeset
   313
    if (k != NULL) {
c62d2e8b2728 7133093: Improve system dictionary performance
coleenp
parents: 46727
diff changeset
   314
      k->methods_do(do_method);
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   315
    }
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   316
  }
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   317
}
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   318
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   319
// Called at the end of the safepoint
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   320
void NonTieredCompPolicy::do_safepoint_work() {
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   321
  if(UseCounterDecay && CounterDecay::is_decay_needed()) {
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   322
    CounterDecay::decay();
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   323
  }
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   324
}
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   325
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   326
void NonTieredCompPolicy::reprofile(ScopeDesc* trap_scope, bool is_osr) {
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   327
  ScopeDesc* sd = trap_scope;
17000
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   328
  MethodCounters* mcs;
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   329
  InvocationCounter* c;
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   330
  for (; !sd->is_top(); sd = sd->sender()) {
17000
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   331
    mcs = sd->method()->method_counters();
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   332
    if (mcs != NULL) {
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   333
      // Reset ICs of inlined methods, since they can trigger compilations also.
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   334
      mcs->invocation_counter()->reset();
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   335
    }
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   336
  }
17000
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   337
  mcs = sd->method()->method_counters();
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   338
  if (mcs != NULL) {
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   339
    c = mcs->invocation_counter();
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   340
    if (is_osr) {
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   341
      // It was an OSR method, so bump the count higher.
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   342
      c->set(c->state(), CompileThreshold);
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   343
    } else {
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   344
      c->reset();
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   345
    }
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   346
    mcs->backedge_counter()->reset();
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   347
  }
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   348
}
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   349
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   350
// This method can be called by any component of the runtime to notify the policy
22551
9bf46d16dcc6 8025856: Fix typos in the GC code
jwilhelm
parents: 19332
diff changeset
   351
// that it's recommended to delay the compilation of this method.
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 11572
diff changeset
   352
void NonTieredCompPolicy::delay_compilation(Method* method) {
17000
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   353
  MethodCounters* mcs = method->method_counters();
17001
1ce8485b1907 8012052: java/lang/invoke/6987555/Test6987555.java crashes with assert(mcs != NULL) failed: MethodCounters cannot be NULL.
jiangli
parents: 17000
diff changeset
   354
  if (mcs != NULL) {
1ce8485b1907 8012052: java/lang/invoke/6987555/Test6987555.java crashes with assert(mcs != NULL) failed: MethodCounters cannot be NULL.
jiangli
parents: 17000
diff changeset
   355
    mcs->invocation_counter()->decay();
1ce8485b1907 8012052: java/lang/invoke/6987555/Test6987555.java crashes with assert(mcs != NULL) failed: MethodCounters cannot be NULL.
jiangli
parents: 17000
diff changeset
   356
    mcs->backedge_counter()->decay();
1ce8485b1907 8012052: java/lang/invoke/6987555/Test6987555.java crashes with assert(mcs != NULL) failed: MethodCounters cannot be NULL.
jiangli
parents: 17000
diff changeset
   357
  }
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   358
}
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   359
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 11572
diff changeset
   360
void NonTieredCompPolicy::disable_compilation(Method* method) {
17000
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   361
  MethodCounters* mcs = method->method_counters();
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   362
  if (mcs != NULL) {
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   363
    mcs->invocation_counter()->set_state(InvocationCounter::wait_for_nothing);
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   364
    mcs->backedge_counter()->set_state(InvocationCounter::wait_for_nothing);
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   365
  }
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   366
}
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   367
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   368
CompileTask* NonTieredCompPolicy::select_task(CompileQueue* compile_queue) {
35547
0ee84aa8e705 8146705: Improve JVMCI support for blocking compilation
dnsimon
parents: 33612
diff changeset
   369
  return select_task_helper(compile_queue);
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   370
}
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   371
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 11572
diff changeset
   372
bool NonTieredCompPolicy::is_mature(Method* method) {
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 11572
diff changeset
   373
  MethodData* mdo = method->method_data();
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   374
  assert(mdo != NULL, "Should be");
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   375
  uint current = mdo->mileage_of(method);
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   376
  uint initial = mdo->creation_mileage();
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   377
  if (current < initial)
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   378
    return true;  // some sort of overflow
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   379
  uint target;
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   380
  if (ProfileMaturityPercentage <= 0)
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   381
    target = (uint) -ProfileMaturityPercentage;  // absolute value
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   382
  else
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   383
    target = (uint)( (ProfileMaturityPercentage * CompileThreshold) / 100 );
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   384
  return (current >= initial + target);
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   385
}
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   386
33593
60764a78fa5c 8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents: 32582
diff changeset
   387
nmethod* NonTieredCompPolicy::event(const methodHandle& method, const methodHandle& inlinee, int branch_bci,
38133
78b95467b9f1 8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents: 37248
diff changeset
   388
                                    int bci, CompLevel comp_level, CompiledMethod* nm, JavaThread* thread) {
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   389
  assert(comp_level == CompLevel_none, "This should be only called from the interpreter");
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   390
  NOT_PRODUCT(trace_frequency_counter_overflow(method, branch_bci, bci));
11572
84afef481892 7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents: 10014
diff changeset
   391
  if (JvmtiExport::can_post_interpreter_events() && thread->is_interp_only_mode()) {
84afef481892 7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents: 10014
diff changeset
   392
    // If certain JVMTI events (e.g. frame pop event) are requested then the
84afef481892 7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents: 10014
diff changeset
   393
    // thread is forced to remain in interpreted code. This is
84afef481892 7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents: 10014
diff changeset
   394
    // implemented partly by a check in the run_compiled_code
84afef481892 7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents: 10014
diff changeset
   395
    // section of the interpreter whether we should skip running
84afef481892 7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents: 10014
diff changeset
   396
    // compiled code, and partly by skipping OSR compiles for
84afef481892 7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents: 10014
diff changeset
   397
    // interpreted-only threads.
84afef481892 7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents: 10014
diff changeset
   398
    if (bci != InvocationEntryBci) {
84afef481892 7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents: 10014
diff changeset
   399
      reset_counter_for_back_branch_event(method);
84afef481892 7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents: 10014
diff changeset
   400
      return NULL;
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   401
    }
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   402
  }
14477
95e66ea71f71 6830717: replay of compilations would help with debugging
minqi
parents: 14391
diff changeset
   403
  if (CompileTheWorld || ReplayCompiles) {
95e66ea71f71 6830717: replay of compilations would help with debugging
minqi
parents: 14391
diff changeset
   404
    // Don't trigger other compiles in testing mode
95e66ea71f71 6830717: replay of compilations would help with debugging
minqi
parents: 14391
diff changeset
   405
    if (bci == InvocationEntryBci) {
95e66ea71f71 6830717: replay of compilations would help with debugging
minqi
parents: 14391
diff changeset
   406
      reset_counter_for_invocation_event(method);
95e66ea71f71 6830717: replay of compilations would help with debugging
minqi
parents: 14391
diff changeset
   407
    } else {
95e66ea71f71 6830717: replay of compilations would help with debugging
minqi
parents: 14391
diff changeset
   408
      reset_counter_for_back_branch_event(method);
95e66ea71f71 6830717: replay of compilations would help with debugging
minqi
parents: 14391
diff changeset
   409
    }
95e66ea71f71 6830717: replay of compilations would help with debugging
minqi
parents: 14391
diff changeset
   410
    return NULL;
95e66ea71f71 6830717: replay of compilations would help with debugging
minqi
parents: 14391
diff changeset
   411
  }
95e66ea71f71 6830717: replay of compilations would help with debugging
minqi
parents: 14391
diff changeset
   412
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   413
  if (bci == InvocationEntryBci) {
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   414
    // when code cache is full, compilation gets switched off, UseCompiler
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   415
    // is set to false
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   416
    if (!method->has_compiled_code() && UseCompiler) {
11572
84afef481892 7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents: 10014
diff changeset
   417
      method_invocation_event(method, thread);
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   418
    } else {
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   419
      // Force counter overflow on method entry, even if no compilation
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   420
      // happened.  (The method_invocation_event call does this also.)
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   421
      reset_counter_for_invocation_event(method);
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   422
    }
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   423
    // compilation at an invocation overflow no longer goes and retries test for
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   424
    // compiled method. We always run the loser of the race as interpreted.
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   425
    // so return NULL
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   426
    return NULL;
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   427
  } else {
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   428
    // counter overflow in a loop => try to do on-stack-replacement
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   429
    nmethod* osr_nm = method->lookup_osr_nmethod_for(bci, CompLevel_highest_tier, true);
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   430
    NOT_PRODUCT(trace_osr_request(method, osr_nm, bci));
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   431
    // when code cache is full, we should not compile any more...
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   432
    if (osr_nm == NULL && UseCompiler) {
11572
84afef481892 7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents: 10014
diff changeset
   433
      method_back_branch_event(method, bci, thread);
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   434
      osr_nm = method->lookup_osr_nmethod_for(bci, CompLevel_highest_tier, true);
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   435
    }
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   436
    if (osr_nm == NULL) {
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   437
      reset_counter_for_back_branch_event(method);
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   438
      return NULL;
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   439
    }
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   440
    return osr_nm;
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   441
  }
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   442
  return NULL;
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   443
}
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   444
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   445
#ifndef PRODUCT
33593
60764a78fa5c 8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents: 32582
diff changeset
   446
void NonTieredCompPolicy::trace_frequency_counter_overflow(const methodHandle& m, int branch_bci, int bci) {
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   447
  if (TraceInvocationCounterOverflow) {
17000
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   448
    MethodCounters* mcs = m->method_counters();
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   449
    assert(mcs != NULL, "MethodCounters cannot be NULL for profiling");
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   450
    InvocationCounter* ic = mcs->invocation_counter();
11bf92e571a2 8010862: The Method counter fields used for profiling can be allocated lazily.
jiangli
parents: 14477
diff changeset
   451
    InvocationCounter* bc = mcs->backedge_counter();
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   452
    ResourceMark rm;
33604
ad1cd9269bd4 8139116: Fixes for warning "format not a string literal"
goetz
parents: 33593
diff changeset
   453
    if (bci == InvocationEntryBci) {
ad1cd9269bd4 8139116: Fixes for warning "format not a string literal"
goetz
parents: 33593
diff changeset
   454
      tty->print("comp-policy cntr ovfl @ %d in entry of ", bci);
ad1cd9269bd4 8139116: Fixes for warning "format not a string literal"
goetz
parents: 33593
diff changeset
   455
    } else {
ad1cd9269bd4 8139116: Fixes for warning "format not a string literal"
goetz
parents: 33593
diff changeset
   456
      tty->print("comp-policy cntr ovfl @ %d in loop of ", bci);
ad1cd9269bd4 8139116: Fixes for warning "format not a string literal"
goetz
parents: 33593
diff changeset
   457
    }
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   458
    m->print_value();
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   459
    tty->cr();
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   460
    ic->print();
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   461
    bc->print();
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   462
    if (ProfileInterpreter) {
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   463
      if (bci != InvocationEntryBci) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 11572
diff changeset
   464
        MethodData* mdo = m->method_data();
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   465
        if (mdo != NULL) {
48418
2207e2917a68 8191854: Null pointer dereference in methodData.hpp:462
dlong
parents: 47216
diff changeset
   466
          ProfileData *pd = mdo->bci_to_data(branch_bci);
2207e2917a68 8191854: Null pointer dereference in methodData.hpp:462
dlong
parents: 47216
diff changeset
   467
          if (pd == NULL) {
2207e2917a68 8191854: Null pointer dereference in methodData.hpp:462
dlong
parents: 47216
diff changeset
   468
            tty->print_cr("back branch count = N/A (missing ProfileData)");
2207e2917a68 8191854: Null pointer dereference in methodData.hpp:462
dlong
parents: 47216
diff changeset
   469
          } else {
2207e2917a68 8191854: Null pointer dereference in methodData.hpp:462
dlong
parents: 47216
diff changeset
   470
            tty->print_cr("back branch count = %d", pd->as_JumpData()->taken());
2207e2917a68 8191854: Null pointer dereference in methodData.hpp:462
dlong
parents: 47216
diff changeset
   471
          }
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   472
        }
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   473
      }
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   474
    }
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   475
  }
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   476
}
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   477
33593
60764a78fa5c 8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents: 32582
diff changeset
   478
void NonTieredCompPolicy::trace_osr_request(const methodHandle& method, nmethod* osr, int bci) {
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   479
  if (TraceOnStackReplacement) {
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   480
    ResourceMark rm;
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   481
    tty->print(osr != NULL ? "Reused OSR entry for " : "Requesting OSR entry for ");
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   482
    method->print_short_name(tty);
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   483
    tty->print_cr(" at bci %d", bci);
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   484
  }
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   485
}
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   486
#endif // !PRODUCT
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 6418
diff changeset
   487
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   488
// SimpleCompPolicy - compile current method
489c9b5090e2 Initial load
duke
parents:
diff changeset
   489
33593
60764a78fa5c 8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents: 32582
diff changeset
   490
void SimpleCompPolicy::method_invocation_event(const methodHandle& m, JavaThread* thread) {
13891
35dabd293e56 7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents: 13728
diff changeset
   491
  const int comp_level = CompLevel_highest_tier;
35dabd293e56 7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents: 13728
diff changeset
   492
  const int hot_count = m->invocation_count();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   493
  reset_counter_for_invocation_event(m);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   494
17126
42a942feeea2 8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents: 17002
diff changeset
   495
  if (is_compilation_enabled() && can_be_compiled(m, comp_level)) {
38133
78b95467b9f1 8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents: 37248
diff changeset
   496
    CompiledMethod* nm = m->code();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   497
    if (nm == NULL ) {
38218
f5ba1dea04eb 8153013: BlockingCompilation test times out
neliasso
parents: 38139
diff changeset
   498
      CompileBroker::compile_method(m, InvocationEntryBci, comp_level, m, hot_count, CompileTask::Reason_InvocationCount, thread);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   499
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   500
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   501
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   502
33593
60764a78fa5c 8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents: 32582
diff changeset
   503
void SimpleCompPolicy::method_back_branch_event(const methodHandle& m, int bci, JavaThread* thread) {
13891
35dabd293e56 7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents: 13728
diff changeset
   504
  const int comp_level = CompLevel_highest_tier;
35dabd293e56 7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents: 13728
diff changeset
   505
  const int hot_count = m->backedge_count();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   506
19332
ee4c8c2af356 8022832: Add WB APIs for OSR compilation
iignatyev
parents: 17126
diff changeset
   507
  if (is_compilation_enabled() && can_be_osr_compiled(m, comp_level)) {
38218
f5ba1dea04eb 8153013: BlockingCompilation test times out
neliasso
parents: 38139
diff changeset
   508
    CompileBroker::compile_method(m, bci, comp_level, m, hot_count, CompileTask::Reason_BackedgeCount, thread);
13891
35dabd293e56 7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents: 13728
diff changeset
   509
    NOT_PRODUCT(trace_osr_completion(m->lookup_osr_nmethod_for(bci, comp_level, true));)
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   510
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   511
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   512
// StackWalkCompPolicy - walk up stack to find a suitable method to compile
489c9b5090e2 Initial load
duke
parents:
diff changeset
   513
489c9b5090e2 Initial load
duke
parents:
diff changeset
   514
#ifdef COMPILER2
489c9b5090e2 Initial load
duke
parents:
diff changeset
   515
const char* StackWalkCompPolicy::_msg = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   516
489c9b5090e2 Initial load
duke
parents:
diff changeset
   517
489c9b5090e2 Initial load
duke
parents:
diff changeset
   518
// Consider m for compilation
33593
60764a78fa5c 8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents: 32582
diff changeset
   519
void StackWalkCompPolicy::method_invocation_event(const methodHandle& m, JavaThread* thread) {
13891
35dabd293e56 7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents: 13728
diff changeset
   520
  const int comp_level = CompLevel_highest_tier;
35dabd293e56 7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents: 13728
diff changeset
   521
  const int hot_count = m->invocation_count();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   522
  reset_counter_for_invocation_event(m);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   523
17126
42a942feeea2 8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents: 17002
diff changeset
   524
  if (is_compilation_enabled() && m->code() == NULL && can_be_compiled(m, comp_level)) {
11572
84afef481892 7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents: 10014
diff changeset
   525
    ResourceMark rm(thread);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   526
    frame       fr     = thread->last_frame();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   527
    assert(fr.is_interpreted_frame(), "must be interpreted");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   528
    assert(fr.interpreter_frame_method() == m(), "bad method");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   529
489c9b5090e2 Initial load
duke
parents:
diff changeset
   530
    if (TraceCompilationPolicy) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   531
      tty->print("method invocation trigger: ");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   532
      m->print_short_name(tty);
24424
2658d7834c6e 8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents: 24013
diff changeset
   533
      tty->print(" ( interpreted " INTPTR_FORMAT ", size=%d ) ", p2i((address)m()), m->code_size());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   534
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   535
    RegisterMap reg_map(thread, false);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   536
    javaVFrame* triggerVF = thread->last_java_vframe(&reg_map);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   537
    // triggerVF is the frame that triggered its counter
31521
f57b2ce43484 8079775: Java 9-fastdebug ia32 Error: Unimplemented with "-XX:CompilationPolicyChoice=1 -XX:-TieredCompilation" options
iveresov
parents: 24424
diff changeset
   538
    RFrame* first = new InterpretedRFrame(triggerVF->fr(), thread, m());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   539
489c9b5090e2 Initial load
duke
parents:
diff changeset
   540
    if (first->top_method()->code() != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   541
      // called obsolete method/nmethod -- no need to recompile
24424
2658d7834c6e 8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents: 24013
diff changeset
   542
      if (TraceCompilationPolicy) tty->print_cr(" --> " INTPTR_FORMAT, p2i(first->top_method()->code()));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   543
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   544
      if (TimeCompilationPolicy) accumulated_time()->start();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   545
      GrowableArray<RFrame*>* stack = new GrowableArray<RFrame*>(50);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   546
      stack->push(first);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   547
      RFrame* top = findTopInlinableFrame(stack);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   548
      if (TimeCompilationPolicy) accumulated_time()->stop();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   549
      assert(top != NULL, "findTopInlinableFrame returned null");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   550
      if (TraceCompilationPolicy) top->print();
13891
35dabd293e56 7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents: 13728
diff changeset
   551
      CompileBroker::compile_method(top->top_method(), InvocationEntryBci, comp_level,
38218
f5ba1dea04eb 8153013: BlockingCompilation test times out
neliasso
parents: 38139
diff changeset
   552
                                    m, hot_count, CompileTask::Reason_InvocationCount, thread);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   553
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   554
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   555
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   556
33593
60764a78fa5c 8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents: 32582
diff changeset
   557
void StackWalkCompPolicy::method_back_branch_event(const methodHandle& m, int bci, JavaThread* thread) {
13891
35dabd293e56 7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents: 13728
diff changeset
   558
  const int comp_level = CompLevel_highest_tier;
35dabd293e56 7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents: 13728
diff changeset
   559
  const int hot_count = m->backedge_count();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   560
19332
ee4c8c2af356 8022832: Add WB APIs for OSR compilation
iignatyev
parents: 17126
diff changeset
   561
  if (is_compilation_enabled() && can_be_osr_compiled(m, comp_level)) {
38218
f5ba1dea04eb 8153013: BlockingCompilation test times out
neliasso
parents: 38139
diff changeset
   562
    CompileBroker::compile_method(m, bci, comp_level, m, hot_count, CompileTask::Reason_BackedgeCount, thread);
13891
35dabd293e56 7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents: 13728
diff changeset
   563
    NOT_PRODUCT(trace_osr_completion(m->lookup_osr_nmethod_for(bci, comp_level, true));)
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   564
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   565
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   566
489c9b5090e2 Initial load
duke
parents:
diff changeset
   567
RFrame* StackWalkCompPolicy::findTopInlinableFrame(GrowableArray<RFrame*>* stack) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   568
  // go up the stack until finding a frame that (probably) won't be inlined
489c9b5090e2 Initial load
duke
parents:
diff changeset
   569
  // into its caller
489c9b5090e2 Initial load
duke
parents:
diff changeset
   570
  RFrame* current = stack->at(0); // current choice for stopping
489c9b5090e2 Initial load
duke
parents:
diff changeset
   571
  assert( current && !current->is_compiled(), "" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   572
  const char* msg = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   573
489c9b5090e2 Initial load
duke
parents:
diff changeset
   574
  while (1) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   575
489c9b5090e2 Initial load
duke
parents:
diff changeset
   576
    // before going up the stack further, check if doing so would get us into
489c9b5090e2 Initial load
duke
parents:
diff changeset
   577
    // compiled code
489c9b5090e2 Initial load
duke
parents:
diff changeset
   578
    RFrame* next = senderOf(current, stack);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   579
    if( !next )               // No next frame up the stack?
489c9b5090e2 Initial load
duke
parents:
diff changeset
   580
      break;                  // Then compile with current frame
489c9b5090e2 Initial load
duke
parents:
diff changeset
   581
31521
f57b2ce43484 8079775: Java 9-fastdebug ia32 Error: Unimplemented with "-XX:CompilationPolicyChoice=1 -XX:-TieredCompilation" options
iveresov
parents: 24424
diff changeset
   582
    Method* m = current->top_method();
f57b2ce43484 8079775: Java 9-fastdebug ia32 Error: Unimplemented with "-XX:CompilationPolicyChoice=1 -XX:-TieredCompilation" options
iveresov
parents: 24424
diff changeset
   583
    Method* next_m = next->top_method();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   584
489c9b5090e2 Initial load
duke
parents:
diff changeset
   585
    if (TraceCompilationPolicy && Verbose) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   586
      tty->print("[caller: ");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   587
      next_m->print_short_name(tty);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   588
      tty->print("] ");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   589
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   590
489c9b5090e2 Initial load
duke
parents:
diff changeset
   591
    if( !Inline ) {           // Inlining turned off
489c9b5090e2 Initial load
duke
parents:
diff changeset
   592
      msg = "Inlining turned off";
489c9b5090e2 Initial load
duke
parents:
diff changeset
   593
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   594
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   595
    if (next_m->is_not_compilable()) { // Did fail to compile this before/
489c9b5090e2 Initial load
duke
parents:
diff changeset
   596
      msg = "caller not compilable";
489c9b5090e2 Initial load
duke
parents:
diff changeset
   597
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   598
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   599
    if (next->num() > MaxRecompilationSearchLength) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   600
      // don't go up too high when searching for recompilees
489c9b5090e2 Initial load
duke
parents:
diff changeset
   601
      msg = "don't go up any further: > MaxRecompilationSearchLength";
489c9b5090e2 Initial load
duke
parents:
diff changeset
   602
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   603
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   604
    if (next->distance() > MaxInterpretedSearchLength) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   605
      // don't go up too high when searching for recompilees
489c9b5090e2 Initial load
duke
parents:
diff changeset
   606
      msg = "don't go up any further: next > MaxInterpretedSearchLength";
489c9b5090e2 Initial load
duke
parents:
diff changeset
   607
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   608
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   609
    // Compiled frame above already decided not to inline;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   610
    // do not recompile him.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   611
    if (next->is_compiled()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   612
      msg = "not going up into optimized code";
489c9b5090e2 Initial load
duke
parents:
diff changeset
   613
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   614
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   615
489c9b5090e2 Initial load
duke
parents:
diff changeset
   616
    // Interpreted frame above us was already compiled.  Do not force
489c9b5090e2 Initial load
duke
parents:
diff changeset
   617
    // a recompile, although if the frame above us runs long enough an
489c9b5090e2 Initial load
duke
parents:
diff changeset
   618
    // OSR might still happen.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   619
    if( current->is_interpreted() && next_m->has_compiled_code() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   620
      msg = "not going up -- already compiled caller";
489c9b5090e2 Initial load
duke
parents:
diff changeset
   621
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   622
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   623
489c9b5090e2 Initial load
duke
parents:
diff changeset
   624
    // Compute how frequent this call site is.  We have current method 'm'.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   625
    // We know next method 'next_m' is interpreted.  Find the call site and
489c9b5090e2 Initial load
duke
parents:
diff changeset
   626
    // check the various invocation counts.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   627
    int invcnt = 0;             // Caller counts
489c9b5090e2 Initial load
duke
parents:
diff changeset
   628
    if (ProfileInterpreter) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   629
      invcnt = next_m->interpreter_invocation_count();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   630
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   631
    int cnt = 0;                // Call site counts
489c9b5090e2 Initial load
duke
parents:
diff changeset
   632
    if (ProfileInterpreter && next_m->method_data() != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   633
      ResourceMark rm;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   634
      int bci = next->top_vframe()->bci();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   635
      ProfileData* data = next_m->method_data()->bci_to_data(bci);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   636
      if (data != NULL && data->is_CounterData())
489c9b5090e2 Initial load
duke
parents:
diff changeset
   637
        cnt = data->as_CounterData()->count();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   638
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   639
489c9b5090e2 Initial load
duke
parents:
diff changeset
   640
    // Caller counts / call-site counts; i.e. is this call site
489c9b5090e2 Initial load
duke
parents:
diff changeset
   641
    // a hot call site for method next_m?
489c9b5090e2 Initial load
duke
parents:
diff changeset
   642
    int freq = (invcnt) ? cnt/invcnt : cnt;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   643
489c9b5090e2 Initial load
duke
parents:
diff changeset
   644
    // Check size and frequency limits
489c9b5090e2 Initial load
duke
parents:
diff changeset
   645
    if ((msg = shouldInline(m, freq, cnt)) != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   646
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   647
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   648
    // Check inlining negative tests
489c9b5090e2 Initial load
duke
parents:
diff changeset
   649
    if ((msg = shouldNotInline(m)) != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   650
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   651
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   652
489c9b5090e2 Initial load
duke
parents:
diff changeset
   653
489c9b5090e2 Initial load
duke
parents:
diff changeset
   654
    // If the caller method is too big or something then we do not want to
489c9b5090e2 Initial load
duke
parents:
diff changeset
   655
    // compile it just to inline a method
17126
42a942feeea2 8012322: Tiered: CompilationPolicy::can_be_compiled(CompLevel_all) mistakenly return false
iignatyev
parents: 17002
diff changeset
   656
    if (!can_be_compiled(next_m, CompLevel_any)) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   657
      msg = "caller cannot be compiled";
489c9b5090e2 Initial load
duke
parents:
diff changeset
   658
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   659
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   660
489c9b5090e2 Initial load
duke
parents:
diff changeset
   661
    if( next_m->name() == vmSymbols::class_initializer_name() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   662
      msg = "do not compile class initializer (OSR ok)";
489c9b5090e2 Initial load
duke
parents:
diff changeset
   663
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   664
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   665
489c9b5090e2 Initial load
duke
parents:
diff changeset
   666
    if (TraceCompilationPolicy && Verbose) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   667
      tty->print("\n\t     check caller: ");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   668
      next_m->print_short_name(tty);
31521
f57b2ce43484 8079775: Java 9-fastdebug ia32 Error: Unimplemented with "-XX:CompilationPolicyChoice=1 -XX:-TieredCompilation" options
iveresov
parents: 24424
diff changeset
   669
      tty->print(" ( interpreted " INTPTR_FORMAT ", size=%d ) ", p2i((address)next_m), next_m->code_size());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   670
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   671
489c9b5090e2 Initial load
duke
parents:
diff changeset
   672
    current = next;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   673
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   674
489c9b5090e2 Initial load
duke
parents:
diff changeset
   675
  assert( !current || !current->is_compiled(), "" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   676
489c9b5090e2 Initial load
duke
parents:
diff changeset
   677
  if (TraceCompilationPolicy && msg) tty->print("(%s)\n", msg);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   678
489c9b5090e2 Initial load
duke
parents:
diff changeset
   679
  return current;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   680
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   681
489c9b5090e2 Initial load
duke
parents:
diff changeset
   682
RFrame* StackWalkCompPolicy::senderOf(RFrame* rf, GrowableArray<RFrame*>* stack) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   683
  RFrame* sender = rf->caller();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   684
  if (sender && sender->num() == stack->length()) stack->push(sender);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   685
  return sender;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   686
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   687
489c9b5090e2 Initial load
duke
parents:
diff changeset
   688
33593
60764a78fa5c 8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents: 32582
diff changeset
   689
const char* StackWalkCompPolicy::shouldInline(const methodHandle& m, float freq, int cnt) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   690
  // Allows targeted inlining
489c9b5090e2 Initial load
duke
parents:
diff changeset
   691
  // positive filter: should send be inlined?  returns NULL (--> yes)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   692
  // or rejection msg
489c9b5090e2 Initial load
duke
parents:
diff changeset
   693
  int max_size = MaxInlineSize;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   694
  int cost = m->code_size();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   695
489c9b5090e2 Initial load
duke
parents:
diff changeset
   696
  // Check for too many throws (and not too huge)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   697
  if (m->interpreter_throwout_count() > InlineThrowCount && cost < InlineThrowMaxSize ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   698
    return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   699
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   700
489c9b5090e2 Initial load
duke
parents:
diff changeset
   701
  // bump the max size if the call is frequent
489c9b5090e2 Initial load
duke
parents:
diff changeset
   702
  if ((freq >= InlineFrequencyRatio) || (cnt >= InlineFrequencyCount)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   703
    if (TraceFrequencyInlining) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   704
      tty->print("(Inlined frequent method)\n");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   705
      m->print();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   706
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   707
    max_size = FreqInlineSize;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   708
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   709
  if (cost > max_size) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   710
    return (_msg = "too big");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   711
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   712
  return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   713
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   714
489c9b5090e2 Initial load
duke
parents:
diff changeset
   715
33593
60764a78fa5c 8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents: 32582
diff changeset
   716
const char* StackWalkCompPolicy::shouldNotInline(const methodHandle& m) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   717
  // negative filter: should send NOT be inlined?  returns NULL (--> inline) or rejection msg
489c9b5090e2 Initial load
duke
parents:
diff changeset
   718
  if (m->is_abstract()) return (_msg = "abstract method");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   719
  // note: we allow ik->is_abstract()
14391
df0a1573d5bd 8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents: 13891
diff changeset
   720
  if (!m->method_holder()->is_initialized()) return (_msg = "method holder not initialized");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   721
  if (m->is_native()) return (_msg = "native method");
38133
78b95467b9f1 8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents: 37248
diff changeset
   722
  CompiledMethod* m_code = m->code();
6418
6671edbd230e 6978355: renaming for 6961697
twisti
parents: 5547
diff changeset
   723
  if (m_code != NULL && m_code->code_size() > InlineSmallCode)
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   724
    return (_msg = "already compiled into a big method");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   725
489c9b5090e2 Initial load
duke
parents:
diff changeset
   726
  // use frequency-based objections only for non-trivial methods
489c9b5090e2 Initial load
duke
parents:
diff changeset
   727
  if (m->code_size() <= MaxTrivialSize) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   728
  if (UseInterpreter) {     // don't use counts with -Xcomp
489c9b5090e2 Initial load
duke
parents:
diff changeset
   729
    if ((m->code() == NULL) && m->was_never_executed()) return (_msg = "never executed");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   730
    if (!m->was_executed_more_than(MIN2(MinInliningThreshold, CompileThreshold >> 1))) return (_msg = "executed < MinInliningThreshold times");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   731
  }
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 11572
diff changeset
   732
  if (Method::has_unloaded_classes_in_signature(m, JavaThread::current())) return (_msg = "unloaded signature classes");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   733
489c9b5090e2 Initial load
duke
parents:
diff changeset
   734
  return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   735
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   736
489c9b5090e2 Initial load
duke
parents:
diff changeset
   737
489c9b5090e2 Initial load
duke
parents:
diff changeset
   738
489c9b5090e2 Initial load
duke
parents:
diff changeset
   739
#endif // COMPILER2