src/hotspot/share/classfile/defaultMethods.cpp
author lfoltan
Mon, 20 Aug 2018 08:25:57 -0400
changeset 51444 3e5d28e6de32
parent 51334 cc2c79d22508
child 53432 1ec56532ae0c
permissions -rw-r--r--
8209301: JVM rename is_anonymous, host_klass to unsafe specific terminology ahead of Unsafe.defineAnonymousClass deprecation Summary: Clean up VM anonymous class terminology. Reviewed-by: coleenp, dholmes, mchung
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
     1
/*
49677
a1a7456dd8b9 8200261: Regression with JVM anonymous class
hseigel
parents: 48463
diff changeset
     2
 * Copyright (c) 2012, 2018, Oracle and/or its affiliates. All rights reserved.
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
     4
 *
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
     7
 * published by the Free Software Foundation.
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
     8
 *
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    13
 * accompanied this code).
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    14
 *
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    18
 *
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    21
 * questions.
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    22
 *
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    23
 */
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    24
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    25
#include "precompiled.hpp"
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    26
#include "classfile/bytecodeAssembler.hpp"
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    27
#include "classfile/defaultMethods.hpp"
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    28
#include "classfile/symbolTable.hpp"
48463
474cec233fb2 8154587: Resolution fails for default method named 'clone'
hseigel
parents: 47554
diff changeset
    29
#include "classfile/systemDictionary.hpp"
33736
1b3950243443 8139564: Convert TraceDefaultMethods to Unified Logging
rprotacio
parents: 29871
diff changeset
    30
#include "logging/log.hpp"
46701
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 42059
diff changeset
    31
#include "logging/logStream.hpp"
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    32
#include "memory/allocation.hpp"
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    33
#include "memory/metadataFactory.hpp"
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    34
#include "memory/resourceArea.hpp"
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 33736
diff changeset
    35
