Mercurial > hg > graal-compiler

annotate src/share/vm/interpreter/rewriter.cpp @ 4559:723df37192d6

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Make it possible again to build a real client libjvm, drop the UseGraal flag. Use the --vm option instead of a special -vm option in the bench command
author Gilles Duboscq <duboscq@ssw.jku.at>
date Fri, 10 Feb 2012 17:04:03 +0100
parents 82af018d61db
children 957c266d8bc5
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1 /*
2142
8012aa3ccede 4926272: methodOopDesc::method_from_bcp is unsafe
never
parents: 2011
diff changeset
2 * Copyright (c) 1998, 2011, Oracle and/or its affiliates. All rights reserved.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
a61af66fc99e Initial load
duke
parents:
diff changeset
4 *
a61af66fc99e Initial load
duke
parents:
diff changeset
5 * This code is free software; you can redistribute it and/or modify it
a61af66fc99e Initial load
duke
parents:
diff changeset
6 * under the terms of the GNU General Public License version 2 only, as
a61af66fc99e Initial load
duke
parents:
diff changeset
7 * published by the Free Software Foundation.
a61af66fc99e Initial load
duke
parents:
diff changeset
8 *
a61af66fc99e Initial load
duke
parents:
diff changeset
9 * This code is distributed in the hope that it will be useful, but WITHOUT
a61af66fc99e Initial load
duke
parents:
diff changeset
10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
a61af66fc99e Initial load
duke
parents:
diff changeset
11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
a61af66fc99e Initial load
duke
parents:
diff changeset
12 * version 2 for more details (a copy is included in the LICENSE file that
a61af66fc99e Initial load
duke
parents:
diff changeset
13 * accompanied this code).
a61af66fc99e Initial load
duke
parents:
diff changeset
14 *
a61af66fc99e Initial load
duke
parents:
diff changeset
15 * You should have received a copy of the GNU General Public License version
a61af66fc99e Initial load
duke
parents:
diff changeset
16 * 2 along with this work; if not, write to the Free Software Foundation,
a61af66fc99e Initial load
duke
parents:
diff changeset
17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
a61af66fc99e Initial load
duke
parents:
diff changeset
18 *
1552
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 1138
diff changeset
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 1138
diff changeset
20 * or visit www.oracle.com if you need additional information or have any
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 1138
diff changeset
21 * questions.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
22 *
a61af66fc99e Initial load
duke
parents:
diff changeset
23 */
a61af66fc99e Initial load
duke
parents:
diff changeset
24
1972
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1660
diff changeset
25 #include "precompiled.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1660
diff changeset
26 #include "interpreter/bytecodes.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1660
diff changeset
27 #include "interpreter/interpreter.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1660
diff changeset
28 #include "interpreter/rewriter.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1660
diff changeset
29 #include "memory/gcLocker.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1660
diff changeset
30 #include "memory/oopFactory.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1660
diff changeset
31 #include "memory/resourceArea.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1660
diff changeset
32 #include "oops/generateOopMap.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1660
diff changeset
33 #include "oops/objArrayOop.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1660
diff changeset
34 #include "oops/oop.inline.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1660
diff changeset
35 #include "prims/methodComparator.hpp"
0
a61af66fc99e Initial load
duke
parents:
diff changeset
36
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
37 // Computes a CPC map (new_index -> original_index) for constant pool entries
0
a61af66fc99e Initial load
duke
parents:
diff changeset
38 // that are referred to by the interpreter at runtime via the constant pool cache.
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
39 // Also computes a CP map (original_index -> new_index).
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
40 // Marks entries in CP which require additional processing.
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
41 void Rewriter::compute_index_maps() {
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
42 const int length = _pool->length();
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
43 init_cp_map(length);
1660
083fde3b838e 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 1602
diff changeset
44 jint tag_mask = 0;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
45 for (int i = 0; i < length; i++) {
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
46 int tag = _pool->tag_at(i).value();
1660
083fde3b838e 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 1602
diff changeset
47 tag_mask |= (1 << tag);
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
48 switch (tag) {
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
49 case JVM_CONSTANT_InterfaceMethodref:
0
a61af66fc99e Initial load
duke
parents:
diff changeset
50 case JVM_CONSTANT_Fieldref : // fall through
a61af66fc99e Initial load
duke
parents:
diff changeset
51 case JVM_CONSTANT_Methodref : // fall through
1602
136b78722a08 6939203: JSR 292 needs method handle constants
jrose
parents: 1579
diff changeset
52 case JVM_CONSTANT_MethodHandle : // fall through
136b78722a08 6939203: JSR 292 needs method handle constants
jrose
parents: 1579
diff changeset
53 case JVM_CONSTANT_MethodType : // fall through
1660
083fde3b838e 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 1602
diff changeset
54 case JVM_CONSTANT_InvokeDynamic : // fall through
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
55 add_cp_cache_entry(i);
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
56 break;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
57 }
a61af66fc99e Initial load
duke
parents:
diff changeset
58 }
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
59
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
60 guarantee((int)_cp_cache_map.length()-1 <= (int)((u2)-1),
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
61 "all cp cache indexes fit in a u2");
1660
083fde3b838e 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 1602
diff changeset
62
083fde3b838e 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 1602
diff changeset
63 _have_invoke_dynamic = ((tag_mask & (1 << JVM_CONSTANT_InvokeDynamic)) != 0);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
64 }
a61af66fc99e Initial load
duke
parents:
diff changeset
65
3404
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
66 // Unrewrite the bytecodes if an error occurs.
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
67 void Rewriter::restore_bytecodes() {
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
68 int len = _methods->length();
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
69
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
70 for (int i = len-1; i >= 0; i--) {
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
71 methodOop method = (methodOop)_methods->obj_at(i);
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
72 scan_method(method, true);
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
73 }
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
74 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
75
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
76 // Creates a constant pool cache given a CPC map
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
77 void Rewriter::make_constant_pool_cache(TRAPS) {
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
78 const int length = _cp_cache_map.length();
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
79 constantPoolCacheOop cache =
2226
c5a923563727 6912621: iCMS: Error: assert(_markBitMap.isMarked(addr + 1),"Missing Printezis bit?")
ysr
parents: 2142
diff changeset
80 oopFactory::new_constantPoolCache(length, CHECK);
c5a923563727 6912621: iCMS: Error: assert(_markBitMap.isMarked(addr + 1),"Missing Printezis bit?")
ysr
parents: 2142
diff changeset
81 No_Safepoint_Verifier nsv;
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
82 cache->initialize(_cp_cache_map);
1660
083fde3b838e 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 1602
diff changeset
83
2011
dad31fc330cd 7001379: bootstrap method data needs to be moved from constant pool to a classfile attribute
jrose
parents: 1972
diff changeset
84 // Don't bother with the next pass if there is no JVM_CONSTANT_InvokeDynamic.
1660
083fde3b838e 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 1602
diff changeset
85 if (_have_invoke_dynamic) {
083fde3b838e 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 1602
diff changeset
86 for (int i = 0; i < length; i++) {
083fde3b838e 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 1602
diff changeset
87 int pool_index = cp_cache_entry_pool_index(i);
083fde3b838e 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 1602
diff changeset
88 if (pool_index >= 0 &&
083fde3b838e 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 1602
diff changeset
89 _pool->tag_at(pool_index).is_invoke_dynamic()) {
083fde3b838e 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 1602
diff changeset
90 int bsm_index = _pool->invoke_dynamic_bootstrap_method_ref_index_at(pool_index);
2460
ed69575596ac 6981791: remove experimental code for JSR 292
jrose
parents: 2226
diff changeset
91 assert(_pool->tag_at(bsm_index).is_method_handle(), "must be a MH constant");
ed69575596ac 6981791: remove experimental code for JSR 292
jrose
parents: 2226
diff changeset
92 // There is a CP cache entry holding the BSM for these calls.
ed69575596ac 6981791: remove experimental code for JSR 292
jrose
parents: 2226
diff changeset
93 int bsm_cache_index = cp_entry_to_cp_cache(bsm_index);
ed69575596ac 6981791: remove experimental code for JSR 292
jrose
parents: 2226
diff changeset
94 cache->entry_at(i)->initialize_bootstrap_method_index_in_cache(bsm_cache_index);
1660
083fde3b838e 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 1602
diff changeset
95 }
083fde3b838e 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 1602
diff changeset
96 }
083fde3b838e 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 1602
diff changeset
97 }
083fde3b838e 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 1602
diff changeset
98
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
99 _pool->set_cache(cache);
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
100 cache->set_constant_pool(_pool());
0
a61af66fc99e Initial load
duke
parents:
diff changeset
101 }
a61af66fc99e Initial load
duke
parents:
diff changeset
102
a61af66fc99e Initial load
duke
parents:
diff changeset
103
a61af66fc99e Initial load
duke
parents:
diff changeset
104
a61af66fc99e Initial load
duke
parents:
diff changeset
105 // The new finalization semantics says that registration of
a61af66fc99e Initial load
duke
parents:
diff changeset
106 // finalizable objects must be performed on successful return from the
a61af66fc99e Initial load
duke
parents:
diff changeset
107 // Object.<init> constructor. We could implement this trivially if
a61af66fc99e Initial load
duke
parents:
diff changeset
108 // <init> were never rewritten but since JVMTI allows this to occur, a
a61af66fc99e Initial load
duke
parents:
diff changeset
109 // more complicated solution is required. A special return bytecode
a61af66fc99e Initial load
duke
parents:
diff changeset
110 // is used only by Object.<init> to signal the finalization
a61af66fc99e Initial load
duke
parents:
diff changeset
111 // registration point. Additionally local 0 must be preserved so it's
a61af66fc99e Initial load
duke
parents:
diff changeset
112 // available to pass to the registration function. For simplicty we
a61af66fc99e Initial load
duke
parents:
diff changeset
113 // require that local 0 is never overwritten so it's available as an
a61af66fc99e Initial load
duke
parents:
diff changeset
114 // argument for registration.
a61af66fc99e Initial load
duke
parents:
diff changeset
115
a61af66fc99e Initial load
duke
parents:
diff changeset
116 void Rewriter::rewrite_Object_init(methodHandle method, TRAPS) {
a61af66fc99e Initial load
duke
parents:
diff changeset
117 RawBytecodeStream bcs(method);
a61af66fc99e Initial load
duke
parents:
diff changeset
118 while (!bcs.is_last_bytecode()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
119 Bytecodes::Code opcode = bcs.raw_next();
a61af66fc99e Initial load
duke
parents:
diff changeset
120 switch (opcode) {
4559
723df37192d6 Make it possible again to build a real client libjvm, drop the UseGraal flag.
Gilles Duboscq <duboscq@ssw.jku.at>
parents: 4138
diff changeset
121 case Bytecodes::_return:
723df37192d6 Make it possible again to build a real client libjvm, drop the UseGraal flag.
Gilles Duboscq <duboscq@ssw.jku.at>
parents: 4138
diff changeset
122 #ifndef GRAAL
723df37192d6 Make it possible again to build a real client libjvm, drop the UseGraal flag.
Gilles Duboscq <duboscq@ssw.jku.at>
parents: 4138
diff changeset
123 *bcs.bcp() = Bytecodes::_return_register_finalizer;
723df37192d6 Make it possible again to build a real client libjvm, drop the UseGraal flag.
Gilles Duboscq <duboscq@ssw.jku.at>
parents: 4138
diff changeset
124 #endif
723df37192d6 Make it possible again to build a real client libjvm, drop the UseGraal flag.
Gilles Duboscq <duboscq@ssw.jku.at>
parents: 4138
diff changeset
125 break;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
126
a61af66fc99e Initial load
duke
parents:
diff changeset
127 case Bytecodes::_istore:
a61af66fc99e Initial load
duke
parents:
diff changeset
128 case Bytecodes::_lstore:
a61af66fc99e Initial load
duke
parents:
diff changeset
129 case Bytecodes::_fstore:
a61af66fc99e Initial load
duke
parents:
diff changeset
130 case Bytecodes::_dstore:
a61af66fc99e Initial load
duke
parents:
diff changeset
131 case Bytecodes::_astore:
a61af66fc99e Initial load
duke
parents:
diff changeset
132 if (bcs.get_index() != 0) continue;
a61af66fc99e Initial load
duke
parents:
diff changeset
133
a61af66fc99e Initial load
duke
parents:
diff changeset
134 // fall through
a61af66fc99e Initial load
duke
parents:
diff changeset
135 case Bytecodes::_istore_0:
a61af66fc99e Initial load
duke
parents:
diff changeset
136 case Bytecodes::_lstore_0:
a61af66fc99e Initial load
duke
parents:
diff changeset
137 case Bytecodes::_fstore_0:
a61af66fc99e Initial load
duke
parents:
diff changeset
138 case Bytecodes::_dstore_0:
a61af66fc99e Initial load
duke
parents:
diff changeset
139 case Bytecodes::_astore_0:
a61af66fc99e Initial load
duke
parents:
diff changeset
140 THROW_MSG(vmSymbols::java_lang_IncompatibleClassChangeError(),
a61af66fc99e Initial load
duke
parents:
diff changeset
141 "can't overwrite local 0 in Object.<init>");
a61af66fc99e Initial load
duke
parents:
diff changeset
142 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
143 }
a61af66fc99e Initial load
duke
parents:
diff changeset
144 }
a61af66fc99e Initial load
duke
parents:
diff changeset
145 }
a61af66fc99e Initial load
duke
parents:
diff changeset
146
a61af66fc99e Initial load
duke
parents:
diff changeset
147
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
148 // Rewrite a classfile-order CP index into a native-order CPC index.
3404
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
149 void Rewriter::rewrite_member_reference(address bcp, int offset, bool reverse) {
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
150 address p = bcp + offset;
3404
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
151 if (!reverse) {
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
152 int cp_index = Bytes::get_Java_u2(p);
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
153 int cache_index = cp_entry_to_cp_cache(cp_index);
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
154 Bytes::put_native_u2(p, cache_index);
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
155 } else {
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
156 int cache_index = Bytes::get_native_u2(p);
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
157 int pool_index = cp_cache_entry_pool_index(cache_index);
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
158 Bytes::put_Java_u2(p, pool_index);
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
159 }
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
160 }
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
161
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
162
3404
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
163 void Rewriter::rewrite_invokedynamic(address bcp, int offset, bool reverse) {
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
164 address p = bcp + offset;
3404
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
165 assert(p[-1] == Bytecodes::_invokedynamic, "not invokedynamic bytecode");
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
166 if (!reverse) {
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
167 int cp_index = Bytes::get_Java_u2(p);
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
168 int cpc = maybe_add_cp_cache_entry(cp_index); // add lazily
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
169 int cpc2 = add_secondary_cp_cache_entry(cpc);
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
170
3404
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
171 // Replace the trailing four bytes with a CPC index for the dynamic
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
172 // call site. Unlike other CPC entries, there is one per bytecode,
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
173 // not just one per distinct CP entry. In other words, the
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
174 // CPC-to-CP relation is many-to-one for invokedynamic entries.
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
175 // This means we must use a larger index size than u2 to address
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
176 // all these entries. That is the main reason invokedynamic
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
177 // must have a five-byte instruction format. (Of course, other JVM
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
178 // implementations can use the bytes for other purposes.)
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
179 Bytes::put_native_u4(p, constantPoolCacheOopDesc::encode_secondary_index(cpc2));
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
180 // Note: We use native_u4 format exclusively for 4-byte indexes.
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
181 } else {
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
182 int cache_index = constantPoolCacheOopDesc::decode_secondary_index(
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
183 Bytes::get_native_u4(p));
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
184 int secondary_index = cp_cache_secondary_entry_main_index(cache_index);
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
185 int pool_index = cp_cache_entry_pool_index(secondary_index);
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
186 assert(_pool->tag_at(pool_index).is_invoke_dynamic(), "wrong index");
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
187 // zero out 4 bytes
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
188 Bytes::put_Java_u4(p, 0);
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
189 Bytes::put_Java_u2(p, pool_index);
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
190 }
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
191 }
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
192
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
193
1602
136b78722a08 6939203: JSR 292 needs method handle constants
jrose
parents: 1579
diff changeset
194 // Rewrite some ldc bytecodes to _fast_aldc
3404
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
195 void Rewriter::maybe_rewrite_ldc(address bcp, int offset, bool is_wide,
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
196 bool reverse) {
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
197 if (!reverse) {
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
198 assert((*bcp) == (is_wide ? Bytecodes::_ldc_w : Bytecodes::_ldc), "not ldc bytecode");
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
199 address p = bcp + offset;
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
200 int cp_index = is_wide ? Bytes::get_Java_u2(p) : (u1)(*p);
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
201 constantTag tag = _pool->tag_at(cp_index).value();
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
202 if (tag.is_method_handle() || tag.is_method_type()) {
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
203 int cache_index = cp_entry_to_cp_cache(cp_index);
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
204 if (is_wide) {
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
205 (*bcp) = Bytecodes::_fast_aldc_w;
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
206 assert(cache_index == (u2)cache_index, "index overflow");
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
207 Bytes::put_native_u2(p, cache_index);
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
208 } else {
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
209 (*bcp) = Bytecodes::_fast_aldc;
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
210 assert(cache_index == (u1)cache_index, "index overflow");
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
211 (*p) = (u1)cache_index;
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
212 }
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
213 }
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
214 } else {
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
215 Bytecodes::Code rewritten_bc =
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
216 (is_wide ? Bytecodes::_fast_aldc_w : Bytecodes::_fast_aldc);
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
217 if ((*bcp) == rewritten_bc) {
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
218 address p = bcp + offset;
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
219 int cache_index = is_wide ? Bytes::get_native_u2(p) : (u1)(*p);
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
220 int pool_index = cp_cache_entry_pool_index(cache_index);
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
221 if (is_wide) {
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
222 (*bcp) = Bytecodes::_ldc_w;
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
223 assert(pool_index == (u2)pool_index, "index overflow");
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
224 Bytes::put_Java_u2(p, pool_index);
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
225 } else {
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
226 (*bcp) = Bytecodes::_ldc;
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
227 assert(pool_index == (u1)pool_index, "index overflow");
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
228 (*p) = (u1)pool_index;
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
229 }
1602
136b78722a08 6939203: JSR 292 needs method handle constants
jrose
parents: 1579
diff changeset
230 }
136b78722a08 6939203: JSR 292 needs method handle constants
jrose
parents: 1579
diff changeset
231 }
136b78722a08 6939203: JSR 292 needs method handle constants
jrose
parents: 1579
diff changeset
232 }
136b78722a08 6939203: JSR 292 needs method handle constants
jrose
parents: 1579
diff changeset
233
136b78722a08 6939203: JSR 292 needs method handle constants
jrose
parents: 1579
diff changeset
234
0
a61af66fc99e Initial load
duke
parents:
diff changeset
235 // Rewrites a method given the index_map information
3404
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
236 void Rewriter::scan_method(methodOop method, bool reverse) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
237
a61af66fc99e Initial load
duke
parents:
diff changeset
238 int nof_jsrs = 0;
a61af66fc99e Initial load
duke
parents:
diff changeset
239 bool has_monitor_bytecodes = false;
a61af66fc99e Initial load
duke
parents:
diff changeset
240
a61af66fc99e Initial load
duke
parents:
diff changeset
241 {
a61af66fc99e Initial load
duke
parents:
diff changeset
242 // We cannot tolerate a GC in this block, because we've
a61af66fc99e Initial load
duke
parents:
diff changeset
243 // cached the bytecodes in 'code_base'. If the methodOop
a61af66fc99e Initial load
duke
parents:
diff changeset
244 // moves, the bytecodes will also move.
a61af66fc99e Initial load
duke
parents:
diff changeset
245 No_Safepoint_Verifier nsv;
a61af66fc99e Initial load
duke
parents:
diff changeset
246 Bytecodes::Code c;
a61af66fc99e Initial load
duke
parents:
diff changeset
247
a61af66fc99e Initial load
duke
parents:
diff changeset
248 // Bytecodes and their length
a61af66fc99e Initial load
duke
parents:
diff changeset
249 const address code_base = method->code_base();
a61af66fc99e Initial load
duke
parents:
diff changeset
250 const int code_length = method->code_size();
a61af66fc99e Initial load
duke
parents:
diff changeset
251
a61af66fc99e Initial load
duke
parents:
diff changeset
252 int bc_length;
a61af66fc99e Initial load
duke
parents:
diff changeset
253 for (int bci = 0; bci < code_length; bci += bc_length) {
a61af66fc99e Initial load
duke
parents:
diff changeset
254 address bcp = code_base + bci;
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
255 int prefix_length = 0;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
256 c = (Bytecodes::Code)(*bcp);
a61af66fc99e Initial load
duke
parents:
diff changeset
257
a61af66fc99e Initial load
duke
parents:
diff changeset
258 // Since we have the code, see if we can get the length
a61af66fc99e Initial load
duke
parents:
diff changeset
259 // directly. Some more complicated bytecodes will report
a61af66fc99e Initial load
duke
parents:
diff changeset
260 // a length of zero, meaning we need to make another method
a61af66fc99e Initial load
duke
parents:
diff changeset
261 // call to calculate the length.
a61af66fc99e Initial load
duke
parents:
diff changeset
262 bc_length = Bytecodes::length_for(c);
a61af66fc99e Initial load
duke
parents:
diff changeset
263 if (bc_length == 0) {
2142
8012aa3ccede 4926272: methodOopDesc::method_from_bcp is unsafe
never
parents: 2011
diff changeset
264 bc_length = Bytecodes::length_at(method, bcp);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
265
a61af66fc99e Initial load
duke
parents:
diff changeset
266 // length_at will put us at the bytecode after the one modified
a61af66fc99e Initial load
duke
parents:
diff changeset
267 // by 'wide'. We don't currently examine any of the bytecodes
a61af66fc99e Initial load
duke
parents:
diff changeset
268 // modified by wide, but in case we do in the future...
a61af66fc99e Initial load
duke
parents:
diff changeset
269 if (c == Bytecodes::_wide) {
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
270 prefix_length = 1;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
271 c = (Bytecodes::Code)bcp[1];
a61af66fc99e Initial load
duke
parents:
diff changeset
272 }
a61af66fc99e Initial load
duke
parents:
diff changeset
273 }
a61af66fc99e Initial load
duke
parents:
diff changeset
274
a61af66fc99e Initial load
duke
parents:
diff changeset
275 assert(bc_length != 0, "impossible bytecode length");
a61af66fc99e Initial load
duke
parents:
diff changeset
276
a61af66fc99e Initial load
duke
parents:
diff changeset
277 switch (c) {
a61af66fc99e Initial load
duke
parents:
diff changeset
278 case Bytecodes::_lookupswitch : {
a61af66fc99e Initial load
duke
parents:
diff changeset
279 #ifndef CC_INTERP
4559
723df37192d6 Make it possible again to build a real client libjvm, drop the UseGraal flag.
Gilles Duboscq <duboscq@ssw.jku.at>
parents: 4138
diff changeset
280 #ifndef GRAAL
2181
Thomas Wuerthinger <wuerthinger@ssw.jku.at>
parents: 2044 2142
diff changeset
281 Bytecode_lookupswitch bc(method, bcp);
1930
Thomas Wuerthinger <wuerthinger@ssw.jku.at>
parents: 1433 1660
diff changeset
282 (*bcp) = (
2181
Thomas Wuerthinger <wuerthinger@ssw.jku.at>
parents: 2044 2142
diff changeset
283 bc.number_of_pairs() < BinarySwitchThreshold
1423
760213a60e8b * rewrite of the code installation
Lukas Stadler <lukas.stadler@oracle.com>
parents: 1138
diff changeset
284 ? Bytecodes::_fast_linearswitch
760213a60e8b * rewrite of the code installation
Lukas Stadler <lukas.stadler@oracle.com>
parents: 1138
diff changeset
285 : Bytecodes::_fast_binaryswitch
760213a60e8b * rewrite of the code installation
Lukas Stadler <lukas.stadler@oracle.com>
parents: 1138
diff changeset
286 );
4559
723df37192d6 Make it possible again to build a real client libjvm, drop the UseGraal flag.
Gilles Duboscq <duboscq@ssw.jku.at>
parents: 4138
diff changeset
287 #endif
0
a61af66fc99e Initial load
duke
parents:
diff changeset
288 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
289 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
290 }
3404
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
291 case Bytecodes::_fast_linearswitch:
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
292 case Bytecodes::_fast_binaryswitch: {
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
293 #ifndef CC_INTERP
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
294 (*bcp) = Bytecodes::_lookupswitch;
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
295 #endif
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
296 break;
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
297 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
298 case Bytecodes::_getstatic : // fall through
a61af66fc99e Initial load
duke
parents:
diff changeset
299 case Bytecodes::_putstatic : // fall through
a61af66fc99e Initial load
duke
parents:
diff changeset
300 case Bytecodes::_getfield : // fall through
a61af66fc99e Initial load
duke
parents:
diff changeset
301 case Bytecodes::_putfield : // fall through
a61af66fc99e Initial load
duke
parents:
diff changeset
302 case Bytecodes::_invokevirtual : // fall through
a61af66fc99e Initial load
duke
parents:
diff changeset
303 case Bytecodes::_invokespecial : // fall through
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
304 case Bytecodes::_invokestatic :
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
305 case Bytecodes::_invokeinterface:
3404
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
306 rewrite_member_reference(bcp, prefix_length+1, reverse);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
307 break;
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
308 case Bytecodes::_invokedynamic:
3404
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
309 rewrite_invokedynamic(bcp, prefix_length+1, reverse);
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
310 break;
1602
136b78722a08 6939203: JSR 292 needs method handle constants
jrose
parents: 1579
diff changeset
311 case Bytecodes::_ldc:
3404
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
312 case Bytecodes::_fast_aldc:
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
313 maybe_rewrite_ldc(bcp, prefix_length+1, false, reverse);
1602
136b78722a08 6939203: JSR 292 needs method handle constants
jrose
parents: 1579
diff changeset
314 break;
136b78722a08 6939203: JSR 292 needs method handle constants
jrose
parents: 1579
diff changeset
315 case Bytecodes::_ldc_w:
3404
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
316 case Bytecodes::_fast_aldc_w:
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
317 maybe_rewrite_ldc(bcp, prefix_length+1, true, reverse);
1602
136b78722a08 6939203: JSR 292 needs method handle constants
jrose
parents: 1579
diff changeset
318 break;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
319 case Bytecodes::_jsr : // fall through
a61af66fc99e Initial load
duke
parents:
diff changeset
320 case Bytecodes::_jsr_w : nof_jsrs++; break;
a61af66fc99e Initial load
duke
parents:
diff changeset
321 case Bytecodes::_monitorenter : // fall through
a61af66fc99e Initial load
duke
parents:
diff changeset
322 case Bytecodes::_monitorexit : has_monitor_bytecodes = true; break;
a61af66fc99e Initial load
duke
parents:
diff changeset
323 }
a61af66fc99e Initial load
duke
parents:
diff changeset
324 }
a61af66fc99e Initial load
duke
parents:
diff changeset
325 }
a61af66fc99e Initial load
duke
parents:
diff changeset
326
a61af66fc99e Initial load
duke
parents:
diff changeset
327 // Update access flags
a61af66fc99e Initial load
duke
parents:
diff changeset
328 if (has_monitor_bytecodes) {
a61af66fc99e Initial load
duke
parents:
diff changeset
329 method->set_has_monitor_bytecodes();
a61af66fc99e Initial load
duke
parents:
diff changeset
330 }
a61af66fc99e Initial load
duke
parents:
diff changeset
331
a61af66fc99e Initial load
duke
parents:
diff changeset
332 // The present of a jsr bytecode implies that the method might potentially
a61af66fc99e Initial load
duke
parents:
diff changeset
333 // have to be rewritten, so we run the oopMapGenerator on the method
a61af66fc99e Initial load
duke
parents:
diff changeset
334 if (nof_jsrs > 0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
335 method->set_has_jsrs();
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
336 // Second pass will revisit this method.
3404
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
337 assert(method->has_jsrs(), "didn't we just set this?");
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
338 }
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
339 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
340
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
341 // After constant pool is created, revisit methods containing jsrs.
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
342 methodHandle Rewriter::rewrite_jsrs(methodHandle method, TRAPS) {
3404
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
343 ResourceMark rm(THREAD);
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
344 ResolveOopMapConflicts romc(method);
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
345 methodHandle original_method = method;
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
346 method = romc.do_potential_rewrite(CHECK_(methodHandle()));
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
347 if (method() != original_method()) {
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
348 // Insert invalid bytecode into original methodOop and set
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
349 // interpreter entrypoint, so that a executing this method
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
350 // will manifest itself in an easy recognizable form.
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
351 address bcp = original_method->bcp_from(0);
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
352 *bcp = (u1)Bytecodes::_shouldnotreachhere;
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
353 int kind = Interpreter::method_kind(original_method);
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
354 original_method->set_interpreter_kind(kind);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
355 }
a61af66fc99e Initial load
duke
parents:
diff changeset
356
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
357 // Update monitor matching info.
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
358 if (romc.monitor_safe()) {
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
359 method->set_guaranteed_monitor_matching();
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
360 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
361
a61af66fc99e Initial load
duke
parents:
diff changeset
362 return method;
a61af66fc99e Initial load
duke
parents:
diff changeset
363 }
a61af66fc99e Initial load
duke
parents:
diff changeset
364
a61af66fc99e Initial load
duke
parents:
diff changeset
365 void Rewriter::rewrite(instanceKlassHandle klass, TRAPS) {
a61af66fc99e Initial load
duke
parents:
diff changeset
366 ResourceMark rm(THREAD);
1138
dd57230ba8fe 6893268: additional dynamic language related optimizations in C2
twisti
parents: 1059
diff changeset
367 Rewriter rw(klass, klass->constants(), klass->methods(), CHECK);
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
368 // (That's all, folks.)
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
369 }
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
370
1138
dd57230ba8fe 6893268: additional dynamic language related optimizations in C2
twisti
parents: 1059
diff changeset
371
dd57230ba8fe 6893268: additional dynamic language related optimizations in C2
twisti
parents: 1059
diff changeset
372 void Rewriter::rewrite(instanceKlassHandle klass, constantPoolHandle cpool, objArrayHandle methods, TRAPS) {
dd57230ba8fe 6893268: additional dynamic language related optimizations in C2
twisti
parents: 1059
diff changeset
373 ResourceMark rm(THREAD);
dd57230ba8fe 6893268: additional dynamic language related optimizations in C2
twisti
parents: 1059
diff changeset
374 Rewriter rw(klass, cpool, methods, CHECK);
dd57230ba8fe 6893268: additional dynamic language related optimizations in C2
twisti
parents: 1059
diff changeset
375 // (That's all, folks.)
dd57230ba8fe 6893268: additional dynamic language related optimizations in C2
twisti
parents: 1059
diff changeset
376 }
dd57230ba8fe 6893268: additional dynamic language related optimizations in C2
twisti
parents: 1059
diff changeset
377
dd57230ba8fe 6893268: additional dynamic language related optimizations in C2
twisti
parents: 1059
diff changeset
378
dd57230ba8fe 6893268: additional dynamic language related optimizations in C2
twisti
parents: 1059
diff changeset
379 Rewriter::Rewriter(instanceKlassHandle klass, constantPoolHandle cpool, objArrayHandle methods, TRAPS)
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
380 : _klass(klass),
1138
dd57230ba8fe 6893268: additional dynamic language related optimizations in C2
twisti
parents: 1059
diff changeset
381 _pool(cpool),
dd57230ba8fe 6893268: additional dynamic language related optimizations in C2
twisti
parents: 1059
diff changeset
382 _methods(methods)
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
383 {
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
384 assert(_pool->cache() == NULL, "constant pool cache must not be set yet");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
385
a61af66fc99e Initial load
duke
parents:
diff changeset
386 // determine index maps for methodOop rewriting
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
387 compute_index_maps();
0
a61af66fc99e Initial load
duke
parents:
diff changeset
388
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
389 if (RegisterFinalizersAtInit && _klass->name() == vmSymbols::java_lang_Object()) {
856
75596850f863 6862576: vmIntrinsics needs cleanup in order to support JSR 292 intrinsics
jrose
parents: 726
diff changeset
390 bool did_rewrite = false;
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
391 int i = _methods->length();
0
a61af66fc99e Initial load
duke
parents:
diff changeset
392 while (i-- > 0) {
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
393 methodOop method = (methodOop)_methods->obj_at(i);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
394 if (method->intrinsic_id() == vmIntrinsics::_Object_init) {
a61af66fc99e Initial load
duke
parents:
diff changeset
395 // rewrite the return bytecodes of Object.<init> to register the
a61af66fc99e Initial load
duke
parents:
diff changeset
396 // object for finalization if needed.
a61af66fc99e Initial load
duke
parents:
diff changeset
397 methodHandle m(THREAD, method);
a61af66fc99e Initial load
duke
parents:
diff changeset
398 rewrite_Object_init(m, CHECK);
856
75596850f863 6862576: vmIntrinsics needs cleanup in order to support JSR 292 intrinsics
jrose
parents: 726
diff changeset
399 did_rewrite = true;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
400 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
401 }
a61af66fc99e Initial load
duke
parents:
diff changeset
402 }
856
75596850f863 6862576: vmIntrinsics needs cleanup in order to support JSR 292 intrinsics
jrose
parents: 726
diff changeset
403 assert(did_rewrite, "must find Object::<init> to rewrite it");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
404 }
a61af66fc99e Initial load
duke
parents:
diff changeset
405
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
406 // rewrite methods, in two passes
3404
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
407 int len = _methods->length();
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
408
3404
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
409 for (int i = len-1; i >= 0; i--) {
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
410 methodOop method = (methodOop)_methods->obj_at(i);
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
411 scan_method(method);
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
412 }
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
413
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
414 // allocate constant pool cache, now that we've seen all the bytecodes
3404
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
415 make_constant_pool_cache(THREAD);
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
416
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
417 // Restore bytecodes to their unrewritten state if there are exceptions
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
418 // rewriting bytecodes or allocating the cpCache
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
419 if (HAS_PENDING_EXCEPTION) {
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
420 restore_bytecodes();
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
421 return;
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
422 }
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
423 }
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
424
3404
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
425 // Relocate jsr/rets in a method. This can't be done with the rewriter
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
426 // stage because it can throw other exceptions, leaving the bytecodes
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
427 // pointing at constant pool cache entries.
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
428 // Link and check jvmti dependencies while we're iterating over the methods.
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
429 // JSR292 code calls with a different set of methods, so two entry points.
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
430 void Rewriter::relocate_and_link(instanceKlassHandle this_oop, TRAPS) {
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
431 objArrayHandle methods(THREAD, this_oop->methods());
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
432 relocate_and_link(this_oop, methods, THREAD);
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
433 }
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
434
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
435 void Rewriter::relocate_and_link(instanceKlassHandle this_oop,
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
436 objArrayHandle methods, TRAPS) {
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
437 int len = methods->length();
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
438 for (int i = len-1; i >= 0; i--) {
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
439 methodHandle m(THREAD, (methodOop)methods->obj_at(i));
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
440
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
441 if (m->has_jsrs()) {
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
442 m = rewrite_jsrs(m, CHECK);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
443 // Method might have gotten rewritten.
3404
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
444 methods->obj_at_put(i, m());
0
a61af66fc99e Initial load
duke
parents:
diff changeset
445 }
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
446
3404
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
447 // Set up method entry points for compiler and interpreter .
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 579
diff changeset
448 m->link_method(m, CHECK);
1574
1eb493f33423 6957080: MethodComparator needs stress testing
jrose
parents: 1565
diff changeset
449
3404
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
450 // This is for JVMTI and unrelated to relocator but the last thing we do
1574
1eb493f33423 6957080: MethodComparator needs stress testing
jrose
parents: 1565
diff changeset
451 #ifdef ASSERT
1eb493f33423 6957080: MethodComparator needs stress testing
jrose
parents: 1565
diff changeset
452 if (StressMethodComparator) {
1eb493f33423 6957080: MethodComparator needs stress testing
jrose
parents: 1565
diff changeset
453 static int nmc = 0;
1eb493f33423 6957080: MethodComparator needs stress testing
jrose
parents: 1565
diff changeset
454 for (int j = i; j >= 0 && j >= i-4; j--) {
1eb493f33423 6957080: MethodComparator needs stress testing
jrose
parents: 1565
diff changeset
455 if ((++nmc % 1000) == 0) tty->print_cr("Have run MethodComparator %d times...", nmc);
3404
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
456 bool z = MethodComparator::methods_EMCP(m(),
2d4b2b833d29 7033141: assert(has_cp_cache(i)) failed: oob
coleenp
parents: 2460
diff changeset
457 (methodOop)methods->obj_at(j));
1574
1eb493f33423 6957080: MethodComparator needs stress testing
jrose
parents: 1565
diff changeset
458 if (j == i && !z) {
1eb493f33423 6957080: MethodComparator needs stress testing
jrose
parents: 1565
diff changeset
459 tty->print("MethodComparator FAIL: "); m->print(); m->print_codes();
1eb493f33423 6957080: MethodComparator needs stress testing
jrose
parents: 1565
diff changeset
460 assert(z, "method must compare equal to itself");
1eb493f33423 6957080: MethodComparator needs stress testing
jrose
parents: 1565
diff changeset
461 }
1eb493f33423 6957080: MethodComparator needs stress testing
jrose
parents: 1565
diff changeset
462 }
1eb493f33423 6957080: MethodComparator needs stress testing
jrose
parents: 1565
diff changeset
463 }
1eb493f33423 6957080: MethodComparator needs stress testing
jrose
parents: 1565
diff changeset
464 #endif //ASSERT
0
a61af66fc99e Initial load
duke
parents:
diff changeset
465 }
a61af66fc99e Initial load
duke
parents:
diff changeset
466 }