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