#include "runtime/handles.inline.hpp"
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    36
#include "runtime/signature.hpp"
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    37
#include "runtime/thread.hpp"
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    38
#include "oops/instanceKlass.hpp"
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    39
#include "oops/klass.hpp"
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    40
#include "oops/method.hpp"
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    41
#include "utilities/accessFlags.hpp"
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    42
#include "utilities/exceptions.hpp"
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    43
#include "utilities/ostream.hpp"
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    44
#include "utilities/pair.hpp"
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    45
#include "utilities/resourceHash.hpp"
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    46
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    47
typedef enum { QUALIFIED, DISQUALIFIED } QualifiedState;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    48
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    49
// Because we use an iterative algorithm when iterating over the type
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    50
// hierarchy, we can't use traditional scoped objects which automatically do
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    51
// cleanup in the destructor when the scope is exited.  PseudoScope (and
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    52
// PseudoScopeMark) provides a similar functionality, but for when you want a
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    53
// scoped object in non-stack memory (such as in resource memory, as we do
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    54
// here).  You've just got to remember to call 'destroy()' on the scope when
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    55
// leaving it (and marks have to be explicitly added).
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    56
class PseudoScopeMark : public ResourceObj {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    57
 public:
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    58
  virtual void destroy() = 0;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    59
};
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    60
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    61
class PseudoScope : public ResourceObj {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    62
 private:
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    63
  GrowableArray<PseudoScopeMark*> _marks;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    64
 public:
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    65
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    66
  static PseudoScope* cast(void* data) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    67
    return static_cast<PseudoScope*>(data);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    68
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    69
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    70
  void add_mark(PseudoScopeMark* psm) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    71
   _marks.append(psm);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    72
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    73
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    74
  void destroy() {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    75
    for (int i = 0; i < _marks.length(); ++i) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    76
      _marks.at(i)->destroy();
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    77
    }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    78
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    79
};
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    80
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    81
static void print_slot(outputStream* str, Symbol* name, Symbol* signature) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    82
  str->print("%s%s", name->as_C_string(), signature->as_C_string());
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    83
}
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    84
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    85
static void print_method(outputStream* str, Method* mo, bool with_class=true) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    86
  if (with_class) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    87
    str->print("%s.", mo->klass_name()->as_C_string());
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    88
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    89
  print_slot(str, mo->name(), mo->signature());
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    90
}
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    91
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    92
/**
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    93
 * Perform a depth-first iteration over the class hierarchy, applying
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    94
 * algorithmic logic as it goes.
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    95
 *
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    96
 * This class is one half of the inheritance hierarchy analysis mechanism.
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    97
 * It is meant to be used in conjunction with another class, the algorithm,
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    98
 * which is indicated by the ALGO template parameter.  This class can be
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
    99
 * paired with any algorithm class that provides the required methods.
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   100
 *
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   101
 * This class contains all the mechanics for iterating over the class hierarchy
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   102
 * starting at a particular root, without recursing (thus limiting stack growth
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   103
 * from this point).  It visits each superclass (if present) and superinterface
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   104
 * in a depth-first manner, with callbacks to the ALGO class as each class is
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   105
 * encountered (visit()), The algorithm can cut-off further exploration of a
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   106
 * particular branch by returning 'false' from a visit() call.
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   107
 *
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   108
 * The ALGO class, must provide a visit() method, which each of which will be
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   109
 * called once for each node in the inheritance tree during the iteration.  In
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   110
 * addition, it can provide a memory block via new_node_data(InstanceKlass*),
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   111
 * which it can use for node-specific storage (and access via the
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   112
 * current_data() and data_at_depth(int) methods).
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   113
 *
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   114
 * Bare minimum needed to be an ALGO class:
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   115
 * class Algo : public HierarchyVisitor<Algo> {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   116
 *   void* new_node_data(InstanceKlass* cls) { return NULL; }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   117
 *   void free_node_data(void* data) { return; }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   118
 *   bool visit() { return true; }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   119
 * };
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   120
 */
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   121
template <class ALGO>
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   122
class HierarchyVisitor : StackObj {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   123
 private:
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   124
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   125
  class Node : public ResourceObj {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   126
   public:
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   127
    InstanceKlass* _class;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   128
    bool _super_was_visited;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   129
    int _interface_index;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   130
    void* _algorithm_data;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   131
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   132
    Node(InstanceKlass* cls, void* data, bool visit_super)
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   133
        : _class(cls), _super_was_visited(!visit_super),
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   134
          _interface_index(0), _algorithm_data(data) {}
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   135
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   136
    int number_of_interfaces() { return _class->local_interfaces()->length(); }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   137
    int interface_index() { return _interface_index; }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   138
    void set_super_visited() { _super_was_visited = true; }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   139
    void increment_visited_interface() { ++_interface_index; }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   140
    void set_all_interfaces_visited() {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   141
      _interface_index = number_of_interfaces();
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   142
    }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   143
    bool has_visited_super() { return _super_was_visited; }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   144
    bool has_visited_all_interfaces() {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   145
      return interface_index() >= number_of_interfaces();
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   146
    }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   147
    InstanceKlass* interface_at(int index) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   148
      return InstanceKlass::cast(_class->local_interfaces()->at(index));
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   149
    }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   150
    InstanceKlass* next_super() { return _class->java_super(); }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   151
    InstanceKlass* next_interface() {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   152
      return interface_at(interface_index());
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   153
    }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   154
  };
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   155
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   156
  bool _cancelled;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   157
  GrowableArray<Node*> _path;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   158
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   159
  Node* current_top() const { return _path.top(); }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   160
  bool has_more_nodes() const { return !_path.is_empty(); }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   161
  void push(InstanceKlass* cls, void* data) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   162
    assert(cls != NULL, "Requires a valid instance class");
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   163
    Node* node = new Node(cls, data, has_super(cls));
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   164
    _path.push(node);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   165
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   166
  void pop() { _path.pop(); }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   167
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   168
  void reset_iteration() {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   169
    _cancelled = false;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   170
    _path.clear();
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   171
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   172
  bool is_cancelled() const { return _cancelled; }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   173
21556
e75cd34a59e0 8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents: 21516
diff changeset
   174
  // This code used to skip interface classes because their only
e75cd34a59e0 8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents: 21516
diff changeset
   175
  // superclass was j.l.Object which would be also covered by class
e75cd34a59e0 8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents: 21516
diff changeset
   176
  // superclass hierarchy walks. Now that the starting point can be
e75cd34a59e0 8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents: 21516
diff changeset
   177
  // an interface, we must ensure we catch j.l.Object as the super.
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   178
  static bool has_super(InstanceKlass* cls) {
21556
e75cd34a59e0 8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents: 21516
diff changeset
   179
    return cls->super() != NULL;
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   180
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   181
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   182
  Node* node_at_depth(int i) const {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   183
    return (i >= _path.length()) ? NULL : _path.at(_path.length() - i - 1);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   184
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   185
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   186
 protected:
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   187
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   188
  // Accessors available to the algorithm
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   189
  int current_depth() const { return _path.length() - 1; }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   190
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   191
  InstanceKlass* class_at_depth(int i) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   192
    Node* n = node_at_depth(i);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   193
    return n == NULL ? NULL : n->_class;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   194
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   195
  InstanceKlass* current_class() { return class_at_depth(0); }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   196
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   197
  void* data_at_depth(int i) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   198
    Node* n = node_at_depth(i);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   199
    return n == NULL ? NULL : n->_algorithm_data;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   200
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   201
  void* current_data() { return data_at_depth(0); }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   202
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   203
  void cancel_iteration() { _cancelled = true; }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   204
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   205
 public:
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   206
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   207
  void run(InstanceKlass* root) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   208
    ALGO* algo = static_cast<ALGO*>(this);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   209
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   210
    reset_iteration();
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   211
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   212
    void* algo_data = algo->new_node_data(root);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   213
    push(root, algo_data);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   214
    bool top_needs_visit = true;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   215
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   216
    do {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   217
      Node* top = current_top();
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   218
      if (top_needs_visit) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   219
        if (algo->visit() == false) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   220
          // algorithm does not want to continue along this path.  Arrange
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   221
          // it so that this state is immediately popped off the stack
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   222
          top->set_super_visited();
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   223
          top->set_all_interfaces_visited();
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   224
        }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   225
        top_needs_visit = false;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   226
      }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   227
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   228
      if (top->has_visited_super() && top->has_visited_all_interfaces()) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   229
        algo->free_node_data(top->_algorithm_data);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   230
        pop();
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   231
      } else {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   232
        InstanceKlass* next = NULL;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   233
        if (top->has_visited_super() == false) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   234
          next = top->next_super();
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   235
          top->set_super_visited();
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   236
        } else {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   237
          next = top->next_interface();
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   238
          top->increment_visited_interface();
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   239
        }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   240
        assert(next != NULL, "Otherwise we shouldn't be here");
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   241
        algo_data = algo->new_node_data(next);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   242
        push(next, algo_data);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   243
        top_needs_visit = true;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   244
      }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   245
    } while (!is_cancelled() && has_more_nodes());
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   246
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   247
};
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   248
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   249
class PrintHierarchy : public HierarchyVisitor<PrintHierarchy> {
33736
1b3950243443 8139564: Convert TraceDefaultMethods to Unified Logging
rprotacio
parents: 29871
diff changeset
   250
 private:
1b3950243443 8139564: Convert TraceDefaultMethods to Unified Logging
rprotacio
parents: 29871
diff changeset
   251
   outputStream* _st;
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   252
 public:
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   253
  bool visit() {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   254
    InstanceKlass* cls = current_class();
33736
1b3950243443 8139564: Convert TraceDefaultMethods to Unified Logging
rprotacio
parents: 29871
diff changeset
   255
    streamIndentor si(_st, current_depth() * 2);
1b3950243443 8139564: Convert TraceDefaultMethods to Unified Logging
rprotacio
parents: 29871
diff changeset
   256
    _st->indent().print_cr("%s", cls->name()->as_C_string());
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   257
    return true;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   258
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   259
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   260
  void* new_node_data(InstanceKlass* cls) { return NULL; }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   261
  void free_node_data(void* data) { return; }
33736
1b3950243443 8139564: Convert TraceDefaultMethods to Unified Logging
rprotacio
parents: 29871
diff changeset
   262
1b3950243443 8139564: Convert TraceDefaultMethods to Unified Logging
rprotacio
parents: 29871
diff changeset
   263
  PrintHierarchy(outputStream* st = tty) : _st(st) {}
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   264
};
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   265
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   266
// Used to register InstanceKlass objects and all related metadata structures
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   267
// (Methods, ConstantPools) as "in-use" by the current thread so that they can't
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   268
// be deallocated by class redefinition while we're using them.  The classes are
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   269
// de-registered when this goes out of scope.
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   270
//
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   271
// Once a class is registered, we need not bother with methodHandles or
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   272
// constantPoolHandles for it's associated metadata.
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   273
class KeepAliveRegistrar : public StackObj {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   274
 private:
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   275
  Thread* _thread;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   276
  GrowableArray<ConstantPool*> _keep_alive;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   277
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   278
 public:
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   279
  KeepAliveRegistrar(Thread* thread) : _thread(thread), _keep_alive(20) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   280
    assert(thread == Thread::current(), "Must be current thread");
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   281
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   282
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   283
  ~KeepAliveRegistrar() {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   284
    for (int i = _keep_alive.length() - 1; i >= 0; --i) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   285
      ConstantPool* cp = _keep_alive.at(i);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   286
      int idx = _thread->metadata_handles()->find_from_end(cp);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   287
      assert(idx > 0, "Must be in the list");
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   288
      _thread->metadata_handles()->remove_at(idx);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   289
    }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   290
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   291
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   292
  // Register a class as 'in-use' by the thread.  It's fine to register a class
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   293
  // multiple times (though perhaps inefficient)
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   294
  void register_class(InstanceKlass* ik) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   295
    ConstantPool* cp = ik->constants();
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   296
    _keep_alive.push(cp);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   297
    _thread->metadata_handles()->push(cp);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   298
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   299
};
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   300
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   301
class KeepAliveVisitor : public HierarchyVisitor<KeepAliveVisitor> {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   302
 private:
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   303
  KeepAliveRegistrar* _registrar;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   304
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   305
 public:
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   306
  KeepAliveVisitor(KeepAliveRegistrar* registrar) : _registrar(registrar) {}
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   307
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   308
  void* new_node_data(InstanceKlass* cls) { return NULL; }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   309
  void free_node_data(void* data) { return; }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   310
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   311
  bool visit() {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   312
    _registrar->register_class(current_class());
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   313
    return true;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   314
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   315
};
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   316
18695
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   317
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   318
// A method family contains a set of all methods that implement a single
18695
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   319
// erased method. As members of the set are collected while walking over the
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   320
// hierarchy, they are tagged with a qualification state.  The qualification
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   321
// state for an erased method is set to disqualified if there exists a path
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   322
// from the root of hierarchy to the method that contains an interleaving
18695
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   323
// erased method defined in an interface.
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   324
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   325
class MethodFamily : public ResourceObj {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   326
 private:
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   327
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   328
  GrowableArray<Pair<Method*,QualifiedState> > _members;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   329
  ResourceHashtable<Method*, int> _member_index;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   330
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   331
  Method* _selected_target;  // Filled in later, if a unique target exists
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   332
  Symbol* _exception_message; // If no unique target is found
20284
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 19966
diff changeset
   333
  Symbol* _exception_name;    // If no unique target is found
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   334
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   335
  bool contains_method(Method* method) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   336
    int* lookup = _member_index.get(method);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   337
    return lookup != NULL;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   338
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   339
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   340
  void add_method(Method* method, QualifiedState state) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   341
    Pair<Method*,QualifiedState> entry(method, state);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   342
    _member_index.put(method, _members.length());
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   343
    _members.append(entry);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   344
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   345
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   346
  void disqualify_method(Method* method) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   347
    int* index = _member_index.get(method);
16379
02ed75a9a421 8009578: [parfait] Null pointer deference in hotspot/src/share/vm/classfile/defaultMethods.cpp
morris
parents: 15601
diff changeset
   348
    guarantee(index != NULL && *index >= 0 && *index < _members.length(), "bad index");
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   349
    _members.at(*index).second = DISQUALIFIED;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   350
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   351
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   352
  Symbol* generate_no_defaults_message(TRAPS) const;
22232
26acfad336c0 8027804: JCK resolveMethod test fails expecting AbstractMethodError
hseigel
parents: 21913
diff changeset
   353
  Symbol* generate_method_message(Symbol *klass_name, Method* method, TRAPS) const;
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   354
  Symbol* generate_conflicts_message(GrowableArray<Method*>* methods, TRAPS) const;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   355
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   356
 public:
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   357
18695
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   358
  MethodFamily()
20284
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 19966
diff changeset
   359
      : _selected_target(NULL), _exception_message(NULL), _exception_name(NULL) {}
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   360
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   361
  void set_target_if_empty(Method* m) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   362
    if (_selected_target == NULL && !m->is_overpass()) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   363
      _selected_target = m;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   364
    }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   365
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   366
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   367
  void record_qualified_method(Method* m) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   368
    // If the method already exists in the set as qualified, this operation is
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   369
    // redundant.  If it already exists as disqualified, then we leave it as
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   370
    // disqualfied.  Thus we only add to the set if it's not already in the
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   371
    // set.
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   372
    if (!contains_method(m)) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   373
      add_method(m, QUALIFIED);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   374
    }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   375
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   376
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   377
  void record_disqualified_method(Method* m) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   378
    // If not in the set, add it as disqualified.  If it's already in the set,
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   379
    // then set the state to disqualified no matter what the previous state was.
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   380
    if (!contains_method(m)) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   381
      add_method(m, DISQUALIFIED);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   382
    } else {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   383
      disqualify_method(m);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   384
    }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   385
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   386
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   387
  bool has_target() const { return _selected_target != NULL; }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   388
  bool throws_exception() { return _exception_message != NULL; }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   389
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   390
  Method* get_selected_target() { return _selected_target; }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   391
  Symbol* get_exception_message() { return _exception_message; }
