hotspot/src/share/vm/runtime/advancedThresholdPolicy.cpp
author thartmann
Thu, 04 Aug 2016 08:58:22 +0200
changeset 40364 5bf8fa4fddfd
parent 38666 5ff19807abd5
child 42650 1f304d0c888b
permissions -rw-r--r--
8162881: Effect of -XX:CICompilerCount depends on ordering of other flags Summary: Value of CICompilerCount should be range checked after ergo. Don't create C2 compiler threads with TieredStopAtLevel=1. Reviewed-by: kvn
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
     1
/*
36597
ee256e343585 8150646: Add support for blocking compiles though whitebox API
simonis
parents: 36556
diff changeset
     2
 * Copyright (c) 2010, 2016, Oracle and/or its affiliates. All rights reserved.
9625
822a93889b58 7043564: compile warning and copyright fixes
iveresov
parents: 8667
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
822a93889b58 7043564: compile warning and copyright fixes
iveresov
parents: 8667
diff changeset
     4
 *
822a93889b58 7043564: compile warning and copyright fixes
iveresov
parents: 8667
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
822a93889b58 7043564: compile warning and copyright fixes
iveresov
parents: 8667
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
822a93889b58 7043564: compile warning and copyright fixes
iveresov
parents: 8667
diff changeset
     7
 * published by the Free Software Foundation.
822a93889b58 7043564: compile warning and copyright fixes
iveresov
parents: 8667
diff changeset
     8
 *
822a93889b58 7043564: compile warning and copyright fixes
iveresov
parents: 8667
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
822a93889b58 7043564: compile warning and copyright fixes
iveresov
parents: 8667
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
822a93889b58 7043564: compile warning and copyright fixes
iveresov
parents: 8667
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
822a93889b58 7043564: compile warning and copyright fixes
iveresov
parents: 8667
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
822a93889b58 7043564: compile warning and copyright fixes
iveresov
parents: 8667
diff changeset
    13
 * accompanied this code).
822a93889b58 7043564: compile warning and copyright fixes
iveresov
parents: 8667
diff changeset
    14
 *
822a93889b58 7043564: compile warning and copyright fixes
iveresov
parents: 8667
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
822a93889b58 7043564: compile warning and copyright fixes
iveresov
parents: 8667
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
822a93889b58 7043564: compile warning and copyright fixes
iveresov
parents: 8667
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
822a93889b58 7043564: compile warning and copyright fixes
iveresov
parents: 8667
diff changeset
    18
 *
822a93889b58 7043564: compile warning and copyright fixes
iveresov
parents: 8667
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
822a93889b58 7043564: compile warning and copyright fixes
iveresov
parents: 8667
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
822a93889b58 7043564: compile warning and copyright fixes
iveresov
parents: 8667
diff changeset
    21
 * questions.
822a93889b58 7043564: compile warning and copyright fixes
iveresov
parents: 8667
diff changeset
    22
 *
822a93889b58 7043564: compile warning and copyright fixes
iveresov
parents: 8667
diff changeset
    23
 */
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    24
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    25
#include "precompiled.hpp"
25715
d5a8dbdc5150 8049325: Introduce and clean up umbrella headers for the files in the cpu subdirectories.
goetz
parents: 24443
diff changeset
    26
#include "code/codeCache.hpp"
33160
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31053
diff changeset
    27
#include "compiler/compileTask.hpp"
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    28
#include "runtime/advancedThresholdPolicy.hpp"
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    29
#include "runtime/simpleThresholdPolicy.inline.hpp"
38666
5ff19807abd5 8152311: [JVMCI] allow JVMCI compiler to change the compilation policy for a method
dnsimon
parents: 38218
diff changeset
    30
#if INCLUDE_JVMCI
5ff19807abd5 8152311: [JVMCI] allow JVMCI compiler to change the compilation policy for a method
dnsimon
parents: 38218
diff changeset
    31
#include "jvmci/jvmciRuntime.hpp"
5ff19807abd5 8152311: [JVMCI] allow JVMCI compiler to change the compilation policy for a method
dnsimon
parents: 38218
diff changeset
    32
