src/hotspot/share/code/dependencies.cpp
author never
Mon, 04 Dec 2017 13:13:44 -0800
changeset 48299 e8f5fc8f5f67
parent 47216 71c04702a3d5
child 49192 6734eeef4283
permissions -rw-r--r--
8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT Reviewed-by: kvn, iveresov, dlong
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     1
/*
46271
979ebd346ecf 8169881: Remove implicit Handle conversions oop->Handle
coleenp
parents: 40362
diff changeset
     2
 * Copyright (c) 2005, 2017, Oracle and/or its affiliates. All rights reserved.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
489c9b5090e2 Initial load
duke
parents:
diff changeset
     4
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
489c9b5090e2 Initial load
duke
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
489c9b5090e2 Initial load
duke
parents:
diff changeset
     7
 * published by the Free Software Foundation.
489c9b5090e2 Initial load
duke
parents:
diff changeset
     8
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
489c9b5090e2 Initial load
duke
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
489c9b5090e2 Initial load
duke
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
489c9b5090e2 Initial load
duke
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
489c9b5090e2 Initial load
duke
parents:
diff changeset
    13
 * accompanied this code).
489c9b5090e2 Initial load
duke
parents:
diff changeset
    14
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
489c9b5090e2 Initial load
duke
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
489c9b5090e2 Initial load
duke
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    18
 *
5547
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 4754
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 4754
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 4754
diff changeset
    21
 * questions.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    22
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
    23
 */
489c9b5090e2 Initial load
duke
parents:
diff changeset
    24
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    25
#include "precompiled.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    26
#include "ci/ciArrayKlass.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    27
#include "ci/ciEnv.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    28
#include "ci/ciKlass.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    29
#include "ci/ciMethod.hpp"
29081
c61eb4914428 8072911: Remove includes of oop.inline.hpp from .hpp files
stefank
parents: 28514
diff changeset
    30
#include "classfile/javaClasses.inline.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    31
#include "code/dependencies.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    32
#include "compiler/compileLog.hpp"
48299
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
    33
#include "compiler/compileBroker.hpp"
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
    34
#include "compiler/compileTask.hpp"
37248
11a660dbbb8e 8132524: Missing includes to resourceArea.hpp
jprovino
parents: 35499
diff changeset
    35
#include "memory/resourceArea.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    36
#include "oops/oop.inline.hpp"
33160
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
    37
#include "oops/objArrayKlass.hpp"
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
    38
#include "runtime/handles.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    39
#include "runtime/handles.inline.hpp"
24351
61b33cc6d3cf 8042195: Introduce umbrella header orderAccess.inline.hpp.
goetz
parents: 23187
diff changeset
    40
#include "runtime/thread.inline.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    41
#include "utilities/copy.hpp"
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    42
489c9b5090e2 Initial load
duke
parents:
diff changeset
    43
489c9b5090e2 Initial load
duke
parents:
diff changeset
    44
#ifdef ASSERT
489c9b5090e2 Initial load
duke
parents:
diff changeset
    45
static bool must_be_in_vm() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    46
  Thread* thread = Thread::current();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    47
  if (thread->is_Java_thread())
489c9b5090e2 Initial load
duke
parents:
diff changeset
    48
    return ((JavaThread*)thread)->thread_state() == _thread_in_vm;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    49
  else
489c9b5090e2 Initial load
duke
parents:
diff changeset
    50
    return true;  //something like this: thread->is_VM_thread();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    51
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    52
#endif //ASSERT
489c9b5090e2 Initial load
duke
parents:
diff changeset
    53
489c9b5090e2 Initial load
duke
parents:
diff changeset
    54
void Dependencies::initialize(ciEnv* env) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    55
  Arena* arena = env->arena();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    56
  _oop_recorder = env->oop_recorder();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    57
  _log = env->log();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    58
  _dep_seen = new(arena) GrowableArray<int>(arena, 500, 0, 0);
33160
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
    59
#if INCLUDE_JVMCI
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
    60
  _using_dep_values = false;
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
    61
#endif
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    62
  DEBUG_ONLY(_deps[end_marker] = NULL);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    63
  for (int i = (int)FIRST_TYPE; i < (int)TYPE_LIMIT; i++) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
    64
    _deps[i] = new(arena) GrowableArray<ciBaseObject*>(arena, 10, 0, 0);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    65
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    66
  _content_bytes = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    67
  _size_in_bytes = (size_t)-1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    68
489c9b5090e2 Initial load
duke
parents:
diff changeset
    69
  assert(TYPE_LIMIT <= (1<<LG2_TYPE_LIMIT), "sanity");
489c9b5090e2 Initial load
duke
parents:
diff changeset
    70
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    71
489c9b5090e2 Initial load
duke
parents:
diff changeset
    72
void Dependencies::assert_evol_method(ciMethod* m) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    73
  assert_common_1(evol_method, m);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    74
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    75
489c9b5090e2 Initial load
duke
parents:
diff changeset
    76