20284
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 19966
diff changeset
   392
  Symbol* get_exception_name() { return _exception_name; }
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   393
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   394
  // Either sets the target or the exception error message
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   395
  void determine_target(InstanceKlass* root, TRAPS) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   396
    if (has_target() || throws_exception()) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   397
      return;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   398
    }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   399
21556
e75cd34a59e0 8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents: 21516
diff changeset
   400
    // Qualified methods are maximally-specific methods
e75cd34a59e0 8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents: 21516
diff changeset
   401
    // These include public, instance concrete (=default) and abstract methods
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   402
    GrowableArray<Method*> qualified_methods;
21516
8fa5308ab970 8027304: Lambda: inheriting abstract + 1 default -> default, not ICCE
acorn
parents: 20710
diff changeset
   403
    int num_defaults = 0;
8fa5308ab970 8027304: Lambda: inheriting abstract + 1 default -> default, not ICCE
acorn
parents: 20710
diff changeset
   404
    int default_index = -1;
21556
e75cd34a59e0 8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents: 21516
diff changeset
   405
    int qualified_index = -1;
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   406
    for (int i = 0; i < _members.length(); ++i) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   407
      Pair<Method*,QualifiedState> entry = _members.at(i);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   408
      if (entry.second == QUALIFIED) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   409
        qualified_methods.append(entry.first);
21556
e75cd34a59e0 8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents: 21516
diff changeset
   410
        qualified_index++;
21516
8fa5308ab970 8027304: Lambda: inheriting abstract + 1 default -> default, not ICCE
acorn
parents: 20710
diff changeset
   411
        if (entry.first->is_default_method()) {
8fa5308ab970 8027304: Lambda: inheriting abstract + 1 default -> default, not ICCE
acorn
parents: 20710
diff changeset
   412
          num_defaults++;
21556
e75cd34a59e0 8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents: 21516
diff changeset
   413
          default_index = qualified_index;
e75cd34a59e0 8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents: 21516
diff changeset
   414
21516
8fa5308ab970 8027304: Lambda: inheriting abstract + 1 default -> default, not ICCE
acorn
parents: 20710
diff changeset
   415
        }
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   416
      }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   417
    }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   418
22232
26acfad336c0 8027804: JCK resolveMethod test fails expecting AbstractMethodError
hseigel
parents: 21913
diff changeset
   419
    if (num_defaults == 0) {
22493
af3de4cee5e5 8031059: invokestatic: ICCE trying to invoke static method when it clashes with an abstract method inherited from an interface
hseigel
parents: 22233
diff changeset
   420
      // If the root klass has a static method with matching name and signature
af3de4cee5e5 8031059: invokestatic: ICCE trying to invoke static method when it clashes with an abstract method inherited from an interface
hseigel
parents: 22233
diff changeset
   421
      // then do not generate an overpass method because it will hide the
af3de4cee5e5 8031059: invokestatic: ICCE trying to invoke static method when it clashes with an abstract method inherited from an interface
hseigel
parents: 22233
diff changeset
   422
      // static method during resolution.
23999
22eb7be3d99d 8033150: invokestatic: IncompatibleClassChangeError trying to invoke static method from a parent in presence of conflicting defaults.
lfoltan
parents: 22493
diff changeset
   423
      if (qualified_methods.length() == 0) {
22eb7be3d99d 8033150: invokestatic: IncompatibleClassChangeError trying to invoke static method from a parent in presence of conflicting defaults.
lfoltan
parents: 22493
diff changeset
   424
        _exception_message = generate_no_defaults_message(CHECK);
22eb7be3d99d 8033150: invokestatic: IncompatibleClassChangeError trying to invoke static method from a parent in presence of conflicting defaults.
lfoltan
parents: 22493
diff changeset
   425
      } else {
22eb7be3d99d 8033150: invokestatic: IncompatibleClassChangeError trying to invoke static method from a parent in presence of conflicting defaults.
lfoltan
parents: 22493
diff changeset
   426
        assert(root != NULL, "Null root class");
22eb7be3d99d 8033150: invokestatic: IncompatibleClassChangeError trying to invoke static method from a parent in presence of conflicting defaults.
lfoltan
parents: 22493
diff changeset
   427
        _exception_message = generate_method_message(root->name(), qualified_methods.at(0), CHECK);
22232
26acfad336c0 8027804: JCK resolveMethod test fails expecting AbstractMethodError
hseigel
parents: 21913
diff changeset
   428
      }
23999
22eb7be3d99d 8033150: invokestatic: IncompatibleClassChangeError trying to invoke static method from a parent in presence of conflicting defaults.
lfoltan
parents: 22493
diff changeset
   429
      _exception_name = vmSymbols::java_lang_AbstractMethodError();
22493
af3de4cee5e5 8031059: invokestatic: ICCE trying to invoke static method when it clashes with an abstract method inherited from an interface
hseigel
parents: 22233
diff changeset
   430
21556
e75cd34a59e0 8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents: 21516
diff changeset
   431
    // If only one qualified method is default, select that
21516
8fa5308ab970 8027304: Lambda: inheriting abstract + 1 default -> default, not ICCE
acorn
parents: 20710
diff changeset
   432
    } else if (num_defaults == 1) {
8fa5308ab970 8027304: Lambda: inheriting abstract + 1 default -> default, not ICCE
acorn
parents: 20710
diff changeset
   433
        _selected_target = qualified_methods.at(default_index);
22493
af3de4cee5e5 8031059: invokestatic: ICCE trying to invoke static method when it clashes with an abstract method inherited from an interface
hseigel
parents: 22233
diff changeset
   434
23999
22eb7be3d99d 8033150: invokestatic: IncompatibleClassChangeError trying to invoke static method from a parent in presence of conflicting defaults.
lfoltan
parents: 22493
diff changeset
   435
    } else if (num_defaults > 1) {
22493
af3de4cee5e5 8031059: invokestatic: ICCE trying to invoke static method when it clashes with an abstract method inherited from an interface
hseigel
parents: 22233
diff changeset
   436
      _exception_message = generate_conflicts_message(&qualified_methods,CHECK);
af3de4cee5e5 8031059: invokestatic: ICCE trying to invoke static method when it clashes with an abstract method inherited from an interface
hseigel
parents: 22233
diff changeset
   437
      _exception_name = vmSymbols::java_lang_IncompatibleClassChangeError();
46701
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 42059
diff changeset
   438
      LogTarget(Debug, defaultmethods) lt;
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 42059
diff changeset
   439
      if (lt.is_enabled()) {
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 42059
diff changeset
   440
        LogStream ls(lt);
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 42059
diff changeset
   441
        _exception_message->print_value_on(&ls);
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 42059
diff changeset
   442
        ls.cr();
20391
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   443
      }
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   444
    }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   445
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   446
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   447
  bool contains_signature(Symbol* query) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   448
    for (int i = 0; i < _members.length(); ++i) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   449
      if (query == _members.at(i).first->signature()) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   450
        return true;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   451
      }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   452
    }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   453
    return false;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   454
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   455
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   456
  void print_selected(outputStream* str, int indent) const {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   457
    assert(has_target(), "Should be called otherwise");
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   458
    streamIndentor si(str, indent * 2);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   459
    str->indent().print("Selected method: ");
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   460
    print_method(str, _selected_target);
19966
64732b96b5f5 8024647: Default method resolution with private superclass method
acorn
parents: 19690
diff changeset
   461
    Klass* method_holder = _selected_target->method_holder();
64732b96b5f5 8024647: Default method resolution with private superclass method
acorn
parents: 19690
diff changeset
   462
    if (!method_holder->is_interface()) {
33736
1b3950243443 8139564: Convert TraceDefaultMethods to Unified Logging
rprotacio
parents: 29871
diff changeset
   463
      str->print(" : in superclass");
19966
64732b96b5f5 8024647: Default method resolution with private superclass method
acorn
parents: 19690
diff changeset
   464
    }
24424
2658d7834c6e 8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents: 23999
diff changeset
   465
    str->cr();
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   466
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   467
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   468
  void print_exception(outputStream* str, int indent) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   469
    assert(throws_exception(), "Should be called otherwise");
20284
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 19966
diff changeset
   470
    assert(_exception_name != NULL, "exception_name should be set");
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   471
    streamIndentor si(str, indent * 2);
20284
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 19966
diff changeset
   472
    str->indent().print_cr("%s: %s", _exception_name->as_C_string(), _exception_message->as_C_string());
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   473
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   474
};
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   475
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   476
Symbol* MethodFamily::generate_no_defaults_message(TRAPS) const {
27680
8ecc0871c18e 8064811: Use THREAD instead of CHECK_NULL in return statements
stefank
parents: 27677
diff changeset
   477
  return SymbolTable::new_symbol("No qualifying defaults found", THREAD);
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   478
}
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   479
22232
26acfad336c0 8027804: JCK resolveMethod test fails expecting AbstractMethodError
hseigel
parents: 21913
diff changeset
   480