#endif
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    33
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    34
#ifdef TIERED
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    35
// Print an event.
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    36
void AdvancedThresholdPolicy::print_specific(EventType type, methodHandle mh, methodHandle imh,
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    37
                                             int bci, CompLevel level) {
13891
35dabd293e56 7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents: 13728
diff changeset
    38
  tty->print(" rate=");
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    39
  if (mh->prev_time() == 0) tty->print("n/a");
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    40
  else tty->print("%f", mh->rate());
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    41
13891
35dabd293e56 7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents: 13728
diff changeset
    42
  tty->print(" k=%.2lf,%.2lf", threshold_scale(CompLevel_full_profile, Tier3LoadFeedback),
35dabd293e56 7200001: failed C1 OSR compile doesn't get recompiled with C2
twisti
parents: 13728
diff changeset
    43
                               threshold_scale(CompLevel_full_optimization, Tier4LoadFeedback));
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    44
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    45
}
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    46
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    47
void AdvancedThresholdPolicy::initialize() {
36556
7f092a7ce938 8150839: Adjust the number of compiler threads for 32-bit platforms
zmajo
parents: 35547
diff changeset
    48
  int count = CICompilerCount;
7f092a7ce938 8150839: Adjust the number of compiler threads for 32-bit platforms
zmajo
parents: 35547
diff changeset
    49
#ifdef _LP64
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    50
  // Turn on ergonomic compiler count selection
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    51
  if (FLAG_IS_DEFAULT(CICompilerCountPerCPU) && FLAG_IS_DEFAULT(CICompilerCount)) {
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    52
    FLAG_SET_DEFAULT(CICompilerCountPerCPU, true);
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    53
  }
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    54
  if (CICompilerCountPerCPU) {
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    55
    // Simple log n seems to grow too slowly for tiered, try something faster: log n * log log n
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    56
    int log_cpu = log2_intptr(os::active_processor_count());
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    57
    int loglog_cpu = log2_intptr(MAX2(log_cpu, 1));
40364
5bf8fa4fddfd 8162881: Effect of -XX:CICompilerCount depends on ordering of other flags
thartmann
parents: 38666
diff changeset
    58
    count = MAX2(log_cpu * loglog_cpu * 3 / 2, 2);
5bf8fa4fddfd 8162881: Effect of -XX:CICompilerCount depends on ordering of other flags
thartmann
parents: 38666
diff changeset
    59
    FLAG_SET_ERGO(intx, CICompilerCount, count);
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    60
  }
36556
7f092a7ce938 8150839: Adjust the number of compiler threads for 32-bit platforms
zmajo
parents: 35547
diff changeset
    61
#else
7f092a7ce938 8150839: Adjust the number of compiler threads for 32-bit platforms
zmajo
parents: 35547
diff changeset
    62
  // On 32-bit systems, the number of compiler threads is limited to 3.
7f092a7ce938 8150839: Adjust the number of compiler threads for 32-bit platforms
zmajo
parents: 35547
diff changeset
    63
  // On these systems, the virtual address space available to the JVM
7f092a7ce938 8150839: Adjust the number of compiler threads for 32-bit platforms
zmajo
parents: 35547
diff changeset
    64
  // is usually limited to 2-4 GB (the exact value depends on the platform).
7f092a7ce938 8150839: Adjust the number of compiler threads for 32-bit platforms
zmajo
parents: 35547
diff changeset
    65
  // As the compilers (especially C2) can consume a large amount of
7f092a7ce938 8150839: Adjust the number of compiler threads for 32-bit platforms
zmajo
parents: 35547
diff changeset
    66
  // memory, scaling the number of compiler threads with the number of
7f092a7ce938 8150839: Adjust the number of compiler threads for 32-bit platforms
zmajo
parents: 35547
diff changeset
    67
  // available cores can result in the exhaustion of the address space
7f092a7ce938 8150839: Adjust the number of compiler threads for 32-bit platforms
zmajo
parents: 35547
diff changeset
    68
  /// available to the VM and thus cause the VM to crash.
7f092a7ce938 8150839: Adjust the number of compiler threads for 32-bit platforms
zmajo
parents: 35547
diff changeset
    69
  if (FLAG_IS_DEFAULT(CICompilerCount)) {
7f092a7ce938 8150839: Adjust the number of compiler threads for 32-bit platforms
zmajo
parents: 35547
diff changeset
    70
    count = 3;
40364
5bf8fa4fddfd 8162881: Effect of -XX:CICompilerCount depends on ordering of other flags
thartmann
parents: 38666
diff changeset
    71
    FLAG_SET_ERGO(intx, CICompilerCount, count);
36556
7f092a7ce938 8150839: Adjust the number of compiler threads for 32-bit platforms
zmajo
parents: 35547
diff changeset
    72
  }
7f092a7ce938 8150839: Adjust the number of compiler threads for 32-bit platforms
zmajo
parents: 35547
diff changeset
    73
#endif
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    74
40364
5bf8fa4fddfd 8162881: Effect of -XX:CICompilerCount depends on ordering of other flags
thartmann
parents: 38666
diff changeset
    75
  if (TieredStopAtLevel < CompLevel_full_optimization) {
5bf8fa4fddfd 8162881: Effect of -XX:CICompilerCount depends on ordering of other flags
thartmann
parents: 38666
diff changeset
    76
    // No C2 compiler thread required
5bf8fa4fddfd 8162881: Effect of -XX:CICompilerCount depends on ordering of other flags
thartmann
parents: 38666
diff changeset
    77
    set_c1_count(count);
5bf8fa4fddfd 8162881: Effect of -XX:CICompilerCount depends on ordering of other flags
thartmann
parents: 38666
diff changeset
    78
  } else {
5bf8fa4fddfd 8162881: Effect of -XX:CICompilerCount depends on ordering of other flags
thartmann
parents: 38666
diff changeset
    79
    set_c1_count(MAX2(count / 3, 1));
5bf8fa4fddfd 8162881: Effect of -XX:CICompilerCount depends on ordering of other flags
thartmann
parents: 38666
diff changeset
    80
    set_c2_count(MAX2(count - c1_count(), 1));
5bf8fa4fddfd 8162881: Effect of -XX:CICompilerCount depends on ordering of other flags
thartmann
parents: 38666
diff changeset
    81
  }
5bf8fa4fddfd 8162881: Effect of -XX:CICompilerCount depends on ordering of other flags
thartmann
parents: 38666
diff changeset
    82
  assert(count == c1_count() + c2_count(), "inconsistent compiler thread count");
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    83
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    84
  // Some inlining tuning
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    85
#ifdef X86
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    86
  if (FLAG_IS_DEFAULT(InlineSmallCode)) {
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    87
    FLAG_SET_DEFAULT(InlineSmallCode, 2000);
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    88
  }
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    89
#endif
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    90
31053
91b652361bce 8085805: aarch64: AdvancedThresholdPolicy lacks tuning of InlineSmallCode size
enevill
parents: 29335
diff changeset
    91
#if defined SPARC || defined AARCH64
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    92
  if (FLAG_IS_DEFAULT(InlineSmallCode)) {
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    93
    FLAG_SET_DEFAULT(InlineSmallCode, 2500);
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    94
  }
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    95
#endif
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    96
17617
4e330bce1812 8012371: Adjust Tiered compile threshold according to available space in code cache
anoll
parents: 17000
diff changeset
    97
  set_increase_threshold_at_ratio();
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    98
  set_start_time(os::javaTimeMillis());
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
    99
}
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   100
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   101
// update_rate() is called from select_task() while holding a compile queue lock.
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 11572
diff changeset
   102
