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