Symbol* MethodFamily::generate_method_message(Symbol *klass_name, Method* method, TRAPS) const {
26acfad336c0 8027804: JCK resolveMethod test fails expecting AbstractMethodError
hseigel
parents: 21913
diff changeset
   481
  stringStream ss;
26acfad336c0 8027804: JCK resolveMethod test fails expecting AbstractMethodError
hseigel
parents: 21913
diff changeset
   482
  ss.print("Method ");
26acfad336c0 8027804: JCK resolveMethod test fails expecting AbstractMethodError
hseigel
parents: 21913
diff changeset
   483
  Symbol* name = method->name();
26acfad336c0 8027804: JCK resolveMethod test fails expecting AbstractMethodError
hseigel
parents: 21913
diff changeset
   484
  Symbol* signature = method->signature();
26acfad336c0 8027804: JCK resolveMethod test fails expecting AbstractMethodError
hseigel
parents: 21913
diff changeset
   485
  ss.write((const char*)klass_name->bytes(), klass_name->utf8_length());
26acfad336c0 8027804: JCK resolveMethod test fails expecting AbstractMethodError
hseigel
parents: 21913
diff changeset
   486
  ss.print(".");
26acfad336c0 8027804: JCK resolveMethod test fails expecting AbstractMethodError
hseigel
parents: 21913
diff changeset
   487
  ss.write((const char*)name->bytes(), name->utf8_length());
26acfad336c0 8027804: JCK resolveMethod test fails expecting AbstractMethodError
hseigel
parents: 21913
diff changeset
   488
  ss.write((const char*)signature->bytes(), signature->utf8_length());
26acfad336c0 8027804: JCK resolveMethod test fails expecting AbstractMethodError
hseigel
parents: 21913
diff changeset
   489
  ss.print(" is abstract");
27677
b1997f2f1b56 8062808: Turn on the -Wreturn-type warning
stefank
parents: 24424
diff changeset
   490
  return SymbolTable::new_symbol(ss.base(), (int)ss.size(), THREAD);
22232
26acfad336c0 8027804: JCK resolveMethod test fails expecting AbstractMethodError
hseigel
parents: 21913
diff changeset
   491
}
26acfad336c0 8027804: JCK resolveMethod test fails expecting AbstractMethodError
hseigel
parents: 21913
diff changeset
   492
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   493
Symbol* MethodFamily::generate_conflicts_message(GrowableArray<Method*>* methods, TRAPS) const {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   494
  stringStream ss;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   495
  ss.print("Conflicting default methods:");
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   496
  for (int i = 0; i < methods->length(); ++i) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   497
    Method* method = methods->at(i);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   498
    Symbol* klass = method->klass_name();
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   499
    Symbol* name = method->name();
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   500
    ss.print(" ");
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   501
    ss.write((const char*)klass->bytes(), klass->utf8_length());
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   502
    ss.print(".");
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   503
    ss.write((const char*)name->bytes(), name->utf8_length());
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   504
  }