void AdvancedThresholdPolicy::update_rate(jlong t, Method* m) {
24443
7aaf1b306b55 8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents: 24321
diff changeset
   103
  // Skip update if counters are absent.
7aaf1b306b55 8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents: 24321
diff changeset
   104
  // Can't allocate them since we are holding compile queue lock.
7aaf1b306b55 8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents: 24321
diff changeset
   105
  if (m->method_counters() == NULL)  return;
7aaf1b306b55 8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents: 24321
diff changeset
   106
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   107
  if (is_old(m)) {
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   108
    // We don't remove old methods from the queue,
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   109
    // so we can just zero the rate.
24443
7aaf1b306b55 8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents: 24321
diff changeset
   110
    m->set_rate(0);
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   111
    return;
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   112
  }
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   113
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   114
  // We don't update the rate if we've just came out of a safepoint.
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   115
  // delta_s is the time since last safepoint in milliseconds.
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   116
  jlong delta_s = t - SafepointSynchronize::end_of_last_safepoint();
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   117
  jlong delta_t = t - (m->prev_time() != 0 ? m->prev_time() : start_time()); // milliseconds since the last measurement
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   118
  // How many events were there since the last time?
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   119
  int event_count = m->invocation_count() + m->backedge_count();
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   120
  int delta_e = event_count - m->prev_event_count();
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   121
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   122
  // We should be running for at least 1ms.
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   123
  if (delta_s >= TieredRateUpdateMinTime) {
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   124
    // And we must've taken the previous point at least 1ms before.
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   125
    if (delta_t >= TieredRateUpdateMinTime && delta_e > 0) {
24443
7aaf1b306b55 8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents: 24321
diff changeset
   126
      m->set_prev_time(t);
7aaf1b306b55 8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents: 24321
diff changeset
   127
      m->set_prev_event_count(event_count);
7aaf1b306b55 8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents: 24321
diff changeset
   128
      m->set_rate((float)delta_e / (float)delta_t); // Rate is events per millisecond
7aaf1b306b55 8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents: 24321
diff changeset
   129
    } else {
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   130
      if (delta_t > TieredRateUpdateMaxTime && delta_e == 0) {
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   131
        // If nothing happened for 25ms, zero the rate. Don't modify prev values.
24443
7aaf1b306b55 8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents: 24321
diff changeset
   132
        m->set_rate(0);
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   133
      }
24443
7aaf1b306b55 8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents: 24321
diff changeset
   134
    }
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   135
  }
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   136
}
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   137
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   138
// Check if this method has been stale from a given number of milliseconds.
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   139
// See select_task().
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 11572
diff changeset
   140
bool AdvancedThresholdPolicy::is_stale(jlong t, jlong timeout, Method* m) {
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   141
  jlong delta_s = t - SafepointSynchronize::end_of_last_safepoint();
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   142
  jlong delta_t = t - m->prev_time();
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   143
  if (delta_t > timeout && delta_s > timeout) {
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   144
    int event_count = m->invocation_count() + m->backedge_count();
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   145
    int delta_e = event_count - m->prev_event_count();
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   146
    // Return true if there were no events.
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   147
    return delta_e == 0;
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   148
  }
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   149
  return false;
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   150
}
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   151
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   152
// We don't remove old methods from the compile queue even if they have
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   153
// very low activity. See select_task().
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 11572
diff changeset
   154
bool AdvancedThresholdPolicy::is_old(Method* method) {
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   155
  return method->invocation_count() > 50000 || method->backedge_count() > 500000;
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   156
}
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   157
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 11572
diff changeset
   158
double AdvancedThresholdPolicy::weight(Method* method) {
35155
db692d3ebbcc 8145096: Undefined behaviour in HotSpot
aph
parents: 33593
diff changeset
   159
  return (double)(method->rate() + 1) *
db692d3ebbcc 8145096: Undefined behaviour in HotSpot
aph
parents: 33593
diff changeset
   160
    (method->invocation_count() + 1) * (method->backedge_count() + 1);
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   161
}
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   162
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   163
// Apply heuristics and return true if x should be compiled before y
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 11572
diff changeset
   164
bool AdvancedThresholdPolicy::compare_methods(Method* x, Method* y) {
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   165
  if (x->highest_comp_level() > y->highest_comp_level()) {
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   166
    // recompilation after deopt
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   167
    return true;
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   168
  } else
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   169
    if (x->highest_comp_level() == y->highest_comp_level()) {
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   170
      if (weight(x) > weight(y)) {
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   171
        return true;
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   172
      }
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   173
    }
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   174
  return false;
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   175
}
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   176
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   177
// Is method profiled enough?
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 11572
diff changeset
   178