void Dependencies::assert_leaf_type(ciKlass* ctxk) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    77
  if (ctxk->is_array_klass()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    78
    // As a special case, support this assertion on an array type,
489c9b5090e2 Initial load
duke
parents:
diff changeset
    79
    // which reduces to an assertion on its element type.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    80
    // Note that this cannot be done with assertions that
489c9b5090e2 Initial load
duke
parents:
diff changeset
    81
    // relate to concreteness or abstractness.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    82
    ciType* elemt = ctxk->as_array_klass()->base_element_type();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    83
    if (!elemt->is_instance_klass())  return;   // Ex:  int[][]
489c9b5090e2 Initial load
duke
parents:
diff changeset
    84
    ctxk = elemt->as_instance_klass();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    85
    //if (ctxk->is_final())  return;            // Ex:  String[][]
489c9b5090e2 Initial load
duke
parents:
diff changeset
    86
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    87
  check_ctxk(ctxk);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    88
  assert_common_1(leaf_type, ctxk);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    89
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    90
489c9b5090e2 Initial load
duke
parents:
diff changeset
    91
void Dependencies::assert_abstract_with_unique_concrete_subtype(ciKlass* ctxk, ciKlass* conck) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    92
  check_ctxk_abstract(ctxk);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    93
  assert_common_2(abstract_with_unique_concrete_subtype, ctxk, conck);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    94
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    95
489c9b5090e2 Initial load
duke
parents:
diff changeset
    96
void Dependencies::assert_abstract_with_no_concrete_subtype(ciKlass* ctxk) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    97
  check_ctxk_abstract(ctxk);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    98
  assert_common_1(abstract_with_no_concrete_subtype, ctxk);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    99
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   100
489c9b5090e2 Initial load
duke
parents:
diff changeset
   101
void Dependencies::assert_concrete_with_no_concrete_subtype(ciKlass* ctxk) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   102
  check_ctxk_concrete(ctxk);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   103
  assert_common_1(concrete_with_no_concrete_subtype, ctxk);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   104
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   105
489c9b5090e2 Initial load
duke
parents:
diff changeset
   106
void Dependencies::assert_unique_concrete_method(ciKlass* ctxk, ciMethod* uniqm) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   107
  check_ctxk(ctxk);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   108
  assert_common_2(unique_concrete_method, ctxk, uniqm);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   109
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   110
489c9b5090e2 Initial load
duke
parents:
diff changeset
   111
void Dependencies::assert_abstract_with_exclusive_concrete_subtypes(ciKlass* ctxk, ciKlass* k1, ciKlass* k2) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   112
  check_ctxk(ctxk);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   113
  assert_common_3(abstract_with_exclusive_concrete_subtypes_2, ctxk, k1, k2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   114
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   115
489c9b5090e2 Initial load
duke
parents:
diff changeset
   116
void Dependencies::assert_exclusive_concrete_methods(ciKlass* ctxk, ciMethod* m1, ciMethod* m2) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   117
  check_ctxk(ctxk);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   118
  assert_common_3(exclusive_concrete_methods_2, ctxk, m1, m2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   119
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   120
489c9b5090e2 Initial load
duke
parents:
diff changeset
   121
void Dependencies::assert_has_no_finalizable_subclasses(ciKlass* ctxk) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   122
  check_ctxk(ctxk);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   123
  assert_common_1(no_finalizable_subclasses, ctxk);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   124
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   125
10503
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
   126
void Dependencies::assert_call_site_target_value(ciCallSite* call_site, ciMethodHandle* method_handle) {
31037
01a5c5fa5681 8079205: CallSite dependency tracking is broken after sun.misc.Cleaner became automatically cleared
vlivanov
parents: 30296
diff changeset
   127
  assert_common_2(call_site_target_value, call_site, method_handle);
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
   128
}
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
   129
33160
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   130
#if INCLUDE_JVMCI
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   131
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   132
Dependencies::Dependencies(Arena* arena, OopRecorder* oop_recorder, CompileLog* log) {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   133
  _oop_recorder = oop_recorder;
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   134
  _log = log;
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   135
  _dep_seen = new(arena) GrowableArray<int>(arena, 500, 0, 0);
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   136
  _using_dep_values = true;
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   137
  DEBUG_ONLY(_dep_values[end_marker] = NULL);
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   138
  for (int i = (int)FIRST_TYPE; i < (int)TYPE_LIMIT; i++) {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   139
    _dep_values[i] = new(arena) GrowableArray<DepValue>(arena, 10, 0, DepValue());
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   140
  }
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   141
  _content_bytes = NULL;
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   142
  _size_in_bytes = (size_t)-1;
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   143
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   144
  assert(TYPE_LIMIT <= (1<<LG2_TYPE_LIMIT), "sanity");
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   145
}
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   146
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   147
void Dependencies::assert_evol_method(Method* m) {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   148
  assert_common_1(evol_method, DepValue(_oop_recorder, m));
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   149
}
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   150
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   151
void Dependencies::assert_has_no_finalizable_subclasses(Klass* ctxk) {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   152
  check_ctxk(ctxk);
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   153
  assert_common_1(no_finalizable_subclasses, DepValue(_oop_recorder, ctxk));
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   154
}
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   155
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   156
void Dependencies::assert_leaf_type(Klass* ctxk) {
33611
9abd65805e19 8139203: Consistent naming for klass type predicates
coleenp
parents: 33602
diff changeset
   157
  if (ctxk->is_array_klass()) {
33160
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   158
    // As a special case, support this assertion on an array type,
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   159
    // which reduces to an assertion on its element type.
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   160
    // Note that this cannot be done with assertions that
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   161
    // relate to concreteness or abstractness.
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   162
    BasicType elemt = ArrayKlass::cast(ctxk)->element_type();
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   163
    if (is_java_primitive(elemt))  return;   // Ex:  int[][]
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   164
    ctxk = ObjArrayKlass::cast(ctxk)->bottom_klass();
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   165
    //if (ctxk->is_final())  return;            // Ex:  String[][]
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   166
  }
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   167
  check_ctxk(ctxk);
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   168
  assert_common_1(leaf_type, DepValue(_oop_recorder, ctxk));
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   169
}
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   170
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   171
void Dependencies::assert_abstract_with_unique_concrete_subtype(Klass* ctxk, Klass* conck) {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   172
  check_ctxk_abstract(ctxk);
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   173
  DepValue ctxk_dv(_oop_recorder, ctxk);
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   174
  DepValue conck_dv(_oop_recorder, conck, &ctxk_dv);
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   175
  assert_common_2(abstract_with_unique_concrete_subtype, ctxk_dv, conck_dv);
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   176
}
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   177
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   178
void Dependencies::assert_unique_concrete_method(Klass* ctxk, Method* uniqm) {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   179
  check_ctxk(ctxk);
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   180
  assert_common_2(unique_concrete_method, DepValue(_oop_recorder, ctxk), DepValue(_oop_recorder, uniqm));
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   181
}
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   182
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   183
void Dependencies::assert_call_site_target_value(oop call_site, oop method_handle) {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   184
  assert_common_2(call_site_target_value, DepValue(_oop_recorder, JNIHandles::make_local(call_site)), DepValue(_oop_recorder, JNIHandles::make_local(method_handle)));
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   185
}
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   186
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   187
#endif // INCLUDE_JVMCI
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   188
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   189
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   190
// Helper function.  If we are adding a new dep. under ctxk2,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   191
// try to find an old dep. under a broader* ctxk1.  If there is
489c9b5090e2 Initial load
duke
parents:
diff changeset
   192
//
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   193
bool Dependencies::maybe_merge_ctxk(GrowableArray<ciBaseObject*>* deps,
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   194
                                    int ctxk_i, ciKlass* ctxk2) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   195
  ciKlass* ctxk1 = deps->at(ctxk_i)->as_metadata()->as_klass();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   196
  if (ctxk2->is_subtype_of(ctxk1)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   197
    return true;  // success, and no need to change
489c9b5090e2 Initial load
duke
parents:
diff changeset
   198
  } else if (ctxk1->is_subtype_of(ctxk2)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   199
    // new context class fully subsumes previous one
489c9b5090e2 Initial load
duke
parents:
diff changeset
   200
    deps->at_put(ctxk_i, ctxk2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   201
    return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   202
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   203
    return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   204
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   205
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   206
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   207
void Dependencies::assert_common_1(DepType dept, ciBaseObject* x) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   208
  assert(dep_args(dept) == 1, "sanity");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   209
  log_dependency(dept, x);
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   210
  GrowableArray<ciBaseObject*>* deps = _deps[dept];
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   211
489c9b5090e2 Initial load
duke
parents:
diff changeset
   212
  // see if the same (or a similar) dep is already recorded
489c9b5090e2 Initial load
duke
parents:
diff changeset
   213
  if (note_dep_seen(dept, x)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   214
    assert(deps->find(x) >= 0, "sanity");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   215
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   216
    deps->append(x);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   217
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   218
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   219
10503
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
   220
void Dependencies::assert_common_2(DepType dept,
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   221
                                   ciBaseObject* x0, ciBaseObject* x1) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   222
  assert(dep_args(dept) == 2, "sanity");
10503
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
   223
  log_dependency(dept, x0, x1);
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   224
  GrowableArray<ciBaseObject*>* deps = _deps[dept];
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   225
489c9b5090e2 Initial load
duke
parents:
diff changeset
   226
  // see if the same (or a similar) dep is already recorded
10503
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
   227
  bool has_ctxk = has_explicit_context_arg(dept);
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
   228
  if (has_ctxk) {
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
   229
    assert(dep_context_arg(dept) == 0, "sanity");
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
   230
    if (note_dep_seen(dept, x1)) {
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
   231
      // look in this bucket for redundant assertions
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
   232
      const int stride = 2;
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
   233
      for (int i = deps->length(); (i -= stride) >= 0; ) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   234
        ciBaseObject* y1 = deps->at(i+1);
10503
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
   235
        if (x1 == y1) {  // same subject; check the context
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   236
          if (maybe_merge_ctxk(deps, i+0, x0->as_metadata()->as_klass())) {
10503
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
   237
            return;
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
   238
          }
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
   239
        }
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
   240
      }
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
   241
    }
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
   242
  } else {
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
   243
    if (note_dep_seen(dept, x0) && note_dep_seen(dept, x1)) {
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
   244
      // look in this bucket for redundant assertions
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
   245
      const int stride = 2;
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
   246
      for (int i = deps->length(); (i -= stride) >= 0; ) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   247
        ciBaseObject* y0 = deps->at(i+0);
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   248
        ciBaseObject* y1 = deps->at(i+1);
10503
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
   249
        if (x0 == y0 && x1 == y1) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   250
          return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   251
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   252
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   253
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   254
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   255
489c9b5090e2 Initial load
duke
parents:
diff changeset
   256
  // append the assertion in the correct bucket:
10503
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
   257
  deps->append(x0);
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
   258
  deps->append(x1);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   259
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   260
10503
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
   261
void Dependencies::assert_common_3(DepType dept,
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   262
                                   ciKlass* ctxk, ciBaseObject* x, ciBaseObject* x2) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   263
  assert(dep_context_arg(dept) == 0, "sanity");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   264
  assert(dep_args(dept) == 3, "sanity");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   265
  log_dependency(dept, ctxk, x, x2);
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   266
  GrowableArray<ciBaseObject*>* deps = _deps[dept];
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   267
489c9b5090e2 Initial load
duke
parents:
diff changeset
   268
  // try to normalize an unordered pair:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   269
  bool swap = false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   270
  switch (dept) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   271
  case abstract_with_exclusive_concrete_subtypes_2:
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   272
    swap = (x->ident() > x2->ident() && x->as_metadata()->as_klass() != ctxk);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   273
    break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   274
  case exclusive_concrete_methods_2:
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   275
    swap = (x->ident() > x2->ident() && x->as_metadata()->as_method()->holder() != ctxk);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   276
    break;
46630
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 46271
diff changeset
   277
  default:
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 46271
diff changeset
   278
    break;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   279
  }
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   280
  if (swap) { ciBaseObject* t = x; x = x2; x2 = t; }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   281
489c9b5090e2 Initial load
duke
parents:
diff changeset
   282
  // see if the same (or a similar) dep is already recorded
489c9b5090e2 Initial load
duke
parents:
diff changeset
   283
  if (note_dep_seen(dept, x) && note_dep_seen(dept, x2)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   284
    // look in this bucket for redundant assertions
489c9b5090e2 Initial load
duke
parents:
diff changeset
   285
    const int stride = 3;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   286
    for (int i = deps->length(); (i -= stride) >= 0; ) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   287
      ciBaseObject* y  = deps->at(i+1);
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   288
      ciBaseObject* y2 = deps->at(i+2);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   289
      if (x == y && x2 == y2) {  // same subjects; check the context
489c9b5090e2 Initial load
duke
parents:
diff changeset
   290
        if (maybe_merge_ctxk(deps, i+0, ctxk)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   291
          return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   292
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   293
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   294
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   295
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   296
  // append the assertion in the correct bucket:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   297
  deps->append(ctxk);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   298
  deps->append(x);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   299
  deps->append(x2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   300
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   301
33160
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   302
#if INCLUDE_JVMCI
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   303
bool Dependencies::maybe_merge_ctxk(GrowableArray<DepValue>* deps,
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   304
                                    int ctxk_i, DepValue ctxk2_dv) {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   305
  Klass* ctxk1 = deps->at(ctxk_i).as_klass(_oop_recorder);
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   306
  Klass* ctxk2 = ctxk2_dv.as_klass(_oop_recorder);
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   307
  if (ctxk2->is_subtype_of(ctxk1)) {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   308
    return true;  // success, and no need to change
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   309
  } else if (ctxk1->is_subtype_of(ctxk2)) {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   310
    // new context class fully subsumes previous one
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   311
    deps->at_put(ctxk_i, ctxk2_dv);
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   312
    return true;
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   313
  } else {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   314
    return false;
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   315
  }
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   316
}
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   317
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   318
void Dependencies::assert_common_1(DepType dept, DepValue x) {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   319
  assert(dep_args(dept) == 1, "sanity");
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   320
  //log_dependency(dept, x);
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   321
  GrowableArray<DepValue>* deps = _dep_values[dept];
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   322
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   323
  // see if the same (or a similar) dep is already recorded
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   324
  if (note_dep_seen(dept, x)) {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   325
    assert(deps->find(x) >= 0, "sanity");
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   326
  } else {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   327
    deps->append(x);
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   328
  }
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   329
}
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   330
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   331
void Dependencies::assert_common_2(DepType dept,
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   332
                                   DepValue x0, DepValue x1) {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   333
  assert(dep_args(dept) == 2, "sanity");
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   334
  //log_dependency(dept, x0, x1);
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   335
  GrowableArray<DepValue>* deps = _dep_values[dept];
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   336
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   337
  // see if the same (or a similar) dep is already recorded
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   338
  bool has_ctxk = has_explicit_context_arg(dept);
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   339
  if (has_ctxk) {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   340
    assert(dep_context_arg(dept) == 0, "sanity");
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   341
    if (note_dep_seen(dept, x1)) {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   342
      // look in this bucket for redundant assertions
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   343
      const int stride = 2;
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   344
      for (int i = deps->length(); (i -= stride) >= 0; ) {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   345
        DepValue y1 = deps->at(i+1);
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   346
        if (x1 == y1) {  // same subject; check the context
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   347
          if (maybe_merge_ctxk(deps, i+0, x0)) {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   348
            return;
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   349
          }
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   350
        }
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   351
      }
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   352
    }
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   353
  } else {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   354
    if (note_dep_seen(dept, x0) && note_dep_seen(dept, x1)) {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   355
      // look in this bucket for redundant assertions
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   356
      const int stride = 2;
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   357
      for (int i = deps->length(); (i -= stride) >= 0; ) {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   358
        DepValue y0 = deps->at(i+0);
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   359
        DepValue y1 = deps->at(i+1);
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   360
        if (x0 == y0 && x1 == y1) {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   361
          return;
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   362
        }
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   363
      }
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   364
    }
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   365
  }
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   366
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   367
  // append the assertion in the correct bucket:
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   368
  deps->append(x0);
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   369
  deps->append(x1);
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   370
}
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   371
#endif // INCLUDE_JVMCI
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   372
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   373
/// Support for encoding dependencies into an nmethod:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   374
489c9b5090e2 Initial load
duke
parents:
diff changeset
   375
void Dependencies::copy_to(nmethod* nm) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   376
  address beg = nm->dependencies_begin();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   377
  address end = nm->dependencies_end();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   378
  guarantee(end - beg >= (ptrdiff_t) size_in_bytes(), "bad sizing");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   379
  Copy::disjoint_words((HeapWord*) content_bytes(),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   380
                       (HeapWord*) beg,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   381
                       size_in_bytes() / sizeof(HeapWord));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   382
  assert(size_in_bytes() % sizeof(HeapWord) == 0, "copy by words");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   383
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   384
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   385
static int sort_dep(ciBaseObject** p1, ciBaseObject** p2, int narg) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   386
  for (int i = 0; i < narg; i++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   387
    int diff = p1[i]->ident() - p2[i]->ident();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   388
    if (diff != 0)  return diff;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   389
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   390
  return 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   391
}
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   392
static int sort_dep_arg_1(ciBaseObject** p1, ciBaseObject** p2)
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   393
{ return sort_dep(p1, p2, 1); }
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   394
static int sort_dep_arg_2(ciBaseObject** p1, ciBaseObject** p2)
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   395
{ return sort_dep(p1, p2, 2); }
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   396
static int sort_dep_arg_3(ciBaseObject** p1, ciBaseObject** p2)
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   397
{ return sort_dep(p1, p2, 3); }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   398
33160
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   399
#if INCLUDE_JVMCI
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   400
// metadata deps are sorted before object deps
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   401
static int sort_dep_value(Dependencies::DepValue* p1, Dependencies::DepValue* p2, int narg) {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   402
  for (int i = 0; i < narg; i++) {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   403
    int diff = p1[i].sort_key() - p2[i].sort_key();
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   404
    if (diff != 0)  return diff;
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   405
  }
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   406
  return 0;
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   407
}
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   408
static int sort_dep_value_arg_1(Dependencies::DepValue* p1, Dependencies::DepValue* p2)
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   409
{ return sort_dep_value(p1, p2, 1); }
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   410
static int sort_dep_value_arg_2(Dependencies::DepValue* p1, Dependencies::DepValue* p2)
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   411
{ return sort_dep_value(p1, p2, 2); }
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   412
static int sort_dep_value_arg_3(Dependencies::DepValue* p1, Dependencies::DepValue* p2)
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   413
{ return sort_dep_value(p1, p2, 3); }
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   414
#endif // INCLUDE_JVMCI
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   415
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   416
void Dependencies::sort_all_deps() {
33160
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   417
#if INCLUDE_JVMCI
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   418
  if (_using_dep_values) {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   419
    for (int deptv = (int)FIRST_TYPE; deptv < (int)TYPE_LIMIT; deptv++) {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   420
      DepType dept = (DepType)deptv;
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   421
      GrowableArray<DepValue>* deps = _dep_values[dept];
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   422
      if (deps->length() <= 1)  continue;
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   423
      switch (dep_args(dept)) {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   424
      case 1: deps->sort(sort_dep_value_arg_1, 1); break;
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   425
      case 2: deps->sort(sort_dep_value_arg_2, 2); break;
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   426
      case 3: deps->sort(sort_dep_value_arg_3, 3); break;
46630
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 46271
diff changeset
   427
      default: ShouldNotReachHere(); break;
33160
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   428
      }
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   429
    }
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   430
    return;
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   431
  }
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   432
#endif // INCLUDE_JVMCI
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   433
  for (int deptv = (int)FIRST_TYPE; deptv < (int)TYPE_LIMIT; deptv++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   434
    DepType dept = (DepType)deptv;
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   435
    GrowableArray<ciBaseObject*>* deps = _deps[dept];
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   436
    if (deps->length() <= 1)  continue;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   437
    switch (dep_args(dept)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   438
    case 1: deps->sort(sort_dep_arg_1, 1); break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   439
    case 2: deps->sort(sort_dep_arg_2, 2); break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   440
    case 3: deps->sort(sort_dep_arg_3, 3); break;
46630
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 46271
diff changeset
   441
    default: ShouldNotReachHere(); break;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   442
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   443
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   444
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   445
489c9b5090e2 Initial load
duke
parents:
diff changeset
   446
size_t Dependencies::estimate_size_in_bytes() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   447
  size_t est_size = 100;
33160
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   448
#if INCLUDE_JVMCI
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   449
  if (_using_dep_values) {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   450
    for (int deptv = (int)FIRST_TYPE; deptv < (int)TYPE_LIMIT; deptv++) {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   451
      DepType dept = (DepType)deptv;
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   452
      GrowableArray<DepValue>* deps = _dep_values[dept];
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   453
      est_size += deps->length() * 2;  // tags and argument(s)
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   454
    }
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   455
    return est_size;
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   456
  }
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   457
#endif // INCLUDE_JVMCI
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   458
  for (int deptv = (int)FIRST_TYPE; deptv < (int)TYPE_LIMIT; deptv++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   459
    DepType dept = (DepType)deptv;
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   460
    GrowableArray<ciBaseObject*>* deps = _deps[dept];
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   461
    est_size += deps->length()*2;  // tags and argument(s)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   462
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   463
  return est_size;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   464
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   465
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   466
ciKlass* Dependencies::ctxk_encoded_as_null(DepType dept, ciBaseObject* x) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   467
  switch (dept) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   468
  case abstract_with_exclusive_concrete_subtypes_2:
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   469
    return x->as_metadata()->as_klass();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   470
  case unique_concrete_method:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   471
  case exclusive_concrete_methods_2:
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   472
    return x->as_metadata()->as_method()->holder();
46630
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 46271
diff changeset
   473
  default:
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 46271
diff changeset
   474
    return NULL;  // let NULL be NULL
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   475
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   476
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   477
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   478
Klass* Dependencies::ctxk_encoded_as_null(DepType dept, Metadata* x) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   479
  assert(must_be_in_vm(), "raw oops here");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   480
  switch (dept) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   481
  case abstract_with_exclusive_concrete_subtypes_2:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   482
    assert(x->is_klass(), "sanity");
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   483
    return (Klass*) x;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   484
  case unique_concrete_method:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   485
  case exclusive_concrete_methods_2:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   486
    assert(x->is_method(), "sanity");
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   487
    return ((Method*)x)->method_holder();
46630
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 46271
diff changeset
   488
  default:
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 46271
diff changeset
   489
    return NULL;  // let NULL be NULL
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   490
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   491
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   492
489c9b5090e2 Initial load
duke
parents:
diff changeset
   493
void Dependencies::encode_content_bytes() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   494
  sort_all_deps();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   495
489c9b5090e2 Initial load
duke
parents:
diff changeset
   496
  // cast is safe, no deps can overflow INT_MAX
489c9b5090e2 Initial load
duke
parents:
diff changeset
   497
  CompressedWriteStream bytes((int)estimate_size_in_bytes());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   498
33160
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   499
#if INCLUDE_JVMCI
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   500
  if (_using_dep_values) {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   501
    for (int deptv = (int)FIRST_TYPE; deptv < (int)TYPE_LIMIT; deptv++) {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   502
      DepType dept = (DepType)deptv;
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   503
      GrowableArray<DepValue>* deps = _dep_values[dept];
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   504
      if (deps->length() == 0)  continue;
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   505
      int stride = dep_args(dept);
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   506
      int ctxkj  = dep_context_arg(dept);  // -1 if no context arg
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   507
      assert(stride > 0, "sanity");
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   508
      for (int i = 0; i < deps->length(); i += stride) {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   509
        jbyte code_byte = (jbyte)dept;
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   510
        int skipj = -1;
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   511
        if (ctxkj >= 0 && ctxkj+1 < stride) {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   512
          Klass*  ctxk = deps->at(i+ctxkj+0).as_klass(_oop_recorder);
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   513
          DepValue x = deps->at(i+ctxkj+1);  // following argument
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   514
          if (ctxk == ctxk_encoded_as_null(dept, x.as_metadata(_oop_recorder))) {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   515
            skipj = ctxkj;  // we win:  maybe one less oop to keep track of
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   516
            code_byte |= default_context_type_bit;
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   517
          }
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   518
        }
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   519
        bytes.write_byte(code_byte);
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   520
        for (int j = 0; j < stride; j++) {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   521
          if (j == skipj)  continue;
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   522
          DepValue v = deps->at(i+j);
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   523
          int idx = v.index();
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   524
          bytes.write_int(idx);
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   525
        }
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   526
      }
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   527
    }
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   528
  } else {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   529
#endif // INCLUDE_JVMCI
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   530
  for (int deptv = (int)FIRST_TYPE; deptv < (int)TYPE_LIMIT; deptv++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   531
    DepType dept = (DepType)deptv;
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   532
    GrowableArray<ciBaseObject*>* deps = _deps[dept];
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   533
    if (deps->length() == 0)  continue;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   534
    int stride = dep_args(dept);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   535
    int ctxkj  = dep_context_arg(dept);  // -1 if no context arg
489c9b5090e2 Initial load
duke
parents:
diff changeset
   536
    assert(stride > 0, "sanity");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   537
    for (int i = 0; i < deps->length(); i += stride) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   538
      jbyte code_byte = (jbyte)dept;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   539
      int skipj = -1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   540
      if (ctxkj >= 0 && ctxkj+1 < stride) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   541
        ciKlass*  ctxk = deps->at(i+ctxkj+0)->as_metadata()->as_klass();
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   542
        ciBaseObject* x     = deps->at(i+ctxkj+1);  // following argument
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   543
        if (ctxk == ctxk_encoded_as_null(dept, x)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   544
          skipj = ctxkj;  // we win:  maybe one less oop to keep track of
489c9b5090e2 Initial load
duke
parents:
diff changeset
   545
          code_byte |= default_context_type_bit;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   546
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   547
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   548
      bytes.write_byte(code_byte);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   549
      for (int j = 0; j < stride; j++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   550
        if (j == skipj)  continue;
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   551
        ciBaseObject* v = deps->at(i+j);
13965
fe09056af6ae 8000232: NPG: SIGSEGV in Dependencies::DepStream::check_klass_dependency on solaris-x64
vlivanov
parents: 13743
diff changeset
   552
        int idx;
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   553
        if (v->is_object()) {
13965
fe09056af6ae 8000232: NPG: SIGSEGV in Dependencies::DepStream::check_klass_dependency on solaris-x64
vlivanov
parents: 13743
diff changeset
   554
          idx = _oop_recorder->find_index(v->as_object()->constant_encoding());
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   555
        } else {
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   556
          ciMetadata* meta = v->as_metadata();
13965
fe09056af6ae 8000232: NPG: SIGSEGV in Dependencies::DepStream::check_klass_dependency on solaris-x64
vlivanov
parents: 13743
diff changeset
   557
          idx = _oop_recorder->find_index(meta->constant_encoding());
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   558
        }
13965
fe09056af6ae 8000232: NPG: SIGSEGV in Dependencies::DepStream::check_klass_dependency on solaris-x64
vlivanov
parents: 13743
diff changeset
   559
        bytes.write_int(idx);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   560
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   561
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   562
  }
33160
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   563
#if INCLUDE_JVMCI
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   564
  }
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   565
#endif
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   566
489c9b5090e2 Initial load
duke
parents:
diff changeset
   567
  // write a sentinel byte to mark the end
489c9b5090e2 Initial load
duke
parents:
diff changeset
   568
  bytes.write_byte(end_marker);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   569
489c9b5090e2 Initial load
duke
parents:
diff changeset
   570
  // round it out to a word boundary
489c9b5090e2 Initial load
duke
parents:
diff changeset
   571
  while (bytes.position() % sizeof(HeapWord) != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   572
    bytes.write_byte(end_marker);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   573
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   574
489c9b5090e2 Initial load
duke
parents:
diff changeset
   575
  // check whether the dept byte encoding really works
489c9b5090e2 Initial load
duke
parents:
diff changeset
   576
  assert((jbyte)default_context_type_bit != 0, "byte overflow");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   577
489c9b5090e2 Initial load
duke
parents:
diff changeset
   578
  _content_bytes = bytes.buffer();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   579
  _size_in_bytes = bytes.position();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   580
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   581
489c9b5090e2 Initial load
duke
parents:
diff changeset
   582
489c9b5090e2 Initial load
duke
parents:
diff changeset
   583
const char* Dependencies::_dep_name[TYPE_LIMIT] = {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   584
  "end_marker",
489c9b5090e2 Initial load
duke
parents:
diff changeset
   585
  "evol_method",
489c9b5090e2 Initial load
duke
parents:
diff changeset
   586
  "leaf_type",
489c9b5090e2 Initial load
duke
parents:
diff changeset
   587
  "abstract_with_unique_concrete_subtype",
489c9b5090e2 Initial load
duke
parents:
diff changeset
   588
  "abstract_with_no_concrete_subtype",
489c9b5090e2 Initial load
duke
parents:
diff changeset
   589
  "concrete_with_no_concrete_subtype",
489c9b5090e2 Initial load
duke
parents:
diff changeset
   590
  "unique_concrete_method",
489c9b5090e2 Initial load
duke
parents:
diff changeset
   591
  "abstract_with_exclusive_concrete_subtypes_2",
489c9b5090e2 Initial load
duke
parents:
diff changeset
   592
  "exclusive_concrete_methods_2",
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
   593
  "no_finalizable_subclasses",
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
   594
  "call_site_target_value"
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   595
};
489c9b5090e2 Initial load
duke
parents:
diff changeset
   596
489c9b5090e2 Initial load
duke
parents:
diff changeset
   597
int Dependencies::_dep_args[TYPE_LIMIT] = {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   598
  -1,// end_marker
489c9b5090e2 Initial load
duke
parents:
diff changeset
   599
  1, // evol_method m
489c9b5090e2 Initial load
duke
parents:
diff changeset
   600
  1, // leaf_type ctxk
489c9b5090e2 Initial load
duke
parents:
diff changeset
   601
  2, // abstract_with_unique_concrete_subtype ctxk, k
489c9b5090e2 Initial load
duke
parents:
diff changeset
   602
  1, // abstract_with_no_concrete_subtype ctxk
489c9b5090e2 Initial load
duke
parents:
diff changeset
   603
  1, // concrete_with_no_concrete_subtype ctxk
489c9b5090e2 Initial load
duke
parents:
diff changeset
   604
  2, // unique_concrete_method ctxk, m
489c9b5090e2 Initial load
duke
parents:
diff changeset
   605
  3, // unique_concrete_subtypes_2 ctxk, k1, k2
489c9b5090e2 Initial load
duke
parents:
diff changeset
   606
  3, // unique_concrete_methods_2 ctxk, m1, m2
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
   607
  1, // no_finalizable_subclasses ctxk
31037
01a5c5fa5681 8079205: CallSite dependency tracking is broken after sun.misc.Cleaner became automatically cleared
vlivanov
parents: 30296
diff changeset
   608
  2  // call_site_target_value call_site, method_handle
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   609
};
489c9b5090e2 Initial load
duke
parents:
diff changeset
   610
489c9b5090e2 Initial load
duke
parents:
diff changeset
   611
const char* Dependencies::dep_name(Dependencies::DepType dept) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   612
  if (!dept_in_mask(dept, all_types))  return "?bad-dep?";
489c9b5090e2 Initial load
duke
parents:
diff changeset
   613
  return _dep_name[dept];
489c9b5090e2 Initial load
duke
parents:
diff changeset
   614
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   615
489c9b5090e2 Initial load
duke
parents:
diff changeset
   616
int Dependencies::dep_args(Dependencies::DepType dept) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   617
  if (!dept_in_mask(dept, all_types))  return -1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   618
  return _dep_args[dept];
489c9b5090e2 Initial load
duke
parents:
diff changeset
   619
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   620
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
   621
void Dependencies::check_valid_dependency_type(DepType dept) {
33105
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 31037
diff changeset
   622
  guarantee(FIRST_TYPE <= dept && dept < TYPE_LIMIT, "invalid dependency type: %d", (int) dept);
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
   623
}
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
   624
48299
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   625
Dependencies::DepType Dependencies::validate_dependencies(CompileTask* task, bool counter_changed, char** failure_detail) {
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   626
  // First, check non-klass dependencies as we might return early and
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   627
  // not check klass dependencies if the system dictionary
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   628
  // modification counter hasn't changed (see below).
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   629
  for (Dependencies::DepStream deps(this); deps.next(); ) {
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   630
    if (deps.is_klass_type())  continue;  // skip klass dependencies
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   631
    Klass* witness = deps.check_dependency();
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   632
    if (witness != NULL) {
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   633
      return deps.type();
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   634
    }
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   635
  }
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   636
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   637
  // Klass dependencies must be checked when the system dictionary
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   638
  // changes.  If logging is enabled all violated dependences will be
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   639
  // recorded in the log.  In debug mode check dependencies even if
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   640
  // the system dictionary hasn't changed to verify that no invalid
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   641
  // dependencies were inserted.  Any violated dependences in this
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   642
  // case are dumped to the tty.
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   643
  if (!counter_changed && !trueInDebug) {
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   644
    return end_marker;
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   645
  }
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   646
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   647
  int klass_violations = 0;
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   648
  DepType result = end_marker;
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   649
  for (Dependencies::DepStream deps(this); deps.next(); ) {
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   650
    if (!deps.is_klass_type())  continue;  // skip non-klass dependencies
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   651
    Klass* witness = deps.check_dependency();
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   652
    if (witness != NULL) {
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   653
      if (klass_violations == 0) {
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   654
        result = deps.type();
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   655
        if (failure_detail != NULL && klass_violations == 0) {
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   656
          // Use a fixed size buffer to prevent the string stream from
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   657
          // resizing in the context of an inner resource mark.
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   658
          char* buffer = NEW_RESOURCE_ARRAY(char, O_BUFLEN);
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   659
          stringStream st(buffer, O_BUFLEN);
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   660
          deps.print_dependency(witness, true, &st);
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   661
          *failure_detail = st.as_string();
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   662
        }
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   663
      }
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   664
      klass_violations++;
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   665
      if (!counter_changed) {
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   666
        // Dependence failed but counter didn't change.  Log a message
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   667
        // describing what failed and allow the assert at the end to
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   668
        // trigger.
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   669
        deps.print_dependency(witness);
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   670
      } else if (xtty == NULL) {
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   671
        // If we're not logging then a single violation is sufficient,
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   672
        // otherwise we want to log all the dependences which were
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   673
        // violated.
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   674
        break;
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   675
      }
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   676
    }
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   677
  }
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   678
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   679
  if (klass_violations != 0) {
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   680
#ifdef ASSERT
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   681
    if (task != NULL && !counter_changed && !PrintCompilation) {
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   682
      // Print out the compile task that failed
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   683
      task->print_tty();
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   684
    }
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   685
#endif
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   686
    assert(counter_changed, "failed dependencies, but counter didn't change");
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   687
  }
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   688
  return result;
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   689
}
e8f5fc8f5f67 8191052: [Graal] java/lang/invoke/CallSiteTest.java intermittently fails with "Failed dependency of type call_site_target_value" when running with Graal as JIT
never
parents: 47216
diff changeset
   690
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   691
// for the sake of the compiler log, print out current dependencies:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   692
void Dependencies::log_all_dependencies() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   693
  if (log() == NULL)  return;
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   694
  ResourceMark rm;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   695
  for (int deptv = (int)FIRST_TYPE; deptv < (int)TYPE_LIMIT; deptv++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   696
    DepType dept = (DepType)deptv;
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   697
    GrowableArray<ciBaseObject*>* deps = _deps[dept];
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   698
    int deplen = deps->length();
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   699
    if (deplen == 0) {
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   700
      continue;
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   701
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   702
    int stride = dep_args(dept);
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   703
    GrowableArray<ciBaseObject*>* ciargs = new GrowableArray<ciBaseObject*>(stride);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   704
    for (int i = 0; i < deps->length(); i += stride) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   705
      for (int j = 0; j < stride; j++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   706
        // flush out the identities before printing
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   707
        ciargs->push(deps->at(i+j));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   708
      }
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   709
      write_dependency_to(log(), dept, ciargs);
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   710
      ciargs->clear();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   711
    }
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   712
    guarantee(deplen == deps->length(), "deps array cannot grow inside nested ResoureMark scope");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   713
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   714
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   715
489c9b5090e2 Initial load
duke
parents:
diff changeset
   716
void Dependencies::write_dependency_to(CompileLog* log,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   717
                                       DepType dept,
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   718
                                       GrowableArray<DepArgument>* args,
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   719
                                       Klass* witness) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   720
  if (log == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   721
    return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   722
  }
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   723
  ResourceMark rm;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   724
  ciEnv* env = ciEnv::current();
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   725
  GrowableArray<ciBaseObject*>* ciargs = new GrowableArray<ciBaseObject*>(args->length());
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   726
  for (GrowableArrayIterator<DepArgument> it = args->begin(); it != args->end(); ++it) {
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   727
    DepArgument arg = *it;
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   728
    if (arg.is_oop()) {
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   729
      ciargs->push(env->get_object(arg.oop_value()));
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   730
    } else {
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   731
      ciargs->push(env->get_metadata(arg.metadata_value()));
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   732
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   733
  }
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   734
  int argslen = ciargs->length();
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   735
  Dependencies::write_dependency_to(log, dept, ciargs, witness);
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   736
  guarantee(argslen == ciargs->length(), "ciargs array cannot grow inside nested ResoureMark scope");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   737
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   738
489c9b5090e2 Initial load
duke
parents:
diff changeset
   739
void Dependencies::write_dependency_to(CompileLog* log,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   740
                                       DepType dept,
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   741
                                       GrowableArray<ciBaseObject*>* args,
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   742
                                       Klass* witness) {
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   743
  if (log == NULL) {
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   744
    return;
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   745
  }
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   746
  ResourceMark rm;
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   747
  GrowableArray<int>* argids = new GrowableArray<int>(args->length());
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   748
  for (GrowableArrayIterator<ciBaseObject*> it = args->begin(); it != args->end(); ++it) {
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   749
    ciBaseObject* obj = *it;
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   750
    if (obj->is_object()) {
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   751
      argids->push(log->identify(obj->as_object()));
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   752
    } else {
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   753
      argids->push(log->identify(obj->as_metadata()));
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   754
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   755
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   756
  if (witness != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   757
    log->begin_elem("dependency_failed");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   758
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   759
    log->begin_elem("dependency");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   760
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   761
  log->print(" type='%s'", dep_name(dept));
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   762
  const int ctxkj = dep_context_arg(dept);  // -1 if no context arg
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   763
  if (ctxkj >= 0 && ctxkj < argids->length()) {
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   764
    log->print(" ctxk='%d'", argids->at(ctxkj));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   765
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   766
  // write remaining arguments, if any.
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   767
  for (int j = 0; j < argids->length(); j++) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   768
    if (j == ctxkj)  continue;  // already logged
489c9b5090e2 Initial load
duke
parents:
diff changeset
   769
    if (j == 1) {
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   770
      log->print(  " x='%d'",    argids->at(j));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   771
    } else {
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   772
      log->print(" x%d='%d'", j, argids->at(j));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   773
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   774
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   775
  if (witness != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   776
    log->object("witness", witness);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   777
    log->stamp();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   778
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   779
  log->end_elem();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   780
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   781
489c9b5090e2 Initial load
duke
parents:
diff changeset
   782
void Dependencies::write_dependency_to(xmlStream* xtty,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   783
                                       DepType dept,
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   784
                                       GrowableArray<DepArgument>* args,
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   785
                                       Klass* witness) {
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   786
  if (xtty == NULL) {
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   787
    return;
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   788
  }
46271
979ebd346ecf 8169881: Remove implicit Handle conversions oop->Handle
coleenp
parents: 40362
diff changeset
   789
  Thread* thread = Thread::current();
979ebd346ecf 8169881: Remove implicit Handle conversions oop->Handle
coleenp
parents: 40362
diff changeset
   790
  HandleMark rm(thread);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   791
  ttyLocker ttyl;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   792
  int ctxkj = dep_context_arg(dept);  // -1 if no context arg
489c9b5090e2 Initial load
duke
parents:
diff changeset
   793
  if (witness != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   794
    xtty->begin_elem("dependency_failed");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   795
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   796
    xtty->begin_elem("dependency");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   797
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   798
  xtty->print(" type='%s'", dep_name(dept));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   799
  if (ctxkj >= 0) {
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   800
    xtty->object("ctxk", args->at(ctxkj).metadata_value());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   801
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   802
  // write remaining arguments, if any.
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   803
  for (int j = 0; j < args->length(); j++) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   804
    if (j == ctxkj)  continue;  // already logged
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   805
    DepArgument arg = args->at(j);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   806
    if (j == 1) {
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   807
      if (arg.is_oop()) {
46271
979ebd346ecf 8169881: Remove implicit Handle conversions oop->Handle
coleenp
parents: 40362
diff changeset
   808
        xtty->object("x", Handle(thread, arg.oop_value()));
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   809
      } else {
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   810
        xtty->object("x", arg.metadata_value());
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   811
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   812
    } else {
46672
dc24cb5e50a1 8184309: Buld warnings from GCC 7.1 on Fedora 26
ysuenaga
parents: 46630
diff changeset
   813
      char xn[12]; sprintf(xn, "x%d", j);
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   814
      if (arg.is_oop()) {
46271
979ebd346ecf 8169881: Remove implicit Handle conversions oop->Handle
coleenp
parents: 40362
diff changeset
   815
        xtty->object(xn, Handle(thread, arg.oop_value()));
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   816
      } else {
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   817
        xtty->object(xn, arg.metadata_value());
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   818
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   819
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   820
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   821
  if (witness != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   822
    xtty->object("witness", witness);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   823
    xtty->stamp();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   824
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   825
  xtty->end_elem();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   826
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   827
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   828
void Dependencies::print_dependency(DepType dept, GrowableArray<DepArgument>* args,
33160
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   829
                                    Klass* witness, outputStream* st) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   830
  ResourceMark rm;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   831
  ttyLocker ttyl;   // keep the following output all in one block
33160
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   832
  st->print_cr("%s of type %s",
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   833
                (witness == NULL)? "Dependency": "Failed dependency",
489c9b5090e2 Initial load
duke
parents:
diff changeset
   834
                dep_name(dept));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   835
  // print arguments
489c9b5090e2 Initial load
duke
parents:
diff changeset
   836
  int ctxkj = dep_context_arg(dept);  // -1 if no context arg
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   837
  for (int j = 0; j < args->length(); j++) {
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   838
    DepArgument arg = args->at(j);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   839
    bool put_star = false;
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   840
    if (arg.is_null())  continue;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   841
    const char* what;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   842
    if (j == ctxkj) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   843
      assert(arg.is_metadata(), "must be");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   844
      what = "context";
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   845
      put_star = !Dependencies::is_concrete_klass((Klass*)arg.metadata_value());
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   846
    } else if (arg.is_method()) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   847
      what = "method ";
28514
da53c1ffc837 8064524: Compiler code generation improvements
drchase
parents: 27700
diff changeset
   848
      put_star = !Dependencies::is_concrete_method((Method*)arg.metadata_value(), NULL);
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   849
    } else if (arg.is_klass()) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   850
      what = "class  ";
489c9b5090e2 Initial load
duke
parents:
diff changeset
   851
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   852
      what = "object ";
489c9b5090e2 Initial load
duke
parents:
diff changeset
   853
    }
33160
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   854
    st->print("  %s = %s", what, (put_star? "*": ""));
26914
a270e0fac50e 8058578: -XX:+TraceDependencies is broken for call_site_target_value dependency type
vlivanov
parents: 25936
diff changeset
   855
    if (arg.is_klass()) {
33160
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   856
      st->print("%s", ((Klass*)arg.metadata_value())->external_name());
26914
a270e0fac50e 8058578: -XX:+TraceDependencies is broken for call_site_target_value dependency type
vlivanov
parents: 25936
diff changeset
   857
    } else if (arg.is_method()) {
33160
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   858
      ((Method*)arg.metadata_value())->print_value_on(st);
26914
a270e0fac50e 8058578: -XX:+TraceDependencies is broken for call_site_target_value dependency type
vlivanov
parents: 25936
diff changeset
   859
    } else if (arg.is_oop()) {
33160
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   860
      arg.oop_value()->print_value_on(st);
26914
a270e0fac50e 8058578: -XX:+TraceDependencies is broken for call_site_target_value dependency type
vlivanov
parents: 25936
diff changeset
   861
    } else {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   862
      ShouldNotReachHere(); // Provide impl for this type.
26914
a270e0fac50e 8058578: -XX:+TraceDependencies is broken for call_site_target_value dependency type
vlivanov
parents: 25936
diff changeset
   863
    }
a270e0fac50e 8058578: -XX:+TraceDependencies is broken for call_site_target_value dependency type
vlivanov
parents: 25936
diff changeset
   864
33160
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   865
    st->cr();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   866
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   867
  if (witness != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   868
    bool put_star = !Dependencies::is_concrete_klass(witness);
33160
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   869
    st->print_cr("  witness = %s%s",
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   870
                  (put_star? "*": ""),
14488
ab48109f7d1b 8001471: Klass::cast() does nothing
hseigel
parents: 14391
diff changeset
   871
                  witness->external_name());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   872
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   873
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   874
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   875
void Dependencies::DepStream::log_dependency(Klass* witness) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   876
  if (_deps == NULL && xtty == NULL)  return;  // fast cutout for runtime
14477
95e66ea71f71 6830717: replay of compilations would help with debugging
minqi
parents: 14391
diff changeset
   877
  ResourceMark rm;
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   878
  const int nargs = argument_count();
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   879
  GrowableArray<DepArgument>* args = new GrowableArray<DepArgument>(nargs);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   880
  for (int j = 0; j < nargs; j++) {
30296
95baefac8485 8057967: CallSite dependency tracking scales devastatingly poorly
vlivanov
parents: 30195
diff changeset
   881
    if (is_oop_argument(j)) {
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   882
      args->push(argument_oop(j));
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   883
    } else {
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   884
      args->push(argument(j));
13965
fe09056af6ae 8000232: NPG: SIGSEGV in Dependencies::DepStream::check_klass_dependency on solaris-x64
vlivanov
parents: 13743
diff changeset
   885
    }
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   886
  }
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   887
  int argslen = args->length();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   888
  if (_deps != NULL && _deps->log() != NULL) {
33160
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   889
    if (ciEnv::current() != NULL) {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   890
      Dependencies::write_dependency_to(_deps->log(), type(), args, witness);
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   891
    } else {
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   892
      // Treat the CompileLog as an xmlstream instead
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   893
      Dependencies::write_dependency_to((xmlStream*)_deps->log(), type(), args, witness);
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   894
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   895
  } else {
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   896
    Dependencies::write_dependency_to(xtty, type(), args, witness);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   897
  }
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   898
  guarantee(argslen == args->length(), "args array cannot grow inside nested ResoureMark scope");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   899
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   900
33160
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   901
void Dependencies::DepStream::print_dependency(Klass* witness, bool verbose, outputStream* st) {
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   902
  ResourceMark rm;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   903
  int nargs = argument_count();
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   904
  GrowableArray<DepArgument>* args = new GrowableArray<DepArgument>(nargs);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   905
  for (int j = 0; j < nargs; j++) {
30296
95baefac8485 8057967: CallSite dependency tracking scales devastatingly poorly
vlivanov
parents: 30195
diff changeset
   906
    if (is_oop_argument(j)) {
26914
a270e0fac50e 8058578: -XX:+TraceDependencies is broken for call_site_target_value dependency type
vlivanov
parents: 25936
diff changeset
   907
      args->push(argument_oop(j));
a270e0fac50e 8058578: -XX:+TraceDependencies is broken for call_site_target_value dependency type
vlivanov
parents: 25936
diff changeset
   908
    } else {
a270e0fac50e 8058578: -XX:+TraceDependencies is broken for call_site_target_value dependency type
vlivanov
parents: 25936
diff changeset
   909
      args->push(argument(j));
a270e0fac50e 8058578: -XX:+TraceDependencies is broken for call_site_target_value dependency type
vlivanov
parents: 25936
diff changeset
   910
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   911
  }
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   912
  int argslen = args->length();
33160
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   913
  Dependencies::print_dependency(type(), args, witness, st);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   914
  if (verbose) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   915
    if (_code != NULL) {
33160
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   916
      st->print("  code: ");
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   917
      _code->print_value_on(st);
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 31037
diff changeset
   918
      st->cr();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   919
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   920
  }
25936
9b693ed74c13 8040920: Uninitialised memory in hotspot/src/share/vm/code/dependencies.cpp
morris
parents: 24484
diff changeset
   921
  guarantee(argslen == args->length(), "args array cannot grow inside nested ResoureMark scope");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   922
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   923
489c9b5090e2 Initial load
duke
parents:
diff changeset
   924
489c9b5090e2 Initial load
duke
parents:
diff changeset
   925
/// Dependency stream support (decodes dependencies from an nmethod):
489c9b5090e2 Initial load
duke
parents:
diff changeset
   926
489c9b5090e2 Initial load
duke
parents:
diff changeset
   927
#ifdef ASSERT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   928
void Dependencies::DepStream::initial_asserts(size_t byte_limit) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   929
  assert(must_be_in_vm(), "raw oops here");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   930
  _byte_limit = byte_limit;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   931
  _type       = (DepType)(end_marker-1);  // defeat "already at end" assert
489c9b5090e2 Initial load
duke
parents:
diff changeset
   932
  assert((_code!=NULL) + (_deps!=NULL) == 1, "one or t'other");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   933
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   934
#endif //ASSERT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   935
489c9b5090e2 Initial load
duke
parents:
diff changeset
   936
bool Dependencies::DepStream::next() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   937
  assert(_type != end_marker, "already at end");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   938
  if (_bytes.position() == 0 && _code != NULL
489c9b5090e2 Initial load
duke
parents:
diff changeset
   939
      && _code->dependencies_size() == 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   940
    // Method has no dependencies at all.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   941
    return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   942
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   943
  int code_byte = (_bytes.read_byte() & 0xFF);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   944
  if (code_byte == end_marker) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   945
    DEBUG_ONLY(_type = end_marker);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   946
    return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   947
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   948
    int ctxk_bit = (code_byte & Dependencies::default_context_type_bit);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   949
    code_byte -= ctxk_bit;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   950
    DepType dept = (DepType)code_byte;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   951
    _type = dept;
10503
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
   952
    Dependencies::check_valid_dependency_type(dept);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   953
    int stride = _dep_args[dept];
489c9b5090e2 Initial load
duke
parents:
diff changeset
   954
    assert(stride == dep_args(dept), "sanity");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   955
    int skipj = -1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   956
    if (ctxk_bit != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   957
      skipj = 0;  // currently the only context argument is at zero
489c9b5090e2 Initial load
duke
parents:
diff changeset
   958
      assert(skipj == dep_context_arg(dept), "zero arg always ctxk");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   959
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   960
    for (int j = 0; j < stride; j++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   961
      _xi[j] = (j == skipj)? 0: _bytes.read_int();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   962
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   963
    DEBUG_ONLY(_xi[stride] = -1);   // help detect overruns
489c9b5090e2 Initial load
duke
parents:
diff changeset
   964
    return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   965
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   966
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   967
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   968
inline Metadata* Dependencies::DepStream::recorded_metadata_at(int i) {
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   969
  Metadata* o = NULL;
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   970
  if (_code != NULL) {
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   971
    o = _code->metadata_at(i);
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   972
  } else {
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   973
    o = _deps->oop_recorder()->metadata_at(i);
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   974
  }
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   975
  return o;
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   976
}
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   977
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   978
inline oop Dependencies::DepStream::recorded_oop_at(int i) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   979
  return (_code != NULL)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   980
         ? _code->oop_at(i)
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   981
    : JNIHandles::resolve(_deps->oop_recorder()->oop_at(i));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   982
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   983
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   984
Metadata* Dependencies::DepStream::argument(int i) {
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   985
  Metadata* result = recorded_metadata_at(argument_index(i));
13965
fe09056af6ae 8000232: NPG: SIGSEGV in Dependencies::DepStream::check_klass_dependency on solaris-x64
vlivanov
parents: 13743
diff changeset
   986
fe09056af6ae 8000232: NPG: SIGSEGV in Dependencies::DepStream::check_klass_dependency on solaris-x64
vlivanov
parents: 13743
diff changeset
   987
  if (result == NULL) { // Explicit context argument can be compressed
fe09056af6ae 8000232: NPG: SIGSEGV in Dependencies::DepStream::check_klass_dependency on solaris-x64
vlivanov
parents: 13743
diff changeset
   988
    int ctxkj = dep_context_arg(type());  // -1 if no explicit context arg
fe09056af6ae 8000232: NPG: SIGSEGV in Dependencies::DepStream::check_klass_dependency on solaris-x64
vlivanov
parents: 13743
diff changeset
   989
    if (ctxkj >= 0 && i == ctxkj && ctxkj+1 < argument_count()) {
fe09056af6ae 8000232: NPG: SIGSEGV in Dependencies::DepStream::check_klass_dependency on solaris-x64
vlivanov
parents: 13743
diff changeset
   990
      result = ctxk_encoded_as_null(type(), argument(ctxkj+1));
fe09056af6ae 8000232: NPG: SIGSEGV in Dependencies::DepStream::check_klass_dependency on solaris-x64
vlivanov
parents: 13743
diff changeset
   991
    }
fe09056af6ae 8000232: NPG: SIGSEGV in Dependencies::DepStream::check_klass_dependency on solaris-x64
vlivanov
parents: 13743
diff changeset
   992
  }
fe09056af6ae 8000232: NPG: SIGSEGV in Dependencies::DepStream::check_klass_dependency on solaris-x64
vlivanov
parents: 13743
diff changeset
   993
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   994
  assert(result == NULL || result->is_klass() || result->is_method(), "must be");
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
   995
  return result;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   996
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   997
22506
0759c126204d 7194669: CodeCache::mark_for_deoptimization should avoid verifying dependencies multiple times
anoll
parents: 20391
diff changeset
   998
/**
0759c126204d 7194669: CodeCache::mark_for_deoptimization should avoid verifying dependencies multiple times
anoll
parents: 20391
diff changeset
   999
 * Returns a unique identifier for each dependency argument.
0759c126204d 7194669: CodeCache::mark_for_deoptimization should avoid verifying dependencies multiple times
anoll
parents: 20391
diff changeset
  1000
 */
0759c126204d 7194669: CodeCache::mark_for_deoptimization should avoid verifying dependencies multiple times
anoll
parents: 20391
diff changeset
  1001
uintptr_t Dependencies::DepStream::get_identifier(int i) {
30296
95baefac8485 8057967: CallSite dependency tracking scales devastatingly poorly
vlivanov
parents: 30195
diff changeset
  1002
  if (is_oop_argument(i)) {
22506
0759c126204d 7194669: CodeCache::mark_for_deoptimization should avoid verifying dependencies multiple times
anoll
parents: 20391
diff changeset
  1003
    return (uintptr_t)(oopDesc*)argument_oop(i);
0759c126204d 7194669: CodeCache::mark_for_deoptimization should avoid verifying dependencies multiple times
anoll
parents: 20391
diff changeset
  1004
  } else {
0759c126204d 7194669: CodeCache::mark_for_deoptimization should avoid verifying dependencies multiple times
anoll
parents: 20391
diff changeset
  1005
    return (uintptr_t)argument(i);
0759c126204d 7194669: CodeCache::mark_for_deoptimization should avoid verifying dependencies multiple times
anoll
parents: 20391
diff changeset
  1006
  }
0759c126204d 7194669: CodeCache::mark_for_deoptimization should avoid verifying dependencies multiple times
anoll
parents: 20391
diff changeset
  1007
}
0759c126204d 7194669: CodeCache::mark_for_deoptimization should avoid verifying dependencies multiple times
anoll
parents: 20391
diff changeset
  1008
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1009
oop Dependencies::DepStream::argument_oop(int i) {
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1010
  oop result = recorded_oop_at(argument_index(i));
46968
9119841280f4 8160399: is_oop_or_null involves undefined behavior
coleenp
parents: 46672
diff changeset
  1011
  assert(oopDesc::is_oop_or_null(result), "must be");
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1012
  return result;
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1013
}
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1014
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1015
Klass* Dependencies::DepStream::context_type() {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1016
  assert(must_be_in_vm(), "raw oops here");
10503
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
  1017
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
  1018
  // Most dependencies have an explicit context type argument.
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
  1019
  {
13965
fe09056af6ae 8000232: NPG: SIGSEGV in Dependencies::DepStream::check_klass_dependency on solaris-x64
vlivanov
parents: 13743
diff changeset
  1020
    int ctxkj = dep_context_arg(type());  // -1 if no explicit context arg
10503
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
  1021
    if (ctxkj >= 0) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1022
      Metadata* k = argument(ctxkj);
13965
fe09056af6ae 8000232: NPG: SIGSEGV in Dependencies::DepStream::check_klass_dependency on solaris-x64
vlivanov
parents: 13743
diff changeset
  1023
      assert(k != NULL && k->is_klass(), "type check");
fe09056af6ae 8000232: NPG: SIGSEGV in Dependencies::DepStream::check_klass_dependency on solaris-x64
vlivanov
parents: 13743
diff changeset
  1024
      return (Klass*)k;
10503
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
  1025
    }
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
  1026
  }
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
  1027
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
  1028
  // Some dependencies are using the klass of the first object
30296
95baefac8485 8057967: CallSite dependency tracking scales devastatingly poorly
vlivanov
parents: 30195
diff changeset
  1029
  // argument as implicit context type.
10503
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
  1030
  {
13965
fe09056af6ae 8000232: NPG: SIGSEGV in Dependencies::DepStream::check_klass_dependency on solaris-x64
vlivanov
parents: 13743
diff changeset
  1031
    int ctxkj = dep_implicit_context_arg(type());
10503
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
  1032
    if (ctxkj >= 0) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1033
      Klass* k = argument_oop(ctxkj)->klass();
13965
fe09056af6ae 8000232: NPG: SIGSEGV in Dependencies::DepStream::check_klass_dependency on solaris-x64
vlivanov
parents: 13743
diff changeset
  1034
      assert(k != NULL && k->is_klass(), "type check");
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1035
      return (Klass*) k;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1036
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1037
  }
10503
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
  1038
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
  1039
  // And some dependencies don't have a context type at all,
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
  1040
  // e.g. evol_method.
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
  1041
  return NULL;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1042
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1043
22506
0759c126204d 7194669: CodeCache::mark_for_deoptimization should avoid verifying dependencies multiple times
anoll
parents: 20391
diff changeset
  1044
// ----------------- DependencySignature --------------------------------------
23187
0f438571f278 8035946: Use ResourceHashtable for dependency checking
anoll
parents: 22921
diff changeset
  1045
bool DependencySignature::equals(DependencySignature const& s1, DependencySignature const& s2) {
0f438571f278 8035946: Use ResourceHashtable for dependency checking
anoll
parents: 22921
diff changeset
  1046
  if ((s1.type() != s2.type()) || (s1.args_count() != s2.args_count())) {
22506
0759c126204d 7194669: CodeCache::mark_for_deoptimization should avoid verifying dependencies multiple times
anoll
parents: 20391
diff changeset
  1047
    return false;
0759c126204d 7194669: CodeCache::mark_for_deoptimization should avoid verifying dependencies multiple times
anoll
parents: 20391
diff changeset
  1048
  }
0759c126204d 7194669: CodeCache::mark_for_deoptimization should avoid verifying dependencies multiple times
anoll
parents: 20391
diff changeset
  1049
23187
0f438571f278 8035946: Use ResourceHashtable for dependency checking
anoll
parents: 22921
diff changeset
  1050
  for (int i = 0; i < s1.args_count(); i++) {
0f438571f278 8035946: Use ResourceHashtable for dependency checking
anoll
parents: 22921
diff changeset
  1051
    if (s1.arg(i) != s2.arg(i)) {
22506
0759c126204d 7194669: CodeCache::mark_for_deoptimization should avoid verifying dependencies multiple times
anoll
parents: 20391
diff changeset
  1052
      return false;
0759c126204d 7194669: CodeCache::mark_for_deoptimization should avoid verifying dependencies multiple times
anoll
parents: 20391
diff changeset
  1053
    }
0759c126204d 7194669: CodeCache::mark_for_deoptimization should avoid verifying dependencies multiple times
anoll
parents: 20391
diff changeset
  1054
  }
0759c126204d 7194669: CodeCache::mark_for_deoptimization should avoid verifying dependencies multiple times
anoll
parents: 20391
diff changeset
  1055
  return true;
0759c126204d 7194669: CodeCache::mark_for_deoptimization should avoid verifying dependencies multiple times
anoll
parents: 20391
diff changeset
  1056
}
0759c126204d 7194669: CodeCache::mark_for_deoptimization should avoid verifying dependencies multiple times
anoll
parents: 20391
diff changeset
  1057
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1058
/// Checking dependencies:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1059
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1060
// This hierarchy walker inspects subtypes of a given type,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1061
// trying to find a "bad" class which breaks a dependency.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1062
// Such a class is called a "witness" to the broken dependency.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1063
// While searching around, we ignore "participants", which
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1064
// are already known to the dependency.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1065
class ClassHierarchyWalker {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1066
 public:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1067
  enum { PARTICIPANT_LIMIT = 3 };
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1068
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1069
 private:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1070
  // optional method descriptor to check for:
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
  1071
  Symbol* _name;
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
  1072
  Symbol* _signature;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1073
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1074
  // special classes which are not allowed to be witnesses:
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1075
  Klass*    _participants[PARTICIPANT_LIMIT+1];
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1076
  int       _num_participants;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1077
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1078
  // cache of method lookups
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1079
  Method* _found_methods[PARTICIPANT_LIMIT+1];
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1080
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1081
  // if non-zero, tells how many witnesses to convert to participants
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1082
  int       _record_witnesses;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1083
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1084
  void initialize(Klass* participant) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1085
    _record_witnesses = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1086
    _participants[0]  = participant;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1087
    _found_methods[0] = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1088
    _num_participants = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1089
    if (participant != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1090
      // Terminating NULL.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1091
      _participants[1] = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1092
      _found_methods[1] = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1093
      _num_participants = 1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1094
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1095
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1096
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1097
  void initialize_from_method(Method* m) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1098
    assert(m != NULL && m->is_method(), "sanity");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1099
    _name      = m->name();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1100
    _signature = m->signature();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1101
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1102
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1103
 public:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1104
  // The walker is initialized to recognize certain methods and/or types
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1105
  // as friendly participants.
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1106
  ClassHierarchyWalker(Klass* participant, Method* m) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1107
    initialize_from_method(m);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1108
    initialize(participant);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1109
  }
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1110
  ClassHierarchyWalker(Method* m) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1111
    initialize_from_method(m);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1112
    initialize(NULL);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1113
  }
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1114
  ClassHierarchyWalker(Klass* participant = NULL) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1115
    _name      = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1116
    _signature = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1117
    initialize(participant);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1118
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1119
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1120
  // This is common code for two searches:  One for concrete subtypes,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1121
  // the other for concrete method implementations and overrides.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1122
  bool doing_subtype_search() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1123
    return _name == NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1124
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1125
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1126
  int num_participants() { return _num_participants; }
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1127
  Klass* participant(int n) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1128
    assert((uint)n <= (uint)_num_participants, "oob");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1129
    return _participants[n];
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1130
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1131
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1132
  // Note:  If n==num_participants, returns NULL.
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1133
  Method* found_method(int n) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1134
    assert((uint)n <= (uint)_num_participants, "oob");
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1135
    Method* fm = _found_methods[n];
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1136
    assert(n == _num_participants || fm != NULL, "proper usage");
30195
fff35af9b018 8069263: assert(fm == NULL || fm->method_holder() == _participants[n]) failed: sanity
roland
parents: 29081
diff changeset
  1137
    if (fm != NULL && fm->method_holder() != _participants[n]) {
fff35af9b018 8069263: assert(fm == NULL || fm->method_holder() == _participants[n]) failed: sanity
roland
parents: 29081
diff changeset
  1138
      // Default methods from interfaces can be added to classes. In
fff35af9b018 8069263: assert(fm == NULL || fm->method_holder() == _participants[n]) failed: sanity
roland
parents: 29081
diff changeset
  1139
      // that case the holder of the method is not the class but the
fff35af9b018 8069263: assert(fm == NULL || fm->method_holder() == _participants[n]) failed: sanity
roland
parents: 29081
diff changeset
  1140
      // interface where it's defined.
fff35af9b018 8069263: assert(fm == NULL || fm->method_holder() == _participants[n]) failed: sanity
roland
parents: 29081
diff changeset
  1141
      assert(fm->is_default_method(), "sanity");
fff35af9b018 8069263: assert(fm == NULL || fm->method_holder() == _participants[n]) failed: sanity
roland
parents: 29081
diff changeset
  1142
      return NULL;
fff35af9b018 8069263: assert(fm == NULL || fm->method_holder() == _participants[n]) failed: sanity
roland
parents: 29081
diff changeset
  1143
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1144
    return fm;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1145
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1146
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1147
#ifdef ASSERT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1148
  // Assert that m is inherited into ctxk, without intervening overrides.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1149
  // (May return true even if this is not true, in corner cases where we punt.)
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1150
  bool check_method_context(Klass* ctxk, Method* m) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1151
    if (m->method_holder() == ctxk)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1152
      return true;  // Quick win.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1153
    if (m->is_private())
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1154
      return false; // Quick lose.  Should not happen.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1155
    if (!(m->is_public() || m->is_protected()))
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1156
      // The override story is complex when packages get involved.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1157
      return true;  // Must punt the assertion to true.
40362
94addd7cfa10 8162477: [JVMCI] assert(wf.check_method_context(ctxk, m)) failed: proper context
never
parents: 37248
diff changeset
  1158
    Method* lm = ctxk->lookup_method(m->name(), m->signature());
94addd7cfa10 8162477: [JVMCI] assert(wf.check_method_context(ctxk, m)) failed: proper context
never
parents: 37248
diff changeset
  1159
    if (lm == NULL && ctxk->is_instance_klass()) {
20391
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 18439
diff changeset
  1160
      // It might be an interface method
40362
94addd7cfa10 8162477: [JVMCI] assert(wf.check_method_context(ctxk, m)) failed: proper context
never
parents: 37248
diff changeset
  1161
      lm = InstanceKlass::cast(ctxk)->lookup_method_in_ordered_interfaces(m->name(),
94addd7cfa10 8162477: [JVMCI] assert(wf.check_method_context(ctxk, m)) failed: proper context
never
parents: 37248
diff changeset
  1162
                                                                          m->signature());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1163
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1164
    if (lm == m)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1165
      // Method m is inherited into ctxk.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1166
      return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1167
    if (lm != NULL) {
10984
bca73db4935e 7105305: assert check_method_context proper context
never
parents: 10503
diff changeset
  1168
      if (!(lm->is_public() || lm->is_protected())) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1169
        // Method is [package-]private, so the override story is complex.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1170
        return true;  // Must punt the assertion to true.
10984
bca73db4935e 7105305: assert check_method_context proper context
never
parents: 10503
diff changeset
  1171
      }
bca73db4935e 7105305: assert check_method_context proper context
never
parents: 10503
diff changeset
  1172
      if (lm->is_static()) {
bca73db4935e 7105305: assert check_method_context proper context
never
parents: 10503
diff changeset
  1173
        // Static methods don't override non-static so punt
bca73db4935e 7105305: assert check_method_context proper context
never
parents: 10503
diff changeset
  1174
        return true;
bca73db4935e 7105305: assert check_method_context proper context
never
parents: 10503
diff changeset
  1175
      }
40362
94addd7cfa10 8162477: [JVMCI] assert(wf.check_method_context(ctxk, m)) failed: proper context
never
parents: 37248
diff changeset
  1176
      if (!Dependencies::is_concrete_method(lm, ctxk) &&
94addd7cfa10 8162477: [JVMCI] assert(wf.check_method_context(ctxk, m)) failed: proper context
never
parents: 37248
diff changeset
  1177
          !Dependencies::is_concrete_method(m, ctxk)) {
94addd7cfa10 8162477: [JVMCI] assert(wf.check_method_context(ctxk, m)) failed: proper context
never
parents: 37248
diff changeset
  1178
        // They are both non-concrete
94addd7cfa10 8162477: [JVMCI] assert(wf.check_method_context(ctxk, m)) failed: proper context
never
parents: 37248
diff changeset
  1179
        if (lm->method_holder()->is_subtype_of(m->method_holder())) {
94addd7cfa10 8162477: [JVMCI] assert(wf.check_method_context(ctxk, m)) failed: proper context
never
parents: 37248
diff changeset
  1180
          // Method m is overridden by lm, but both are non-concrete.
94addd7cfa10 8162477: [JVMCI] assert(wf.check_method_context(ctxk, m)) failed: proper context
never
parents: 37248
diff changeset
  1181
          return true;
94addd7cfa10 8162477: [JVMCI] assert(wf.check_method_context(ctxk, m)) failed: proper context
never
parents: 37248
diff changeset
  1182
        }
94addd7cfa10 8162477: [JVMCI] assert(wf.check_method_context(ctxk, m)) failed: proper context
never
parents: 37248
diff changeset
  1183
        if (lm->method_holder()->is_interface() && m->method_holder()->is_interface() &&
94addd7cfa10 8162477: [JVMCI] assert(wf.check_method_context(ctxk, m)) failed: proper context
never
parents: 37248
diff changeset
  1184
            ctxk->is_subtype_of(m->method_holder()) && ctxk->is_subtype_of(lm->method_holder())) {
94addd7cfa10 8162477: [JVMCI] assert(wf.check_method_context(ctxk, m)) failed: proper context
never
parents: 37248
diff changeset
  1185
          // Interface method defined in multiple super interfaces
94addd7cfa10 8162477: [JVMCI] assert(wf.check_method_context(ctxk, m)) failed: proper context
never
parents: 37248
diff changeset
  1186
          return true;
94addd7cfa10 8162477: [JVMCI] assert(wf.check_method_context(ctxk, m)) failed: proper context
never
parents: 37248
diff changeset
  1187
        }
94addd7cfa10 8162477: [JVMCI] assert(wf.check_method_context(ctxk, m)) failed: proper context
never
parents: 37248
diff changeset
  1188
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1189
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1190
    ResourceMark rm;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1191
    tty->print_cr("Dependency method not found in the associated context:");
14488
ab48109f7d1b 8001471: Klass::cast() does nothing
hseigel
parents: 14391
diff changeset
  1192
    tty->print_cr("  context = %s", ctxk->external_name());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1193
    tty->print(   "  method = "); m->print_short_name(tty); tty->cr();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1194
    if (lm != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1195
      tty->print( "  found = "); lm->print_short_name(tty); tty->cr();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1196
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1197
    return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1198
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1199
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1200
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1201
  void add_participant(Klass* participant) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1202
    assert(_num_participants + _record_witnesses < PARTICIPANT_LIMIT, "oob");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1203
    int np = _num_participants++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1204
    _participants[np] = participant;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1205
    _participants[np+1] = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1206
    _found_methods[np+1] = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1207
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1208
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1209
  void record_witnesses(int add) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1210
    if (add > PARTICIPANT_LIMIT)  add = PARTICIPANT_LIMIT;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1211
    assert(_num_participants + add < PARTICIPANT_LIMIT, "oob");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1212
    _record_witnesses = add;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1213
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1214
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1215
  bool is_witness(Klass* k) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1216
    if (doing_subtype_search()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1217
      return Dependencies::is_concrete_klass(k);
33611
9abd65805e19 8139203: Consistent naming for klass type predicates
coleenp
parents: 33602
diff changeset
  1218
    } else if (!k->is_instance_klass()) {
27700
a402738ebfcf 8050079: crash while compiling java.lang.ref.Finalizer::runFinalizer
thartmann
parents: 26914
diff changeset
  1219
      return false; // no methods to find in an array type
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1220
    } else {
28514
da53c1ffc837 8064524: Compiler code generation improvements
drchase
parents: 27700
diff changeset
  1221
      // Search class hierarchy first.
da53c1ffc837 8064524: Compiler code generation improvements
drchase
parents: 27700
diff changeset
  1222
      Method* m = InstanceKlass::cast(k)->find_instance_method(_name, _signature);
da53c1ffc837 8064524: Compiler code generation improvements
drchase
parents: 27700
diff changeset
  1223
      if (!Dependencies::is_concrete_method(m, k)) {
da53c1ffc837 8064524: Compiler code generation improvements
drchase
parents: 27700
diff changeset
  1224
        // Check interface defaults also, if any exist.
da53c1ffc837 8064524: Compiler code generation improvements
drchase
parents: 27700
diff changeset
  1225
        Array<Method*>* default_methods = InstanceKlass::cast(k)->default_methods();
da53c1ffc837 8064524: Compiler code generation improvements
drchase
parents: 27700
diff changeset
  1226
        if (default_methods == NULL)
da53c1ffc837 8064524: Compiler code generation improvements
drchase
parents: 27700
diff changeset
  1227
            return false;
da53c1ffc837 8064524: Compiler code generation improvements
drchase
parents: 27700
diff changeset
  1228
        m = InstanceKlass::cast(k)->find_method(default_methods, _name, _signature);
da53c1ffc837 8064524: Compiler code generation improvements
drchase
parents: 27700
diff changeset
  1229
        if (!Dependencies::is_concrete_method(m, NULL))
da53c1ffc837 8064524: Compiler code generation improvements
drchase
parents: 27700
diff changeset
  1230
            return false;
da53c1ffc837 8064524: Compiler code generation improvements
drchase
parents: 27700
diff changeset
  1231
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1232
      _found_methods[_num_participants] = m;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1233
      // Note:  If add_participant(k) is called,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1234
      // the method m will already be memoized for it.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1235
      return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1236
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1237
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1238
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1239
  bool is_participant(Klass* k) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1240
    if (k == _participants[0]) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1241
      return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1242
    } else if (_num_participants <= 1) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1243
      return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1244
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1245
      return in_list(k, &_participants[1]);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1246
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1247
  }
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1248
  bool ignore_witness(Klass* witness) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1249
    if (_record_witnesses == 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1250
      return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1251
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1252
      --_record_witnesses;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1253
      add_participant(witness);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1254
      return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1255
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1256
  }
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1257
  static bool in_list(Klass* x, Klass** list) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1258
    for (int i = 0; ; i++) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1259
      Klass* y = list[i];
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1260
      if (y == NULL)  break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1261
      if (y == x)  return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1262
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1263
    return false;  // not in list
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1264
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1265
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1266
 private:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1267
  // the actual search method:
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1268
  Klass* find_witness_anywhere(Klass* context_type,
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1269
                                 bool participants_hide_witnesses,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1270
                                 bool top_level_call = true);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1271
  // the spot-checking version:
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1272
  Klass* find_witness_in(KlassDepChange& changes,
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1273
                         Klass* context_type,
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1274
                           bool participants_hide_witnesses);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1275
 public:
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1276
  Klass* find_witness_subtype(Klass* context_type, KlassDepChange* changes = NULL) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1277
    assert(doing_subtype_search(), "must set up a subtype search");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1278
    // When looking for unexpected concrete types,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1279
    // do not look beneath expected ones.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1280
    const bool participants_hide_witnesses = true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1281
    // CX > CC > C' is OK, even if C' is new.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1282
    // CX > { CC,  C' } is not OK if C' is new, and C' is the witness.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1283
    if (changes != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1284
      return find_witness_in(*changes, context_type, participants_hide_witnesses);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1285
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1286
      return find_witness_anywhere(context_type, participants_hide_witnesses);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1287
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1288
  }
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1289
  Klass* find_witness_definer(Klass* context_type, KlassDepChange* changes = NULL) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1290
    assert(!doing_subtype_search(), "must set up a method definer search");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1291
    // When looking for unexpected concrete methods,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1292
    // look beneath expected ones, to see if there are overrides.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1293
    const bool participants_hide_witnesses = true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1294
    // CX.m > CC.m > C'.m is not OK, if C'.m is new, and C' is the witness.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1295
    if (changes != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1296
      return find_witness_in(*changes, context_type, !participants_hide_witnesses);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1297
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1298
      return find_witness_anywhere(context_type, !participants_hide_witnesses);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1299
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1300
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1301
};
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1302
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1303
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1304
static int deps_find_witness_calls = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1305
static int deps_find_witness_steps = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1306
static int deps_find_witness_recursions = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1307
static int deps_find_witness_singles = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1308
static int deps_find_witness_print = 0; // set to -1 to force a final print
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1309
static bool count_find_witness_calls() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1310
  if (TraceDependencies || LogCompilation) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1311
    int pcount = deps_find_witness_print + 1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1312
    bool final_stats      = (pcount == 0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1313
    bool initial_call     = (pcount == 1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1314
    bool occasional_print = ((pcount & ((1<<10) - 1)) == 0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1315
    if (pcount < 0)  pcount = 1; // crude overflow protection
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1316
    deps_find_witness_print = pcount;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1317
    if (VerifyDependencies && initial_call) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1318
      tty->print_cr("Warning:  TraceDependencies results may be inflated by VerifyDependencies");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1319
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1320
    if (occasional_print || final_stats) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1321
      // Every now and then dump a little info about dependency searching.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1322
      if (xtty != NULL) {
4754
8aef16f24e16 6614597: Performance variability in jvm2008 xml.validation
kvn
parents: 4493
diff changeset
  1323
       ttyLocker ttyl;
8aef16f24e16 6614597: Performance variability in jvm2008 xml.validation
kvn
parents: 4493
diff changeset
  1324
       xtty->elem("deps_find_witness calls='%d' steps='%d' recursions='%d' singles='%d'",
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1325
                   deps_find_witness_calls,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1326
                   deps_find_witness_steps,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1327
                   deps_find_witness_recursions,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1328
                   deps_find_witness_singles);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1329
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1330
      if (final_stats || (TraceDependencies && WizardMode)) {
4754
8aef16f24e16 6614597: Performance variability in jvm2008 xml.validation
kvn
parents: 4493
diff changeset
  1331
        ttyLocker ttyl;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1332
        tty->print_cr("Dependency check (find_witness) "
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1333
                      "calls=%d, steps=%d (avg=%.1f), recursions=%d, singles=%d",
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1334
                      deps_find_witness_calls,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1335
                      deps_find_witness_steps,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1336
                      (double)deps_find_witness_steps / deps_find_witness_calls,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1337
                      deps_find_witness_recursions,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1338
                      deps_find_witness_singles);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1339
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1340
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1341
    return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1342
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1343
  return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1344
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1345
#else
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1346
#define count_find_witness_calls() (0)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1347
#endif //PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1348
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1349
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1350
Klass* ClassHierarchyWalker::find_witness_in(KlassDepChange& changes,
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1351
                                               Klass* context_type,
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1352
                                               bool participants_hide_witnesses) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1353
  assert(changes.involves_context(context_type), "irrelevant dependency");
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1354
  Klass* new_type = changes.new_type();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1355
18073
f02460441ddc 8014431: cleanup warnings indicated by the -Wunused-value compiler option on linux
ccheung
parents: 14490
diff changeset
  1356
  (void)count_find_witness_calls();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1357
  NOT_PRODUCT(deps_find_witness_singles++);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1358
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1359
  // Current thread must be in VM (not native mode, as in CI):
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1360
  assert(must_be_in_vm(), "raw oops here");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1361
  // Must not move the class hierarchy during this check:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1362
  assert_locked_or_safepoint(Compile_lock);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1363
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1364
  int nof_impls = InstanceKlass::cast(context_type)->nof_implementors();
203
2e348cc9121a 6610906: inexplicable IncompatibleClassChangeError
jrose
parents: 1
diff changeset
  1365
  if (nof_impls > 1) {
2e348cc9121a 6610906: inexplicable IncompatibleClassChangeError
jrose
parents: 1
diff changeset
  1366
    // Avoid this case: *I.m > { A.m, C }; B.m > C
2e348cc9121a 6610906: inexplicable IncompatibleClassChangeError
jrose
parents: 1
diff changeset
  1367
    // %%% Until this is fixed more systematically, bail out.
2e348cc9121a 6610906: inexplicable IncompatibleClassChangeError
jrose
parents: 1
diff changeset
  1368
    // See corresponding comment in find_witness_anywhere.
2e348cc9121a 6610906: inexplicable IncompatibleClassChangeError
jrose
parents: 1
diff changeset
  1369
    return context_type;
2e348cc9121a 6610906: inexplicable IncompatibleClassChangeError
jrose
parents: 1
diff changeset
  1370
  }
2e348cc9121a 6610906: inexplicable IncompatibleClassChangeError
jrose
parents: 1
diff changeset
  1371
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1372
  assert(!is_participant(new_type), "only old classes are participants");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1373
  if (participants_hide_witnesses) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1374
    // If the new type is a subtype of a participant, we are done.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1375
    for (int i = 0; i < num_participants(); i++) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1376
      Klass* part = participant(i);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1377
      if (part == NULL)  continue;
14488
ab48109f7d1b 8001471: Klass::cast() does nothing
hseigel
parents: 14391
diff changeset
  1378
      assert(changes.involves_context(part) == new_type->is_subtype_of(part),
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1379
             "correct marking of participants, b/c new_type is unique");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1380
      if (changes.involves_context(part)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1381
        // new guy is protected from this check by previous participant
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1382
        return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1383
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1384
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1385
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1386
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1387
  if (is_witness(new_type) &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1388
      !ignore_witness(new_type)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1389
    return new_type;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1390
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1391
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1392
  return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1393
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1394
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1395
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1396
// Walk hierarchy under a context type, looking for unexpected types.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1397
// Do not report participant types, and recursively walk beneath
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1398
// them only if participants_hide_witnesses is false.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1399
// If top_level_call is false, skip testing the context type,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1400
// because the caller has already considered it.
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1401
Klass* ClassHierarchyWalker::find_witness_anywhere(Klass* context_type,
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1402
                                                     bool participants_hide_witnesses,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1403
                                                     bool top_level_call) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1404
  // Current thread must be in VM (not native mode, as in CI):
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1405
  assert(must_be_in_vm(), "raw oops here");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1406
  // Must not move the class hierarchy during this check:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1407
  assert_locked_or_safepoint(Compile_lock);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1408
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1409
  bool do_counts = count_find_witness_calls();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1410
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1411
  // Check the root of the sub-hierarchy first.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1412
  if (top_level_call) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1413
    if (do_counts) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1414
      NOT_PRODUCT(deps_find_witness_calls++);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1415
      NOT_PRODUCT(deps_find_witness_steps++);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1416
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1417
    if (is_participant(context_type)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1418
      if (participants_hide_witnesses)  return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1419
      // else fall through to search loop...
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1420
    } else if (is_witness(context_type) && !ignore_witness(context_type)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1421
      // The context is an abstract class or interface, to start with.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1422
      return context_type;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1423
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1424
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1425
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1426
  // Now we must check each implementor and each subclass.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1427
  // Use a short worklist to avoid blowing the stack.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1428
  // Each worklist entry is a *chain* of subklass siblings to process.
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1429
  const int CHAINMAX = 100;  // >= 1 + InstanceKlass::implementors_limit
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1430
  Klass* chains[CHAINMAX];
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1431
  int    chaini = 0;  // index into worklist
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1432
  Klass* chain;       // scratch variable
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1433
#define ADD_SUBCLASS_CHAIN(k)                     {  \
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1434
    assert(chaini < CHAINMAX, "oob");                \
27700
a402738ebfcf 8050079: crash while compiling java.lang.ref.Finalizer::runFinalizer
thartmann
parents: 26914
diff changeset
  1435
    chain = k->subklass();                           \
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1436
    if (chain != NULL)  chains[chaini++] = chain;    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1437
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1438
  // Look for non-abstract subclasses.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1439
  // (Note:  Interfaces do not have subclasses.)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1440
  ADD_SUBCLASS_CHAIN(context_type);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1441
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1442
  // If it is an interface, search its direct implementors.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1443
  // (Their subclasses are additional indirect implementors.
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1444
  // See InstanceKlass::add_implementor.)
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1445
  // (Note:  nof_implementors is always zero for non-interfaces.)
27700
a402738ebfcf 8050079: crash while compiling java.lang.ref.Finalizer::runFinalizer
thartmann
parents: 26914
diff changeset
  1446
  if (top_level_call) {
a402738ebfcf 8050079: crash while compiling java.lang.ref.Finalizer::runFinalizer
thartmann
parents: 26914
diff changeset
  1447
    int nof_impls = InstanceKlass::cast(context_type)->nof_implementors();
a402738ebfcf 8050079: crash while compiling java.lang.ref.Finalizer::runFinalizer
thartmann
parents: 26914
diff changeset
  1448
    if (nof_impls > 1) {
a402738ebfcf 8050079: crash while compiling java.lang.ref.Finalizer::runFinalizer
thartmann
parents: 26914
diff changeset
  1449
      // Avoid this case: *I.m > { A.m, C }; B.m > C
a402738ebfcf 8050079: crash while compiling java.lang.ref.Finalizer::runFinalizer
thartmann
parents: 26914
diff changeset
  1450
      // Here, I.m has 2 concrete implementations, but m appears unique
a402738ebfcf 8050079: crash while compiling java.lang.ref.Finalizer::runFinalizer
thartmann
parents: 26914
diff changeset
  1451
      // as A.m, because the search misses B.m when checking C.
a402738ebfcf 8050079: crash while compiling java.lang.ref.Finalizer::runFinalizer
thartmann
parents: 26914
diff changeset
  1452
      // The inherited method B.m was getting missed by the walker
a402738ebfcf 8050079: crash while compiling java.lang.ref.Finalizer::runFinalizer
thartmann
parents: 26914
diff changeset
  1453
      // when interface 'I' was the starting point.
a402738ebfcf 8050079: crash while compiling java.lang.ref.Finalizer::runFinalizer
thartmann
parents: 26914
diff changeset
  1454
      // %%% Until this is fixed more systematically, bail out.
a402738ebfcf 8050079: crash while compiling java.lang.ref.Finalizer::runFinalizer
thartmann
parents: 26914
diff changeset
  1455
      // (Old CHA had the same limitation.)
a402738ebfcf 8050079: crash while compiling java.lang.ref.Finalizer::runFinalizer
thartmann
parents: 26914
diff changeset
  1456
      return context_type;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1457
    }
27700
a402738ebfcf 8050079: crash while compiling java.lang.ref.Finalizer::runFinalizer
thartmann
parents: 26914
diff changeset
  1458
    if (nof_impls > 0) {
a402738ebfcf 8050079: crash while compiling java.lang.ref.Finalizer::runFinalizer
thartmann
parents: 26914
diff changeset
  1459
      Klass* impl = InstanceKlass::cast(context_type)->implementor();
a402738ebfcf 8050079: crash while compiling java.lang.ref.Finalizer::runFinalizer
thartmann
parents: 26914
diff changeset
  1460
      assert(impl != NULL, "just checking");
a402738ebfcf 8050079: crash while compiling java.lang.ref.Finalizer::runFinalizer
thartmann
parents: 26914
diff changeset
  1461
      // If impl is the same as the context_type, then more than one
a402738ebfcf 8050079: crash while compiling java.lang.ref.Finalizer::runFinalizer
thartmann
parents: 26914
diff changeset
  1462
      // implementor has seen. No exact info in this case.
a402738ebfcf 8050079: crash while compiling java.lang.ref.Finalizer::runFinalizer
thartmann
parents: 26914
diff changeset
  1463
      if (impl == context_type) {
a402738ebfcf 8050079: crash while compiling java.lang.ref.Finalizer::runFinalizer
thartmann
parents: 26914
diff changeset
  1464
        return context_type;  // report an inexact witness to this sad affair
a402738ebfcf 8050079: crash while compiling java.lang.ref.Finalizer::runFinalizer
thartmann
parents: 26914
diff changeset
  1465
      }
a402738ebfcf 8050079: crash while compiling java.lang.ref.Finalizer::runFinalizer
thartmann
parents: 26914
diff changeset
  1466
      if (do_counts)
a402738ebfcf 8050079: crash while compiling java.lang.ref.Finalizer::runFinalizer
thartmann
parents: 26914
diff changeset
  1467
        { NOT_PRODUCT(deps_find_witness_steps++); }
a402738ebfcf 8050079: crash while compiling java.lang.ref.Finalizer::runFinalizer
thartmann
parents: 26914
diff changeset
  1468
      if (is_participant(impl)) {
a402738ebfcf 8050079: crash while compiling java.lang.ref.Finalizer::runFinalizer
thartmann
parents: 26914
diff changeset
  1469
        if (!participants_hide_witnesses) {
a402738ebfcf 8050079: crash while compiling java.lang.ref.Finalizer::runFinalizer
thartmann
parents: 26914
diff changeset
  1470
          ADD_SUBCLASS_CHAIN(impl);
a402738ebfcf 8050079: crash while compiling java.lang.ref.Finalizer::runFinalizer
thartmann
parents: 26914
diff changeset
  1471
        }
a402738ebfcf 8050079: crash while compiling java.lang.ref.Finalizer::runFinalizer
thartmann
parents: 26914
diff changeset
  1472
      } else if (is_witness(impl) && !ignore_witness(impl)) {
a402738ebfcf 8050079: crash while compiling java.lang.ref.Finalizer::runFinalizer
thartmann
parents: 26914
diff changeset
  1473
        return impl;
a402738ebfcf 8050079: crash while compiling java.lang.ref.Finalizer::runFinalizer
thartmann
parents: 26914
diff changeset
  1474
      } else {
12369
48fd3da4025c 7154670: The instanceKlass _implementors[] and _nof_implementors are not needed for non-interface klass.
jiangli
parents: 11440
diff changeset
  1475
        ADD_SUBCLASS_CHAIN(impl);
48fd3da4025c 7154670: The instanceKlass _implementors[] and _nof_implementors are not needed for non-interface klass.
jiangli
parents: 11440
diff changeset
  1476
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1477
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1478
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1479
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1480
  // Recursively process each non-trivial sibling chain.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1481
  while (chaini > 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1482
    Klass* chain = chains[--chaini];
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1483
    for (Klass* sub = chain; sub != NULL; sub = sub->next_sibling()) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1484
      if (do_counts) { NOT_PRODUCT(deps_find_witness_steps++); }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1485
      if (is_participant(sub)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1486
        if (participants_hide_witnesses)  continue;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1487
        // else fall through to process this guy's subclasses
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1488
      } else if (is_witness(sub) && !ignore_witness(sub)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1489
        return sub;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1490
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1491
      if (chaini < (VerifyDependencies? 2: CHAINMAX)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1492
        // Fast path.  (Partially disabled if VerifyDependencies.)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1493
        ADD_SUBCLASS_CHAIN(sub);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1494
      } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1495
        // Worklist overflow.  Do a recursive call.  Should be rare.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1496
        // The recursive call will have its own worklist, of course.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1497
        // (Note that sub has already been tested, so that there is
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1498
        // no need for the recursive call to re-test.  That's handy,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1499
        // since the recursive call sees sub as the context_type.)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1500
        if (do_counts) { NOT_PRODUCT(deps_find_witness_recursions++); }
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1501
        Klass* witness = find_witness_anywhere(sub,
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1502
                                                 participants_hide_witnesses,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1503
                                                 /*top_level_call=*/ false);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1504
        if (witness != NULL)  return witness;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1505
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1506
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1507
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1508
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1509
  // No witness found.  The dependency remains unbroken.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1510
  return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1511
#undef ADD_SUBCLASS_CHAIN
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1512
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1513
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1514
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1515
bool Dependencies::is_concrete_klass(Klass* k) {
14488
ab48109f7d1b 8001471: Klass::cast() does nothing
hseigel
parents: 14391
diff changeset
  1516
  if (k->is_abstract())  return false;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1517
  // %%% We could treat classes which are concrete but
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1518
  // have not yet been instantiated as virtually abstract.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1519
  // This would require a deoptimization barrier on first instantiation.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1520
  //if (k->is_not_instantiated())  return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1521
  return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1522
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1523
28514
da53c1ffc837 8064524: Compiler code generation improvements
drchase
parents: 27700
diff changeset
  1524
bool Dependencies::is_concrete_method(Method* m, Klass * k) {
da53c1ffc837 8064524: Compiler code generation improvements
drchase
parents: 27700
diff changeset
  1525
  // NULL is not a concrete method,
da53c1ffc837 8064524: Compiler code generation improvements
drchase
parents: 27700
diff changeset
  1526
  // statics are irrelevant to virtual call sites,
da53c1ffc837 8064524: Compiler code generation improvements
drchase
parents: 27700
diff changeset
  1527
  // abstract methods are not concrete,
da53c1ffc837 8064524: Compiler code generation improvements
drchase
parents: 27700
diff changeset
  1528
  // overpass (error) methods are not concrete if k is abstract
da53c1ffc837 8064524: Compiler code generation improvements
drchase
parents: 27700
diff changeset
  1529
  //
da53c1ffc837 8064524: Compiler code generation improvements
drchase
parents: 27700
diff changeset
  1530
  // note "true" is conservative answer --
da53c1ffc837 8064524: Compiler code generation improvements
drchase
parents: 27700
diff changeset
  1531
  //     overpass clause is false if k == NULL, implies return true if
da53c1ffc837 8064524: Compiler code generation improvements
drchase
parents: 27700
diff changeset
  1532
  //     answer depends on overpass clause.
da53c1ffc837 8064524: Compiler code generation improvements
drchase
parents: 27700
diff changeset
  1533
  return ! ( m == NULL || m -> is_static() || m -> is_abstract() ||
46630
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 46271
diff changeset
  1534
             (m->is_overpass() && k != NULL && k -> is_abstract()) );
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1535
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1536
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1537
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1538
Klass* Dependencies::find_finalizable_subclass(Klass* k) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1539
  if (k->is_interface())  return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1540
  if (k->has_finalizer()) return k;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1541
  k = k->subklass();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1542
  while (k != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1543
    Klass* result = find_finalizable_subclass(k);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1544
    if (result != NULL) return result;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1545
    k = k->next_sibling();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1546
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1547
  return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1548
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1549
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1550
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1551
bool Dependencies::is_concrete_klass(ciInstanceKlass* k) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1552
  if (k->is_abstract())  return false;
10984
bca73db4935e 7105305: assert check_method_context proper context
never
parents: 10503
diff changeset
  1553
  // We could also return false if k does not yet appear to be
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1554
  // instantiated, if the VM version supports this distinction also.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1555
  //if (k->is_not_instantiated())  return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1556
  return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1557
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1558
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1559
bool Dependencies::has_finalizable_subclass(ciInstanceKlass* k) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1560
  return k->has_finalizable_subclass();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1561
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1562
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1563
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1564
// Any use of the contents (bytecodes) of a method must be
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1565
// marked by an "evol_method" dependency, if those contents
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1566
// can change.  (Note: A method is always dependent on itself.)
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1567
Klass* Dependencies::check_evol_method(Method* m) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1568
  assert(must_be_in_vm(), "raw oops here");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1569
  // Did somebody do a JVMTI RedefineClasses while our backs were turned?
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1570
  // Or is there a now a breakpoint?
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1571
  // (Assumes compiled code cannot handle bkpts; change if UseFastBreakpoints.)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1572
  if (m->is_old()
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1573
      || m->number_of_breakpoints() > 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1574
    return m->method_holder();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1575
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1576
    return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1577
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1578
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1579
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1580
// This is a strong assertion:  It is that the given type
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1581
// has no subtypes whatever.  It is most useful for
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1582
// optimizing checks on reflected types or on array types.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1583
// (Checks on types which are derived from real instances
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1584
// can be optimized more strongly than this, because we
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1585
// know that the checked type comes from a concrete type,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1586
// and therefore we can disregard abstract types.)
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1587
Klass* Dependencies::check_leaf_type(Klass* ctxk) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1588
  assert(must_be_in_vm(), "raw oops here");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1589
  assert_locked_or_safepoint(Compile_lock);
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1590
  InstanceKlass* ctx = InstanceKlass::cast(ctxk);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1591
  Klass* sub = ctx->subklass();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1592
  if (sub != NULL) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1593
    return sub;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1594
  } else if (ctx->nof_implementors() != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1595
    // if it is an interface, it must be unimplemented
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1596
    // (if it is not an interface, nof_implementors is always zero)
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1597
    Klass* impl = ctx->implementor();
12369
48fd3da4025c 7154670: The instanceKlass _implementors[] and _nof_implementors are not needed for non-interface klass.
jiangli
parents: 11440
diff changeset
  1598
    assert(impl != NULL, "must be set");
48fd3da4025c 7154670: The instanceKlass _implementors[] and _nof_implementors are not needed for non-interface klass.
jiangli
parents: 11440
diff changeset
  1599
    return impl;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1600
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1601
    return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1602
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1603
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1604
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1605
// Test the assertion that conck is the only concrete subtype* of ctxk.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1606
// The type conck itself is allowed to have have further concrete subtypes.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1607
// This allows the compiler to narrow occurrences of ctxk by conck,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1608
// when dealing with the types of actual instances.
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1609
Klass* Dependencies::check_abstract_with_unique_concrete_subtype(Klass* ctxk,
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1610
                                                                   Klass* conck,
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1611
                                                                   KlassDepChange* changes) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1612
  ClassHierarchyWalker wf(conck);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1613
  return wf.find_witness_subtype(ctxk, changes);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1614
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1615
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1616
// If a non-concrete class has no concrete subtypes, it is not (yet)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1617
// instantiatable.  This can allow the compiler to make some paths go
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1618
// dead, if they are gated by a test of the type.
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1619
Klass* Dependencies::check_abstract_with_no_concrete_subtype(Klass* ctxk,
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1620
                                                               KlassDepChange* changes) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1621
  // Find any concrete subtype, with no participants:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1622
  ClassHierarchyWalker wf;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1623
  return wf.find_witness_subtype(ctxk, changes);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1624
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1625
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1626
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1627
// If a concrete class has no concrete subtypes, it can always be
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1628
// exactly typed.  This allows the use of a cheaper type test.
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1629
Klass* Dependencies::check_concrete_with_no_concrete_subtype(Klass* ctxk,
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1630
                                                               KlassDepChange* changes) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1631
  // Find any concrete subtype, with only the ctxk as participant:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1632
  ClassHierarchyWalker wf(ctxk);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1633
  return wf.find_witness_subtype(ctxk, changes);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1634
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1635
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1636
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1637
// Find the unique concrete proper subtype of ctxk, or NULL if there
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1638
// is more than one concrete proper subtype.  If there are no concrete
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1639
// proper subtypes, return ctxk itself, whether it is concrete or not.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1640
// The returned subtype is allowed to have have further concrete subtypes.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1641
// That is, return CC1 for CX > CC1 > CC2, but NULL for CX > { CC1, CC2 }.
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1642
Klass* Dependencies::find_unique_concrete_subtype(Klass* ctxk) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1643
  ClassHierarchyWalker wf(ctxk);   // Ignore ctxk when walking.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1644
  wf.record_witnesses(1);          // Record one other witness when walking.
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1645
  Klass* wit = wf.find_witness_subtype(ctxk);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1646
  if (wit != NULL)  return NULL;   // Too many witnesses.
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1647
  Klass* conck = wf.participant(0);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1648
  if (conck == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1649
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1650
    // Make sure the dependency mechanism will pass this discovery:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1651
    if (VerifyDependencies) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1652
      // Turn off dependency tracing while actually testing deps.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1653
      FlagSetting fs(TraceDependencies, false);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1654
      if (!Dependencies::is_concrete_klass(ctxk)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1655
        guarantee(NULL ==
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1656
                  (void *)check_abstract_with_no_concrete_subtype(ctxk),
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1657
                  "verify dep.");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1658
      } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1659
        guarantee(NULL ==
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1660
                  (void *)check_concrete_with_no_concrete_subtype(ctxk),
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1661
                  "verify dep.");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1662
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1663
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1664
#endif //PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1665
    return ctxk;                   // Return ctxk as a flag for "no subtypes".
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1666
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1667
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1668
    // Make sure the dependency mechanism will pass this discovery:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1669
    if (VerifyDependencies) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1670
      // Turn off dependency tracing while actually testing deps.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1671
      FlagSetting fs(TraceDependencies, false);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1672
      if (!Dependencies::is_concrete_klass(ctxk)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1673
        guarantee(NULL == (void *)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1674
                  check_abstract_with_unique_concrete_subtype(ctxk, conck),
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1675
                  "verify dep.");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1676
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1677
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1678
#endif //PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1679
    return conck;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1680
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1681
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1682
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1683
// Test the assertion that the k[12] are the only concrete subtypes of ctxk,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1684
// except possibly for further subtypes of k[12] themselves.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1685
// The context type must be abstract.  The types k1 and k2 are themselves
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1686
// allowed to have further concrete subtypes.
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1687
Klass* Dependencies::check_abstract_with_exclusive_concrete_subtypes(
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1688
                                                Klass* ctxk,
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1689
                                                Klass* k1,
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1690
                                                Klass* k2,
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1691
                                                KlassDepChange* changes) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1692
  ClassHierarchyWalker wf;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1693
  wf.add_participant(k1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1694
  wf.add_participant(k2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1695
  return wf.find_witness_subtype(ctxk, changes);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1696
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1697
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1698
// Search ctxk for concrete implementations.  If there are klen or fewer,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1699
// pack them into the given array and return the number.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1700
// Otherwise, return -1, meaning the given array would overflow.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1701
// (Note that a return of 0 means there are exactly no concrete subtypes.)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1702
// In this search, if ctxk is concrete, it will be reported alone.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1703
// For any type CC reported, no proper subtypes of CC will be reported.
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1704
int Dependencies::find_exclusive_concrete_subtypes(Klass* ctxk,
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1705
                                                   int klen,
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1706
                                                   Klass* karray[]) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1707
  ClassHierarchyWalker wf;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1708
  wf.record_witnesses(klen);
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1709
  Klass* wit = wf.find_witness_subtype(ctxk);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1710
  if (wit != NULL)  return -1;  // Too many witnesses.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1711
  int num = wf.num_participants();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1712
  assert(num <= klen, "oob");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1713
  // Pack the result array with the good news.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1714
  for (int i = 0; i < num; i++)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1715
    karray[i] = wf.participant(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1716
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1717
  // Make sure the dependency mechanism will pass this discovery:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1718
  if (VerifyDependencies) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1719
    // Turn off dependency tracing while actually testing deps.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1720
    FlagSetting fs(TraceDependencies, false);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1721
    switch (Dependencies::is_concrete_klass(ctxk)? -1: num) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1722
    case -1: // ctxk was itself concrete
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1723
      guarantee(num == 1 && karray[0] == ctxk, "verify dep.");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1724
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1725
    case 0:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1726
      guarantee(NULL == (void *)check_abstract_with_no_concrete_subtype(ctxk),
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1727
                "verify dep.");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1728
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1729
    case 1:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1730
      guarantee(NULL == (void *)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1731
                check_abstract_with_unique_concrete_subtype(ctxk, karray[0]),
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1732
                "verify dep.");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1733
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1734
    case 2:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1735
      guarantee(NULL == (void *)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1736
                check_abstract_with_exclusive_concrete_subtypes(ctxk,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1737
                                                                karray[0],
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1738
                                                                karray[1]),
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1739
                "verify dep.");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1740
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1741
    default:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1742
      ShouldNotReachHere();  // klen > 2 yet supported
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1743
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1744
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1745
#endif //PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1746
  return num;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1747
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1748
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1749
// If a class (or interface) has a unique concrete method uniqm, return NULL.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1750
// Otherwise, return a class that contains an interfering method.
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1751
Klass* Dependencies::check_unique_concrete_method(Klass* ctxk, Method* uniqm,
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1752
                                                    KlassDepChange* changes) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1753
  // Here is a missing optimization:  If uniqm->is_final(),
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1754
  // we don't really need to search beneath it for overrides.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1755
  // This is probably not important, since we don't use dependencies
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1756
  // to track final methods.  (They can't be "definalized".)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1757
  ClassHierarchyWalker wf(uniqm->method_holder(), uniqm);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1758
  return wf.find_witness_definer(ctxk, changes);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1759
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1760
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1761
// Find the set of all non-abstract methods under ctxk that match m.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1762
// (The method m must be defined or inherited in ctxk.)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1763
// Include m itself in the set, unless it is abstract.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1764
// If this set has exactly one element, return that element.
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1765
Method* Dependencies::find_unique_concrete_method(Klass* ctxk, Method* m) {
24484
4a37a32e36a9 6563994: assert(wf.check_method_context(ctxk, m), "proper context") failed
bharadwaj
parents: 24351
diff changeset
  1766
  // Return NULL if m is marked old; must have been a redefined method.
4a37a32e36a9 6563994: assert(wf.check_method_context(ctxk, m), "proper context") failed
bharadwaj
parents: 24351
diff changeset
  1767
  if (m->is_old()) {
4a37a32e36a9 6563994: assert(wf.check_method_context(ctxk, m), "proper context") failed
bharadwaj
parents: 24351
diff changeset
  1768
    return NULL;
4a37a32e36a9 6563994: assert(wf.check_method_context(ctxk, m), "proper context") failed
bharadwaj
parents: 24351
diff changeset
  1769
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1770
  ClassHierarchyWalker wf(m);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1771
  assert(wf.check_method_context(ctxk, m), "proper context");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1772
  wf.record_witnesses(1);
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1773
  Klass* wit = wf.find_witness_definer(ctxk);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1774
  if (wit != NULL)  return NULL;  // Too many witnesses.
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1775
  Method* fm = wf.found_method(0);  // Will be NULL if num_parts == 0.
28514
da53c1ffc837 8064524: Compiler code generation improvements
drchase
parents: 27700
diff changeset
  1776
  if (Dependencies::is_concrete_method(m, ctxk)) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1777
    if (fm == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1778
      // It turns out that m was always the only implementation.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1779
      fm = m;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1780
    } else if (fm != m) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1781
      // Two conflicting implementations after all.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1782
      // (This can happen if m is inherited into ctxk and fm overrides it.)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1783
      return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1784
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1785
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1786
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1787
  // Make sure the dependency mechanism will pass this discovery:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1788
  if (VerifyDependencies && fm != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1789
    guarantee(NULL == (void *)check_unique_concrete_method(ctxk, fm),
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1790
              "verify dep.");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1791
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1792
#endif //PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1793
  return fm;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1794
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1795
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1796
Klass* Dependencies::check_exclusive_concrete_methods(Klass* ctxk,
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1797
                                                        Method* m1,
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1798
                                                        Method* m2,
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1799
                                                        KlassDepChange* changes) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1800
  ClassHierarchyWalker wf(m1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1801
  wf.add_participant(m1->method_holder());
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1802
  wf.add_participant(m2->method_holder());
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1803
  return wf.find_witness_definer(ctxk, changes);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1804
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1805
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1806
Klass* Dependencies::check_has_no_finalizable_subclasses(Klass* ctxk, KlassDepChange* changes) {
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1807
  Klass* search_at = ctxk;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1808
  if (changes != NULL)
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1809
    search_at = changes->new_type(); // just look at the new bit
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1810
  return find_finalizable_subclass(search_at);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1811
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1812
31037
01a5c5fa5681 8079205: CallSite dependency tracking is broken after sun.misc.Cleaner became automatically cleared
vlivanov
parents: 30296
diff changeset
  1813
Klass* Dependencies::check_call_site_target_value(oop call_site, oop method_handle, CallSiteDepChange* changes) {
01a5c5fa5681 8079205: CallSite dependency tracking is broken after sun.misc.Cleaner became automatically cleared
vlivanov
parents: 30296
diff changeset
  1814
  assert(!oopDesc::is_null(call_site), "sanity");
01a5c5fa5681 8079205: CallSite dependency tracking is broken after sun.misc.Cleaner became automatically cleared
vlivanov
parents: 30296
diff changeset
  1815
  assert(!oopDesc::is_null(method_handle), "sanity");
30296
95baefac8485 8057967: CallSite dependency tracking scales devastatingly poorly
vlivanov
parents: 30195
diff changeset
  1816
  assert(call_site->is_a(SystemDictionary::CallSite_klass()),     "sanity");
95baefac8485 8057967: CallSite dependency tracking scales devastatingly poorly
vlivanov
parents: 30195
diff changeset
  1817
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1818
  if (changes == NULL) {
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1819
    // Validate all CallSites
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1820
    if (java_lang_invoke_CallSite::target(call_site) != method_handle)
10503
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
  1821
      return call_site->klass();  // assertion failed
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1822
  } else {
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1823
    // Validate the given CallSite
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1824
    if (call_site == changes->call_site() && java_lang_invoke_CallSite::target(call_site) != changes->method_handle()) {
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1825
      assert(method_handle != changes->method_handle(), "must be");
10503
04b74421bdea 7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents: 10265
diff changeset
  1826
      return call_site->klass();  // assertion failed
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1827
    }
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1828
  }
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1829
  return NULL;  // assertion still valid
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1830
}
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1831
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1832
void Dependencies::DepStream::trace_and_log_witness(Klass* witness) {
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1833
  if (witness != NULL) {
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1834
    if (TraceDependencies) {
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1835
      print_dependency(witness, /*verbose=*/ true);
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1836
    }
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1837
    // The following is a no-op unless logging is enabled:
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1838
    log_dependency(witness);
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1839
  }
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1840
}
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1841
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1842
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1843
Klass* Dependencies::DepStream::check_klass_dependency(KlassDepChange* changes) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1844
  assert_locked_or_safepoint(Compile_lock);
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1845
  Dependencies::check_valid_dependency_type(type());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1846
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1847
  Klass* witness = NULL;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1848
  switch (type()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1849
  case evol_method:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1850
    witness = check_evol_method(method_argument(0));
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1851
    break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1852
  case leaf_type:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1853
    witness = check_leaf_type(context_type());
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1854
    break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1855
  case abstract_with_unique_concrete_subtype:
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1856
    witness = check_abstract_with_unique_concrete_subtype(context_type(), type_argument(1), changes);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1857
    break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1858
  case abstract_with_no_concrete_subtype:
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1859
    witness = check_abstract_with_no_concrete_subtype(context_type(), changes);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1860
    break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1861
  case concrete_with_no_concrete_subtype:
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1862
    witness = check_concrete_with_no_concrete_subtype(context_type(), changes);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1863
    break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1864
  case unique_concrete_method:
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1865
    witness = check_unique_concrete_method(context_type(), method_argument(1), changes);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1866
    break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1867
  case abstract_with_exclusive_concrete_subtypes_2:
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1868
    witness = check_abstract_with_exclusive_concrete_subtypes(context_type(), type_argument(1), type_argument(2), changes);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1869
    break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1870
  case exclusive_concrete_methods_2:
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1871
    witness = check_exclusive_concrete_methods(context_type(), method_argument(1), method_argument(2), changes);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1872
    break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1873
  case no_finalizable_subclasses:
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1874
    witness = check_has_no_finalizable_subclasses(context_type(), changes);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1875
    break;
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1876
  default:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1877
    witness = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1878
    break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1879
  }
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1880
  trace_and_log_witness(witness);
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1881
  return witness;
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1882
}
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1883
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1884
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1885
Klass* Dependencies::DepStream::check_call_site_dependency(CallSiteDepChange* changes) {
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1886
  assert_locked_or_safepoint(Compile_lock);
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1887
  Dependencies::check_valid_dependency_type(type());
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1888
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1889
  Klass* witness = NULL;
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1890
  switch (type()) {
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1891
  case call_site_target_value:
31037
01a5c5fa5681 8079205: CallSite dependency tracking is broken after sun.misc.Cleaner became automatically cleared
vlivanov
parents: 30296
diff changeset
  1892
    witness = check_call_site_target_value(argument_oop(0), argument_oop(1), changes);
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1893
    break;
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1894
  default:
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1895
    witness = NULL;
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1896
    break;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1897
  }
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1898
  trace_and_log_witness(witness);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1899
  return witness;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1900
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1901
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1902
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1903
Klass* Dependencies::DepStream::spot_check_dependency_at(DepChange& changes) {
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1904
  // Handle klass dependency
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1905
  if (changes.is_klass_change() && changes.as_klass_change()->involves_context(context_type()))
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1906
    return check_klass_dependency(changes.as_klass_change());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1907
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1908
  // Handle CallSite dependency
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1909
  if (changes.is_call_site_change())
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1910
    return check_call_site_dependency(changes.as_call_site_change());
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1911
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1912
  // irrelevant dependency; skip it
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1913
  return NULL;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1914
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1915
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1916
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1917
void DepChange::print() {
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1918
  int nsup = 0, nint = 0;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1919
  for (ContextStream str(*this); str.next(); ) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1920
    Klass* k = str.klass();
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1921
    switch (str.change_type()) {
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1922
    case Change_new_type:
33602
16053580a684 8139163: InstanceKlass::cast passes through NULL
coleenp
parents: 33198
diff changeset
  1923
      tty->print_cr("  dependee = %s", k->external_name());
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1924
      break;
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1925
    case Change_new_sub:
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1926
      if (!WizardMode) {
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1927
        ++nsup;
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1928
      } else {
33602
16053580a684 8139163: InstanceKlass::cast passes through NULL
coleenp
parents: 33198
diff changeset
  1929
        tty->print_cr("  context super = %s", k->external_name());
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1930
      }
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1931
      break;
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1932
    case Change_new_impl:
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1933
      if (!WizardMode) {
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1934
        ++nint;
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1935
      } else {
33602
16053580a684 8139163: InstanceKlass::cast passes through NULL
coleenp
parents: 33198
diff changeset
  1936
        tty->print_cr("  context interface = %s", k->external_name());
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1937
      }
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1938
      break;
46630
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 46271
diff changeset
  1939
    default:
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 46271
diff changeset
  1940
      break;
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1941
    }
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1942
  }
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1943
  if (nsup + nint != 0) {
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1944
    tty->print_cr("  context supers = %d, interfaces = %d", nsup, nint);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1945
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1946
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1947
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1948
void DepChange::ContextStream::start() {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1949
  Klass* new_type = _changes.is_klass_change() ? _changes.as_klass_change()->new_type() : (Klass*) NULL;
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1950
  _change_type = (new_type == NULL ? NO_CHANGE : Start_Klass);
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1951
  _klass = new_type;
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1952
  _ti_base = NULL;
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1953
  _ti_index = 0;
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1954
  _ti_limit = 0;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1955
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1956
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1957
bool DepChange::ContextStream::next() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1958
  switch (_change_type) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1959
  case Start_Klass:             // initial state; _klass is the new type
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1960
    _ti_base = InstanceKlass::cast(_klass)->transitive_interfaces();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1961
    _ti_index = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1962
    _change_type = Change_new_type;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1963
    return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1964
  case Change_new_type:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1965
    // fall through:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1966
    _change_type = Change_new_sub;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1967
  case Change_new_sub:
227
1d53148d4a4c 6598190: JPRT tests fail when run with -XX:+CheckUnhandledOops
sbohne
parents: 1
diff changeset
  1968
    // 6598190: brackets workaround Sun Studio C++ compiler bug 6629277
1d53148d4a4c 6598190: JPRT tests fail when run with -XX:+CheckUnhandledOops
sbohne
parents: 1
diff changeset
  1969
    {
33602
16053580a684 8139163: InstanceKlass::cast passes through NULL
coleenp
parents: 33198
diff changeset
  1970
      _klass = _klass->super();
227
1d53148d4a4c 6598190: JPRT tests fail when run with -XX:+CheckUnhandledOops
sbohne
parents: 1
diff changeset
  1971
      if (_klass != NULL) {
1d53148d4a4c 6598190: JPRT tests fail when run with -XX:+CheckUnhandledOops
sbohne
parents: 1
diff changeset
  1972
        return true;
1d53148d4a4c 6598190: JPRT tests fail when run with -XX:+CheckUnhandledOops
sbohne
parents: 1
diff changeset
  1973
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1974
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1975
    // else set up _ti_limit and fall through:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1976
    _ti_limit = (_ti_base == NULL) ? 0 : _ti_base->length();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1977
    _change_type = Change_new_impl;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1978
  case Change_new_impl:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1979
    if (_ti_index < _ti_limit) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  1980
      _klass = _ti_base->at(_ti_index++);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1981
      return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1982
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1983
    // fall through:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1984
    _change_type = NO_CHANGE;  // iterator is exhausted
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1985
  case NO_CHANGE:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1986
    break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1987
  default:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1988
    ShouldNotReachHere();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1989
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1990
  return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1991
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1992
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1993
void KlassDepChange::initialize() {
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1994
  // entire transaction must be under this lock:
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1995
  assert_lock_strong(Compile_lock);
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1996
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1997
  // Mark all dependee and all its superclasses
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  1998
  // Mark transitive interfaces
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1999
  for (ContextStream str(*this); str.next(); ) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  2000
    Klass* d = str.klass();
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  2001
    assert(!InstanceKlass::cast(d)->is_marked_dependent(), "checking");
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  2002
    InstanceKlass::cast(d)->set_is_marked_dependent(true);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2003
  }
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  2004
}
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  2005
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  2006
KlassDepChange::~KlassDepChange() {
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  2007
  // Unmark all dependee and all its superclasses
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  2008
  // Unmark transitive interfaces
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  2009
  for (ContextStream str(*this); str.next(); ) {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  2010
    Klass* d = str.klass();
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  2011
    InstanceKlass::cast(d)->set_is_marked_dependent(false);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2012
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2013
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2014
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  2015
bool KlassDepChange::involves_context(Klass* k) {
33611
9abd65805e19 8139203: Consistent naming for klass type predicates
coleenp
parents: 33602
diff changeset
  2016
  if (k == NULL || !k->is_instance_klass()) {
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  2017
    return false;
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  2018
  }
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 12369
diff changeset
  2019
  InstanceKlass* ik = InstanceKlass::cast(k);
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  2020
  bool is_contained = ik->is_marked_dependent();
14488
ab48109f7d1b 8001471: Klass::cast() does nothing
hseigel
parents: 14391
diff changeset
  2021
  assert(is_contained == new_type()->is_subtype_of(k),
10265
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  2022
         "correct marking of potential context types");
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  2023
  return is_contained;
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  2024
}
4c869854aebd 7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents: 8921
diff changeset
  2025
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2026
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2027
void Dependencies::print_statistics() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2028
  if (deps_find_witness_print != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2029
    // Call one final time, to flush out the data.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2030
    deps_find_witness_print = -1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2031
    count_find_witness_calls();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2032
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2033
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2034
#endif
35498
392b50de06c6 8146401: Clean up oop.hpp: add inline directives and fix header files
goetz
parents: 33611
diff changeset
  2035
392b50de06c6 8146401: Clean up oop.hpp: add inline directives and fix header files
goetz
parents: 33611
diff changeset
  2036
CallSiteDepChange::CallSiteDepChange(Handle call_site, Handle method_handle) :
392b50de06c6 8146401: Clean up oop.hpp: add inline directives and fix header files
goetz
parents: 33611
diff changeset
  2037
  _call_site(call_site),
392b50de06c6 8146401: Clean up oop.hpp: add inline directives and fix header files
goetz
parents: 33611
diff changeset
  2038
  _method_handle(method_handle) {
392b50de06c6 8146401: Clean up oop.hpp: add inline directives and fix header files
goetz
parents: 33611
diff changeset
  2039
  assert(_call_site()->is_a(SystemDictionary::CallSite_klass()), "must be");
392b50de06c6 8146401: Clean up oop.hpp: add inline directives and fix header files
goetz
parents: 33611
diff changeset
  2040
  assert(_method_handle.is_null() || _method_handle()->is_a(SystemDictionary::MethodHandle_klass()), "must be");
392b50de06c6 8146401: Clean up oop.hpp: add inline directives and fix header files
goetz
parents: 33611
diff changeset
  2041
}