27677
b1997f2f1b56 8062808: Turn on the -Wreturn-type warning
stefank
parents: 24424
diff changeset
   505
  return SymbolTable::new_symbol(ss.base(), (int)ss.size(), THREAD);
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   506
}
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   507
18695
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   508
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   509
class StateRestorer;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   510
18695
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   511
// StatefulMethodFamily is a wrapper around a MethodFamily that maintains the
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   512
// qualification state during hierarchy visitation, and applies that state
18695
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   513
// when adding members to the MethodFamily
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   514
class StatefulMethodFamily : public ResourceObj {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   515
  friend class StateRestorer;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   516
 private:
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   517
  QualifiedState _qualification_state;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   518
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   519
  void set_qualification_state(QualifiedState state) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   520
    _qualification_state = state;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   521
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   522
18695
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   523
 protected:
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   524
  MethodFamily* _method_family;
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   525
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   526
 public:
18695
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   527
  StatefulMethodFamily() {
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   528
   _method_family = new MethodFamily();
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   529
   _qualification_state = QUALIFIED;
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   530
  }
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   531
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   532
  StatefulMethodFamily(MethodFamily* mf) {
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   533
   _method_family = mf;
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   534
   _qualification_state = QUALIFIED;
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   535
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   536
18695
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   537
  void set_target_if_empty(Method* m) { _method_family->set_target_if_empty(m); }
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   538
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   539
  MethodFamily* get_method_family() { return _method_family; }
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   540
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   541
  StateRestorer* record_method_and_dq_further(Method* mo);
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   542
};
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   543
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   544
class StateRestorer : public PseudoScopeMark {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   545
 private:
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   546
  StatefulMethodFamily* _method;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   547
  QualifiedState _state_to_restore;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   548
 public:
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   549
  StateRestorer(StatefulMethodFamily* dm, QualifiedState state)
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   550
      : _method(dm), _state_to_restore(state) {}
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   551
  ~StateRestorer() { destroy(); }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   552
  void restore_state() { _method->set_qualification_state(_state_to_restore); }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   553
  virtual void destroy() { restore_state(); }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   554
};
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   555
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   556
StateRestorer* StatefulMethodFamily::record_method_and_dq_further(Method* mo) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   557
  StateRestorer* mark = new StateRestorer(this, _qualification_state);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   558
  if (_qualification_state == QUALIFIED) {
18695
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   559
    _method_family->record_qualified_method(mo);
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   560
  } else {
18695
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   561
    _method_family->record_disqualified_method(mo);
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   562
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   563
  // Everything found "above"??? this method in the hierarchy walk is set to
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   564
  // disqualified
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   565
  set_qualification_state(DISQUALIFIED);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   566
  return mark;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   567
}
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   568
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   569
// Represents a location corresponding to a vtable slot for methods that
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   570
// neither the class nor any of it's ancestors provide an implementaion.
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   571
// Default methods may be present to fill this slot.
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   572
class EmptyVtableSlot : public ResourceObj {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   573
 private:
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   574
  Symbol* _name;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   575
  Symbol* _signature;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   576
  int _size_of_parameters;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   577
  MethodFamily* _binding;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   578
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   579
 public:
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   580
  EmptyVtableSlot(Method* method)
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   581
      : _name(method->name()), _signature(method->signature()),
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   582
        _size_of_parameters(method->size_of_parameters()), _binding(NULL) {}
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   583
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   584
  Symbol* name() const { return _name; }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   585
  Symbol* signature() const { return _signature; }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   586
  int size_of_parameters() const { return _size_of_parameters; }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   587
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   588
  void bind_family(MethodFamily* lm) { _binding = lm; }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   589
  bool is_bound() { return _binding != NULL; }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   590
  MethodFamily* get_binding() { return _binding; }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   591
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   592
  void print_on(outputStream* str) const {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   593
    print_slot(str, name(), signature());
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   594
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   595
};
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   596
20391
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   597
static bool already_in_vtable_slots(GrowableArray<EmptyVtableSlot*>* slots, Method* m) {
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   598
  bool found = false;
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   599
  for (int j = 0; j < slots->length(); ++j) {
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   600
    if (slots->at(j)->name() == m->name() &&
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   601
        slots->at(j)->signature() == m->signature() ) {
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   602
      found = true;
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   603
      break;
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   604
    }
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   605
  }
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   606
  return found;
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   607
}
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   608
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   609
static GrowableArray<EmptyVtableSlot*>* find_empty_vtable_slots(
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 33736
diff changeset
   610
    InstanceKlass* klass, const GrowableArray<Method*>* mirandas, TRAPS) {
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   611
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   612
  assert(klass != NULL, "Must be valid class");
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   613
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   614
  GrowableArray<EmptyVtableSlot*>* slots = new GrowableArray<EmptyVtableSlot*>();
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   615
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   616
  // All miranda methods are obvious candidates
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   617
  for (int i = 0; i < mirandas->length(); ++i) {
20391
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   618
    Method* m = mirandas->at(i);
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   619
    if (!already_in_vtable_slots(slots, m)) {
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   620
      slots->append(new EmptyVtableSlot(m));
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   621
    }
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   622
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   623
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   624
  // Also any overpasses in our superclasses, that we haven't implemented.
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   625
  // (can't use the vtable because it is not guaranteed to be initialized yet)
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   626
  InstanceKlass* super = klass->java_super();
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   627
  while (super != NULL) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   628
    for (int i = 0; i < super->methods()->length(); ++i) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   629
      Method* m = super->methods()->at(i);
21913
0e2fd7282ac6 8028438: static superclass method masks default methods
acorn
parents: 21556
diff changeset
   630
      if (m->is_overpass() || m->is_static()) {
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   631
        // m is a method that would have been a miranda if not for the
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   632
        // default method processing that occurred on behalf of our superclass,
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   633
        // so it's a method we want to re-examine in this new context.  That is,
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   634
        // unless we have a real implementation of it in the current class.
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   635
        Method* impl = klass->lookup_method(m->name(), m->signature());
21913
0e2fd7282ac6 8028438: static superclass method masks default methods
acorn
parents: 21556
diff changeset
   636
        if (impl == NULL || impl->is_overpass() || impl->is_static()) {
20391
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   637
          if (!already_in_vtable_slots(slots, m)) {
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   638
            slots->append(new EmptyVtableSlot(m));
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   639
          }
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   640
        }
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   641
      }
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   642
    }
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   643
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   644
    // also any default methods in our superclasses
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   645
    if (super->default_methods() != NULL) {
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   646
      for (int i = 0; i < super->default_methods()->length(); ++i) {
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   647
        Method* m = super->default_methods()->at(i);
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   648
        // m is a method that would have been a miranda if not for the
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   649
        // default method processing that occurred on behalf of our superclass,
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   650
        // so it's a method we want to re-examine in this new context.  That is,
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   651
        // unless we have a real implementation of it in the current class.
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   652
        Method* impl = klass->lookup_method(m->name(), m->signature());
21913
0e2fd7282ac6 8028438: static superclass method masks default methods
acorn
parents: 21556
diff changeset
   653
        if (impl == NULL || impl->is_overpass() || impl->is_static()) {
20391
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   654
          if (!already_in_vtable_slots(slots, m)) {
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   655
            slots->append(new EmptyVtableSlot(m));
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   656
          }
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   657
        }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   658
      }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   659
    }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   660
    super = super->java_super();
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   661
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   662
46701
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 42059
diff changeset
   663
  LogTarget(Debug, defaultmethods) lt;
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 42059
diff changeset
   664
  if (lt.is_enabled()) {
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 42059
diff changeset
   665
    lt.print("Slots that need filling:");
33736
1b3950243443 8139564: Convert TraceDefaultMethods to Unified Logging
rprotacio
parents: 29871
diff changeset
   666
    ResourceMark rm;
46701
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 42059
diff changeset
   667
    LogStream ls(lt);
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 42059
diff changeset
   668
    streamIndentor si(&ls);
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   669
    for (int i = 0; i < slots->length(); ++i) {
46701
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 42059
diff changeset
   670
      ls.indent();
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 42059
diff changeset
   671
      slots->at(i)->print_on(&ls);
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 42059
diff changeset
   672
      ls.cr();
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   673
    }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   674
  }
33736
1b3950243443 8139564: Convert TraceDefaultMethods to Unified Logging
rprotacio
parents: 29871
diff changeset
   675
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   676
  return slots;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   677
}
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   678
18695
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   679
// Iterates over the superinterface type hierarchy looking for all methods
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   680
// with a specific erased signature.
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   681
class FindMethodsByErasedSig : public HierarchyVisitor<FindMethodsByErasedSig> {
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   682
 private:
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   683
  // Context data
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   684
  Symbol* _method_name;
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   685
  Symbol* _method_signature;
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   686
  StatefulMethodFamily*  _family;
48463
474cec233fb2 8154587: Resolution fails for default method named 'clone'
hseigel
parents: 47554
diff changeset
   687
  bool _cur_class_is_interface;
18695
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   688
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   689
 public:
48463
474cec233fb2 8154587: Resolution fails for default method named 'clone'
hseigel
parents: 47554
diff changeset
   690
  FindMethodsByErasedSig(Symbol* name, Symbol* signature, bool is_interf) :
51334
cc2c79d22508 8208671: Runtime, JFR, Serviceability changes to allow enabling -Wreorder
tschatzl
parents: 49677
diff changeset
   691
      _method_name(name), _method_signature(signature), _family(NULL),
cc2c79d22508 8208671: Runtime, JFR, Serviceability changes to allow enabling -Wreorder
tschatzl
parents: 49677
diff changeset
   692
      _cur_class_is_interface(is_interf) {}
18695
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   693
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   694
  void get_discovered_family(MethodFamily** family) {
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   695
      if (_family != NULL) {
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   696
        *family = _family->get_method_family();
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   697
      } else {
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   698
        *family = NULL;
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   699
      }
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   700
  }
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   701
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   702
  void* new_node_data(InstanceKlass* cls) { return new PseudoScope(); }
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   703
  void free_node_data(void* node_data) {
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   704
    PseudoScope::cast(node_data)->destroy();
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   705
  }
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   706
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   707
  // Find all methods on this hierarchy that match this
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   708
  // method's erased (name, signature)
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   709
  bool visit() {
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   710
    PseudoScope* scope = PseudoScope::cast(current_data());
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   711
    InstanceKlass* iklass = current_class();
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   712
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   713
    Method* m = iklass->find_method(_method_name, _method_signature);
48463
474cec233fb2 8154587: Resolution fails for default method named 'clone'
hseigel
parents: 47554
diff changeset
   714
    // Private interface methods are not candidates for default methods.
474cec233fb2 8154587: Resolution fails for default method named 'clone'
hseigel
parents: 47554
diff changeset
   715
    // invokespecial to private interface methods doesn't use default method logic.
474cec233fb2 8154587: Resolution fails for default method named 'clone'
hseigel
parents: 47554
diff changeset
   716
    // Private class methods are not candidates for default methods.
474cec233fb2 8154587: Resolution fails for default method named 'clone'
hseigel
parents: 47554
diff changeset
   717
    // Private methods do not override default methods, so need to perform
474cec233fb2 8154587: Resolution fails for default method named 'clone'
hseigel
parents: 47554
diff changeset
   718
    // default method inheritance without including private methods.
474cec233fb2 8154587: Resolution fails for default method named 'clone'
hseigel
parents: 47554
diff changeset
   719
    // The overpasses are your supertypes' errors, we do not include them.
474cec233fb2 8154587: Resolution fails for default method named 'clone'
hseigel
parents: 47554
diff changeset
   720
    // Non-public methods in java.lang.Object are not candidates for default
474cec233fb2 8154587: Resolution fails for default method named 'clone'
hseigel
parents: 47554
diff changeset
   721
    // methods.
474cec233fb2 8154587: Resolution fails for default method named 'clone'
hseigel
parents: 47554
diff changeset
   722
    // Future: take access controls into account for superclass methods
474cec233fb2 8154587: Resolution fails for default method named 'clone'
hseigel
parents: 47554
diff changeset
   723
    if (m != NULL && !m->is_static() && !m->is_overpass() && !m->is_private() &&
474cec233fb2 8154587: Resolution fails for default method named 'clone'
hseigel
parents: 47554
diff changeset
   724
     (!_cur_class_is_interface || !SystemDictionary::is_nonpublic_Object_method(m))) {
18695
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   725
      if (_family == NULL) {
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   726
        _family = new StatefulMethodFamily();
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   727
      }
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   728
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   729
      if (iklass->is_interface()) {
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   730
        StateRestorer* restorer = _family->record_method_and_dq_further(m);
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   731
        scope->add_mark(restorer);
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   732
      } else {
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   733
        // This is the rule that methods in classes "win" (bad word) over
48463
474cec233fb2 8154587: Resolution fails for default method named 'clone'
hseigel
parents: 47554
diff changeset
   734
        // methods in interfaces. This works because of single inheritance.
474cec233fb2 8154587: Resolution fails for default method named 'clone'
hseigel
parents: 47554
diff changeset
   735
        // Private methods in classes do not "win", they will be found
29871
77e7fe063568 8065366: Better private method resolution
acorn
parents: 29316
diff changeset
   736
        // first on searching, but overriding for invokevirtual needs
77e7fe063568 8065366: Better private method resolution
acorn
parents: 29316
diff changeset
   737
        // to find default method candidates for the same signature
18695
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   738
        _family->set_target_if_empty(m);
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   739
      }
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   740
    }
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   741
    return true;
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   742
  }
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   743
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   744
};
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   745
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   746
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   747
20391
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   748
static void create_defaults_and_exceptions(
18695
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   749
    GrowableArray<EmptyVtableSlot*>* slots, InstanceKlass* klass, TRAPS);
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   750
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   751
static void generate_erased_defaults(
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   752
     InstanceKlass* klass, GrowableArray<EmptyVtableSlot*>* empty_slots,
48463
474cec233fb2 8154587: Resolution fails for default method named 'clone'
hseigel
parents: 47554
diff changeset
   753
     EmptyVtableSlot* slot, bool is_intf, TRAPS) {
18695
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   754
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   755
  // sets up a set of methods with the same exact erased signature
48463
474cec233fb2 8154587: Resolution fails for default method named 'clone'
hseigel
parents: 47554
diff changeset
   756
  FindMethodsByErasedSig visitor(slot->name(), slot->signature(), is_intf);
18695
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   757
  visitor.run(klass);
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   758
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   759
  MethodFamily* family;
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   760
  visitor.get_discovered_family(&family);
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   761
  if (family != NULL) {
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   762
    family->determine_target(klass, CHECK);
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   763
    slot->bind_family(family);
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   764
  }
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   765
}
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   766
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   767
static void merge_in_new_methods(InstanceKlass* klass,
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   768
    GrowableArray<Method*>* new_methods, TRAPS);
20391
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   769
static void create_default_methods( InstanceKlass* klass,
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   770
    GrowableArray<Method*>* new_methods, TRAPS);
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   771
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   772
// This is the guts of the default methods implementation.  This is called just
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   773
// after the classfile has been parsed if some ancestor has default methods.
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   774
//
47554
bc112140e089 8186092: Unnecessary loader constraints produced when there are multiple defaults
hseigel
parents: 47216
diff changeset
   775
// First it finds any name/signature slots that need any implementation (either
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   776
// because they are miranda or a superclass's implementation is an overpass
19681
1b35da7b1d85 8012294: remove generic handling for default methods
acorn
parents: 18695
diff changeset
   777
// itself).  For each slot, iterate over the hierarchy, to see if they contain a
1b35da7b1d85 8012294: remove generic handling for default methods
acorn
parents: 18695
diff changeset
   778
// signature that matches the slot we are looking at.
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   779
//
47554
bc112140e089 8186092: Unnecessary loader constraints produced when there are multiple defaults
hseigel
parents: 47216
diff changeset
   780
// For each slot filled, we either record the default method candidate in the
bc112140e089 8186092: Unnecessary loader constraints produced when there are multiple defaults
hseigel
parents: 47216
diff changeset
   781
// klass default_methods list or, only to handle exception cases, we create an
bc112140e089 8186092: Unnecessary loader constraints produced when there are multiple defaults
hseigel
parents: 47216
diff changeset
   782
// overpass method that throws an exception and add it to the klass methods list.
19681
1b35da7b1d85 8012294: remove generic handling for default methods
acorn
parents: 18695
diff changeset
   783
// The JVM does not create bridges nor handle generic signatures here.
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   784
void DefaultMethods::generate_default_methods(
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 33736
diff changeset
   785
    InstanceKlass* klass, const GrowableArray<Method*>* mirandas, TRAPS) {
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 33736
diff changeset
   786
  assert(klass != NULL, "invariant");
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   787
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   788
  // This resource mark is the bound for all memory allocation that takes
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   789
  // place during default method processing.  After this goes out of scope,
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   790
  // all (Resource) objects' memory will be reclaimed.  Be careful if adding an
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   791
  // embedded resource mark under here as that memory can't be used outside
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   792
  // whatever scope it's in.
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   793
  ResourceMark rm(THREAD);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   794
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   795
  // Keep entire hierarchy alive for the duration of the computation
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 33736
diff changeset
   796
  constantPoolHandle cp(THREAD, klass->constants());
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   797
  KeepAliveRegistrar keepAlive(THREAD);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   798
  KeepAliveVisitor loadKeepAlive(&keepAlive);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   799
  loadKeepAlive.run(klass);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   800
46701
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 42059
diff changeset
   801
  LogTarget(Debug, defaultmethods) lt;
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 42059
diff changeset
   802
  if (lt.is_enabled()) {
33736
1b3950243443 8139564: Convert TraceDefaultMethods to Unified Logging
rprotacio
parents: 29871
diff changeset
   803
    ResourceMark rm;
46701
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 42059
diff changeset
   804
    lt.print("%s %s requires default method processing",
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 42059
diff changeset
   805
             klass->is_interface() ? "Interface" : "Class",
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 42059
diff changeset
   806
             klass->name()->as_klass_external_name());
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 42059
diff changeset
   807
    LogStream ls(lt);
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 42059
diff changeset
   808
    PrintHierarchy printer(&ls);
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   809
    printer.run(klass);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   810
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   811
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   812
  GrowableArray<EmptyVtableSlot*>* empty_slots =
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   813
      find_empty_vtable_slots(klass, mirandas, CHECK);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   814
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   815
  for (int i = 0; i < empty_slots->length(); ++i) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   816
    EmptyVtableSlot* slot = empty_slots->at(i);
46701
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 42059
diff changeset
   817
    LogTarget(Debug, defaultmethods) lt;
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 42059
diff changeset
   818
    if (lt.is_enabled()) {
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 42059
diff changeset
   819
      LogStream ls(lt);
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 42059
diff changeset
   820
      streamIndentor si(&ls, 2);
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 42059
diff changeset
   821
      ls.indent().print("Looking for default methods for slot ");
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 42059
diff changeset
   822
      slot->print_on(&ls);
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 42059
diff changeset
   823
      ls.cr();
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   824
    }
48463
474cec233fb2 8154587: Resolution fails for default method named 'clone'
hseigel
parents: 47554
diff changeset
   825
    generate_erased_defaults(klass, empty_slots, slot, klass->is_interface(), CHECK);
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   826
  }
33736
1b3950243443 8139564: Convert TraceDefaultMethods to Unified Logging
rprotacio
parents: 29871
diff changeset
   827
  log_debug(defaultmethods)("Creating defaults and overpasses...");
20391
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   828
  create_defaults_and_exceptions(empty_slots, klass, CHECK);
33736
1b3950243443 8139564: Convert TraceDefaultMethods to Unified Logging
rprotacio
parents: 29871
diff changeset
   829
  log_debug(defaultmethods)("Default method processing complete");
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   830
}
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   831
20284
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 19966
diff changeset
   832
static int assemble_method_error(
595a25ab9474 8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents: 19966
diff changeset
   833
    BytecodeConstantPool* cp, BytecodeBuffer* buffer, Symbol* errorName, Symbol* message, TRAPS) {
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   834
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   835
  Symbol* init = vmSymbols::object_initializer_name();
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   836
  Symbol* sig = vmSymbols::string_void_signature();
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   837
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   838
  BytecodeAssembler assem(buffer, cp);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   839
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   840
  assem._new(errorName);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   841
  assem.dup();
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   842
  assem.load_string(message);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   843
  assem.invokespecial(errorName, init, sig);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   844
  assem.athrow();
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   845
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   846
  return 3; // max stack size: [ exception, exception, string ]
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   847
}
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   848
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   849
static Method* new_method(
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   850
    BytecodeConstantPool* cp, BytecodeBuffer* bytecodes, Symbol* name,
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   851
    Symbol* sig, AccessFlags flags, int max_stack, int params,
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   852
    ConstMethod::MethodType mt, TRAPS) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   853
18695
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   854
  address code_start = 0;
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   855
  int code_length = 0;
15601
df8faef6efaf 8007320: NPG: move method annotations
coleenp
parents: 15219
diff changeset
   856
  InlineTableSizes sizes;
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   857
18695
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   858
  if (bytecodes != NULL && bytecodes->length() > 0) {
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   859
    code_start = static_cast<address>(bytecodes->adr_at(0));
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   860
    code_length = bytecodes->length();
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   861
  }
be902722fe0a 8013635: VM should no longer create bridges for generic signatures.
acorn
parents: 17859
diff changeset
   862
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   863
  Method* m = Method::allocate(cp->pool_holder()->class_loader_data(),
15601
df8faef6efaf 8007320: NPG: move method annotations
coleenp
parents: 15219
diff changeset
   864
                               code_length, flags, &sizes,
15102
0a86564e5f61 8004728: Add hotspot support for parameter reflection
coleenp
parents: 14586
diff changeset
   865
                               mt, CHECK_NULL);
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   866
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   867
  m->set_constants(NULL); // This will get filled in later
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   868
  m->set_name_index(cp->utf8(name));
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   869
  m->set_signature_index(cp->utf8(sig));
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   870
  ResultTypeFinder rtf(sig);
37480
291ee208fb72 8132051: Better byte behavior
coleenp
parents: 37242
diff changeset
   871
  m->constMethod()->set_result_type(rtf.type());
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   872
  m->set_size_of_parameters(params);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   873
  m->set_max_stack(max_stack);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   874
  m->set_max_locals(params);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   875
  m->constMethod()->set_stackmap_data(NULL);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   876
  m->set_code(code_start);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   877
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   878
  return m;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   879
}
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   880
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   881
static void switchover_constant_pool(BytecodeConstantPool* bpool,
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   882
    InstanceKlass* klass, GrowableArray<Method*>* new_methods, TRAPS) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   883
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   884
  if (new_methods->length() > 0) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   885
    ConstantPool* cp = bpool->create_constant_pool(CHECK);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   886
    if (cp != klass->constants()) {
49677
a1a7456dd8b9 8200261: Regression with JVM anonymous class
hseigel
parents: 48463
diff changeset
   887
      // Copy resolved anonymous class into new constant pool.
51444
3e5d28e6de32 8209301: JVM rename is_anonymous, host_klass to unsafe specific terminology ahead of Unsafe.defineAnonymousClass deprecation
lfoltan
parents: 51334
diff changeset
   888
      if (klass->is_unsafe_anonymous()) {
49677
a1a7456dd8b9 8200261: Regression with JVM anonymous class
hseigel
parents: 48463
diff changeset
   889
        cp->klass_at_put(klass->this_class_index(), klass);
a1a7456dd8b9 8200261: Regression with JVM anonymous class
hseigel
parents: 48463
diff changeset
   890
      }
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   891
      klass->class_loader_data()->add_to_deallocate_list(klass->constants());
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   892
      klass->set_constants(cp);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   893
      cp->set_pool_holder(klass);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   894
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   895
      for (int i = 0; i < new_methods->length(); ++i) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   896
        new_methods->at(i)->set_constants(cp);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   897
      }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   898
      for (int i = 0; i < klass->methods()->length(); ++i) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   899
        Method* mo = klass->methods()->at(i);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   900
        mo->set_constants(cp);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   901
      }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   902
    }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   903
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   904
}
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   905
20391
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   906
// Create default_methods list for the current class.
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   907
// With the VM only processing erased signatures, the VM only
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   908
// creates an overpass in a conflict case or a case with no candidates.
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   909
// This allows virtual methods to override the overpass, but ensures
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   910
// that a local method search will find the exception rather than an abstract
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   911
// or default method that is not a valid candidate.
47554
bc112140e089 8186092: Unnecessary loader constraints produced when there are multiple defaults
hseigel
parents: 47216
diff changeset
   912