bool AdvancedThresholdPolicy::is_method_profiled(Method* method) {
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 11572
diff changeset
   179
  MethodData* mdo = method->method_data();
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   180
  if (mdo != NULL) {
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   181
    int i = mdo->invocation_count_delta();
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   182
    int b = mdo->backedge_count_delta();
28650
772aaab2582f 8059606: Enable per-method usage of CompileThresholdScaling (per-method compilation thresholds)
zmajo
parents: 27643
diff changeset
   183
    return call_predicate_helper<CompLevel_full_profile>(i, b, 1, method);
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   184
  }
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   185
  return false;
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   186
}
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   187
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   188
// Called with the queue locked and with at least one element
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   189
CompileTask* AdvancedThresholdPolicy::select_task(CompileQueue* compile_queue) {
35547
0ee84aa8e705 8146705: Improve JVMCI support for blocking compilation
dnsimon
parents: 35155
diff changeset
   190
  CompileTask *max_blocking_task = NULL;
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   191
  CompileTask *max_task = NULL;
13952
e3cf184080bc 8000213: NPG: Should have renamed arrayKlass and typeArrayKlass
coleenp
parents: 13891
diff changeset
   192
  Method* max_method = NULL;
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   193
  jlong t = os::javaTimeMillis();
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   194
  // Iterate through the queue and find a method with a maximum rate.
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   195
  for (CompileTask* task = compile_queue->first(); task != NULL;) {
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   196
    CompileTask* next_task = task->next();
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 11572
diff changeset
   197
    Method* method = task->method();
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 11572
diff changeset
   198
    update_rate(t, method);
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   199
    if (max_task == NULL) {
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   200
      max_task = task;
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   201
      max_method = method;
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   202
    } else {
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   203
      // If a method has been stale for some time, remove it from the queue.
38218
f5ba1dea04eb 8153013: BlockingCompilation test times out
neliasso
parents: 38133
diff changeset
   204
      // Blocking tasks and tasks submitted from whitebox API don't become stale
f5ba1dea04eb 8153013: BlockingCompilation test times out
neliasso
parents: 38133
diff changeset
   205
      if (task->can_become_stale() && is_stale(t, TieredCompileTaskTimeout, method) && !is_old(method)) {
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   206
        if (PrintTieredEvents) {
10014
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   207
          print_event(REMOVE_FROM_QUEUE, method, method, task->osr_bci(), (CompLevel)task->comp_level());
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   208
        }
33160
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31053
diff changeset
   209
        task->log_task_dequeued("stale");
24443
7aaf1b306b55 8023461: Thread holding lock at safepoint that vm can block on: MethodCompileQueue_lock
vlivanov
parents: 24321
diff changeset
   210
        compile_queue->remove_and_mark_stale(task);
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   211
        method->clear_queued_for_compilation();
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   212
        task = next_task;
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   213
        continue;
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   214
      }
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   215
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   216
      // Select a method with a higher rate
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 11572
diff changeset
   217
      if (compare_methods(method, max_method)) {
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   218
        max_task = task;
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   219
        max_method = method;
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   220
      }
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   221
    }
36597
ee256e343585 8150646: Add support for blocking compiles though whitebox API
simonis
parents: 36556
diff changeset
   222
ee256e343585 8150646: Add support for blocking compiles though whitebox API
simonis
parents: 36556
diff changeset
   223
    if (task->is_blocking()) {
35547
0ee84aa8e705 8146705: Improve JVMCI support for blocking compilation
dnsimon
parents: 35155
diff changeset
   224
      if (max_blocking_task == NULL || compare_methods(method, max_blocking_task->method())) {
0ee84aa8e705 8146705: Improve JVMCI support for blocking compilation
dnsimon
parents: 35155
diff changeset
   225
        max_blocking_task = task;
0ee84aa8e705 8146705: Improve JVMCI support for blocking compilation
dnsimon
parents: 35155
diff changeset
   226
      }
0ee84aa8e705 8146705: Improve JVMCI support for blocking compilation
dnsimon
parents: 35155
diff changeset
   227
    }
36597
ee256e343585 8150646: Add support for blocking compiles though whitebox API
simonis
parents: 36556
diff changeset
   228
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   229
    task = next_task;
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   230
  }
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   231
36597
ee256e343585 8150646: Add support for blocking compiles though whitebox API
simonis
parents: 36556
diff changeset
   232
  if (max_blocking_task != NULL) {
ee256e343585 8150646: Add support for blocking compiles though whitebox API
simonis
parents: 36556
diff changeset
   233
    // In blocking compilation mode, the CompileBroker will make
ee256e343585 8150646: Add support for blocking compiles though whitebox API
simonis
parents: 36556
diff changeset
   234
    // compilations submitted by a JVMCI compiler thread non-blocking. These
ee256e343585 8150646: Add support for blocking compiles though whitebox API
simonis
parents: 36556
diff changeset
   235
    // compilations should be scheduled after all blocking compilations
ee256e343585 8150646: Add support for blocking compiles though whitebox API
simonis
parents: 36556
diff changeset
   236
    // to service non-compiler related compilations sooner and reduce the
ee256e343585 8150646: Add support for blocking compiles though whitebox API
simonis
parents: 36556
diff changeset
   237
    // chance of such compilations timing out.
ee256e343585 8150646: Add support for blocking compiles though whitebox API
simonis
parents: 36556
diff changeset
   238
    max_task = max_blocking_task;
ee256e343585 8150646: Add support for blocking compiles though whitebox API
simonis
parents: 36556
diff changeset
   239
    max_method = max_task->method();
33160
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31053
diff changeset
   240
  }
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31053
diff changeset
   241
10250
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   242
  if (max_task->comp_level() == CompLevel_full_profile && TieredStopAtLevel > CompLevel_full_profile
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 11572
diff changeset
   243
      && is_method_profiled(max_method)) {
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   244
    max_task->set_comp_level(CompLevel_limited_profile);
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   245
    if (PrintTieredEvents) {
10014
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   246
      print_event(UPDATE_IN_QUEUE, max_method, max_method, max_task->osr_bci(), (CompLevel)max_task->comp_level());
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   247
    }
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   248
  }
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   249
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   250
  return max_task;
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   251
}
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   252
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   253
double AdvancedThresholdPolicy::threshold_scale(CompLevel level, int feedback_k) {
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   254
  double queue_size = CompileBroker::queue_size(level);
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   255
  int comp_count = compiler_count(level);
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   256
  double k = queue_size / (feedback_k * comp_count) + 1;
17617
4e330bce1812 8012371: Adjust Tiered compile threshold according to available space in code cache
anoll
parents: 17000
diff changeset
   257
4e330bce1812 8012371: Adjust Tiered compile threshold according to available space in code cache
anoll
parents: 17000
diff changeset
   258
  // Increase C1 compile threshold when the code cache is filled more
4e330bce1812 8012371: Adjust Tiered compile threshold according to available space in code cache
anoll
parents: 17000
diff changeset
   259
  // than specified by IncreaseFirstTierCompileThresholdAt percentage.
4e330bce1812 8012371: Adjust Tiered compile threshold according to available space in code cache
anoll
parents: 17000
diff changeset
   260
  // The main intention is to keep enough free space for C2 compiled code
4e330bce1812 8012371: Adjust Tiered compile threshold according to available space in code cache
anoll
parents: 17000
diff changeset
   261
  // to achieve peak performance if the code cache is under stress.
4e330bce1812 8012371: Adjust Tiered compile threshold according to available space in code cache
anoll
parents: 17000
diff changeset
   262
  if ((TieredStopAtLevel == CompLevel_full_optimization) && (level != CompLevel_full_optimization))  {
26796
666464578742 8015774: Add support for multiple code heaps
thartmann
parents: 25715
diff changeset
   263
    double current_reverse_free_ratio = CodeCache::reverse_free_ratio(CodeCache::get_code_blob_type(level));
17617
4e330bce1812 8012371: Adjust Tiered compile threshold according to available space in code cache
anoll
parents: 17000
diff changeset
   264
    if (current_reverse_free_ratio > _increase_threshold_at_ratio) {
4e330bce1812 8012371: Adjust Tiered compile threshold according to available space in code cache
anoll
parents: 17000
diff changeset
   265
      k *= exp(current_reverse_free_ratio - _increase_threshold_at_ratio);
4e330bce1812 8012371: Adjust Tiered compile threshold according to available space in code cache
anoll
parents: 17000
diff changeset
   266
    }
4e330bce1812 8012371: Adjust Tiered compile threshold according to available space in code cache
anoll
parents: 17000
diff changeset
   267
  }
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   268
  return k;
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   269
}
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   270
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   271
// Call and loop predicates determine whether a transition to a higher
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   272
// compilation level should be performed (pointers to predicate functions
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   273
// are passed to common()).
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   274
// Tier?LoadFeedback is basically a coefficient that determines of
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   275
// how many methods per compiler thread can be in the queue before
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   276
// the threshold values double.
28650
772aaab2582f 8059606: Enable per-method usage of CompileThresholdScaling (per-method compilation thresholds)
zmajo
parents: 27643
diff changeset
   277
