author | coleenp |
Tue, 14 May 2019 11:29:18 -0400 | |
changeset 54847 | 59ea39bb2809 |
parent 54786 | ebf733a324d4 |
child 58722 | cba8afa5cfed |
permissions | -rw-r--r-- |
1 | 1 |
/* |
54124
5d48ae032588
8219579: Remove redundant signature parsing from the verifier
hseigel
parents:
51997
diff
changeset
|
2 |
* Copyright (c) 1997, 2019, Oracle and/or its affiliates. All rights reserved. |
1 | 3 |
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
4 |
* |
|
5 |
* This code is free software; you can redistribute it and/or modify it |
|
6 |
* under the terms of the GNU General Public License version 2 only, as |
|
7 |
* published by the Free Software Foundation. |
|
8 |
* |
|
9 |
* This code is distributed in the hope that it will be useful, but WITHOUT |
|
10 |
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
|
11 |
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
|
12 |
* version 2 for more details (a copy is included in the LICENSE file that |
|
13 |
* accompanied this code). |
|
14 |
* |
|
15 |
* You should have received a copy of the GNU General Public License version |
|
16 |
* 2 along with this work; if not, write to the Free Software Foundation, |
|
17 |
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
|
18 |
* |
|
5547
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
5426
diff
changeset
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
5426
diff
changeset
|
20 |
* or visit www.oracle.com if you need additional information or have any |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
5426
diff
changeset
|
21 |
* questions. |
1 | 22 |
* |
23 |
*/ |
|
24 |
||
7397 | 25 |
#include "precompiled.hpp" |
26 |
#include "classfile/symbolTable.hpp" |
|
27 |
#include "classfile/systemDictionary.hpp" |
|
28 |
#include "memory/oopFactory.hpp" |
|
37248 | 29 |
#include "memory/resourceArea.hpp" |
54786 | 30 |
#include "memory/universe.hpp" |
7397 | 31 |
#include "oops/instanceKlass.hpp" |
32 |
#include "oops/oop.inline.hpp" |
|
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7397
diff
changeset
|
33 |
#include "oops/symbol.hpp" |
7397 | 34 |
#include "oops/typeArrayKlass.hpp" |
35 |
#include "runtime/signature.hpp" |
|
1 | 36 |
|
37 |
// Implementation of SignatureIterator |
|
38 |
||
39 |
// Signature syntax: |
|
40 |
// |
|
41 |
// Signature = "(" {Parameter} ")" ReturnType. |
|
42 |
// Parameter = FieldType. |
|
43 |
// ReturnType = FieldType | "V". |
|
44 |
// FieldType = "B" | "C" | "D" | "F" | "I" | "J" | "S" | "Z" | "L" ClassName ";" | "[" FieldType. |
|
45 |
// ClassName = string. |
|
46 |
||
47 |
||
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7397
diff
changeset
|
48 |
SignatureIterator::SignatureIterator(Symbol* signature) { |
1 | 49 |
_signature = signature; |
50 |
_parameter_index = 0; |
|
51 |
} |
|
52 |
||
53 |
void SignatureIterator::expect(char c) { |
|
51997
9ce37fa2e179
8209138: Symbol constructor uses u1 as the element type of its name argument
hseigel
parents:
49354
diff
changeset
|
54 |
if (_signature->char_at(_index) != c) fatal("expecting %c", c); |
1 | 55 |
_index++; |
56 |
} |
|
57 |
||
58 |
int SignatureIterator::parse_type() { |
|
59 |
// Note: This function could be simplified by using "return T_XXX_size;" |
|
60 |
// instead of the assignment and the break statements. However, it |
|
61 |
// seems that the product build for win32_i486 with MS VC++ 6.0 doesn't |
|
62 |
// work (stack underflow for some tests) - this seems to be a VC++ 6.0 |
|
63 |
// compiler bug (was problem - gri 4/27/2000). |
|
64 |
int size = -1; |
|
51997
9ce37fa2e179
8209138: Symbol constructor uses u1 as the element type of its name argument
hseigel
parents:
49354
diff
changeset
|
65 |
switch(_signature->char_at(_index)) { |
1 | 66 |
case 'B': do_byte (); if (_parameter_index < 0 ) _return_type = T_BYTE; |
67 |
_index++; size = T_BYTE_size ; break; |
|
68 |
case 'C': do_char (); if (_parameter_index < 0 ) _return_type = T_CHAR; |
|
69 |
_index++; size = T_CHAR_size ; break; |
|
70 |
case 'D': do_double(); if (_parameter_index < 0 ) _return_type = T_DOUBLE; |
|
71 |
_index++; size = T_DOUBLE_size ; break; |
|
72 |
case 'F': do_float (); if (_parameter_index < 0 ) _return_type = T_FLOAT; |
|
73 |
_index++; size = T_FLOAT_size ; break; |
|
74 |
case 'I': do_int (); if (_parameter_index < 0 ) _return_type = T_INT; |
|
75 |
_index++; size = T_INT_size ; break; |
|
76 |
case 'J': do_long (); if (_parameter_index < 0 ) _return_type = T_LONG; |
|
77 |
_index++; size = T_LONG_size ; break; |
|
78 |
case 'S': do_short (); if (_parameter_index < 0 ) _return_type = T_SHORT; |
|
79 |
_index++; size = T_SHORT_size ; break; |
|
80 |
case 'Z': do_bool (); if (_parameter_index < 0 ) _return_type = T_BOOLEAN; |
|
81 |
_index++; size = T_BOOLEAN_size; break; |
|
82 |
case 'V': do_void (); if (_parameter_index < 0 ) _return_type = T_VOID; |
|
83 |
_index++; size = T_VOID_size; ; break; |
|
84 |
case 'L': |
|
85 |
{ int begin = ++_index; |
|
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7397
diff
changeset
|
86 |
Symbol* sig = _signature; |
51997
9ce37fa2e179
8209138: Symbol constructor uses u1 as the element type of its name argument
hseigel
parents:
49354
diff
changeset
|
87 |
while (sig->char_at(_index++) != ';') ; |
1 | 88 |
do_object(begin, _index); |
89 |
} |
|
90 |
if (_parameter_index < 0 ) _return_type = T_OBJECT; |
|
91 |
size = T_OBJECT_size; |
|
92 |
break; |
|
93 |
case '[': |
|
94 |
{ int begin = ++_index; |
|
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7397
diff
changeset
|
95 |
Symbol* sig = _signature; |
51997
9ce37fa2e179
8209138: Symbol constructor uses u1 as the element type of its name argument
hseigel
parents:
49354
diff
changeset
|
96 |
while (sig->char_at(_index) == '[') { |
1 | 97 |
_index++; |
98 |
} |
|
51997
9ce37fa2e179
8209138: Symbol constructor uses u1 as the element type of its name argument
hseigel
parents:
49354
diff
changeset
|
99 |
if (sig->char_at(_index) == 'L') { |
9ce37fa2e179
8209138: Symbol constructor uses u1 as the element type of its name argument
hseigel
parents:
49354
diff
changeset
|
100 |
while (sig->char_at(_index++) != ';') ; |
1 | 101 |
} else { |
102 |
_index++; |
|
103 |
} |
|
104 |
do_array(begin, _index); |
|
105 |
if (_parameter_index < 0 ) _return_type = T_ARRAY; |
|
106 |
} |
|
107 |
size = T_ARRAY_size; |
|
108 |
break; |
|
109 |
default: |
|
110 |
ShouldNotReachHere(); |
|
111 |
break; |
|
112 |
} |
|
113 |
assert(size >= 0, "size must be set"); |
|
114 |
return size; |
|
115 |
} |
|
116 |
||
117 |
||
118 |
void SignatureIterator::check_signature_end() { |
|
119 |
if (_index < _signature->utf8_length()) { |
|
120 |
tty->print_cr("too many chars in signature"); |
|
121 |
_signature->print_value_on(tty); |
|
122 |
tty->print_cr(" @ %d", _index); |
|
123 |
} |
|
124 |
} |
|
125 |
||
126 |
||
127 |
void SignatureIterator::iterate_parameters() { |
|
128 |
// Parse parameters |
|
129 |
_index = 0; |
|
130 |
_parameter_index = 0; |
|
131 |
expect('('); |
|
51997
9ce37fa2e179
8209138: Symbol constructor uses u1 as the element type of its name argument
hseigel
parents:
49354
diff
changeset
|
132 |
while (_signature->char_at(_index) != ')') _parameter_index += parse_type(); |
1 | 133 |
expect(')'); |
134 |
_parameter_index = 0; |
|
135 |
} |
|
136 |
||
22551 | 137 |
// Optimized version of iterate_parameters when fingerprint is known |
1 | 138 |
void SignatureIterator::iterate_parameters( uint64_t fingerprint ) { |
139 |
uint64_t saved_fingerprint = fingerprint; |
|
140 |
||
141 |
// Check for too many arguments |
|
27471 | 142 |
if (fingerprint == (uint64_t)CONST64(-1)) { |
1 | 143 |
SignatureIterator::iterate_parameters(); |
144 |
return; |
|
145 |
} |
|
146 |
||
147 |
assert(fingerprint, "Fingerprint should not be 0"); |
|
148 |
||
149 |
_parameter_index = 0; |
|
150 |
fingerprint = fingerprint >> (static_feature_size + result_feature_size); |
|
151 |
while ( 1 ) { |
|
152 |
switch ( fingerprint & parameter_feature_mask ) { |
|
153 |
case bool_parm: |
|
154 |
do_bool(); |
|
155 |
_parameter_index += T_BOOLEAN_size; |
|
156 |
break; |
|
157 |
case byte_parm: |
|
158 |
do_byte(); |
|
159 |
_parameter_index += T_BYTE_size; |
|
160 |
break; |
|
161 |
case char_parm: |
|
162 |
do_char(); |
|
163 |
_parameter_index += T_CHAR_size; |
|
164 |
break; |
|
165 |
case short_parm: |
|
166 |
do_short(); |
|
167 |
_parameter_index += T_SHORT_size; |
|
168 |
break; |
|
169 |
case int_parm: |
|
170 |
do_int(); |
|
171 |
_parameter_index += T_INT_size; |
|
172 |
break; |
|
173 |
case obj_parm: |
|
174 |
do_object(0, 0); |
|
175 |
_parameter_index += T_OBJECT_size; |
|
176 |
break; |
|
177 |
case long_parm: |
|
178 |
do_long(); |
|
179 |
_parameter_index += T_LONG_size; |
|
180 |
break; |
|
181 |
case float_parm: |
|
182 |
do_float(); |
|
183 |
_parameter_index += T_FLOAT_size; |
|
184 |
break; |
|
185 |
case double_parm: |
|
186 |
do_double(); |
|
187 |
_parameter_index += T_DOUBLE_size; |
|
188 |
break; |
|
189 |
case done_parm: |
|
190 |
return; |
|
191 |
default: |
|
33148
68fa8b6c4340
8042893: compiler: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
33105
diff
changeset
|
192 |
tty->print_cr("*** parameter is " UINT64_FORMAT, fingerprint & parameter_feature_mask); |
1 | 193 |
tty->print_cr("*** fingerprint is " PTR64_FORMAT, saved_fingerprint); |
194 |
ShouldNotReachHere(); |
|
195 |
break; |
|
196 |
} |
|
197 |
fingerprint >>= parameter_feature_size; |
|
198 |
} |
|
199 |
} |
|
200 |
||
201 |
||
202 |
void SignatureIterator::iterate_returntype() { |
|
203 |
// Ignore parameters |
|
204 |
_index = 0; |
|
205 |
expect('('); |
|
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7397
diff
changeset
|
206 |
Symbol* sig = _signature; |
41545 | 207 |
// Need to skip over each type in the signature's argument list until a |
208 |
// closing ')' is found., then get the return type. We cannot just scan |
|
209 |
// for the first ')' because ')' is a legal character in a type name. |
|
51997
9ce37fa2e179
8209138: Symbol constructor uses u1 as the element type of its name argument
hseigel
parents:
49354
diff
changeset
|
210 |
while (sig->char_at(_index) != ')') { |
9ce37fa2e179
8209138: Symbol constructor uses u1 as the element type of its name argument
hseigel
parents:
49354
diff
changeset
|
211 |
switch(sig->char_at(_index)) { |
41545 | 212 |
case 'B': |
213 |
case 'C': |
|
214 |
case 'D': |
|
215 |
case 'F': |
|
216 |
case 'I': |
|
217 |
case 'J': |
|
218 |
case 'S': |
|
219 |
case 'Z': |
|
220 |
case 'V': |
|
221 |
{ |
|
222 |
_index++; |
|
223 |
} |
|
224 |
break; |
|
225 |
case 'L': |
|
226 |
{ |
|
51997
9ce37fa2e179
8209138: Symbol constructor uses u1 as the element type of its name argument
hseigel
parents:
49354
diff
changeset
|
227 |
while (sig->char_at(_index++) != ';') ; |
41545 | 228 |
} |
229 |
break; |
|
230 |
case '[': |
|
231 |
{ |
|
54133
829bf950287e
8220366: Optimize Symbol handling in ClassVerifier and SignatureStream
redestad
parents:
54124
diff
changeset
|
232 |
while (sig->char_at(++_index) == '[') ; |
51997
9ce37fa2e179
8209138: Symbol constructor uses u1 as the element type of its name argument
hseigel
parents:
49354
diff
changeset
|
233 |
if (sig->char_at(_index) == 'L') { |
9ce37fa2e179
8209138: Symbol constructor uses u1 as the element type of its name argument
hseigel
parents:
49354
diff
changeset
|
234 |
while (sig->char_at(_index++) != ';') ; |
41545 | 235 |
} else { |
236 |
_index++; |
|
237 |
} |
|
238 |
} |
|
239 |
break; |
|
240 |
default: |
|
241 |
ShouldNotReachHere(); |
|
242 |
break; |
|
243 |
} |
|
244 |
} |
|
1 | 245 |
expect(')'); |
246 |
// Parse return type |
|
247 |
_parameter_index = -1; |
|
248 |
parse_type(); |
|
249 |
check_signature_end(); |
|
250 |
_parameter_index = 0; |
|
251 |
} |
|
252 |
||
253 |
||
254 |
void SignatureIterator::iterate() { |
|
255 |
// Parse parameters |
|
256 |
_parameter_index = 0; |
|
257 |
_index = 0; |
|
258 |
expect('('); |
|
51997
9ce37fa2e179
8209138: Symbol constructor uses u1 as the element type of its name argument
hseigel
parents:
49354
diff
changeset
|
259 |
while (_signature->char_at(_index) != ')') _parameter_index += parse_type(); |
1 | 260 |
expect(')'); |
261 |
// Parse return type |
|
262 |
_parameter_index = -1; |
|
263 |
parse_type(); |
|
264 |
check_signature_end(); |
|
265 |
_parameter_index = 0; |
|
266 |
} |
|
267 |
||
268 |
||
269 |
// Implementation of SignatureStream |
|
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7397
diff
changeset
|
270 |
SignatureStream::SignatureStream(Symbol* signature, bool is_method) : |
54133
829bf950287e
8220366: Optimize Symbol handling in ClassVerifier and SignatureStream
redestad
parents:
54124
diff
changeset
|
271 |
_signature(signature), _at_return_type(false), _previous_name(NULL), _names(NULL) { |
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7397
diff
changeset
|
272 |
_begin = _end = (is_method ? 1 : 0); // skip first '(' in method signatures |
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7397
diff
changeset
|
273 |
next(); |
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7397
diff
changeset
|
274 |
} |
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7397
diff
changeset
|
275 |
|
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7397
diff
changeset
|
276 |
SignatureStream::~SignatureStream() { |
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7397
diff
changeset
|
277 |
// decrement refcount for names created during signature parsing |
54133
829bf950287e
8220366: Optimize Symbol handling in ClassVerifier and SignatureStream
redestad
parents:
54124
diff
changeset
|
278 |
if (_names != NULL) { |
829bf950287e
8220366: Optimize Symbol handling in ClassVerifier and SignatureStream
redestad
parents:
54124
diff
changeset
|
279 |
for (int i = 0; i < _names->length(); i++) { |
829bf950287e
8220366: Optimize Symbol handling in ClassVerifier and SignatureStream
redestad
parents:
54124
diff
changeset
|
280 |
_names->at(i)->decrement_refcount(); |
829bf950287e
8220366: Optimize Symbol handling in ClassVerifier and SignatureStream
redestad
parents:
54124
diff
changeset
|
281 |
} |
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7397
diff
changeset
|
282 |
} |
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7397
diff
changeset
|
283 |
} |
1 | 284 |
|
285 |
bool SignatureStream::is_done() const { |
|
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7397
diff
changeset
|
286 |
return _end > _signature->utf8_length(); |
1 | 287 |
} |
288 |
||
289 |
||
290 |
void SignatureStream::next_non_primitive(int t) { |
|
291 |
switch (t) { |
|
292 |
case 'L': { |
|
293 |
_type = T_OBJECT; |
|
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7397
diff
changeset
|
294 |
Symbol* sig = _signature; |
51997
9ce37fa2e179
8209138: Symbol constructor uses u1 as the element type of its name argument
hseigel
parents:
49354
diff
changeset
|
295 |
while (sig->char_at(_end++) != ';'); |
1 | 296 |
break; |
297 |
} |
|
298 |
case '[': { |
|
299 |
_type = T_ARRAY; |
|
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7397
diff
changeset
|
300 |
Symbol* sig = _signature; |
51997
9ce37fa2e179
8209138: Symbol constructor uses u1 as the element type of its name argument
hseigel
parents:
49354
diff
changeset
|
301 |
char c = sig->char_at(_end); |
9ce37fa2e179
8209138: Symbol constructor uses u1 as the element type of its name argument
hseigel
parents:
49354
diff
changeset
|
302 |
while ('0' <= c && c <= '9') c = sig->char_at(_end++); |
9ce37fa2e179
8209138: Symbol constructor uses u1 as the element type of its name argument
hseigel
parents:
49354
diff
changeset
|
303 |
while (sig->char_at(_end) == '[') { |
1 | 304 |
_end++; |
51997
9ce37fa2e179
8209138: Symbol constructor uses u1 as the element type of its name argument
hseigel
parents:
49354
diff
changeset
|
305 |
c = sig->char_at(_end); |
9ce37fa2e179
8209138: Symbol constructor uses u1 as the element type of its name argument
hseigel
parents:
49354
diff
changeset
|
306 |
while ('0' <= c && c <= '9') c = sig->char_at(_end++); |
1 | 307 |
} |
51997
9ce37fa2e179
8209138: Symbol constructor uses u1 as the element type of its name argument
hseigel
parents:
49354
diff
changeset
|
308 |
switch(sig->char_at(_end)) { |
1 | 309 |
case 'B': |
310 |
case 'C': |
|
311 |
case 'D': |
|
312 |
case 'F': |
|
313 |
case 'I': |
|
314 |
case 'J': |
|
315 |
case 'S': |
|
316 |
case 'Z':_end++; break; |
|
317 |
default: { |
|
51997
9ce37fa2e179
8209138: Symbol constructor uses u1 as the element type of its name argument
hseigel
parents:
49354
diff
changeset
|
318 |
while (sig->char_at(_end++) != ';'); |
1 | 319 |
break; |
320 |
} |
|
321 |
} |
|
322 |
break; |
|
323 |
} |
|
324 |
case ')': _end++; next(); _at_return_type = true; break; |
|
325 |
default : ShouldNotReachHere(); |
|
326 |
} |
|
327 |
} |
|
328 |
||
329 |
||
330 |
bool SignatureStream::is_object() const { |
|
331 |
return _type == T_OBJECT |
|
332 |
|| _type == T_ARRAY; |
|
333 |
} |
|
334 |
||
335 |
bool SignatureStream::is_array() const { |
|
336 |
return _type == T_ARRAY; |
|
337 |
} |
|
338 |
||
54847
59ea39bb2809
8223657: Remove unused THREAD argument from SymbolTable functions
coleenp
parents:
54786
diff
changeset
|
339 |
Symbol* SignatureStream::as_symbol() { |
1 | 340 |
// Create a symbol from for string _begin _end |
341 |
int begin = _begin; |
|
342 |
int end = _end; |
|
343 |
||
51997
9ce37fa2e179
8209138: Symbol constructor uses u1 as the element type of its name argument
hseigel
parents:
49354
diff
changeset
|
344 |
if ( _signature->char_at(_begin) == 'L' |
9ce37fa2e179
8209138: Symbol constructor uses u1 as the element type of its name argument
hseigel
parents:
49354
diff
changeset
|
345 |
&& _signature->char_at(_end-1) == ';') { |
1 | 346 |
begin++; |
347 |
end--; |
|
348 |
} |
|
349 |
||
54133
829bf950287e
8220366: Optimize Symbol handling in ClassVerifier and SignatureStream
redestad
parents:
54124
diff
changeset
|
350 |
const char* symbol_chars = (const char*)_signature->base() + begin; |
829bf950287e
8220366: Optimize Symbol handling in ClassVerifier and SignatureStream
redestad
parents:
54124
diff
changeset
|
351 |
int len = end - begin; |
829bf950287e
8220366: Optimize Symbol handling in ClassVerifier and SignatureStream
redestad
parents:
54124
diff
changeset
|
352 |
|
829bf950287e
8220366: Optimize Symbol handling in ClassVerifier and SignatureStream
redestad
parents:
54124
diff
changeset
|
353 |
// Quick check for common symbols in signatures |
829bf950287e
8220366: Optimize Symbol handling in ClassVerifier and SignatureStream
redestad
parents:
54124
diff
changeset
|
354 |
assert((vmSymbols::java_lang_String()->utf8_length() == 16 && vmSymbols::java_lang_Object()->utf8_length() == 16), "sanity"); |
829bf950287e
8220366: Optimize Symbol handling in ClassVerifier and SignatureStream
redestad
parents:
54124
diff
changeset
|
355 |
if (len == 16 && |
829bf950287e
8220366: Optimize Symbol handling in ClassVerifier and SignatureStream
redestad
parents:
54124
diff
changeset
|
356 |
strncmp(symbol_chars, "java/lang/", 10) == 0) { |
829bf950287e
8220366: Optimize Symbol handling in ClassVerifier and SignatureStream
redestad
parents:
54124
diff
changeset
|
357 |
if (strncmp("String", symbol_chars + 10, 6) == 0) { |
829bf950287e
8220366: Optimize Symbol handling in ClassVerifier and SignatureStream
redestad
parents:
54124
diff
changeset
|
358 |
return vmSymbols::java_lang_String(); |
829bf950287e
8220366: Optimize Symbol handling in ClassVerifier and SignatureStream
redestad
parents:
54124
diff
changeset
|
359 |
} else if (strncmp("Object", symbol_chars + 10, 6) == 0) { |
829bf950287e
8220366: Optimize Symbol handling in ClassVerifier and SignatureStream
redestad
parents:
54124
diff
changeset
|
360 |
return vmSymbols::java_lang_Object(); |
829bf950287e
8220366: Optimize Symbol handling in ClassVerifier and SignatureStream
redestad
parents:
54124
diff
changeset
|
361 |
} |
829bf950287e
8220366: Optimize Symbol handling in ClassVerifier and SignatureStream
redestad
parents:
54124
diff
changeset
|
362 |
} |
829bf950287e
8220366: Optimize Symbol handling in ClassVerifier and SignatureStream
redestad
parents:
54124
diff
changeset
|
363 |
|
829bf950287e
8220366: Optimize Symbol handling in ClassVerifier and SignatureStream
redestad
parents:
54124
diff
changeset
|
364 |
Symbol* name = _previous_name; |
829bf950287e
8220366: Optimize Symbol handling in ClassVerifier and SignatureStream
redestad
parents:
54124
diff
changeset
|
365 |
if (name != NULL && name->equals(symbol_chars, len)) { |
829bf950287e
8220366: Optimize Symbol handling in ClassVerifier and SignatureStream
redestad
parents:
54124
diff
changeset
|
366 |
return name; |
829bf950287e
8220366: Optimize Symbol handling in ClassVerifier and SignatureStream
redestad
parents:
54124
diff
changeset
|
367 |
} |
829bf950287e
8220366: Optimize Symbol handling in ClassVerifier and SignatureStream
redestad
parents:
54124
diff
changeset
|
368 |
|
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7397
diff
changeset
|
369 |
// Save names for cleaning up reference count at the end of |
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7397
diff
changeset
|
370 |
// SignatureStream scope. |
54847
59ea39bb2809
8223657: Remove unused THREAD argument from SymbolTable functions
coleenp
parents:
54786
diff
changeset
|
371 |
name = SymbolTable::new_symbol(symbol_chars, len); |
54133
829bf950287e
8220366: Optimize Symbol handling in ClassVerifier and SignatureStream
redestad
parents:
54124
diff
changeset
|
372 |
if (!name->is_permanent()) { |
829bf950287e
8220366: Optimize Symbol handling in ClassVerifier and SignatureStream
redestad
parents:
54124
diff
changeset
|
373 |
if (_names == NULL) { |
829bf950287e
8220366: Optimize Symbol handling in ClassVerifier and SignatureStream
redestad
parents:
54124
diff
changeset
|
374 |
_names = new GrowableArray<Symbol*>(10); |
829bf950287e
8220366: Optimize Symbol handling in ClassVerifier and SignatureStream
redestad
parents:
54124
diff
changeset
|
375 |
} |
829bf950287e
8220366: Optimize Symbol handling in ClassVerifier and SignatureStream
redestad
parents:
54124
diff
changeset
|
376 |
_names->push(name); // save new symbol for decrementing later |
829bf950287e
8220366: Optimize Symbol handling in ClassVerifier and SignatureStream
redestad
parents:
54124
diff
changeset
|
377 |
} |
829bf950287e
8220366: Optimize Symbol handling in ClassVerifier and SignatureStream
redestad
parents:
54124
diff
changeset
|
378 |
_previous_name = name; |
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7397
diff
changeset
|
379 |
return name; |
1 | 380 |
} |
381 |
||
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
8921
diff
changeset
|
382 |
Klass* SignatureStream::as_klass(Handle class_loader, Handle protection_domain, |
54847
59ea39bb2809
8223657: Remove unused THREAD argument from SymbolTable functions
coleenp
parents:
54786
diff
changeset
|
383 |
FailureMode failure_mode, TRAPS) { |
5421
e294db54fc0d
6939196: method handle signatures off the boot class path get linkage errors
jrose
parents:
1
diff
changeset
|
384 |
if (!is_object()) return NULL; |
54847
59ea39bb2809
8223657: Remove unused THREAD argument from SymbolTable functions
coleenp
parents:
54786
diff
changeset
|
385 |
Symbol* name = as_symbol(); |
5421
e294db54fc0d
6939196: method handle signatures off the boot class path get linkage errors
jrose
parents:
1
diff
changeset
|
386 |
if (failure_mode == ReturnNull) { |
e294db54fc0d
6939196: method handle signatures off the boot class path get linkage errors
jrose
parents:
1
diff
changeset
|
387 |
return SystemDictionary::resolve_or_null(name, class_loader, protection_domain, THREAD); |
e294db54fc0d
6939196: method handle signatures off the boot class path get linkage errors
jrose
parents:
1
diff
changeset
|
388 |
} else { |
e294db54fc0d
6939196: method handle signatures off the boot class path get linkage errors
jrose
parents:
1
diff
changeset
|
389 |
bool throw_error = (failure_mode == NCDFError); |
e294db54fc0d
6939196: method handle signatures off the boot class path get linkage errors
jrose
parents:
1
diff
changeset
|
390 |
return SystemDictionary::resolve_or_fail(name, class_loader, protection_domain, throw_error, THREAD); |
e294db54fc0d
6939196: method handle signatures off the boot class path get linkage errors
jrose
parents:
1
diff
changeset
|
391 |
} |
e294db54fc0d
6939196: method handle signatures off the boot class path get linkage errors
jrose
parents:
1
diff
changeset
|
392 |
} |
e294db54fc0d
6939196: method handle signatures off the boot class path get linkage errors
jrose
parents:
1
diff
changeset
|
393 |
|
e294db54fc0d
6939196: method handle signatures off the boot class path get linkage errors
jrose
parents:
1
diff
changeset
|
394 |
oop SignatureStream::as_java_mirror(Handle class_loader, Handle protection_domain, |
e294db54fc0d
6939196: method handle signatures off the boot class path get linkage errors
jrose
parents:
1
diff
changeset
|
395 |
FailureMode failure_mode, TRAPS) { |
48836
423bcbb288ff
8196601: IllegalAccessError: cannot access class jdk.jfr.internal.handlers.EventHandler
lfoltan
parents:
48826
diff
changeset
|
396 |
if (!is_object()) |
423bcbb288ff
8196601: IllegalAccessError: cannot access class jdk.jfr.internal.handlers.EventHandler
lfoltan
parents:
48826
diff
changeset
|
397 |
return Universe::java_mirror(type()); |
423bcbb288ff
8196601: IllegalAccessError: cannot access class jdk.jfr.internal.handlers.EventHandler
lfoltan
parents:
48826
diff
changeset
|
398 |
Klass* klass = as_klass(class_loader, protection_domain, failure_mode, CHECK_NULL); |
423bcbb288ff
8196601: IllegalAccessError: cannot access class jdk.jfr.internal.handlers.EventHandler
lfoltan
parents:
48826
diff
changeset
|
399 |
if (klass == NULL) return NULL; |
423bcbb288ff
8196601: IllegalAccessError: cannot access class jdk.jfr.internal.handlers.EventHandler
lfoltan
parents:
48826
diff
changeset
|
400 |
return klass->java_mirror(); |
5421
e294db54fc0d
6939196: method handle signatures off the boot class path get linkage errors
jrose
parents:
1
diff
changeset
|
401 |
} |
1 | 402 |
|
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7397
diff
changeset
|
403 |
Symbol* SignatureStream::as_symbol_or_null() { |
1 | 404 |
// Create a symbol from for string _begin _end |
405 |
ResourceMark rm; |
|
406 |
||
407 |
int begin = _begin; |
|
408 |
int end = _end; |
|
409 |
||
51997
9ce37fa2e179
8209138: Symbol constructor uses u1 as the element type of its name argument
hseigel
parents:
49354
diff
changeset
|
410 |
if ( _signature->char_at(_begin) == 'L' |
9ce37fa2e179
8209138: Symbol constructor uses u1 as the element type of its name argument
hseigel
parents:
49354
diff
changeset
|
411 |
&& _signature->char_at(_end-1) == ';') { |
1 | 412 |
begin++; |
413 |
end--; |
|
414 |
} |
|
415 |
||
416 |
char* buffer = NEW_RESOURCE_ARRAY(char, end - begin); |
|
417 |
for (int index = begin; index < end; index++) { |
|
51997
9ce37fa2e179
8209138: Symbol constructor uses u1 as the element type of its name argument
hseigel
parents:
49354
diff
changeset
|
418 |
buffer[index - begin] = _signature->char_at(index); |
1 | 419 |
} |
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7397
diff
changeset
|
420 |
Symbol* result = SymbolTable::probe(buffer, end - begin); |
1 | 421 |
return result; |
422 |
} |
|
423 |
||
20702
bbe0fcde6e13
8023657: New type profiling points: arguments to call
roland
parents:
14488
diff
changeset
|
424 |
int SignatureStream::reference_parameter_count() { |
bbe0fcde6e13
8023657: New type profiling points: arguments to call
roland
parents:
14488
diff
changeset
|
425 |
int args_count = 0; |
bbe0fcde6e13
8023657: New type profiling points: arguments to call
roland
parents:
14488
diff
changeset
|
426 |
for ( ; !at_return_type(); next()) { |
bbe0fcde6e13
8023657: New type profiling points: arguments to call
roland
parents:
14488
diff
changeset
|
427 |
if (is_object()) { |
bbe0fcde6e13
8023657: New type profiling points: arguments to call
roland
parents:
14488
diff
changeset
|
428 |
args_count++; |
bbe0fcde6e13
8023657: New type profiling points: arguments to call
roland
parents:
14488
diff
changeset
|
429 |
} |
bbe0fcde6e13
8023657: New type profiling points: arguments to call
roland
parents:
14488
diff
changeset
|
430 |
} |
bbe0fcde6e13
8023657: New type profiling points: arguments to call
roland
parents:
14488
diff
changeset
|
431 |
return args_count; |
bbe0fcde6e13
8023657: New type profiling points: arguments to call
roland
parents:
14488
diff
changeset
|
432 |
} |
bbe0fcde6e13
8023657: New type profiling points: arguments to call
roland
parents:
14488
diff
changeset
|
433 |
|
54124
5d48ae032588
8219579: Remove redundant signature parsing from the verifier
hseigel
parents:
51997
diff
changeset
|
434 |
#ifdef ASSERT |
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7397
diff
changeset
|
435 |
bool SignatureVerifier::is_valid_method_signature(Symbol* sig) { |
1 | 436 |
const char* method_sig = (const char*)sig->bytes(); |
437 |
ssize_t len = sig->utf8_length(); |
|
438 |
ssize_t index = 0; |
|
439 |
if (method_sig != NULL && len > 1 && method_sig[index] == '(') { |
|
440 |
++index; |
|
441 |
while (index < len && method_sig[index] != ')') { |
|
442 |
ssize_t res = is_valid_type(&method_sig[index], len - index); |
|
443 |
if (res == -1) { |
|
444 |
return false; |
|
445 |
} else { |
|
446 |
index += res; |
|
447 |
} |
|
448 |
} |
|
449 |
if (index < len && method_sig[index] == ')') { |
|
450 |
// check the return type |
|
451 |
++index; |
|
452 |
return (is_valid_type(&method_sig[index], len - index) == (len - index)); |
|
453 |
} |
|
454 |
} |
|
455 |
return false; |
|
456 |
} |
|
457 |
||
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7397
diff
changeset
|
458 |
bool SignatureVerifier::is_valid_type_signature(Symbol* sig) { |
1 | 459 |
const char* type_sig = (const char*)sig->bytes(); |
460 |
ssize_t len = sig->utf8_length(); |
|
461 |
return (type_sig != NULL && len >= 1 && |
|
462 |
(is_valid_type(type_sig, len) == len)); |
|
463 |
} |
|
464 |
||
465 |
// Checks to see if the type (not to go beyond 'limit') refers to a valid type. |
|
466 |
// Returns -1 if it is not, or the index of the next character that is not part |
|
467 |
// of the type. The type encoding may end before 'limit' and that's ok. |
|
468 |
ssize_t SignatureVerifier::is_valid_type(const char* type, ssize_t limit) { |
|
469 |
ssize_t index = 0; |
|
470 |
||
471 |
// Iterate over any number of array dimensions |
|
472 |
while (index < limit && type[index] == '[') ++index; |
|
473 |
if (index >= limit) { |
|
474 |
return -1; |
|
475 |
} |
|
476 |
switch (type[index]) { |
|
477 |
case 'B': case 'C': case 'D': case 'F': case 'I': |
|
478 |
case 'J': case 'S': case 'Z': case 'V': |
|
479 |
return index + 1; |
|
480 |
case 'L': |
|
481 |
for (index = index + 1; index < limit; ++index) { |
|
482 |
char c = type[index]; |
|
54133
829bf950287e
8220366: Optimize Symbol handling in ClassVerifier and SignatureStream
redestad
parents:
54124
diff
changeset
|
483 |
switch (c) { |
829bf950287e
8220366: Optimize Symbol handling in ClassVerifier and SignatureStream
redestad
parents:
54124
diff
changeset
|
484 |
case ';': |
829bf950287e
8220366: Optimize Symbol handling in ClassVerifier and SignatureStream
redestad
parents:
54124
diff
changeset
|
485 |
return index + 1; |
829bf950287e
8220366: Optimize Symbol handling in ClassVerifier and SignatureStream
redestad
parents:
54124
diff
changeset
|
486 |
case '\0': case '.': case '[': |
829bf950287e
8220366: Optimize Symbol handling in ClassVerifier and SignatureStream
redestad
parents:
54124
diff
changeset
|
487 |
return -1; |
829bf950287e
8220366: Optimize Symbol handling in ClassVerifier and SignatureStream
redestad
parents:
54124
diff
changeset
|
488 |
default: ; // fall through |
1 | 489 |
} |
490 |
} |
|
491 |
// fall through |
|
492 |
default: ; // fall through |
|
493 |
} |
|
494 |
return -1; |
|
495 |
} |
|
54124
5d48ae032588
8219579: Remove redundant signature parsing from the verifier
hseigel
parents:
51997
diff
changeset
|
496 |
#endif // ASSERT |