//
bc112140e089 8186092: Unnecessary loader constraints produced when there are multiple defaults
hseigel
parents: 47216
diff changeset
   913
// Note that if overpass method are ever created that are not exception
bc112140e089 8186092: Unnecessary loader constraints produced when there are multiple defaults
hseigel
parents: 47216
diff changeset
   914
// throwing methods then the loader constraint checking logic for vtable and
bc112140e089 8186092: Unnecessary loader constraints produced when there are multiple defaults
hseigel
parents: 47216
diff changeset
   915
// itable creation needs to be changed to check loader constraints for the
bc112140e089 8186092: Unnecessary loader constraints produced when there are multiple defaults
hseigel
parents: 47216
diff changeset
   916
// overpass methods that do not throw exceptions.
20391
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   917
static void create_defaults_and_exceptions(
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   918
    GrowableArray<EmptyVtableSlot*>* slots,
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   919
    InstanceKlass* klass, TRAPS) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   920
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   921
  GrowableArray<Method*> overpasses;
20391
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   922
  GrowableArray<Method*> defaults;
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   923
  BytecodeConstantPool bpool(klass->constants());
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   924
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   925
  for (int i = 0; i < slots->length(); ++i) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   926
    EmptyVtableSlot* slot = slots->at(i);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   927
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   928
    if (slot->is_bound()) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   929
      MethodFamily* method = slot->get_binding();
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   930
      BytecodeBuffer buffer;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   931