bool AdvancedThresholdPolicy::loop_predicate(int i, int b, CompLevel cur_level, Method* method) {
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   278
  switch(cur_level) {
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   279
  case CompLevel_none:
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   280
  case CompLevel_limited_profile: {
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   281
    double k = threshold_scale(CompLevel_full_profile, Tier3LoadFeedback);
28650
772aaab2582f 8059606: Enable per-method usage of CompileThresholdScaling (per-method compilation thresholds)
zmajo
parents: 27643
diff changeset
   282
    return loop_predicate_helper<CompLevel_none>(i, b, k, method);
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   283
  }
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   284
  case CompLevel_full_profile: {
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   285
    double k = threshold_scale(CompLevel_full_optimization, Tier4LoadFeedback);
28650
772aaab2582f 8059606: Enable per-method usage of CompileThresholdScaling (per-method compilation thresholds)
zmajo
parents: 27643
diff changeset
   286
    return loop_predicate_helper<CompLevel_full_profile>(i, b, k, method);
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   287
  }
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   288
  default:
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   289
    return true;
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   290
  }
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   291
}
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   292
28650
772aaab2582f 8059606: Enable per-method usage of CompileThresholdScaling (per-method compilation thresholds)
zmajo
parents: 27643
diff changeset
   293
bool AdvancedThresholdPolicy::call_predicate(int i, int b, CompLevel cur_level, Method* method) {
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   294
  switch(cur_level) {
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   295
  case CompLevel_none:
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   296
  case CompLevel_limited_profile: {
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   297
    double k = threshold_scale(CompLevel_full_profile, Tier3LoadFeedback);
28650
772aaab2582f 8059606: Enable per-method usage of CompileThresholdScaling (per-method compilation thresholds)
zmajo
parents: 27643
diff changeset
   298
    return call_predicate_helper<CompLevel_none>(i, b, k, method);
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   299
  }
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   300
  case CompLevel_full_profile: {
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   301
    double k = threshold_scale(CompLevel_full_optimization, Tier4LoadFeedback);
28650
772aaab2582f 8059606: Enable per-method usage of CompileThresholdScaling (per-method compilation thresholds)
zmajo
parents: 27643
diff changeset
   302
    return call_predicate_helper<CompLevel_full_profile>(i, b, k, method);
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   303
  }
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   304
  default:
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   305
    return true;
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   306
  }
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   307
}
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   308
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   309
// If a method is old enough and is still in the interpreter we would want to
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   310
// start profiling without waiting for the compiled method to arrive.
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   311
// We also take the load on compilers into the account.
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 11572
diff changeset
   312
bool AdvancedThresholdPolicy::should_create_mdo(Method* method, CompLevel cur_level) {
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   313
  if (cur_level == CompLevel_none &&
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   314
      CompileBroker::queue_size(CompLevel_full_optimization) <=
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   315
      Tier3DelayOn * compiler_count(CompLevel_full_optimization)) {
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   316
    int i = method->invocation_count();
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   317
    int b = method->backedge_count();
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   318
    double k = Tier0ProfilingStartPercentage / 100.0;
28650
772aaab2582f 8059606: Enable per-method usage of CompileThresholdScaling (per-method compilation thresholds)
zmajo
parents: 27643
diff changeset
   319
    return call_predicate_helper<CompLevel_none>(i, b, k, method) || loop_predicate_helper<CompLevel_none>(i, b, k, method);
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   320
  }
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   321
  return false;
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   322
}
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   323
10014
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   324
// Inlining control: if we're compiling a profiled method with C1 and the callee
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   325
// is known to have OSRed in a C2 version, don't inline it.
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   326
bool AdvancedThresholdPolicy::should_not_inline(ciEnv* env, ciMethod* callee) {
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   327
  CompLevel comp_level = (CompLevel)env->comp_level();
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   328
  if (comp_level == CompLevel_full_profile ||
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   329
      comp_level == CompLevel_limited_profile) {
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   330
    return callee->highest_osr_comp_level() == CompLevel_full_optimization;
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   331
  }
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   332
  return false;
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   333
}
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   334
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   335
// Create MDO if necessary.
11572
84afef481892 7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents: 11424
diff changeset
   336
