annotate src/share/vm/ci/ciField.cpp @ 17716:cdb71841f4bc

6498581: ThreadInterruptTest3 produces wrong output on Windows Summary: There is race condition between os::interrupt and os::is_interrupted on Windows. In JVM_Sleep(Thread.sleep), check if thread gets interrupted, it may see interrupted but not really interrupted so cause spurious waking up (early return from sleep). Fix by checking if interrupt event really gets set thus prevent false return. For intrinsic of _isInterrupted, on Windows, go fastpath only on bit not set. Reviewed-by: acorn, kvn Contributed-by: david.holmes@oracle.com, yumin.qi@oracle.com
author minqi
date Wed, 26 Feb 2014 15:20:41 -0800
parents b2e698d2276c
children cefad50507d8 63a4eb8bcd23 00cf2b6f51b9
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1 /*
12264
b2e698d2276c 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 12190
diff changeset
2 * Copyright (c) 1999, 2013, 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: 1173
diff changeset
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 1173
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: 1173
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: 1682
diff changeset
25 #include "precompiled.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1682
diff changeset
26 #include "ci/ciField.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1682
diff changeset
27 #include "ci/ciInstanceKlass.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1682
diff changeset
28 #include "ci/ciUtilities.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1682
diff changeset
29 #include "classfile/systemDictionary.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1682
diff changeset
30 #include "gc_interface/collectedHeap.inline.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1682
diff changeset
31 #include "interpreter/linkResolver.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1682
diff changeset
32 #include "memory/universe.inline.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1682
diff changeset
33 #include "oops/oop.inline.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1682
diff changeset
34 #include "oops/oop.inline2.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1682
diff changeset
35 #include "runtime/fieldDescriptor.hpp"
0
a61af66fc99e Initial load
duke
parents:
diff changeset
36
a61af66fc99e Initial load
duke
parents:
diff changeset
37 // ciField
a61af66fc99e Initial load
duke
parents:
diff changeset
38 //
a61af66fc99e Initial load
duke
parents:
diff changeset
39 // This class represents the result of a field lookup in the VM.
a61af66fc99e Initial load
duke
parents:
diff changeset
40 // The lookup may not succeed, in which case the information in
a61af66fc99e Initial load
duke
parents:
diff changeset
41 // the ciField will be incomplete.
a61af66fc99e Initial load
duke
parents:
diff changeset
42
a61af66fc99e Initial load
duke
parents:
diff changeset
43 // The ciObjectFactory cannot create circular data structures in one query.
a61af66fc99e Initial load
duke
parents:
diff changeset
44 // To avoid vicious circularities, we initialize ciField::_type to NULL
a61af66fc99e Initial load
duke
parents:
diff changeset
45 // for reference types and derive it lazily from the ciField::_signature.
a61af66fc99e Initial load
duke
parents:
diff changeset
46 // Primitive types are eagerly initialized, and basic layout queries
a61af66fc99e Initial load
duke
parents:
diff changeset
47 // can succeed without initialization, using only the BasicType of the field.
a61af66fc99e Initial load
duke
parents:
diff changeset
48
a61af66fc99e Initial load
duke
parents:
diff changeset
49 // Notes on bootstrapping and shared CI objects: A field is shared if and
a61af66fc99e Initial load
duke
parents:
diff changeset
50 // only if it is (a) non-static and (b) declared by a shared instance klass.
a61af66fc99e Initial load
duke
parents:
diff changeset
51 // This allows non-static field lists to be cached on shared types.
a61af66fc99e Initial load
duke
parents:
diff changeset
52 // Because the _type field is lazily initialized, however, there is a
a61af66fc99e Initial load
duke
parents:
diff changeset
53 // special restriction that a shared field cannot cache an unshared type.
a61af66fc99e Initial load
duke
parents:
diff changeset
54 // This puts a small performance penalty on shared fields with unshared
a61af66fc99e Initial load
duke
parents:
diff changeset
55 // types, such as StackTraceElement[] Throwable.stackTrace.
a61af66fc99e Initial load
duke
parents:
diff changeset
56 // (Throwable is shared because ClassCastException is shared, but
a61af66fc99e Initial load
duke
parents:
diff changeset
57 // StackTraceElement is not presently shared.)
a61af66fc99e Initial load
duke
parents:
diff changeset
58
a61af66fc99e Initial load
duke
parents:
diff changeset
59 // It is not a vicious circularity for a ciField to recursively create
a61af66fc99e Initial load
duke
parents:
diff changeset
60 // the ciSymbols necessary to represent its name and signature.
a61af66fc99e Initial load
duke
parents:
diff changeset
61 // Therefore, these items are created eagerly, and the name and signature
a61af66fc99e Initial load
duke
parents:
diff changeset
62 // of a shared field are themselves shared symbols. This somewhat
a61af66fc99e Initial load
duke
parents:
diff changeset
63 // pollutes the set of shared CI objects: It grows from 50 to 93 items,
a61af66fc99e Initial load
duke
parents:
diff changeset
64 // with all of the additional 43 being uninteresting shared ciSymbols.
a61af66fc99e Initial load
duke
parents:
diff changeset
65 // This adds at most one step to the binary search, an amount which
a61af66fc99e Initial load
duke
parents:
diff changeset
66 // decreases for complex compilation tasks.
a61af66fc99e Initial load
duke
parents:
diff changeset
67
a61af66fc99e Initial load
duke
parents:
diff changeset
68 // ------------------------------------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
69 // ciField::ciField
6151
e778c29768e6 7152811: Issues in client compiler
never
parents: 3785
diff changeset
70 ciField::ciField(ciInstanceKlass* klass, int index): _known_to_link_with_put(NULL), _known_to_link_with_get(NULL) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
71 ASSERT_IN_VM;
a61af66fc99e Initial load
duke
parents:
diff changeset
72 CompilerThread *thread = CompilerThread::current();
a61af66fc99e Initial load
duke
parents:
diff changeset
73
a61af66fc99e Initial load
duke
parents:
diff changeset
74 assert(ciObjectFactory::is_initialized(), "not a shared field");
a61af66fc99e Initial load
duke
parents:
diff changeset
75
a61af66fc99e Initial load
duke
parents:
diff changeset
76 assert(klass->get_instanceKlass()->is_linked(), "must be linked before using its constan-pool");
a61af66fc99e Initial load
duke
parents:
diff changeset
77
a61af66fc99e Initial load
duke
parents:
diff changeset
78 constantPoolHandle cpool(thread, klass->get_instanceKlass()->constants());
a61af66fc99e Initial load
duke
parents:
diff changeset
79
a61af66fc99e Initial load
duke
parents:
diff changeset
80 // Get the field's name, signature, and type.
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
81 Symbol* name = cpool->name_ref_at(index);
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
82 _name = ciEnv::current(thread)->get_symbol(name);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
83
a61af66fc99e Initial load
duke
parents:
diff changeset
84 int nt_index = cpool->name_and_type_ref_index_at(index);
a61af66fc99e Initial load
duke
parents:
diff changeset
85 int sig_index = cpool->signature_ref_index_at(nt_index);
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
86 Symbol* signature = cpool->symbol_at(sig_index);
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
87 _signature = ciEnv::current(thread)->get_symbol(signature);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
88
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
89 BasicType field_type = FieldType::basic_type(signature);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
90
a61af66fc99e Initial load
duke
parents:
diff changeset
91 // If the field is a pointer type, get the klass of the
a61af66fc99e Initial load
duke
parents:
diff changeset
92 // field.
a61af66fc99e Initial load
duke
parents:
diff changeset
93 if (field_type == T_OBJECT || field_type == T_ARRAY) {
a61af66fc99e Initial load
duke
parents:
diff changeset
94 bool ignore;
a61af66fc99e Initial load
duke
parents:
diff changeset
95 // This is not really a class reference; the index always refers to the
a61af66fc99e Initial load
duke
parents:
diff changeset
96 // field's type signature, as a symbol. Linkage checks do not apply.
1138
dd57230ba8fe 6893268: additional dynamic language related optimizations in C2
twisti
parents: 0
diff changeset
97 _type = ciEnv::current(thread)->get_klass_by_index(cpool, sig_index, ignore, klass);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
98 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
99 _type = ciType::make(field_type);
a61af66fc99e Initial load
duke
parents:
diff changeset
100 }
a61af66fc99e Initial load
duke
parents:
diff changeset
101
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
102 _name = (ciSymbol*)ciEnv::current(thread)->get_symbol(name);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
103
a61af66fc99e Initial load
duke
parents:
diff changeset
104 // Get the field's declared holder.
a61af66fc99e Initial load
duke
parents:
diff changeset
105 //
a61af66fc99e Initial load
duke
parents:
diff changeset
106 // Note: we actually create a ciInstanceKlass for this klass,
a61af66fc99e Initial load
duke
parents:
diff changeset
107 // even though we may not need to.
a61af66fc99e Initial load
duke
parents:
diff changeset
108 int holder_index = cpool->klass_ref_index_at(index);
a61af66fc99e Initial load
duke
parents:
diff changeset
109 bool holder_is_accessible;
a61af66fc99e Initial load
duke
parents:
diff changeset
110 ciInstanceKlass* declared_holder =
1138
dd57230ba8fe 6893268: additional dynamic language related optimizations in C2
twisti
parents: 0
diff changeset
111 ciEnv::current(thread)->get_klass_by_index(cpool, holder_index,
dd57230ba8fe 6893268: additional dynamic language related optimizations in C2
twisti
parents: 0
diff changeset
112 holder_is_accessible,
dd57230ba8fe 6893268: additional dynamic language related optimizations in C2
twisti
parents: 0
diff changeset
113 klass)->as_instance_klass();
0
a61af66fc99e Initial load
duke
parents:
diff changeset
114
a61af66fc99e Initial load
duke
parents:
diff changeset
115 // The declared holder of this field may not have been loaded.
a61af66fc99e Initial load
duke
parents:
diff changeset
116 // Bail out with partial field information.
a61af66fc99e Initial load
duke
parents:
diff changeset
117 if (!holder_is_accessible) {
12264
b2e698d2276c 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 12190
diff changeset
118 // _type has already been set.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
119 // The default values for _flags and _constant_value will suffice.
a61af66fc99e Initial load
duke
parents:
diff changeset
120 // We need values for _holder, _offset, and _is_constant,
a61af66fc99e Initial load
duke
parents:
diff changeset
121 _holder = declared_holder;
a61af66fc99e Initial load
duke
parents:
diff changeset
122 _offset = -1;
a61af66fc99e Initial load
duke
parents:
diff changeset
123 _is_constant = false;
a61af66fc99e Initial load
duke
parents:
diff changeset
124 return;
a61af66fc99e Initial load
duke
parents:
diff changeset
125 }
a61af66fc99e Initial load
duke
parents:
diff changeset
126
6725
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6153
diff changeset
127 InstanceKlass* loaded_decl_holder = declared_holder->get_instanceKlass();
0
a61af66fc99e Initial load
duke
parents:
diff changeset
128
a61af66fc99e Initial load
duke
parents:
diff changeset
129 // Perform the field lookup.
a61af66fc99e Initial load
duke
parents:
diff changeset
130 fieldDescriptor field_desc;
6725
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6153
diff changeset
131 Klass* canonical_holder =
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
132 loaded_decl_holder->find_field(name, signature, &field_desc);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
133 if (canonical_holder == NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
134 // Field lookup failed. Will be detected by will_link.
a61af66fc99e Initial load
duke
parents:
diff changeset
135 _holder = declared_holder;
a61af66fc99e Initial load
duke
parents:
diff changeset
136 _offset = -1;
a61af66fc99e Initial load
duke
parents:
diff changeset
137 _is_constant = false;
a61af66fc99e Initial load
duke
parents:
diff changeset
138 return;
a61af66fc99e Initial load
duke
parents:
diff changeset
139 }
a61af66fc99e Initial load
duke
parents:
diff changeset
140
a61af66fc99e Initial load
duke
parents:
diff changeset
141 assert(canonical_holder == field_desc.field_holder(), "just checking");
a61af66fc99e Initial load
duke
parents:
diff changeset
142 initialize_from(&field_desc);
a61af66fc99e Initial load
duke
parents:
diff changeset
143 }
a61af66fc99e Initial load
duke
parents:
diff changeset
144
6151
e778c29768e6 7152811: Issues in client compiler
never
parents: 3785
diff changeset
145 ciField::ciField(fieldDescriptor *fd): _known_to_link_with_put(NULL), _known_to_link_with_get(NULL) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
146 ASSERT_IN_VM;
a61af66fc99e Initial load
duke
parents:
diff changeset
147
a61af66fc99e Initial load
duke
parents:
diff changeset
148 // Get the field's name, signature, and type.
a61af66fc99e Initial load
duke
parents:
diff changeset
149 ciEnv* env = CURRENT_ENV;
2177
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
150 _name = env->get_symbol(fd->name());
3582bf76420e 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 1972
diff changeset
151 _signature = env->get_symbol(fd->signature());
0
a61af66fc99e Initial load
duke
parents:
diff changeset
152
a61af66fc99e Initial load
duke
parents:
diff changeset
153 BasicType field_type = fd->field_type();
a61af66fc99e Initial load
duke
parents:
diff changeset
154
a61af66fc99e Initial load
duke
parents:
diff changeset
155 // If the field is a pointer type, get the klass of the
a61af66fc99e Initial load
duke
parents:
diff changeset
156 // field.
a61af66fc99e Initial load
duke
parents:
diff changeset
157 if (field_type == T_OBJECT || field_type == T_ARRAY) {
a61af66fc99e Initial load
duke
parents:
diff changeset
158 _type = NULL; // must call compute_type on first access
a61af66fc99e Initial load
duke
parents:
diff changeset
159 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
160 _type = ciType::make(field_type);
a61af66fc99e Initial load
duke
parents:
diff changeset
161 }
a61af66fc99e Initial load
duke
parents:
diff changeset
162
a61af66fc99e Initial load
duke
parents:
diff changeset
163 initialize_from(fd);
a61af66fc99e Initial load
duke
parents:
diff changeset
164
a61af66fc99e Initial load
duke
parents:
diff changeset
165 // Either (a) it is marked shared, or else (b) we are done bootstrapping.
a61af66fc99e Initial load
duke
parents:
diff changeset
166 assert(is_shared() || ciObjectFactory::is_initialized(),
a61af66fc99e Initial load
duke
parents:
diff changeset
167 "bootstrap classes must not create & cache unshared fields");
a61af66fc99e Initial load
duke
parents:
diff changeset
168 }
a61af66fc99e Initial load
duke
parents:
diff changeset
169
1173
73b22f919c34 6912065: final fields in objects need to support inlining optimizations for JSR 292
jrose
parents: 1142
diff changeset
170 static bool trust_final_non_static_fields(ciInstanceKlass* holder) {
73b22f919c34 6912065: final fields in objects need to support inlining optimizations for JSR 292
jrose
parents: 1142
diff changeset
171 if (holder == NULL)
73b22f919c34 6912065: final fields in objects need to support inlining optimizations for JSR 292
jrose
parents: 1142
diff changeset
172 return false;
73b22f919c34 6912065: final fields in objects need to support inlining optimizations for JSR 292
jrose
parents: 1142
diff changeset
173 if (holder->name() == ciSymbol::java_lang_System())
73b22f919c34 6912065: final fields in objects need to support inlining optimizations for JSR 292
jrose
parents: 1142
diff changeset
174 // Never trust strangely unstable finals: System.out, etc.
73b22f919c34 6912065: final fields in objects need to support inlining optimizations for JSR 292
jrose
parents: 1142
diff changeset
175 return false;
73b22f919c34 6912065: final fields in objects need to support inlining optimizations for JSR 292
jrose
parents: 1142
diff changeset
176 // Even if general trusting is disabled, trust system-built closures in these packages.
2357
8033953d67ff 7012648: move JSR 292 to package java.lang.invoke and adjust names
jrose
parents: 2261
diff changeset
177 if (holder->is_in_package("java/lang/invoke") || holder->is_in_package("sun/invoke"))
1173
73b22f919c34 6912065: final fields in objects need to support inlining optimizations for JSR 292
jrose
parents: 1142
diff changeset
178 return true;
73b22f919c34 6912065: final fields in objects need to support inlining optimizations for JSR 292
jrose
parents: 1142
diff changeset
179 return TrustFinalNonStaticFields;
73b22f919c34 6912065: final fields in objects need to support inlining optimizations for JSR 292
jrose
parents: 1142
diff changeset
180 }
73b22f919c34 6912065: final fields in objects need to support inlining optimizations for JSR 292
jrose
parents: 1142
diff changeset
181
0
a61af66fc99e Initial load
duke
parents:
diff changeset
182 void ciField::initialize_from(fieldDescriptor* fd) {
a61af66fc99e Initial load
duke
parents:
diff changeset
183 // Get the flags, offset, and canonical holder of the field.
a61af66fc99e Initial load
duke
parents:
diff changeset
184 _flags = ciFlags(fd->access_flags());
a61af66fc99e Initial load
duke
parents:
diff changeset
185 _offset = fd->offset();
6725
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6153
diff changeset
186 _holder = CURRENT_ENV->get_instance_klass(fd->field_holder());
0
a61af66fc99e Initial load
duke
parents:
diff changeset
187
a61af66fc99e Initial load
duke
parents:
diff changeset
188 // Check to see if the field is constant.
12190
edb5ab0f3fe5 8001107: @Stable annotation for constant folding of lazily evaluated variables
vlivanov
parents: 7421
diff changeset
189 bool is_final = this->is_final();
edb5ab0f3fe5 8001107: @Stable annotation for constant folding of lazily evaluated variables
vlivanov
parents: 7421
diff changeset
190 bool is_stable = FoldStableValues && this->is_stable();
edb5ab0f3fe5 8001107: @Stable annotation for constant folding of lazily evaluated variables
vlivanov
parents: 7421
diff changeset
191 if (_holder->is_initialized() && (is_final || is_stable)) {
1138
dd57230ba8fe 6893268: additional dynamic language related optimizations in C2
twisti
parents: 0
diff changeset
192 if (!this->is_static()) {
2357
8033953d67ff 7012648: move JSR 292 to package java.lang.invoke and adjust names
jrose
parents: 2261
diff changeset
193 // A field can be constant if it's a final static field or if
8033953d67ff 7012648: move JSR 292 to package java.lang.invoke and adjust names
jrose
parents: 2261
diff changeset
194 // it's a final non-static field of a trusted class (classes in
8033953d67ff 7012648: move JSR 292 to package java.lang.invoke and adjust names
jrose
parents: 2261
diff changeset
195 // java.lang.invoke and sun.invoke packages and subpackages).
12190
edb5ab0f3fe5 8001107: @Stable annotation for constant folding of lazily evaluated variables
vlivanov
parents: 7421
diff changeset
196 if (is_stable || trust_final_non_static_fields(_holder)) {
1138
dd57230ba8fe 6893268: additional dynamic language related optimizations in C2
twisti
parents: 0
diff changeset
197 _is_constant = true;
dd57230ba8fe 6893268: additional dynamic language related optimizations in C2
twisti
parents: 0
diff changeset
198 return;
dd57230ba8fe 6893268: additional dynamic language related optimizations in C2
twisti
parents: 0
diff changeset
199 }
dd57230ba8fe 6893268: additional dynamic language related optimizations in C2
twisti
parents: 0
diff changeset
200 _is_constant = false;
dd57230ba8fe 6893268: additional dynamic language related optimizations in C2
twisti
parents: 0
diff changeset
201 return;
dd57230ba8fe 6893268: additional dynamic language related optimizations in C2
twisti
parents: 0
diff changeset
202 }
dd57230ba8fe 6893268: additional dynamic language related optimizations in C2
twisti
parents: 0
diff changeset
203
0
a61af66fc99e Initial load
duke
parents:
diff changeset
204 // This field just may be constant. The only cases where it will
a61af66fc99e Initial load
duke
parents:
diff changeset
205 // not be constant are:
a61af66fc99e Initial load
duke
parents:
diff changeset
206 //
a61af66fc99e Initial load
duke
parents:
diff changeset
207 // 1. The field holds a non-perm-space oop. The field is, strictly
a61af66fc99e Initial load
duke
parents:
diff changeset
208 // speaking, constant but we cannot embed non-perm-space oops into
a61af66fc99e Initial load
duke
parents:
diff changeset
209 // generated code. For the time being we need to consider the
a61af66fc99e Initial load
duke
parents:
diff changeset
210 // field to be not constant.
a61af66fc99e Initial load
duke
parents:
diff changeset
211 // 2. The field is a *special* static&final field whose value
a61af66fc99e Initial load
duke
parents:
diff changeset
212 // may change. The three examples are java.lang.System.in,
a61af66fc99e Initial load
duke
parents:
diff changeset
213 // java.lang.System.out, and java.lang.System.err.
a61af66fc99e Initial load
duke
parents:
diff changeset
214
6725
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6153
diff changeset
215 KlassHandle k = _holder->get_Klass();
1142
4ce7240d622c 6914300: ciEnv should export all well known classes
never
parents: 1138
diff changeset
216 assert( SystemDictionary::System_klass() != NULL, "Check once per vm");
2261
1957c1478794 7019819: bare oop in ciField
never
parents: 2177
diff changeset
217 if( k() == SystemDictionary::System_klass() ) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
218 // Check offsets for case 2: System.in, System.out, or System.err
a61af66fc99e Initial load
duke
parents:
diff changeset
219 if( _offset == java_lang_System::in_offset_in_bytes() ||
a61af66fc99e Initial load
duke
parents:
diff changeset
220 _offset == java_lang_System::out_offset_in_bytes() ||
a61af66fc99e Initial load
duke
parents:
diff changeset
221 _offset == java_lang_System::err_offset_in_bytes() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
222 _is_constant = false;
a61af66fc99e Initial load
duke
parents:
diff changeset
223 return;
a61af66fc99e Initial load
duke
parents:
diff changeset
224 }
a61af66fc99e Initial load
duke
parents:
diff changeset
225 }
a61af66fc99e Initial load
duke
parents:
diff changeset
226
2376
c7f3d0b4570f 7017732: move static fields into Class to prepare for perm gen removal
never
parents: 2357
diff changeset
227 Handle mirror = k->java_mirror();
c7f3d0b4570f 7017732: move static fields into Class to prepare for perm gen removal
never
parents: 2357
diff changeset
228
0
a61af66fc99e Initial load
duke
parents:
diff changeset
229 switch(type()->basic_type()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
230 case T_BYTE:
2376
c7f3d0b4570f 7017732: move static fields into Class to prepare for perm gen removal
never
parents: 2357
diff changeset
231 _constant_value = ciConstant(type()->basic_type(), mirror->byte_field(_offset));
0
a61af66fc99e Initial load
duke
parents:
diff changeset
232 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
233 case T_CHAR:
2376
c7f3d0b4570f 7017732: move static fields into Class to prepare for perm gen removal
never
parents: 2357
diff changeset
234 _constant_value = ciConstant(type()->basic_type(), mirror->char_field(_offset));
0
a61af66fc99e Initial load
duke
parents:
diff changeset
235 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
236 case T_SHORT:
2376
c7f3d0b4570f 7017732: move static fields into Class to prepare for perm gen removal
never
parents: 2357
diff changeset
237 _constant_value = ciConstant(type()->basic_type(), mirror->short_field(_offset));
0
a61af66fc99e Initial load
duke
parents:
diff changeset
238 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
239 case T_BOOLEAN:
2376
c7f3d0b4570f 7017732: move static fields into Class to prepare for perm gen removal
never
parents: 2357
diff changeset
240 _constant_value = ciConstant(type()->basic_type(), mirror->bool_field(_offset));
0
a61af66fc99e Initial load
duke
parents:
diff changeset
241 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
242 case T_INT:
2376
c7f3d0b4570f 7017732: move static fields into Class to prepare for perm gen removal
never
parents: 2357
diff changeset
243 _constant_value = ciConstant(type()->basic_type(), mirror->int_field(_offset));
0
a61af66fc99e Initial load
duke
parents:
diff changeset
244 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
245 case T_FLOAT:
2376
c7f3d0b4570f 7017732: move static fields into Class to prepare for perm gen removal
never
parents: 2357
diff changeset
246 _constant_value = ciConstant(mirror->float_field(_offset));
0
a61af66fc99e Initial load
duke
parents:
diff changeset
247 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
248 case T_DOUBLE:
2376
c7f3d0b4570f 7017732: move static fields into Class to prepare for perm gen removal
never
parents: 2357
diff changeset
249 _constant_value = ciConstant(mirror->double_field(_offset));
0
a61af66fc99e Initial load
duke
parents:
diff changeset
250 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
251 case T_LONG:
2376
c7f3d0b4570f 7017732: move static fields into Class to prepare for perm gen removal
never
parents: 2357
diff changeset
252 _constant_value = ciConstant(mirror->long_field(_offset));
0
a61af66fc99e Initial load
duke
parents:
diff changeset
253 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
254 case T_OBJECT:
a61af66fc99e Initial load
duke
parents:
diff changeset
255 case T_ARRAY:
a61af66fc99e Initial load
duke
parents:
diff changeset
256 {
2376
c7f3d0b4570f 7017732: move static fields into Class to prepare for perm gen removal
never
parents: 2357
diff changeset
257 oop o = mirror->obj_field(_offset);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
258
a61af66fc99e Initial load
duke
parents:
diff changeset
259 // A field will be "constant" if it is known always to be
a61af66fc99e Initial load
duke
parents:
diff changeset
260 // a non-null reference to an instance of a particular class,
a61af66fc99e Initial load
duke
parents:
diff changeset
261 // or to a particular array. This can happen even if the instance
a61af66fc99e Initial load
duke
parents:
diff changeset
262 // or array is not perm. In such a case, an "unloaded" ciArray
a61af66fc99e Initial load
duke
parents:
diff changeset
263 // or ciInstance is created. The compiler may be able to use
a61af66fc99e Initial load
duke
parents:
diff changeset
264 // information about the object's class (which is exact) or length.
a61af66fc99e Initial load
duke
parents:
diff changeset
265
a61af66fc99e Initial load
duke
parents:
diff changeset
266 if (o == NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
267 _constant_value = ciConstant(type()->basic_type(), ciNullObject::make());
a61af66fc99e Initial load
duke
parents:
diff changeset
268 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
269 _constant_value = ciConstant(type()->basic_type(), CURRENT_ENV->get_object(o));
a61af66fc99e Initial load
duke
parents:
diff changeset
270 assert(_constant_value.as_object() == CURRENT_ENV->get_object(o), "check interning");
a61af66fc99e Initial load
duke
parents:
diff changeset
271 }
a61af66fc99e Initial load
duke
parents:
diff changeset
272 }
a61af66fc99e Initial load
duke
parents:
diff changeset
273 }
12190
edb5ab0f3fe5 8001107: @Stable annotation for constant folding of lazily evaluated variables
vlivanov
parents: 7421
diff changeset
274 if (is_stable && _constant_value.is_null_or_zero()) {
edb5ab0f3fe5 8001107: @Stable annotation for constant folding of lazily evaluated variables
vlivanov
parents: 7421
diff changeset
275 // It is not a constant after all; treat it as uninitialized.
edb5ab0f3fe5 8001107: @Stable annotation for constant folding of lazily evaluated variables
vlivanov
parents: 7421
diff changeset
276 _is_constant = false;
edb5ab0f3fe5 8001107: @Stable annotation for constant folding of lazily evaluated variables
vlivanov
parents: 7421
diff changeset
277 } else {
edb5ab0f3fe5 8001107: @Stable annotation for constant folding of lazily evaluated variables
vlivanov
parents: 7421
diff changeset
278 _is_constant = true;
edb5ab0f3fe5 8001107: @Stable annotation for constant folding of lazily evaluated variables
vlivanov
parents: 7421
diff changeset
279 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
280 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
281 _is_constant = false;
a61af66fc99e Initial load
duke
parents:
diff changeset
282 }
a61af66fc99e Initial load
duke
parents:
diff changeset
283 }
a61af66fc99e Initial load
duke
parents:
diff changeset
284
a61af66fc99e Initial load
duke
parents:
diff changeset
285 // ------------------------------------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
286 // ciField::compute_type
a61af66fc99e Initial load
duke
parents:
diff changeset
287 //
a61af66fc99e Initial load
duke
parents:
diff changeset
288 // Lazily compute the type, if it is an instance klass.
a61af66fc99e Initial load
duke
parents:
diff changeset
289 ciType* ciField::compute_type() {
a61af66fc99e Initial load
duke
parents:
diff changeset
290 GUARDED_VM_ENTRY(return compute_type_impl();)
a61af66fc99e Initial load
duke
parents:
diff changeset
291 }
a61af66fc99e Initial load
duke
parents:
diff changeset
292
a61af66fc99e Initial load
duke
parents:
diff changeset
293 ciType* ciField::compute_type_impl() {
3785
ddd894528dbc 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 2376
diff changeset
294 ciKlass* type = CURRENT_ENV->get_klass_by_name_impl(_holder, constantPoolHandle(), _signature, false);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
295 if (!type->is_primitive_type() && is_shared()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
296 // We must not cache a pointer to an unshared type, in a shared field.
a61af66fc99e Initial load
duke
parents:
diff changeset
297 bool type_is_also_shared = false;
a61af66fc99e Initial load
duke
parents:
diff changeset
298 if (type->is_type_array_klass()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
299 type_is_also_shared = true; // int[] etc. are explicitly bootstrapped
a61af66fc99e Initial load
duke
parents:
diff changeset
300 } else if (type->is_instance_klass()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
301 type_is_also_shared = type->as_instance_klass()->is_shared();
a61af66fc99e Initial load
duke
parents:
diff changeset
302 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
303 // Currently there is no 'shared' query for array types.
a61af66fc99e Initial load
duke
parents:
diff changeset
304 type_is_also_shared = !ciObjectFactory::is_initialized();
a61af66fc99e Initial load
duke
parents:
diff changeset
305 }
a61af66fc99e Initial load
duke
parents:
diff changeset
306 if (!type_is_also_shared)
a61af66fc99e Initial load
duke
parents:
diff changeset
307 return type; // Bummer.
a61af66fc99e Initial load
duke
parents:
diff changeset
308 }
a61af66fc99e Initial load
duke
parents:
diff changeset
309 _type = type;
a61af66fc99e Initial load
duke
parents:
diff changeset
310 return type;
a61af66fc99e Initial load
duke
parents:
diff changeset
311 }
a61af66fc99e Initial load
duke
parents:
diff changeset
312
a61af66fc99e Initial load
duke
parents:
diff changeset
313
a61af66fc99e Initial load
duke
parents:
diff changeset
314 // ------------------------------------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
315 // ciField::will_link
a61af66fc99e Initial load
duke
parents:
diff changeset
316 //
a61af66fc99e Initial load
duke
parents:
diff changeset
317 // Can a specific access to this field be made without causing
a61af66fc99e Initial load
duke
parents:
diff changeset
318 // link errors?
a61af66fc99e Initial load
duke
parents:
diff changeset
319 bool ciField::will_link(ciInstanceKlass* accessing_klass,
a61af66fc99e Initial load
duke
parents:
diff changeset
320 Bytecodes::Code bc) {
a61af66fc99e Initial load
duke
parents:
diff changeset
321 VM_ENTRY_MARK;
6151
e778c29768e6 7152811: Issues in client compiler
never
parents: 3785
diff changeset
322 assert(bc == Bytecodes::_getstatic || bc == Bytecodes::_putstatic ||
e778c29768e6 7152811: Issues in client compiler
never
parents: 3785
diff changeset
323 bc == Bytecodes::_getfield || bc == Bytecodes::_putfield,
e778c29768e6 7152811: Issues in client compiler
never
parents: 3785
diff changeset
324 "unexpected bytecode");
e778c29768e6 7152811: Issues in client compiler
never
parents: 3785
diff changeset
325
0
a61af66fc99e Initial load
duke
parents:
diff changeset
326 if (_offset == -1) {
a61af66fc99e Initial load
duke
parents:
diff changeset
327 // at creation we couldn't link to our holder so we need to
a61af66fc99e Initial load
duke
parents:
diff changeset
328 // maintain that stance, otherwise there's no safe way to use this
a61af66fc99e Initial load
duke
parents:
diff changeset
329 // ciField.
a61af66fc99e Initial load
duke
parents:
diff changeset
330 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
331 }
a61af66fc99e Initial load
duke
parents:
diff changeset
332
6151
e778c29768e6 7152811: Issues in client compiler
never
parents: 3785
diff changeset
333 // Check for static/nonstatic mismatch
e778c29768e6 7152811: Issues in client compiler
never
parents: 3785
diff changeset
334 bool is_static = (bc == Bytecodes::_getstatic || bc == Bytecodes::_putstatic);
e778c29768e6 7152811: Issues in client compiler
never
parents: 3785
diff changeset
335 if (is_static != this->is_static()) {
e778c29768e6 7152811: Issues in client compiler
never
parents: 3785
diff changeset
336 return false;
e778c29768e6 7152811: Issues in client compiler
never
parents: 3785
diff changeset
337 }
e778c29768e6 7152811: Issues in client compiler
never
parents: 3785
diff changeset
338
e778c29768e6 7152811: Issues in client compiler
never
parents: 3785
diff changeset
339 // Get and put can have different accessibility rules
e778c29768e6 7152811: Issues in client compiler
never
parents: 3785
diff changeset
340 bool is_put = (bc == Bytecodes::_putfield || bc == Bytecodes::_putstatic);
e778c29768e6 7152811: Issues in client compiler
never
parents: 3785
diff changeset
341 if (is_put) {
e778c29768e6 7152811: Issues in client compiler
never
parents: 3785
diff changeset
342 if (_known_to_link_with_put == accessing_klass) {
e778c29768e6 7152811: Issues in client compiler
never
parents: 3785
diff changeset
343 return true;
e778c29768e6 7152811: Issues in client compiler
never
parents: 3785
diff changeset
344 }
6153
aa07e41a9f80 7160677: missing else in fix for 7152811
never
parents: 6151
diff changeset
345 } else {
6151
e778c29768e6 7152811: Issues in client compiler
never
parents: 3785
diff changeset
346 if (_known_to_link_with_get == accessing_klass) {
e778c29768e6 7152811: Issues in client compiler
never
parents: 3785
diff changeset
347 return true;
e778c29768e6 7152811: Issues in client compiler
never
parents: 3785
diff changeset
348 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
349 }
a61af66fc99e Initial load
duke
parents:
diff changeset
350
12264
b2e698d2276c 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 12190
diff changeset
351 fieldDescriptor result;
b2e698d2276c 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 12190
diff changeset
352 LinkResolver::resolve_field(result, _holder->get_instanceKlass(),
b2e698d2276c 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 12190
diff changeset
353 _name->get_symbol(), _signature->get_symbol(),
b2e698d2276c 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 12190
diff changeset
354 accessing_klass->get_Klass(), bc, true, false,
b2e698d2276c 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 12190
diff changeset
355 KILL_COMPILE_ON_FATAL_(false));
0
a61af66fc99e Initial load
duke
parents:
diff changeset
356
a61af66fc99e Initial load
duke
parents:
diff changeset
357 // update the hit-cache, unless there is a problem with memory scoping:
6151
e778c29768e6 7152811: Issues in client compiler
never
parents: 3785
diff changeset
358 if (accessing_klass->is_shared() || !is_shared()) {
e778c29768e6 7152811: Issues in client compiler
never
parents: 3785
diff changeset
359 if (is_put) {
e778c29768e6 7152811: Issues in client compiler
never
parents: 3785
diff changeset
360 _known_to_link_with_put = accessing_klass;
e778c29768e6 7152811: Issues in client compiler
never
parents: 3785
diff changeset
361 } else {
e778c29768e6 7152811: Issues in client compiler
never
parents: 3785
diff changeset
362 _known_to_link_with_get = accessing_klass;
e778c29768e6 7152811: Issues in client compiler
never
parents: 3785
diff changeset
363 }
e778c29768e6 7152811: Issues in client compiler
never
parents: 3785
diff changeset
364 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
365
a61af66fc99e Initial load
duke
parents:
diff changeset
366 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
367 }
a61af66fc99e Initial load
duke
parents:
diff changeset
368
a61af66fc99e Initial load
duke
parents:
diff changeset
369 // ------------------------------------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
370 // ciField::print
a61af66fc99e Initial load
duke
parents:
diff changeset
371 void ciField::print() {
7421
ad5dd04754ee 8005031: Some cleanup in c2 to prepare for incremental inlining support
roland
parents: 6725
diff changeset
372 tty->print("<ciField name=");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
373 _holder->print_name();
a61af66fc99e Initial load
duke
parents:
diff changeset
374 tty->print(".");
a61af66fc99e Initial load
duke
parents:
diff changeset
375 _name->print_symbol();
7421
ad5dd04754ee 8005031: Some cleanup in c2 to prepare for incremental inlining support
roland
parents: 6725
diff changeset
376 tty->print(" signature=");
ad5dd04754ee 8005031: Some cleanup in c2 to prepare for incremental inlining support
roland
parents: 6725
diff changeset
377 _signature->print_symbol();
0
a61af66fc99e Initial load
duke
parents:
diff changeset
378 tty->print(" offset=%d type=", _offset);
12190
edb5ab0f3fe5 8001107: @Stable annotation for constant folding of lazily evaluated variables
vlivanov
parents: 7421
diff changeset
379 if (_type != NULL)
edb5ab0f3fe5 8001107: @Stable annotation for constant folding of lazily evaluated variables
vlivanov
parents: 7421
diff changeset
380 _type->print_name();
edb5ab0f3fe5 8001107: @Stable annotation for constant folding of lazily evaluated variables
vlivanov
parents: 7421
diff changeset
381 else
edb5ab0f3fe5 8001107: @Stable annotation for constant folding of lazily evaluated variables
vlivanov
parents: 7421
diff changeset
382 tty->print("(reference)");
edb5ab0f3fe5 8001107: @Stable annotation for constant folding of lazily evaluated variables
vlivanov
parents: 7421
diff changeset
383 tty->print(" flags=%04x", flags().as_int());
0
a61af66fc99e Initial load
duke
parents:
diff changeset
384 tty->print(" is_constant=%s", bool_to_str(_is_constant));
1682
e5dfb3ccb88b 6969569: assert(is_static() && is_constant()) failed: illegal call to constant_value()
kvn
parents: 1552
diff changeset
385 if (_is_constant && is_static()) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
386 tty->print(" constant_value=");
a61af66fc99e Initial load
duke
parents:
diff changeset
387 _constant_value.print();
a61af66fc99e Initial load
duke
parents:
diff changeset
388 }
a61af66fc99e Initial load
duke
parents:
diff changeset
389 tty->print(">");
a61af66fc99e Initial load
duke
parents:
diff changeset
390 }
a61af66fc99e Initial load
duke
parents:
diff changeset
391
a61af66fc99e Initial load
duke
parents:
diff changeset
392 // ------------------------------------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
393 // ciField::print_name_on
a61af66fc99e Initial load
duke
parents:
diff changeset
394 //
a61af66fc99e Initial load
duke
parents:
diff changeset
395 // Print the name of this field
a61af66fc99e Initial load
duke
parents:
diff changeset
396 void ciField::print_name_on(outputStream* st) {
a61af66fc99e Initial load
duke
parents:
diff changeset
397 name()->print_symbol_on(st);
a61af66fc99e Initial load
duke
parents:
diff changeset
398 }