46701
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 42059
diff changeset
   932
      LogTarget(Debug, defaultmethods) lt;
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 42059
diff changeset
   933
      if (lt.is_enabled()) {
41669
2091069b6851 8081800: AbstractMethodError when evaluating a private method in an interface via debugger
dholmes
parents: 37480
diff changeset
   934
        ResourceMark rm(THREAD);
46701
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 42059
diff changeset
   935
        LogStream ls(lt);
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 42059
diff changeset
   936
        ls.print("for slot: ");
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 42059
diff changeset
   937
        slot->print_on(&ls);
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 42059
diff changeset
   938
        ls.cr();
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   939
        if (method->has_target()) {
46701
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 42059
diff changeset
   940
          method->print_selected(&ls, 1);
20391
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   941
        } else if (method->throws_exception()) {
46701
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 42059
diff changeset
   942
          method->print_exception(&ls, 1);
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   943
        }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   944
      }
20391
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   945
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   946
      if (method->has_target()) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   947
        Method* selected = method->get_selected_target();
19966
64732b96b5f5 8024647: Default method resolution with private superclass method
acorn
parents: 19690
diff changeset
   948
        if (selected->method_holder()->is_interface()) {
41669
2091069b6851 8081800: AbstractMethodError when evaluating a private method in an interface via debugger
dholmes
parents: 37480
diff changeset
   949
          assert(!selected->is_private(), "pushing private interface method as default");
20391
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   950
          defaults.push(selected);
19966
64732b96b5f5 8024647: Default method resolution with private superclass method
acorn
parents: 19690
diff changeset
   951
        }
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   952
      } else if (method->throws_exception()) {
20391
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   953
        int max_stack = assemble_method_error(&bpool, &buffer,
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   954
           method->get_exception_name(), method->get_exception_message(), CHECK);
19966
64732b96b5f5 8024647: Default method resolution with private superclass method
acorn
parents: 19690
diff changeset
   955
        AccessFlags flags = accessFlags_from(
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   956
          JVM_ACC_PUBLIC | JVM_ACC_SYNTHETIC | JVM_ACC_BRIDGE);
20391
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   957
         Method* m = new_method(&bpool, &buffer, slot->name(), slot->signature(),
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   958
          flags, max_stack, slot->size_of_parameters(),
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   959
          ConstMethod::OVERPASS, CHECK);
20391
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   960
        // We push to the methods list:
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   961
        // overpass methods which are exception throwing methods
19966
64732b96b5f5 8024647: Default method resolution with private superclass method
acorn
parents: 19690
diff changeset
   962
        if (m != NULL) {
64732b96b5f5 8024647: Default method resolution with private superclass method
acorn
parents: 19690
diff changeset
   963
          overpasses.push(m);
64732b96b5f5 8024647: Default method resolution with private superclass method
acorn
parents: 19690
diff changeset
   964
        }
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   965
      }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   966
    }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   967
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   968
33736
1b3950243443 8139564: Convert TraceDefaultMethods to Unified Logging
rprotacio
parents: 29871
diff changeset
   969
