author | bobv |
Tue, 29 Aug 2017 15:53:04 -0400 | |
changeset 47100 | 17953cf82d51 |
parent 45766 | 4b5557c9b656 |
permissions | -rw-r--r-- |
29337
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
1 |
/* |
39254
fb4492288b01
8156760: VM crashes if -XX:-ReduceInitialCardMarks is set
thartmann
parents:
32576
diff
changeset
|
2 |
* Copyright (c) 2016, Oracle and/or its affiliates. All rights reserved. |
29337
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
3 |
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
4 |
* |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
5 |
* This code is free software; you can redistribute it and/or modify it |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
6 |
* under the terms of the GNU General Public License version 2 only, as |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
7 |
* published by the Free Software Foundation. |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
8 |
* |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
9 |
* This code is distributed in the hope that it will be useful, but WITHOUT |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
10 |
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
11 |
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
12 |
* version 2 for more details (a copy is included in the LICENSE file that |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
13 |
* accompanied this code). |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
14 |
* |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
15 |
* You should have received a copy of the GNU General Public License version |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
16 |
* 2 along with this work; if not, write to the Free Software Foundation, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
17 |
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
18 |
* |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
20 |
* or visit www.oracle.com if you need additional information or have any |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
21 |
* questions. |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
22 |
* |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
23 |
*/ |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
24 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
25 |
#include "precompiled.hpp" |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
26 |
#include "opto/arraycopynode.hpp" |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
27 |
#include "opto/graphKit.hpp" |
45427
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
28 |
#include "runtime/sharedRuntime.hpp" |
29337
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
29 |
|
42086
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
40365
diff
changeset
|
30 |
ArrayCopyNode::ArrayCopyNode(Compile* C, bool alloc_tightly_coupled, bool has_negative_length_guard) |
29337
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
31 |
: CallNode(arraycopy_type(), NULL, TypeRawPtr::BOTTOM), |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
32 |
_alloc_tightly_coupled(alloc_tightly_coupled), |
42086
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
40365
diff
changeset
|
33 |
_has_negative_length_guard(has_negative_length_guard), |
29337
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
34 |
_kind(None), |
30629
b6e5ad2f18d5
8076188: Optimize arraycopy out for non escaping destination
roland
parents:
29360
diff
changeset
|
35 |
_arguments_validated(false), |
b6e5ad2f18d5
8076188: Optimize arraycopy out for non escaping destination
roland
parents:
29360
diff
changeset
|
36 |
_src_type(TypeOopPtr::BOTTOM), |
b6e5ad2f18d5
8076188: Optimize arraycopy out for non escaping destination
roland
parents:
29360
diff
changeset
|
37 |
_dest_type(TypeOopPtr::BOTTOM) { |
29337
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
38 |
init_class_id(Class_ArrayCopy); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
39 |
init_flags(Flag_is_macro); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
40 |
C->add_macro_node(this); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
41 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
42 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
43 |
uint ArrayCopyNode::size_of() const { return sizeof(*this); } |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
44 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
45 |
ArrayCopyNode* ArrayCopyNode::make(GraphKit* kit, bool may_throw, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
46 |
Node* src, Node* src_offset, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
47 |
Node* dest, Node* dest_offset, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
48 |
Node* length, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
49 |
bool alloc_tightly_coupled, |
42086
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
40365
diff
changeset
|
50 |
bool has_negative_length_guard, |
29337
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
51 |
Node* src_klass, Node* dest_klass, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
52 |
Node* src_length, Node* dest_length) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
53 |
|
42086
feac795f345d
8159611: C2: ArrayCopy elimination skips required parameter checks
simonis
parents:
40365
diff
changeset
|
54 |
ArrayCopyNode* ac = new ArrayCopyNode(kit->C, alloc_tightly_coupled, has_negative_length_guard); |
29337
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
55 |
Node* prev_mem = kit->set_predefined_input_for_runtime_call(ac); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
56 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
57 |
ac->init_req(ArrayCopyNode::Src, src); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
58 |
ac->init_req(ArrayCopyNode::SrcPos, src_offset); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
59 |
ac->init_req(ArrayCopyNode::Dest, dest); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
60 |
ac->init_req(ArrayCopyNode::DestPos, dest_offset); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
61 |
ac->init_req(ArrayCopyNode::Length, length); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
62 |
ac->init_req(ArrayCopyNode::SrcLen, src_length); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
63 |
ac->init_req(ArrayCopyNode::DestLen, dest_length); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
64 |
ac->init_req(ArrayCopyNode::SrcKlass, src_klass); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
65 |
ac->init_req(ArrayCopyNode::DestKlass, dest_klass); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
66 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
67 |
if (may_throw) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
68 |
ac->set_req(TypeFunc::I_O , kit->i_o()); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
69 |
kit->add_safepoint_edges(ac, false); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
70 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
71 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
72 |
return ac; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
73 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
74 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
75 |
void ArrayCopyNode::connect_outputs(GraphKit* kit) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
76 |
kit->set_all_memory_call(this, true); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
77 |
kit->set_control(kit->gvn().transform(new ProjNode(this,TypeFunc::Control))); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
78 |
kit->set_i_o(kit->gvn().transform(new ProjNode(this, TypeFunc::I_O))); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
79 |
kit->make_slow_call_ex(this, kit->env()->Throwable_klass(), true); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
80 |
kit->set_all_memory_call(this); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
81 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
82 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
83 |
#ifndef PRODUCT |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
84 |
const char* ArrayCopyNode::_kind_names[] = {"arraycopy", "arraycopy, validated arguments", "clone", "oop array clone", "CopyOf", "CopyOfRange"}; |
32084
7743e6943cdf
8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents:
31233
diff
changeset
|
85 |
|
29337
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
86 |
void ArrayCopyNode::dump_spec(outputStream *st) const { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
87 |
CallNode::dump_spec(st); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
88 |
st->print(" (%s%s)", _kind_names[_kind], _alloc_tightly_coupled ? ", tightly coupled allocation" : ""); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
89 |
} |
32084
7743e6943cdf
8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents:
31233
diff
changeset
|
90 |
|
7743e6943cdf
8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents:
31233
diff
changeset
|
91 |
void ArrayCopyNode::dump_compact_spec(outputStream* st) const { |
7743e6943cdf
8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents:
31233
diff
changeset
|
92 |
st->print("%s%s", _kind_names[_kind], _alloc_tightly_coupled ? ",tight" : ""); |
7743e6943cdf
8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents:
31233
diff
changeset
|
93 |
} |
29337
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
94 |
#endif |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
95 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
96 |
intptr_t ArrayCopyNode::get_length_if_constant(PhaseGVN *phase) const { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
97 |
// check that length is constant |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
98 |
Node* length = in(ArrayCopyNode::Length); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
99 |
const Type* length_type = phase->type(length); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
100 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
101 |
if (length_type == Type::TOP) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
102 |
return -1; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
103 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
104 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
105 |
assert(is_clonebasic() || is_arraycopy() || is_copyof() || is_copyofrange(), "unexpected array copy type"); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
106 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
107 |
return is_clonebasic() ? length->find_intptr_t_con(-1) : length->find_int_con(-1); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
108 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
109 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
110 |
int ArrayCopyNode::get_count(PhaseGVN *phase) const { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
111 |
Node* src = in(ArrayCopyNode::Src); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
112 |
const Type* src_type = phase->type(src); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
113 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
114 |
if (is_clonebasic()) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
115 |
if (src_type->isa_instptr()) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
116 |
const TypeInstPtr* inst_src = src_type->is_instptr(); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
117 |
ciInstanceKlass* ik = inst_src->klass()->as_instance_klass(); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
118 |
// ciInstanceKlass::nof_nonstatic_fields() doesn't take injected |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
119 |
// fields into account. They are rare anyway so easier to simply |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
120 |
// skip instances with injected fields. |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
121 |
if ((!inst_src->klass_is_exact() && (ik->is_interface() || ik->has_subklass())) || ik->has_injected_fields()) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
122 |
return -1; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
123 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
124 |
int nb_fields = ik->nof_nonstatic_fields(); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
125 |
return nb_fields; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
126 |
} else { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
127 |
const TypeAryPtr* ary_src = src_type->isa_aryptr(); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
128 |
assert (ary_src != NULL, "not an array or instance?"); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
129 |
// clone passes a length as a rounded number of longs. If we're |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
130 |
// cloning an array we'll do it element by element. If the |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
131 |
// length input to ArrayCopyNode is constant, length of input |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
132 |
// array must be too. |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
133 |
|
29345
e8f6ba59748f
8073792: assert((get_length_if_constant(phase) == -1) == !ary_src->size()->is_con()) failed: inconsistent
roland
parents:
29340
diff
changeset
|
134 |
assert((get_length_if_constant(phase) == -1) == !ary_src->size()->is_con() || |
e8f6ba59748f
8073792: assert((get_length_if_constant(phase) == -1) == !ary_src->size()->is_con()) failed: inconsistent
roland
parents:
29340
diff
changeset
|
135 |
phase->is_IterGVN(), "inconsistent"); |
29337
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
136 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
137 |
if (ary_src->size()->is_con()) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
138 |
return ary_src->size()->get_con(); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
139 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
140 |
return -1; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
141 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
142 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
143 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
144 |
return get_length_if_constant(phase); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
145 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
146 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
147 |
Node* ArrayCopyNode::try_clone_instance(PhaseGVN *phase, bool can_reshape, int count) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
148 |
if (!is_clonebasic()) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
149 |
return NULL; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
150 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
151 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
152 |
Node* src = in(ArrayCopyNode::Src); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
153 |
Node* dest = in(ArrayCopyNode::Dest); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
154 |
Node* ctl = in(TypeFunc::Control); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
155 |
Node* in_mem = in(TypeFunc::Memory); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
156 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
157 |
const Type* src_type = phase->type(src); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
158 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
159 |
assert(src->is_AddP(), "should be base + off"); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
160 |
assert(dest->is_AddP(), "should be base + off"); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
161 |
Node* base_src = src->in(AddPNode::Base); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
162 |
Node* base_dest = dest->in(AddPNode::Base); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
163 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
164 |
MergeMemNode* mem = MergeMemNode::make(in_mem); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
165 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
166 |
const TypeInstPtr* inst_src = src_type->isa_instptr(); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
167 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
168 |
if (inst_src == NULL) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
169 |
return NULL; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
170 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
171 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
172 |
if (!inst_src->klass_is_exact()) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
173 |
ciInstanceKlass* ik = inst_src->klass()->as_instance_klass(); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
174 |
assert(!ik->is_interface() && !ik->has_subklass(), "inconsistent klass hierarchy"); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
175 |
phase->C->dependencies()->assert_leaf_type(ik); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
176 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
177 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
178 |
ciInstanceKlass* ik = inst_src->klass()->as_instance_klass(); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
179 |
assert(ik->nof_nonstatic_fields() <= ArrayCopyLoadStoreMaxElem, "too many fields"); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
180 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
181 |
for (int i = 0; i < count; i++) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
182 |
ciField* field = ik->nonstatic_field_at(i); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
183 |
int fieldidx = phase->C->alias_type(field)->index(); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
184 |
const TypePtr* adr_type = phase->C->alias_type(field)->adr_type(); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
185 |
Node* off = phase->MakeConX(field->offset()); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
186 |
Node* next_src = phase->transform(new AddPNode(base_src,base_src,off)); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
187 |
Node* next_dest = phase->transform(new AddPNode(base_dest,base_dest,off)); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
188 |
BasicType bt = field->layout_type(); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
189 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
190 |
const Type *type; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
191 |
if (bt == T_OBJECT) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
192 |
if (!field->type()->is_loaded()) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
193 |
type = TypeInstPtr::BOTTOM; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
194 |
} else { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
195 |
ciType* field_klass = field->type(); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
196 |
type = TypeOopPtr::make_from_klass(field_klass->as_klass()); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
197 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
198 |
} else { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
199 |
type = Type::get_const_basic_type(bt); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
200 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
201 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
202 |
Node* v = LoadNode::make(*phase, ctl, mem->memory_at(fieldidx), next_src, adr_type, type, bt, MemNode::unordered); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
203 |
v = phase->transform(v); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
204 |
Node* s = StoreNode::make(*phase, ctl, mem->memory_at(fieldidx), next_dest, adr_type, v, bt, MemNode::unordered); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
205 |
s = phase->transform(s); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
206 |
mem->set_memory_at(fieldidx, s); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
207 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
208 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
209 |
if (!finish_transform(phase, can_reshape, ctl, mem)) { |
39254
fb4492288b01
8156760: VM crashes if -XX:-ReduceInitialCardMarks is set
thartmann
parents:
32576
diff
changeset
|
210 |
// Return NodeSentinel to indicate that the transform failed |
fb4492288b01
8156760: VM crashes if -XX:-ReduceInitialCardMarks is set
thartmann
parents:
32576
diff
changeset
|
211 |
return NodeSentinel; |
29337
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
212 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
213 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
214 |
return mem; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
215 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
216 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
217 |
bool ArrayCopyNode::prepare_array_copy(PhaseGVN *phase, bool can_reshape, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
218 |
Node*& adr_src, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
219 |
Node*& base_src, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
220 |
Node*& adr_dest, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
221 |
Node*& base_dest, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
222 |
BasicType& copy_type, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
223 |
const Type*& value_type, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
224 |
bool& disjoint_bases) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
225 |
Node* src = in(ArrayCopyNode::Src); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
226 |
Node* dest = in(ArrayCopyNode::Dest); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
227 |
const Type* src_type = phase->type(src); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
228 |
const TypeAryPtr* ary_src = src_type->isa_aryptr(); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
229 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
230 |
if (is_arraycopy() || is_copyofrange() || is_copyof()) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
231 |
const Type* dest_type = phase->type(dest); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
232 |
const TypeAryPtr* ary_dest = dest_type->isa_aryptr(); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
233 |
Node* src_offset = in(ArrayCopyNode::SrcPos); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
234 |
Node* dest_offset = in(ArrayCopyNode::DestPos); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
235 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
236 |
// newly allocated object is guaranteed to not overlap with source object |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
237 |
disjoint_bases = is_alloc_tightly_coupled(); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
238 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
239 |
if (ary_src == NULL || ary_src->klass() == NULL || |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
240 |
ary_dest == NULL || ary_dest->klass() == NULL) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
241 |
// We don't know if arguments are arrays |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
242 |
return false; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
243 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
244 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
245 |
BasicType src_elem = ary_src->klass()->as_array_klass()->element_type()->basic_type(); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
246 |
BasicType dest_elem = ary_dest->klass()->as_array_klass()->element_type()->basic_type(); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
247 |
if (src_elem == T_ARRAY) src_elem = T_OBJECT; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
248 |
if (dest_elem == T_ARRAY) dest_elem = T_OBJECT; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
249 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
250 |
if (src_elem != dest_elem || dest_elem == T_VOID) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
251 |
// We don't know if arguments are arrays of the same type |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
252 |
return false; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
253 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
254 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
255 |
if (dest_elem == T_OBJECT && (!is_alloc_tightly_coupled() || !GraphKit::use_ReduceInitialCardMarks())) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
256 |
// It's an object array copy but we can't emit the card marking |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
257 |
// that is needed |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
258 |
return false; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
259 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
260 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
261 |
value_type = ary_src->elem(); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
262 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
263 |
base_src = src; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
264 |
base_dest = dest; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
265 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
266 |
uint shift = exact_log2(type2aelembytes(dest_elem)); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
267 |
uint header = arrayOopDesc::base_offset_in_bytes(dest_elem); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
268 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
269 |
adr_src = src; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
270 |
adr_dest = dest; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
271 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
272 |
src_offset = Compile::conv_I2X_index(phase, src_offset, ary_src->size()); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
273 |
dest_offset = Compile::conv_I2X_index(phase, dest_offset, ary_dest->size()); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
274 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
275 |
Node* src_scale = phase->transform(new LShiftXNode(src_offset, phase->intcon(shift))); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
276 |
Node* dest_scale = phase->transform(new LShiftXNode(dest_offset, phase->intcon(shift))); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
277 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
278 |
adr_src = phase->transform(new AddPNode(base_src, adr_src, src_scale)); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
279 |
adr_dest = phase->transform(new AddPNode(base_dest, adr_dest, dest_scale)); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
280 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
281 |
adr_src = new AddPNode(base_src, adr_src, phase->MakeConX(header)); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
282 |
adr_dest = new AddPNode(base_dest, adr_dest, phase->MakeConX(header)); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
283 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
284 |
adr_src = phase->transform(adr_src); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
285 |
adr_dest = phase->transform(adr_dest); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
286 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
287 |
copy_type = dest_elem; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
288 |
} else { |
44322
051426f34a5e
8176505: Wrong assertion 'should be an array copy/clone' in arraycopynode.cpp
simonis
parents:
42086
diff
changeset
|
289 |
assert(ary_src != NULL, "should be a clone"); |
051426f34a5e
8176505: Wrong assertion 'should be an array copy/clone' in arraycopynode.cpp
simonis
parents:
42086
diff
changeset
|
290 |
assert(is_clonebasic(), "should be"); |
29337
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
291 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
292 |
disjoint_bases = true; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
293 |
assert(src->is_AddP(), "should be base + off"); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
294 |
assert(dest->is_AddP(), "should be base + off"); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
295 |
adr_src = src; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
296 |
base_src = src->in(AddPNode::Base); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
297 |
adr_dest = dest; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
298 |
base_dest = dest->in(AddPNode::Base); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
299 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
300 |
assert(phase->type(src->in(AddPNode::Offset))->is_intptr_t()->get_con() == phase->type(dest->in(AddPNode::Offset))->is_intptr_t()->get_con(), "same start offset?"); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
301 |
BasicType elem = ary_src->klass()->as_array_klass()->element_type()->basic_type(); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
302 |
if (elem == T_ARRAY) elem = T_OBJECT; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
303 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
304 |
int diff = arrayOopDesc::base_offset_in_bytes(elem) - phase->type(src->in(AddPNode::Offset))->is_intptr_t()->get_con(); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
305 |
assert(diff >= 0, "clone should not start after 1st array element"); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
306 |
if (diff > 0) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
307 |
adr_src = phase->transform(new AddPNode(base_src, adr_src, phase->MakeConX(diff))); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
308 |
adr_dest = phase->transform(new AddPNode(base_dest, adr_dest, phase->MakeConX(diff))); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
309 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
310 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
311 |
copy_type = elem; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
312 |
value_type = ary_src->elem(); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
313 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
314 |
return true; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
315 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
316 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
317 |
const TypePtr* ArrayCopyNode::get_address_type(PhaseGVN *phase, Node* n) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
318 |
const Type* at = phase->type(n); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
319 |
assert(at != Type::TOP, "unexpected type"); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
320 |
const TypePtr* atp = at->isa_ptr(); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
321 |
// adjust atp to be the correct array element address type |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
322 |
atp = atp->add_offset(Type::OffsetBot); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
323 |
return atp; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
324 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
325 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
326 |
void ArrayCopyNode::array_copy_test_overlap(PhaseGVN *phase, bool can_reshape, bool disjoint_bases, int count, Node*& forward_ctl, Node*& backward_ctl) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
327 |
Node* ctl = in(TypeFunc::Control); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
328 |
if (!disjoint_bases && count > 1) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
329 |
Node* src_offset = in(ArrayCopyNode::SrcPos); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
330 |
Node* dest_offset = in(ArrayCopyNode::DestPos); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
331 |
assert(src_offset != NULL && dest_offset != NULL, "should be"); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
332 |
Node* cmp = phase->transform(new CmpINode(src_offset, dest_offset)); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
333 |
Node *bol = phase->transform(new BoolNode(cmp, BoolTest::lt)); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
334 |
IfNode *iff = new IfNode(ctl, bol, PROB_FAIR, COUNT_UNKNOWN); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
335 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
336 |
phase->transform(iff); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
337 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
338 |
forward_ctl = phase->transform(new IfFalseNode(iff)); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
339 |
backward_ctl = phase->transform(new IfTrueNode(iff)); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
340 |
} else { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
341 |
forward_ctl = ctl; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
342 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
343 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
344 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
345 |
Node* ArrayCopyNode::array_copy_forward(PhaseGVN *phase, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
346 |
bool can_reshape, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
347 |
Node* forward_ctl, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
348 |
Node* start_mem_src, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
349 |
Node* start_mem_dest, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
350 |
const TypePtr* atp_src, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
351 |
const TypePtr* atp_dest, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
352 |
Node* adr_src, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
353 |
Node* base_src, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
354 |
Node* adr_dest, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
355 |
Node* base_dest, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
356 |
BasicType copy_type, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
357 |
const Type* value_type, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
358 |
int count) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
359 |
Node* mem = phase->C->top(); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
360 |
if (!forward_ctl->is_top()) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
361 |
// copy forward |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
362 |
mem = start_mem_dest; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
363 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
364 |
if (count > 0) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
365 |
Node* v = LoadNode::make(*phase, forward_ctl, start_mem_src, adr_src, atp_src, value_type, copy_type, MemNode::unordered); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
366 |
v = phase->transform(v); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
367 |
mem = StoreNode::make(*phase, forward_ctl, mem, adr_dest, atp_dest, v, copy_type, MemNode::unordered); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
368 |
mem = phase->transform(mem); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
369 |
for (int i = 1; i < count; i++) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
370 |
Node* off = phase->MakeConX(type2aelembytes(copy_type) * i); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
371 |
Node* next_src = phase->transform(new AddPNode(base_src,adr_src,off)); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
372 |
Node* next_dest = phase->transform(new AddPNode(base_dest,adr_dest,off)); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
373 |
v = LoadNode::make(*phase, forward_ctl, mem, next_src, atp_src, value_type, copy_type, MemNode::unordered); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
374 |
v = phase->transform(v); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
375 |
mem = StoreNode::make(*phase, forward_ctl,mem,next_dest,atp_dest,v, copy_type, MemNode::unordered); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
376 |
mem = phase->transform(mem); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
377 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
378 |
} else if(can_reshape) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
379 |
PhaseIterGVN* igvn = phase->is_IterGVN(); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
380 |
igvn->_worklist.push(adr_src); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
381 |
igvn->_worklist.push(adr_dest); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
382 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
383 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
384 |
return mem; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
385 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
386 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
387 |
Node* ArrayCopyNode::array_copy_backward(PhaseGVN *phase, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
388 |
bool can_reshape, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
389 |
Node* backward_ctl, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
390 |
Node* start_mem_src, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
391 |
Node* start_mem_dest, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
392 |
const TypePtr* atp_src, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
393 |
const TypePtr* atp_dest, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
394 |
Node* adr_src, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
395 |
Node* base_src, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
396 |
Node* adr_dest, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
397 |
Node* base_dest, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
398 |
BasicType copy_type, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
399 |
const Type* value_type, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
400 |
int count) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
401 |
Node* mem = phase->C->top(); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
402 |
if (!backward_ctl->is_top()) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
403 |
// copy backward |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
404 |
mem = start_mem_dest; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
405 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
406 |
if (count > 0) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
407 |
for (int i = count-1; i >= 1; i--) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
408 |
Node* off = phase->MakeConX(type2aelembytes(copy_type) * i); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
409 |
Node* next_src = phase->transform(new AddPNode(base_src,adr_src,off)); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
410 |
Node* next_dest = phase->transform(new AddPNode(base_dest,adr_dest,off)); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
411 |
Node* v = LoadNode::make(*phase, backward_ctl, mem, next_src, atp_src, value_type, copy_type, MemNode::unordered); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
412 |
v = phase->transform(v); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
413 |
mem = StoreNode::make(*phase, backward_ctl,mem,next_dest,atp_dest,v, copy_type, MemNode::unordered); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
414 |
mem = phase->transform(mem); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
415 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
416 |
Node* v = LoadNode::make(*phase, backward_ctl, mem, adr_src, atp_src, value_type, copy_type, MemNode::unordered); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
417 |
v = phase->transform(v); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
418 |
mem = StoreNode::make(*phase, backward_ctl, mem, adr_dest, atp_dest, v, copy_type, MemNode::unordered); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
419 |
mem = phase->transform(mem); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
420 |
} else if(can_reshape) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
421 |
PhaseIterGVN* igvn = phase->is_IterGVN(); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
422 |
igvn->_worklist.push(adr_src); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
423 |
igvn->_worklist.push(adr_dest); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
424 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
425 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
426 |
return mem; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
427 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
428 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
429 |
bool ArrayCopyNode::finish_transform(PhaseGVN *phase, bool can_reshape, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
430 |
Node* ctl, Node *mem) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
431 |
if (can_reshape) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
432 |
PhaseIterGVN* igvn = phase->is_IterGVN(); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
433 |
igvn->set_delay_transform(false); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
434 |
if (is_clonebasic()) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
435 |
Node* out_mem = proj_out(TypeFunc::Memory); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
436 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
437 |
if (out_mem->outcnt() != 1 || !out_mem->raw_out(0)->is_MergeMem() || |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
438 |
out_mem->raw_out(0)->outcnt() != 1 || !out_mem->raw_out(0)->raw_out(0)->is_MemBar()) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
439 |
assert(!GraphKit::use_ReduceInitialCardMarks(), "can only happen with card marking"); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
440 |
return false; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
441 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
442 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
443 |
igvn->replace_node(out_mem->raw_out(0), mem); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
444 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
445 |
Node* out_ctl = proj_out(TypeFunc::Control); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
446 |
igvn->replace_node(out_ctl, ctl); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
447 |
} else { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
448 |
// replace fallthrough projections of the ArrayCopyNode by the |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
449 |
// new memory, control and the input IO. |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
450 |
CallProjections callprojs; |
31233
7033a9f3e2f1
8086016: closed/java/text/Format/NumberFormat/BigDecimalCompatibilityTest.java is crashing
roland
parents:
31044
diff
changeset
|
451 |
extract_projections(&callprojs, true, false); |
29337
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
452 |
|
31233
7033a9f3e2f1
8086016: closed/java/text/Format/NumberFormat/BigDecimalCompatibilityTest.java is crashing
roland
parents:
31044
diff
changeset
|
453 |
if (callprojs.fallthrough_ioproj != NULL) { |
7033a9f3e2f1
8086016: closed/java/text/Format/NumberFormat/BigDecimalCompatibilityTest.java is crashing
roland
parents:
31044
diff
changeset
|
454 |
igvn->replace_node(callprojs.fallthrough_ioproj, in(TypeFunc::I_O)); |
7033a9f3e2f1
8086016: closed/java/text/Format/NumberFormat/BigDecimalCompatibilityTest.java is crashing
roland
parents:
31044
diff
changeset
|
455 |
} |
7033a9f3e2f1
8086016: closed/java/text/Format/NumberFormat/BigDecimalCompatibilityTest.java is crashing
roland
parents:
31044
diff
changeset
|
456 |
if (callprojs.fallthrough_memproj != NULL) { |
7033a9f3e2f1
8086016: closed/java/text/Format/NumberFormat/BigDecimalCompatibilityTest.java is crashing
roland
parents:
31044
diff
changeset
|
457 |
igvn->replace_node(callprojs.fallthrough_memproj, mem); |
7033a9f3e2f1
8086016: closed/java/text/Format/NumberFormat/BigDecimalCompatibilityTest.java is crashing
roland
parents:
31044
diff
changeset
|
458 |
} |
7033a9f3e2f1
8086016: closed/java/text/Format/NumberFormat/BigDecimalCompatibilityTest.java is crashing
roland
parents:
31044
diff
changeset
|
459 |
if (callprojs.fallthrough_catchproj != NULL) { |
7033a9f3e2f1
8086016: closed/java/text/Format/NumberFormat/BigDecimalCompatibilityTest.java is crashing
roland
parents:
31044
diff
changeset
|
460 |
igvn->replace_node(callprojs.fallthrough_catchproj, ctl); |
7033a9f3e2f1
8086016: closed/java/text/Format/NumberFormat/BigDecimalCompatibilityTest.java is crashing
roland
parents:
31044
diff
changeset
|
461 |
} |
29337
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
462 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
463 |
// The ArrayCopyNode is not disconnected. It still has the |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
464 |
// projections for the exception case. Replace current |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
465 |
// ArrayCopyNode with a dummy new one with a top() control so |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
466 |
// that this part of the graph stays consistent but is |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
467 |
// eventually removed. |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
468 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
469 |
set_req(0, phase->C->top()); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
470 |
remove_dead_region(phase, can_reshape); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
471 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
472 |
} else { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
473 |
if (in(TypeFunc::Control) != ctl) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
474 |
// we can't return new memory and control from Ideal at parse time |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
475 |
assert(!is_clonebasic(), "added control for clone?"); |
29340
e5ee51840b93
8073624: Fix waring "converting to non-pointer type 'bool' from NULL" in arraycopynode.cpp
simonis
parents:
29337
diff
changeset
|
476 |
return false; |
29337
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
477 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
478 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
479 |
return true; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
480 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
481 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
482 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
483 |
Node *ArrayCopyNode::Ideal(PhaseGVN *phase, bool can_reshape) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
484 |
if (remove_dead_region(phase, can_reshape)) return this; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
485 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
486 |
if (StressArrayCopyMacroNode && !can_reshape) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
487 |
phase->record_for_igvn(this); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
488 |
return NULL; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
489 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
490 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
491 |
// See if it's a small array copy and we can inline it as |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
492 |
// loads/stores |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
493 |
// Here we can only do: |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
494 |
// - arraycopy if all arguments were validated before and we don't |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
495 |
// need card marking |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
496 |
// - clone for which we don't need to do card marking |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
497 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
498 |
if (!is_clonebasic() && !is_arraycopy_validated() && |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
499 |
!is_copyofrange_validated() && !is_copyof_validated()) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
500 |
return NULL; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
501 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
502 |
|
29360
dd9daceb5f3d
8073957: assert(ary_src != 0) failed: not an array or instance?
roland
parents:
29345
diff
changeset
|
503 |
assert(in(TypeFunc::Control) != NULL && |
dd9daceb5f3d
8073957: assert(ary_src != 0) failed: not an array or instance?
roland
parents:
29345
diff
changeset
|
504 |
in(TypeFunc::Memory) != NULL && |
dd9daceb5f3d
8073957: assert(ary_src != 0) failed: not an array or instance?
roland
parents:
29345
diff
changeset
|
505 |
in(ArrayCopyNode::Src) != NULL && |
dd9daceb5f3d
8073957: assert(ary_src != 0) failed: not an array or instance?
roland
parents:
29345
diff
changeset
|
506 |
in(ArrayCopyNode::Dest) != NULL && |
dd9daceb5f3d
8073957: assert(ary_src != 0) failed: not an array or instance?
roland
parents:
29345
diff
changeset
|
507 |
in(ArrayCopyNode::Length) != NULL && |
dd9daceb5f3d
8073957: assert(ary_src != 0) failed: not an array or instance?
roland
parents:
29345
diff
changeset
|
508 |
((in(ArrayCopyNode::SrcPos) != NULL && in(ArrayCopyNode::DestPos) != NULL) || |
dd9daceb5f3d
8073957: assert(ary_src != 0) failed: not an array or instance?
roland
parents:
29345
diff
changeset
|
509 |
is_clonebasic()), "broken inputs"); |
dd9daceb5f3d
8073957: assert(ary_src != 0) failed: not an array or instance?
roland
parents:
29345
diff
changeset
|
510 |
|
dd9daceb5f3d
8073957: assert(ary_src != 0) failed: not an array or instance?
roland
parents:
29345
diff
changeset
|
511 |
if (in(TypeFunc::Control)->is_top() || |
dd9daceb5f3d
8073957: assert(ary_src != 0) failed: not an array or instance?
roland
parents:
29345
diff
changeset
|
512 |
in(TypeFunc::Memory)->is_top() || |
dd9daceb5f3d
8073957: assert(ary_src != 0) failed: not an array or instance?
roland
parents:
29345
diff
changeset
|
513 |
phase->type(in(ArrayCopyNode::Src)) == Type::TOP || |
dd9daceb5f3d
8073957: assert(ary_src != 0) failed: not an array or instance?
roland
parents:
29345
diff
changeset
|
514 |
phase->type(in(ArrayCopyNode::Dest)) == Type::TOP || |
dd9daceb5f3d
8073957: assert(ary_src != 0) failed: not an array or instance?
roland
parents:
29345
diff
changeset
|
515 |
(in(ArrayCopyNode::SrcPos) != NULL && in(ArrayCopyNode::SrcPos)->is_top()) || |
dd9daceb5f3d
8073957: assert(ary_src != 0) failed: not an array or instance?
roland
parents:
29345
diff
changeset
|
516 |
(in(ArrayCopyNode::DestPos) != NULL && in(ArrayCopyNode::DestPos)->is_top())) { |
29337
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
517 |
return NULL; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
518 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
519 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
520 |
int count = get_count(phase); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
521 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
522 |
if (count < 0 || count > ArrayCopyLoadStoreMaxElem) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
523 |
return NULL; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
524 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
525 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
526 |
Node* mem = try_clone_instance(phase, can_reshape, count); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
527 |
if (mem != NULL) { |
39254
fb4492288b01
8156760: VM crashes if -XX:-ReduceInitialCardMarks is set
thartmann
parents:
32576
diff
changeset
|
528 |
return (mem == NodeSentinel) ? NULL : mem; |
29337
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
529 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
530 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
531 |
Node* adr_src = NULL; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
532 |
Node* base_src = NULL; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
533 |
Node* adr_dest = NULL; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
534 |
Node* base_dest = NULL; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
535 |
BasicType copy_type = T_ILLEGAL; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
536 |
const Type* value_type = NULL; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
537 |
bool disjoint_bases = false; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
538 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
539 |
if (!prepare_array_copy(phase, can_reshape, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
540 |
adr_src, base_src, adr_dest, base_dest, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
541 |
copy_type, value_type, disjoint_bases)) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
542 |
return NULL; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
543 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
544 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
545 |
Node* src = in(ArrayCopyNode::Src); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
546 |
Node* dest = in(ArrayCopyNode::Dest); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
547 |
const TypePtr* atp_src = get_address_type(phase, src); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
548 |
const TypePtr* atp_dest = get_address_type(phase, dest); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
549 |
uint alias_idx_src = phase->C->get_alias_index(atp_src); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
550 |
uint alias_idx_dest = phase->C->get_alias_index(atp_dest); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
551 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
552 |
Node *in_mem = in(TypeFunc::Memory); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
553 |
Node *start_mem_src = in_mem; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
554 |
Node *start_mem_dest = in_mem; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
555 |
if (in_mem->is_MergeMem()) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
556 |
start_mem_src = in_mem->as_MergeMem()->memory_at(alias_idx_src); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
557 |
start_mem_dest = in_mem->as_MergeMem()->memory_at(alias_idx_dest); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
558 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
559 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
560 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
561 |
if (can_reshape) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
562 |
assert(!phase->is_IterGVN()->delay_transform(), "cannot delay transforms"); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
563 |
phase->is_IterGVN()->set_delay_transform(true); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
564 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
565 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
566 |
Node* backward_ctl = phase->C->top(); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
567 |
Node* forward_ctl = phase->C->top(); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
568 |
array_copy_test_overlap(phase, can_reshape, disjoint_bases, count, forward_ctl, backward_ctl); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
569 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
570 |
Node* forward_mem = array_copy_forward(phase, can_reshape, forward_ctl, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
571 |
start_mem_src, start_mem_dest, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
572 |
atp_src, atp_dest, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
573 |
adr_src, base_src, adr_dest, base_dest, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
574 |
copy_type, value_type, count); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
575 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
576 |
Node* backward_mem = array_copy_backward(phase, can_reshape, backward_ctl, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
577 |
start_mem_src, start_mem_dest, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
578 |
atp_src, atp_dest, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
579 |
adr_src, base_src, adr_dest, base_dest, |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
580 |
copy_type, value_type, count); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
581 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
582 |
Node* ctl = NULL; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
583 |
if (!forward_ctl->is_top() && !backward_ctl->is_top()) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
584 |
ctl = new RegionNode(3); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
585 |
mem = new PhiNode(ctl, Type::MEMORY, atp_dest); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
586 |
ctl->init_req(1, forward_ctl); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
587 |
mem->init_req(1, forward_mem); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
588 |
ctl->init_req(2, backward_ctl); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
589 |
mem->init_req(2, backward_mem); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
590 |
ctl = phase->transform(ctl); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
591 |
mem = phase->transform(mem); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
592 |
} else if (!forward_ctl->is_top()) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
593 |
ctl = forward_ctl; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
594 |
mem = forward_mem; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
595 |
} else { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
596 |
assert(!backward_ctl->is_top(), "no copy?"); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
597 |
ctl = backward_ctl; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
598 |
mem = backward_mem; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
599 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
600 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
601 |
if (can_reshape) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
602 |
assert(phase->is_IterGVN()->delay_transform(), "should be delaying transforms"); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
603 |
phase->is_IterGVN()->set_delay_transform(false); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
604 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
605 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
606 |
MergeMemNode* out_mem = MergeMemNode::make(in_mem); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
607 |
out_mem->set_memory_at(alias_idx_dest, mem); |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
608 |
mem = out_mem; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
609 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
610 |
if (!finish_transform(phase, can_reshape, ctl, mem)) { |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
611 |
return NULL; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
612 |
} |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
613 |
|
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
614 |
return mem; |
ef2be52deeaf
6912521: System.arraycopy works slower than the simple loop for little lengths
roland
parents:
diff
changeset
|
615 |
} |
30629
b6e5ad2f18d5
8076188: Optimize arraycopy out for non escaping destination
roland
parents:
29360
diff
changeset
|
616 |
|
b6e5ad2f18d5
8076188: Optimize arraycopy out for non escaping destination
roland
parents:
29360
diff
changeset
|
617 |
bool ArrayCopyNode::may_modify(const TypeOopPtr *t_oop, PhaseTransform *phase) { |
31044
9fecc7e87949
8080699: Assert failed: Not a Java pointer in JCK test
roland
parents:
30629
diff
changeset
|
618 |
Node* dest = in(ArrayCopyNode::Dest); |
9fecc7e87949
8080699: Assert failed: Not a Java pointer in JCK test
roland
parents:
30629
diff
changeset
|
619 |
if (dest->is_top()) { |
9fecc7e87949
8080699: Assert failed: Not a Java pointer in JCK test
roland
parents:
30629
diff
changeset
|
620 |
return false; |
9fecc7e87949
8080699: Assert failed: Not a Java pointer in JCK test
roland
parents:
30629
diff
changeset
|
621 |
} |
9fecc7e87949
8080699: Assert failed: Not a Java pointer in JCK test
roland
parents:
30629
diff
changeset
|
622 |
const TypeOopPtr* dest_t = phase->type(dest)->is_oopptr(); |
30629
b6e5ad2f18d5
8076188: Optimize arraycopy out for non escaping destination
roland
parents:
29360
diff
changeset
|
623 |
assert(!dest_t->is_known_instance() || _dest_type->is_known_instance(), "result of EA not recorded"); |
31044
9fecc7e87949
8080699: Assert failed: Not a Java pointer in JCK test
roland
parents:
30629
diff
changeset
|
624 |
assert(in(ArrayCopyNode::Src)->is_top() || !phase->type(in(ArrayCopyNode::Src))->is_oopptr()->is_known_instance() || |
9fecc7e87949
8080699: Assert failed: Not a Java pointer in JCK test
roland
parents:
30629
diff
changeset
|
625 |
_src_type->is_known_instance(), "result of EA not recorded"); |
30629
b6e5ad2f18d5
8076188: Optimize arraycopy out for non escaping destination
roland
parents:
29360
diff
changeset
|
626 |
|
b6e5ad2f18d5
8076188: Optimize arraycopy out for non escaping destination
roland
parents:
29360
diff
changeset
|
627 |
if (_dest_type != TypeOopPtr::BOTTOM || t_oop->is_known_instance()) { |
b6e5ad2f18d5
8076188: Optimize arraycopy out for non escaping destination
roland
parents:
29360
diff
changeset
|
628 |
assert(_dest_type == TypeOopPtr::BOTTOM || _dest_type->is_known_instance(), "result of EA is known instance"); |
b6e5ad2f18d5
8076188: Optimize arraycopy out for non escaping destination
roland
parents:
29360
diff
changeset
|
629 |
return t_oop->instance_id() == _dest_type->instance_id(); |
b6e5ad2f18d5
8076188: Optimize arraycopy out for non escaping destination
roland
parents:
29360
diff
changeset
|
630 |
} |
b6e5ad2f18d5
8076188: Optimize arraycopy out for non escaping destination
roland
parents:
29360
diff
changeset
|
631 |
|
b6e5ad2f18d5
8076188: Optimize arraycopy out for non escaping destination
roland
parents:
29360
diff
changeset
|
632 |
return CallNode::may_modify_arraycopy_helper(dest_t, t_oop, phase); |
b6e5ad2f18d5
8076188: Optimize arraycopy out for non escaping destination
roland
parents:
29360
diff
changeset
|
633 |
} |
32370
38b7b5772b4f
8130847: Cloned object's fields observed as null after C2 escape analysis
roland
parents:
32084
diff
changeset
|
634 |
|
45427
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
635 |
bool ArrayCopyNode::may_modify_helper(const TypeOopPtr *t_oop, Node* n, PhaseTransform *phase, CallNode*& call) { |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
636 |
if (n != NULL && |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
637 |
n->is_Call() && |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
638 |
n->as_Call()->may_modify(t_oop, phase) && |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
639 |
(n->as_Call()->is_ArrayCopy() || n->as_Call()->is_call_to_arraycopystub())) { |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
640 |
call = n->as_Call(); |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
641 |
return true; |
32370
38b7b5772b4f
8130847: Cloned object's fields observed as null after C2 escape analysis
roland
parents:
32084
diff
changeset
|
642 |
} |
38b7b5772b4f
8130847: Cloned object's fields observed as null after C2 escape analysis
roland
parents:
32084
diff
changeset
|
643 |
return false; |
38b7b5772b4f
8130847: Cloned object's fields observed as null after C2 escape analysis
roland
parents:
32084
diff
changeset
|
644 |
} |
38b7b5772b4f
8130847: Cloned object's fields observed as null after C2 escape analysis
roland
parents:
32084
diff
changeset
|
645 |
|
45427
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
646 |
static Node* step_over_gc_barrier(Node* c) { |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
647 |
if (UseG1GC && !GraphKit::use_ReduceInitialCardMarks() && |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
648 |
c != NULL && c->is_Region() && c->req() == 3) { |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
649 |
for (uint i = 1; i < c->req(); i++) { |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
650 |
if (c->in(i) != NULL && c->in(i)->is_Region() && |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
651 |
c->in(i)->req() == 3) { |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
652 |
Node* r = c->in(i); |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
653 |
for (uint j = 1; j < r->req(); j++) { |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
654 |
if (r->in(j) != NULL && r->in(j)->is_Proj() && |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
655 |
r->in(j)->in(0) != NULL && |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
656 |
r->in(j)->in(0)->Opcode() == Op_CallLeaf && |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
657 |
r->in(j)->in(0)->as_Call()->entry_point() == CAST_FROM_FN_PTR(address, SharedRuntime::g1_wb_post)) { |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
658 |
Node* call = r->in(j)->in(0); |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
659 |
c = c->in(i == 1 ? 2 : 1); |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
660 |
if (c != NULL) { |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
661 |
c = c->in(0); |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
662 |
if (c != NULL) { |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
663 |
c = c->in(0); |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
664 |
assert(call->in(0) == NULL || |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
665 |
call->in(0)->in(0) == NULL || |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
666 |
call->in(0)->in(0)->in(0) == NULL || |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
667 |
call->in(0)->in(0)->in(0)->in(0) == NULL || |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
668 |
call->in(0)->in(0)->in(0)->in(0)->in(0) == NULL || |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
669 |
c == call->in(0)->in(0)->in(0)->in(0)->in(0), "bad barrier shape"); |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
670 |
return c; |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
671 |
} |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
672 |
} |
32370
38b7b5772b4f
8130847: Cloned object's fields observed as null after C2 escape analysis
roland
parents:
32084
diff
changeset
|
673 |
} |
38b7b5772b4f
8130847: Cloned object's fields observed as null after C2 escape analysis
roland
parents:
32084
diff
changeset
|
674 |
} |
38b7b5772b4f
8130847: Cloned object's fields observed as null after C2 escape analysis
roland
parents:
32084
diff
changeset
|
675 |
} |
38b7b5772b4f
8130847: Cloned object's fields observed as null after C2 escape analysis
roland
parents:
32084
diff
changeset
|
676 |
} |
38b7b5772b4f
8130847: Cloned object's fields observed as null after C2 escape analysis
roland
parents:
32084
diff
changeset
|
677 |
} |
45427
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
678 |
return c; |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
679 |
} |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
680 |
|
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
681 |
bool ArrayCopyNode::may_modify(const TypeOopPtr *t_oop, MemBarNode* mb, PhaseTransform *phase, ArrayCopyNode*& ac) { |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
682 |
|
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
683 |
Node* c = mb->in(0); |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
684 |
|
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
685 |
// step over g1 gc barrier if we're at a clone with ReduceInitialCardMarks off |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
686 |
c = step_over_gc_barrier(c); |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
687 |
|
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
688 |
CallNode* call = NULL; |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
689 |
if (c != NULL && c->is_Region()) { |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
690 |
for (uint i = 1; i < c->req(); i++) { |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
691 |
if (c->in(i) != NULL) { |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
692 |
Node* n = c->in(i)->in(0); |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
693 |
if (may_modify_helper(t_oop, n, phase, call)) { |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
694 |
ac = call->isa_ArrayCopy(); |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
695 |
assert(c == mb->in(0), "only for clone"); |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
696 |
return true; |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
697 |
} |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
698 |
} |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
699 |
} |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
700 |
} else if (may_modify_helper(t_oop, c->in(0), phase, call)) { |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
701 |
ac = call->isa_ArrayCopy(); |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
702 |
assert(c == mb->in(0) || (ac != NULL && ac->is_clonebasic() && !GraphKit::use_ReduceInitialCardMarks()), "only for clone"); |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
703 |
return true; |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
704 |
} |
32370
38b7b5772b4f
8130847: Cloned object's fields observed as null after C2 escape analysis
roland
parents:
32084
diff
changeset
|
705 |
|
38b7b5772b4f
8130847: Cloned object's fields observed as null after C2 escape analysis
roland
parents:
32084
diff
changeset
|
706 |
return false; |
38b7b5772b4f
8130847: Cloned object's fields observed as null after C2 escape analysis
roland
parents:
32084
diff
changeset
|
707 |
} |
38b7b5772b4f
8130847: Cloned object's fields observed as null after C2 escape analysis
roland
parents:
32084
diff
changeset
|
708 |
|
38b7b5772b4f
8130847: Cloned object's fields observed as null after C2 escape analysis
roland
parents:
32084
diff
changeset
|
709 |
// Does this array copy modify offsets between offset_lo and offset_hi |
38b7b5772b4f
8130847: Cloned object's fields observed as null after C2 escape analysis
roland
parents:
32084
diff
changeset
|
710 |
// in the destination array |
38b7b5772b4f
8130847: Cloned object's fields observed as null after C2 escape analysis
roland
parents:
32084
diff
changeset
|
711 |
// if must_modify is false, return true if the copy could write |
38b7b5772b4f
8130847: Cloned object's fields observed as null after C2 escape analysis
roland
parents:
32084
diff
changeset
|
712 |
// between offset_lo and offset_hi |
38b7b5772b4f
8130847: Cloned object's fields observed as null after C2 escape analysis
roland
parents:
32084
diff
changeset
|
713 |
// if must_modify is true, return true if the copy is guaranteed to |
38b7b5772b4f
8130847: Cloned object's fields observed as null after C2 escape analysis
roland
parents:
32084
diff
changeset
|
714 |
// write between offset_lo and offset_hi |
45427
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
715 |
bool ArrayCopyNode::modifies(intptr_t offset_lo, intptr_t offset_hi, PhaseTransform* phase, bool must_modify) const { |
32370
38b7b5772b4f
8130847: Cloned object's fields observed as null after C2 escape analysis
roland
parents:
32084
diff
changeset
|
716 |
assert(_kind == ArrayCopy || _kind == CopyOf || _kind == CopyOfRange, "only for real array copies"); |
38b7b5772b4f
8130847: Cloned object's fields observed as null after C2 escape analysis
roland
parents:
32084
diff
changeset
|
717 |
|
45427
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
718 |
Node* dest = in(Dest); |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
719 |
Node* dest_pos = in(DestPos); |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
720 |
Node* len = in(Length); |
32370
38b7b5772b4f
8130847: Cloned object's fields observed as null after C2 escape analysis
roland
parents:
32084
diff
changeset
|
721 |
|
38b7b5772b4f
8130847: Cloned object's fields observed as null after C2 escape analysis
roland
parents:
32084
diff
changeset
|
722 |
const TypeInt *dest_pos_t = phase->type(dest_pos)->isa_int(); |
38b7b5772b4f
8130847: Cloned object's fields observed as null after C2 escape analysis
roland
parents:
32084
diff
changeset
|
723 |
const TypeInt *len_t = phase->type(len)->isa_int(); |
38b7b5772b4f
8130847: Cloned object's fields observed as null after C2 escape analysis
roland
parents:
32084
diff
changeset
|
724 |
const TypeAryPtr* ary_t = phase->type(dest)->isa_aryptr(); |
38b7b5772b4f
8130847: Cloned object's fields observed as null after C2 escape analysis
roland
parents:
32084
diff
changeset
|
725 |
|
45427
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
726 |
if (dest_pos_t == NULL || len_t == NULL || ary_t == NULL) { |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
727 |
return !must_modify; |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
728 |
} |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
729 |
|
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
730 |
BasicType ary_elem = ary_t->klass()->as_array_klass()->element_type()->basic_type(); |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
731 |
uint header = arrayOopDesc::base_offset_in_bytes(ary_elem); |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
732 |
uint elemsize = type2aelembytes(ary_elem); |
32370
38b7b5772b4f
8130847: Cloned object's fields observed as null after C2 escape analysis
roland
parents:
32084
diff
changeset
|
733 |
|
45427
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
734 |
jlong dest_pos_plus_len_lo = (((jlong)dest_pos_t->_lo) + len_t->_lo) * elemsize + header; |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
735 |
jlong dest_pos_plus_len_hi = (((jlong)dest_pos_t->_hi) + len_t->_hi) * elemsize + header; |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
736 |
jlong dest_pos_lo = ((jlong)dest_pos_t->_lo) * elemsize + header; |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
737 |
jlong dest_pos_hi = ((jlong)dest_pos_t->_hi) * elemsize + header; |
32370
38b7b5772b4f
8130847: Cloned object's fields observed as null after C2 escape analysis
roland
parents:
32084
diff
changeset
|
738 |
|
45427
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
739 |
if (must_modify) { |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
740 |
if (offset_lo >= dest_pos_hi && offset_hi < dest_pos_plus_len_lo) { |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
741 |
return true; |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
742 |
} |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
743 |
} else { |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
744 |
if (offset_hi >= dest_pos_lo && offset_lo < dest_pos_plus_len_hi) { |
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
745 |
return true; |
32370
38b7b5772b4f
8130847: Cloned object's fields observed as null after C2 escape analysis
roland
parents:
32084
diff
changeset
|
746 |
} |
38b7b5772b4f
8130847: Cloned object's fields observed as null after C2 escape analysis
roland
parents:
32084
diff
changeset
|
747 |
} |
38b7b5772b4f
8130847: Cloned object's fields observed as null after C2 escape analysis
roland
parents:
32084
diff
changeset
|
748 |
return false; |
38b7b5772b4f
8130847: Cloned object's fields observed as null after C2 escape analysis
roland
parents:
32084
diff
changeset
|
749 |
} |
45427
64e07017ce01
8179678: ArrayCopy with same src and dst can cause incorrect execution or compiler crash
roland
parents:
44322
diff
changeset
|
750 |