void AdvancedThresholdPolicy::create_mdo(methodHandle mh, JavaThread* THREAD) {
29335
6af23ac5a762 8067012: Don't create MDO for constant getters
ppunegov
parents: 28650
diff changeset
   337
  if (mh->is_native() ||
6af23ac5a762 8067012: Don't create MDO for constant getters
ppunegov
parents: 28650
diff changeset
   338
      mh->is_abstract() ||
6af23ac5a762 8067012: Don't create MDO for constant getters
ppunegov
parents: 28650
diff changeset
   339
      mh->is_accessor() ||
6af23ac5a762 8067012: Don't create MDO for constant getters
ppunegov
parents: 28650
diff changeset
   340
      mh->is_constant_getter()) {
6af23ac5a762 8067012: Don't create MDO for constant getters
ppunegov
parents: 28650
diff changeset
   341
    return;
6af23ac5a762 8067012: Don't create MDO for constant getters
ppunegov
parents: 28650
diff changeset
   342
  }
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   343
  if (mh->method_data() == NULL) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 11572
diff changeset
   344
    Method::build_interpreter_method_data(mh, CHECK_AND_CLEAR);
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   345
  }
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   346
}
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   347
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   348
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   349
/*
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   350
 * Method states:
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   351
 *   0 - interpreter (CompLevel_none)
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   352
 *   1 - pure C1 (CompLevel_simple)
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   353
 *   2 - C1 with invocation and backedge counting (CompLevel_limited_profile)
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   354
 *   3 - C1 with full profiling (CompLevel_full_profile)
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   355
 *   4 - C2 (CompLevel_full_optimization)
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   356
 *
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   357
 * Common state transition patterns:
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   358
 * a. 0 -> 3 -> 4.
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   359
 *    The most common path. But note that even in this straightforward case
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   360
 *    profiling can start at level 0 and finish at level 3.
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   361
 *
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   362
 * b. 0 -> 2 -> 3 -> 4.
22551
9bf46d16dcc6 8025856: Fix typos in the GC code
jwilhelm
parents: 17617
diff changeset
   363
 *    This case occurs when the load on C2 is deemed too high. So, instead of transitioning
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   364
 *    into state 3 directly and over-profiling while a method is in the C2 queue we transition to
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   365
 *    level 2 and wait until the load on C2 decreases. This path is disabled for OSRs.
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   366
 *
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   367
 * c. 0 -> (3->2) -> 4.
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   368
 *    In this case we enqueue a method for compilation at level 3, but the C1 queue is long enough
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   369
 *    to enable the profiling to fully occur at level 0. In this case we change the compilation level
27643
fe8f95a2d9bc 8056071: compiler/whitebox/IsMethodCompilableTest.java fails with 'method() is not compilable after 3 iterations'
thartmann
parents: 26796
diff changeset
   370
 *    of the method to 2 while the request is still in-queue, because it'll allow it to run much faster
fe8f95a2d9bc 8056071: compiler/whitebox/IsMethodCompilableTest.java fails with 'method() is not compilable after 3 iterations'
thartmann
parents: 26796
diff changeset
   371
 *    without full profiling while c2 is compiling.
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   372
 *
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   373
 * d. 0 -> 3 -> 1 or 0 -> 2 -> 1.
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   374
 *    After a method was once compiled with C1 it can be identified as trivial and be compiled to
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   375
 *    level 1. These transition can also occur if a method can't be compiled with C2 but can with C1.
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   376
 *
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   377
 * e. 0 -> 4.
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   378
 *    This can happen if a method fails C1 compilation (it will still be profiled in the interpreter)
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   379
 *    or because of a deopt that didn't require reprofiling (compilation won't happen in this case because
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   380
 *    the compiled version already exists).
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   381
 *
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   382
 * Note that since state 0 can be reached from any other state via deoptimization different loops
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   383
 * are possible.
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   384
 *
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   385
 */
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   386
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   387
// Common transition function. Given a predicate determines if a method should transition to another level.
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 11572
diff changeset
   388
CompLevel AdvancedThresholdPolicy::common(Predicate p, Method* method, CompLevel cur_level, bool disable_feedback) {
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   389
  CompLevel next_level = cur_level;
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   390
  int i = method->invocation_count();
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   391
  int b = method->backedge_count();
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   392
10250
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   393
  if (is_trivial(method)) {
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   394
    next_level = CompLevel_simple;
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   395
  } else {
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   396
    switch(cur_level) {
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   397
    case CompLevel_none:
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   398
      // If we were at full profile level, would we switch to full opt?
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   399
      if (common(p, method, CompLevel_full_profile, disable_feedback) == CompLevel_full_optimization) {
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   400
        next_level = CompLevel_full_optimization;
28650
772aaab2582f 8059606: Enable per-method usage of CompileThresholdScaling (per-method compilation thresholds)
zmajo
parents: 27643
diff changeset
   401
      } else if ((this->*p)(i, b, cur_level, method)) {
33160
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31053
diff changeset
   402
#if INCLUDE_JVMCI
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31053
diff changeset
   403
        if (UseJVMCICompiler) {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31053
diff changeset
   404
          // Since JVMCI takes a while to warm up, its queue inevitably backs up during
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31053
diff changeset
   405
          // early VM execution.
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31053
diff changeset
   406
          next_level = CompLevel_full_profile;
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31053
diff changeset
   407
          break;
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31053
diff changeset
   408
        }
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31053
diff changeset
   409
#endif
10250
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   410
        // C1-generated fully profiled code is about 30% slower than the limited profile
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   411
        // code that has only invocation and backedge counters. The observation is that
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   412
        // if C2 queue is large enough we can spend too much time in the fully profiled code
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   413
        // while waiting for C2 to pick the method from the queue. To alleviate this problem
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   414
        // we introduce a feedback on the C2 queue size. If the C2 queue is sufficiently long
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   415
        // we choose to compile a limited profiled version and then recompile with full profiling
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   416
        // when the load on C2 goes down.
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   417
        if (!disable_feedback && CompileBroker::queue_size(CompLevel_full_optimization) >
33160
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31053
diff changeset
   418
            Tier3DelayOn * compiler_count(CompLevel_full_optimization)) {
10250
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   419
          next_level = CompLevel_limited_profile;
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   420
        } else {
10250
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   421
          next_level = CompLevel_full_profile;
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   422
        }
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   423
      }
10250
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   424
      break;
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   425
    case CompLevel_limited_profile:
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   426
      if (is_method_profiled(method)) {
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   427
        // Special case: we got here because this method was fully profiled in the interpreter.
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   428
        next_level = CompLevel_full_optimization;
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   429
      } else {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 11572
diff changeset
   430
        MethodData* mdo = method->method_data();
10250
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   431
        if (mdo != NULL) {
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   432
          if (mdo->would_profile()) {
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   433
            if (disable_feedback || (CompileBroker::queue_size(CompLevel_full_optimization) <=
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   434
                                     Tier3DelayOff * compiler_count(CompLevel_full_optimization) &&
28650
772aaab2582f 8059606: Enable per-method usage of CompileThresholdScaling (per-method compilation thresholds)
zmajo
parents: 27643
diff changeset
   435
                                     (this->*p)(i, b, cur_level, method))) {
10250
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   436
              next_level = CompLevel_full_profile;
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   437
            }
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   438
          } else {
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   439
            next_level = CompLevel_full_optimization;
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   440
          }
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   441
        }
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   442
      }
10250
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   443
      break;
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   444
    case CompLevel_full_profile:
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   445
      {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 11572
diff changeset
   446
        MethodData* mdo = method->method_data();
10250
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   447
        if (mdo != NULL) {
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   448
          if (mdo->would_profile()) {
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   449
            int mdo_i = mdo->invocation_count_delta();
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   450
            int mdo_b = mdo->backedge_count_delta();
28650
772aaab2582f 8059606: Enable per-method usage of CompileThresholdScaling (per-method compilation thresholds)
zmajo
parents: 27643
diff changeset
   451
            if ((this->*p)(mdo_i, mdo_b, cur_level, method)) {
10250
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   452
              next_level = CompLevel_full_optimization;
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   453
            }
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   454
          } else {
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   455
            next_level = CompLevel_full_optimization;
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   456
          }
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   457
        }
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   458
      }
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   459
      break;
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   460
    }
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   461
  }
10250
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   462
  return MIN2(next_level, (CompLevel)TieredStopAtLevel);
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   463
}
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   464
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   465
// Determine if a method should be compiled with a normal entry point at a different level.
38666
5ff19807abd5 8152311: [JVMCI] allow JVMCI compiler to change the compilation policy for a method
dnsimon
parents: 38218
diff changeset
   466