1b3950243443 8139564: Convert TraceDefaultMethods to Unified Logging
rprotacio
parents: 29871
diff changeset
   970
  log_debug(defaultmethods)("Created %d overpass methods", overpasses.length());
1b3950243443 8139564: Convert TraceDefaultMethods to Unified Logging
rprotacio
parents: 29871
diff changeset
   971
  log_debug(defaultmethods)("Created %d default  methods", defaults.length());
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   972
20391
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   973
  if (overpasses.length() > 0) {
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   974
    switchover_constant_pool(&bpool, klass, &overpasses, CHECK);
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   975
    merge_in_new_methods(klass, &overpasses, CHECK);
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   976
  }
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   977
  if (defaults.length() > 0) {
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   978
    create_default_methods(klass, &defaults, CHECK);
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   979
  }
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   980
}
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   981
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   982
static void create_default_methods( InstanceKlass* klass,
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   983
    GrowableArray<Method*>* new_methods, TRAPS) {
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   984
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   985
  int new_size = new_methods->length();
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   986
  Array<Method*>* total_default_methods = MetadataFactory::new_array<Method*>(
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   987
      klass->class_loader_data(), new_size, NULL, CHECK);
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   988
  for (int index = 0; index < new_size; index++ ) {
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   989
    total_default_methods->at_put(index, new_methods->at(index));
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   990
  }
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   991
  Method::sort_methods(total_default_methods, false, false);
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   992
7b146c5ebb18 8009130: Lambda: Fix access controls, loader constraints.
acorn
parents: 20300
diff changeset
   993
  klass->set_default_methods(total_default_methods);
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   994
}
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   995
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   996
static void sort_methods(GrowableArray<Method*>* methods) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   997
  // Note that this must sort using the same key as is used for sorting
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   998
  // methods in InstanceKlass.
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
   999
  bool sorted = true;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1000
  for (int i = methods->length() - 1; i > 0; --i) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1001
    for (int j = 0; j < i; ++j) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1002
      Method* m1 = methods->at(j);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1003
      Method* m2 = methods->at(j + 1);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1004
      if ((uintptr_t)m1->name() > (uintptr_t)m2->name()) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1005
        methods->at_put(j, m2);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1006
        methods->at_put(j + 1, m1);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1007
        sorted = false;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1008
      }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1009
    }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1010
    if (sorted) break;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1011
    sorted = true;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1012
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1013
#ifdef ASSERT
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1014
  uintptr_t prev = 0;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1015
  for (int i = 0; i < methods->length(); ++i) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1016
    Method* mh = methods->at(i);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1017
    uintptr_t nv = (uintptr_t)mh->name();
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1018
    assert(nv >= prev, "Incorrect overpass method ordering");
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1019
    prev = nv;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1020
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1021
#endif
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1022
}
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1023
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1024
static void merge_in_new_methods(InstanceKlass* klass,
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1025
    GrowableArray<Method*>* new_methods, TRAPS) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1026
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1027
  enum { ANNOTATIONS, PARAMETERS, DEFAULTS, NUM_ARRAYS };
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1028
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1029
  Array<Method*>* original_methods = klass->methods();
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1030
  Array<int>* original_ordering = klass->method_ordering();
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1031
  Array<int>* merged_ordering = Universe::the_empty_int_array();
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1032
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1033
  int new_size = klass->methods()->length() + new_methods->length();
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1034
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1035
  Array<Method*>* merged_methods = MetadataFactory::new_array<Method*>(
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1036
      klass->class_loader_data(), new_size, NULL, CHECK);
15601
df8faef6efaf 8007320: NPG: move method annotations
coleenp
parents: 15219
diff changeset
  1037
22233
f0028de67b30 8030633: nsk/jvmti/RedefineClasses/StressRedefine failed invalid method ordering length on Solaris
coleenp
parents: 22232
diff changeset
  1038
  // original_ordering might be empty if this class has no methods of its own
f0028de67b30 8030633: nsk/jvmti/RedefineClasses/StressRedefine failed invalid method ordering length on Solaris
coleenp
parents: 22232
diff changeset
  1039
  if (JvmtiExport::can_maintain_original_method_order() || DumpSharedSpaces) {
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1040
    merged_ordering = MetadataFactory::new_array<int>(
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1041
        klass->class_loader_data(), new_size, CHECK);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1042
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1043
  int method_order_index = klass->methods()->length();
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1044
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1045
  sort_methods(new_methods);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1046
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1047
  // Perform grand merge of existing methods and new methods
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1048
  int orig_idx = 0;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1049
  int new_idx = 0;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1050
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1051
  for (int i = 0; i < new_size; ++i) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1052
    Method* orig_method = NULL;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1053
    Method* new_method = NULL;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1054
    if (orig_idx < original_methods->length()) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1055
      orig_method = original_methods->at(orig_idx);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1056
    }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1057
    if (new_idx < new_methods->length()) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1058
      new_method = new_methods->at(new_idx);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1059
    }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1060
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1061
    if (orig_method != NULL &&
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1062
        (new_method == NULL || orig_method->name() < new_method->name())) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1063
      merged_methods->at_put(i, orig_method);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1064
      original_methods->at_put(orig_idx, NULL);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1065
      if (merged_ordering->length() > 0) {
22233
f0028de67b30 8030633: nsk/jvmti/RedefineClasses/StressRedefine failed invalid method ordering length on Solaris
coleenp
parents: 22232
diff changeset
  1066
        assert(original_ordering != NULL && original_ordering->length() > 0,
f0028de67b30 8030633: nsk/jvmti/RedefineClasses/StressRedefine failed invalid method ordering length on Solaris
coleenp
parents: 22232
diff changeset
  1067
               "should have original order information for this method");
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1068
        merged_ordering->at_put(i, original_ordering->at(orig_idx));
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1069
      }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1070
      ++orig_idx;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1071
    } else {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1072
      merged_methods->at_put(i, new_method);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1073
      if (merged_ordering->length() > 0) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1074
        merged_ordering->at_put(i, method_order_index++);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1075
      }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1076
      ++new_idx;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1077
    }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1078
    // update idnum for new location
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1079
    merged_methods->at(i)->set_method_idnum(i);
29316
5287df8a8972 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 27680
diff changeset
  1080
    merged_methods->at(i)->set_orig_method_idnum(i);
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1081
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1082
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1083
  // Verify correct order
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1084
#ifdef ASSERT
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1085
  uintptr_t prev = 0;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1086
  for (int i = 0; i < merged_methods->length(); ++i) {
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1087
    Method* mo = merged_methods->at(i);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1088
    uintptr_t nv = (uintptr_t)mo->name();
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1089
    assert(nv >= prev, "Incorrect method ordering");
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1090
    prev = nv;
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1091
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1092
#endif
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1093
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1094
  // Replace klass methods with new merged lists
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1095
  klass->set_methods(merged_methods);
17859
cda7f55ca4dc 8015436: compiler/ciReplay/TestSA.sh fails with assert() index is out of bounds
sspitsyn
parents: 16379
diff changeset
  1096
  klass->set_initial_method_idnum(new_size);
22233
f0028de67b30 8030633: nsk/jvmti/RedefineClasses/StressRedefine failed invalid method ordering length on Solaris
coleenp
parents: 22232
diff changeset
  1097
  klass->set_method_ordering(merged_ordering);
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1098
22233
f0028de67b30 8030633: nsk/jvmti/RedefineClasses/StressRedefine failed invalid method ordering length on Solaris
coleenp
parents: 22232
diff changeset
  1099
  // Free metadata
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1100
  ClassLoaderData* cld = klass->class_loader_data();
22233
f0028de67b30 8030633: nsk/jvmti/RedefineClasses/StressRedefine failed invalid method ordering length on Solaris
coleenp
parents: 22232
diff changeset
  1101
  if (original_methods->length() > 0) {
21556
e75cd34a59e0 8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents: 21516
diff changeset
  1102
    MetadataFactory::free_array(cld, original_methods);
e75cd34a59e0 8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents: 21516
diff changeset
  1103
  }
22233
f0028de67b30 8030633: nsk/jvmti/RedefineClasses/StressRedefine failed invalid method ordering length on Solaris
coleenp
parents: 22232
diff changeset
  1104
  if (original_ordering != NULL && original_ordering->length() > 0) {
14385
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1105
    MetadataFactory::free_array(cld, original_ordering);
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1106
  }
959bbcc16725 7200776: Implement default methods in interfaces
kamg
parents:
diff changeset
  1107
}