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