CompLevel AdvancedThresholdPolicy::call_event(Method* method, CompLevel cur_level, JavaThread * thread) {
10013
714ad59b56cb 7058689: Tiered: Reprofiling doesn't happen in presence of level 4 OSR methods
iveresov
parents: 9625
diff changeset
   467
  CompLevel osr_level = MIN2((CompLevel) method->highest_osr_comp_level(),
10250
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   468
                             common(&AdvancedThresholdPolicy::loop_predicate, method, cur_level, true));
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   469
  CompLevel next_level = common(&AdvancedThresholdPolicy::call_predicate, method, cur_level);
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   470
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   471
  // If OSR method level is greater than the regular method level, the levels should be
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   472
  // equalized by raising the regular method level in order to avoid OSRs during each
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   473
  // invocation of the method.
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   474
  if (osr_level == CompLevel_full_optimization && cur_level == CompLevel_full_profile) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 11572
diff changeset
   475
    MethodData* mdo = method->method_data();
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   476
    guarantee(mdo != NULL, "MDO should not be NULL");
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   477
    if (mdo->invocation_count() >= 1) {
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   478
      next_level = CompLevel_full_optimization;
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   479
    }
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   480
  } else {
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   481
    next_level = MAX2(osr_level, next_level);
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   482
  }
38666
5ff19807abd5 8152311: [JVMCI] allow JVMCI compiler to change the compilation policy for a method
dnsimon
parents: 38218
diff changeset
   483
#if INCLUDE_JVMCI
5ff19807abd5 8152311: [JVMCI] allow JVMCI compiler to change the compilation policy for a method
dnsimon
parents: 38218
diff changeset
   484
  if (UseJVMCICompiler) {
5ff19807abd5 8152311: [JVMCI] allow JVMCI compiler to change the compilation policy for a method
dnsimon
parents: 38218
diff changeset
   485
    next_level = JVMCIRuntime::adjust_comp_level(method, false, next_level, thread);
5ff19807abd5 8152311: [JVMCI] allow JVMCI compiler to change the compilation policy for a method
dnsimon
parents: 38218
diff changeset
   486
  }
5ff19807abd5 8152311: [JVMCI] allow JVMCI compiler to change the compilation policy for a method
dnsimon
parents: 38218
diff changeset
   487
#endif
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   488
  return next_level;
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   489
}
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   490
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   491
// Determine if we should do an OSR compilation of a given method.
38666
5ff19807abd5 8152311: [JVMCI] allow JVMCI compiler to change the compilation policy for a method
dnsimon
parents: 38218
diff changeset
   492
CompLevel AdvancedThresholdPolicy::loop_event(Method* method, CompLevel cur_level, JavaThread * thread) {
10250
0794cd144834 7066339: Tiered: policy should make consistent decisions about osr levels
iveresov
parents: 10014
diff changeset
   493
  CompLevel next_level = common(&AdvancedThresholdPolicy::loop_predicate, method, cur_level, true);
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   494
  if (cur_level == CompLevel_none) {
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   495
    // If there is a live OSR method that means that we deopted to the interpreter
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   496
    // for the transition.
10013
714ad59b56cb 7058689: Tiered: Reprofiling doesn't happen in presence of level 4 OSR methods
iveresov
parents: 9625
diff changeset
   497
    CompLevel osr_level = MIN2((CompLevel)method->highest_osr_comp_level(), next_level);
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   498
    if (osr_level > CompLevel_none) {
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   499
      return osr_level;
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   500
    }
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   501
  }
38666
5ff19807abd5 8152311: [JVMCI] allow JVMCI compiler to change the compilation policy for a method
dnsimon
parents: 38218
diff changeset
   502
#if INCLUDE_JVMCI
5ff19807abd5 8152311: [JVMCI] allow JVMCI compiler to change the compilation policy for a method
dnsimon
parents: 38218
diff changeset
   503
  if (UseJVMCICompiler) {
5ff19807abd5 8152311: [JVMCI] allow JVMCI compiler to change the compilation policy for a method
dnsimon
parents: 38218
diff changeset
   504
    next_level = JVMCIRuntime::adjust_comp_level(method, true, next_level, thread);
5ff19807abd5 8152311: [JVMCI] allow JVMCI compiler to change the compilation policy for a method
dnsimon
parents: 38218
diff changeset
   505
  }
5ff19807abd5 8152311: [JVMCI] allow JVMCI compiler to change the compilation policy for a method
dnsimon
parents: 38218
diff changeset
   506
#endif
10013
714ad59b56cb 7058689: Tiered: Reprofiling doesn't happen in presence of level 4 OSR methods
iveresov
parents: 9625
diff changeset
   507
  return next_level;
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   508
}
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   509
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   510
// Update the rate and submit compile
33593
60764a78fa5c 8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents: 33160
diff changeset
   511
void AdvancedThresholdPolicy::submit_compile(const methodHandle& mh, int bci, CompLevel level, JavaThread* thread) {
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   512
  int hot_count = (bci == InvocationEntryBci) ? mh->invocation_count() : mh->backedge_count();
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   513
  update_rate(os::javaTimeMillis(), mh());
38218
f5ba1dea04eb 8153013: BlockingCompilation test times out
neliasso
parents: 38133
diff changeset
   514
  CompileBroker::compile_method(mh, bci, level, mh, hot_count, CompileTask::Reason_Tiered, thread);
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   515
}
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   516
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   517
// Handle the invocation event.
33593
60764a78fa5c 8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents: 33160
diff changeset
   518
void AdvancedThresholdPolicy::method_invocation_event(const methodHandle& mh, const methodHandle& imh,
38133
78b95467b9f1 8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents: 36597
diff changeset
   519
                                                      CompLevel level, CompiledMethod* nm, JavaThread* thread) {
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   520
  if (should_create_mdo(mh(), level)) {
11572
84afef481892 7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents: 11424
diff changeset
   521
    create_mdo(mh, thread);
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   522
  }
24321
621f7e09fc0a 8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents: 24013
diff changeset
   523
  if (is_compilation_enabled() && !CompileBroker::compilation_is_in_queue(mh)) {
38666
5ff19807abd5 8152311: [JVMCI] allow JVMCI compiler to change the compilation policy for a method
dnsimon
parents: 38218
diff changeset
   524
    CompLevel next_level = call_event(mh(), level, thread);
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   525
    if (next_level != level) {
11572
84afef481892 7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents: 11424
diff changeset
   526
      compile(mh, InvocationEntryBci, next_level, thread);
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   527
    }
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   528
  }
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   529
}
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   530
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   531
// Handle the back branch event. Notice that we can compile the method
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   532
// with a regular entry from here.
33593
60764a78fa5c 8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents: 33160
diff changeset
   533
void AdvancedThresholdPolicy::method_back_branch_event(const methodHandle& mh, const methodHandle& imh,
38133
78b95467b9f1 8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents: 36597
diff changeset
   534
                                                       int bci, CompLevel level, CompiledMethod* nm, JavaThread* thread) {
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   535
  if (should_create_mdo(mh(), level)) {
11572
84afef481892 7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents: 11424
diff changeset
   536
    create_mdo(mh, thread);
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   537
  }
10014
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   538
  // Check if MDO should be created for the inlined method
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   539
  if (should_create_mdo(imh(), level)) {
11572
84afef481892 7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents: 11424
diff changeset
   540
    create_mdo(imh, thread);
10014
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   541
  }
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   542
10014
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   543
  if (is_compilation_enabled()) {
38666
5ff19807abd5 8152311: [JVMCI] allow JVMCI compiler to change the compilation policy for a method
dnsimon
parents: 38218
diff changeset
   544
    CompLevel next_osr_level = loop_event(imh(), level, thread);
10014
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   545
    CompLevel max_osr_level = (CompLevel)imh->highest_osr_comp_level();
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   546
    // At the very least compile the OSR version
24321
621f7e09fc0a 8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents: 24013
diff changeset
   547
    if (!CompileBroker::compilation_is_in_queue(imh) && (next_osr_level != level)) {
11572
84afef481892 7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents: 11424
diff changeset
   548
      compile(imh, bci, next_osr_level, thread);
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   549
    }
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   550
10014
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   551
    // Use loop event as an opportunity to also check if there's been
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   552
    // enough calls.
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   553
    CompLevel cur_level, next_level;
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   554
    if (mh() != imh()) { // If there is an enclosing method
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   555
      guarantee(nm != NULL, "Should have nmethod here");
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   556
      cur_level = comp_level(mh());
38666
5ff19807abd5 8152311: [JVMCI] allow JVMCI compiler to change the compilation policy for a method
dnsimon
parents: 38218
diff changeset
   557
      next_level = call_event(mh(), cur_level, thread);
10014
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   558
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   559
      if (max_osr_level == CompLevel_full_optimization) {
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   560
        // The inlinee OSRed to full opt, we need to modify the enclosing method to avoid deopts
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   561
        bool make_not_entrant = false;
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   562
        if (nm->is_osr_method()) {
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   563
          // This is an osr method, just make it not entrant and recompile later if needed
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   564
          make_not_entrant = true;
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   565
        } else {
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   566
          if (next_level != CompLevel_full_optimization) {
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   567
            // next_level is not full opt, so we need to recompile the
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   568
            // enclosing method without the inlinee
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   569
            cur_level = CompLevel_none;
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   570
            make_not_entrant = true;
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   571
          }
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   572
        }
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   573
        if (make_not_entrant) {
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   574
          if (PrintTieredEvents) {
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   575
            int osr_bci = nm->is_osr_method() ? nm->osr_entry_bci() : InvocationEntryBci;
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   576
            print_event(MAKE_NOT_ENTRANT, mh(), mh(), osr_bci, level);
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   577
          }
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   578
          nm->make_not_entrant();
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   579
        }
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   580
      }
24321
621f7e09fc0a 8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents: 24013
diff changeset
   581
      if (!CompileBroker::compilation_is_in_queue(mh)) {
10014
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   582
        // Fix up next_level if necessary to avoid deopts
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   583
        if (next_level == CompLevel_limited_profile && max_osr_level == CompLevel_full_profile) {
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   584
          next_level = CompLevel_full_profile;
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   585
        }
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   586
        if (cur_level != next_level) {
11572
84afef481892 7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents: 11424
diff changeset
   587
          compile(mh, InvocationEntryBci, next_level, thread);
10014
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   588
        }
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   589
      }
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   590
    } else {
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   591
      cur_level = comp_level(imh());
38666
5ff19807abd5 8152311: [JVMCI] allow JVMCI compiler to change the compilation policy for a method
dnsimon
parents: 38218
diff changeset
   592
      next_level = call_event(imh(), cur_level, thread);
24321
621f7e09fc0a 8040798: compiler/startup/SmallCodeCacheStartup.java timed out in RT_Baseline
anoll
parents: 24013
diff changeset
   593
      if (!CompileBroker::compilation_is_in_queue(imh) && (next_level != cur_level)) {
11572
84afef481892 7131259: compile_method and CompilationPolicy::event shouldn't be declared TRAPS
iveresov
parents: 11424
diff changeset
   594
        compile(imh, InvocationEntryBci, next_level, thread);
10014
a5c2141ee857 7057120: Tiered: Allow C1 to inline methods with loops
iveresov
parents: 10013
diff changeset
   595
      }
8667
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   596
    }
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   597
  }
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   598
}
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   599
b32929355d27 7020403: Add AdvancedCompilationPolicy for tiered
iveresov
parents:
diff changeset
   600
#endif // TIERED