annotate src/share/vm/opto/mulnode.cpp @ 3249:e1162778c1c8

7009266: G1: assert(obj->is_oop_or_null(true )) failed: Error Summary: A referent object that is only weakly reachable at the start of concurrent marking but is re-attached to the strongly reachable object graph during marking may not be marked as live. This can cause the reference object to be processed prematurely and leave dangling pointers to the referent object. Implement a read barrier for the java.lang.ref.Reference::referent field by intrinsifying the Reference.get() method, and intercepting accesses though JNI, reflection, and Unsafe, so that when a non-null referent object is read it is also logged in an SATB buffer. Reviewed-by: kvn, iveresov, never, tonyp, dholmes
author johnc
date Thu, 07 Apr 2011 09:53:20 -0700
parents f95d63e2154a
children c7b60b601eb4
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1 /*
1972
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1552
diff changeset
2 * Copyright (c) 1997, 2010, Oracle and/or its affiliates. All rights reserved.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
a61af66fc99e Initial load
duke
parents:
diff changeset
4 *
a61af66fc99e Initial load
duke
parents:
diff changeset
5 * This code is free software; you can redistribute it and/or modify it
a61af66fc99e Initial load
duke
parents:
diff changeset
6 * under the terms of the GNU General Public License version 2 only, as
a61af66fc99e Initial load
duke
parents:
diff changeset
7 * published by the Free Software Foundation.
a61af66fc99e Initial load
duke
parents:
diff changeset
8 *
a61af66fc99e Initial load
duke
parents:
diff changeset
9 * This code is distributed in the hope that it will be useful, but WITHOUT
a61af66fc99e Initial load
duke
parents:
diff changeset
10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
a61af66fc99e Initial load
duke
parents:
diff changeset
11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
a61af66fc99e Initial load
duke
parents:
diff changeset
12 * version 2 for more details (a copy is included in the LICENSE file that
a61af66fc99e Initial load
duke
parents:
diff changeset
13 * accompanied this code).
a61af66fc99e Initial load
duke
parents:
diff changeset
14 *
a61af66fc99e Initial load
duke
parents:
diff changeset
15 * You should have received a copy of the GNU General Public License version
a61af66fc99e Initial load
duke
parents:
diff changeset
16 * 2 along with this work; if not, write to the Free Software Foundation,
a61af66fc99e Initial load
duke
parents:
diff changeset
17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
a61af66fc99e Initial load
duke
parents:
diff changeset
18 *
1552
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 897
diff changeset
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 897
diff changeset
20 * or visit www.oracle.com if you need additional information or have any
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 897
diff changeset
21 * questions.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
22 *
a61af66fc99e Initial load
duke
parents:
diff changeset
23 */
a61af66fc99e Initial load
duke
parents:
diff changeset
24
1972
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1552
diff changeset
25 #include "precompiled.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1552
diff changeset
26 #include "memory/allocation.inline.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1552
diff changeset
27 #include "opto/addnode.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1552
diff changeset
28 #include "opto/connode.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1552
diff changeset
29 #include "opto/memnode.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1552
diff changeset
30 #include "opto/mulnode.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1552
diff changeset
31 #include "opto/phaseX.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1552
diff changeset
32 #include "opto/subnode.hpp"
0
a61af66fc99e Initial load
duke
parents:
diff changeset
33
1972
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1552
diff changeset
34 // Portions of code courtesy of Clifford Click
0
a61af66fc99e Initial load
duke
parents:
diff changeset
35
a61af66fc99e Initial load
duke
parents:
diff changeset
36
a61af66fc99e Initial load
duke
parents:
diff changeset
37 //=============================================================================
a61af66fc99e Initial load
duke
parents:
diff changeset
38 //------------------------------hash-------------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
39 // Hash function over MulNodes. Needs to be commutative; i.e., I swap
a61af66fc99e Initial load
duke
parents:
diff changeset
40 // (commute) inputs to MulNodes willy-nilly so the hash function must return
a61af66fc99e Initial load
duke
parents:
diff changeset
41 // the same value in the presence of edge swapping.
a61af66fc99e Initial load
duke
parents:
diff changeset
42 uint MulNode::hash() const {
a61af66fc99e Initial load
duke
parents:
diff changeset
43 return (uintptr_t)in(1) + (uintptr_t)in(2) + Opcode();
a61af66fc99e Initial load
duke
parents:
diff changeset
44 }
a61af66fc99e Initial load
duke
parents:
diff changeset
45
a61af66fc99e Initial load
duke
parents:
diff changeset
46 //------------------------------Identity---------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
47 // Multiplying a one preserves the other argument
a61af66fc99e Initial load
duke
parents:
diff changeset
48 Node *MulNode::Identity( PhaseTransform *phase ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
49 register const Type *one = mul_id(); // The multiplicative identity
a61af66fc99e Initial load
duke
parents:
diff changeset
50 if( phase->type( in(1) )->higher_equal( one ) ) return in(2);
a61af66fc99e Initial load
duke
parents:
diff changeset
51 if( phase->type( in(2) )->higher_equal( one ) ) return in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
52
a61af66fc99e Initial load
duke
parents:
diff changeset
53 return this;
a61af66fc99e Initial load
duke
parents:
diff changeset
54 }
a61af66fc99e Initial load
duke
parents:
diff changeset
55
a61af66fc99e Initial load
duke
parents:
diff changeset
56 //------------------------------Ideal------------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
57 // We also canonicalize the Node, moving constants to the right input,
a61af66fc99e Initial load
duke
parents:
diff changeset
58 // and flatten expressions (so that 1+x+2 becomes x+3).
a61af66fc99e Initial load
duke
parents:
diff changeset
59 Node *MulNode::Ideal(PhaseGVN *phase, bool can_reshape) {
a61af66fc99e Initial load
duke
parents:
diff changeset
60 const Type *t1 = phase->type( in(1) );
a61af66fc99e Initial load
duke
parents:
diff changeset
61 const Type *t2 = phase->type( in(2) );
a61af66fc99e Initial load
duke
parents:
diff changeset
62 Node *progress = NULL; // Progress flag
a61af66fc99e Initial load
duke
parents:
diff changeset
63 // We are OK if right is a constant, or right is a load and
a61af66fc99e Initial load
duke
parents:
diff changeset
64 // left is a non-constant.
a61af66fc99e Initial load
duke
parents:
diff changeset
65 if( !(t2->singleton() ||
a61af66fc99e Initial load
duke
parents:
diff changeset
66 (in(2)->is_Load() && !(t1->singleton() || in(1)->is_Load())) ) ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
67 if( t1->singleton() || // Left input is a constant?
a61af66fc99e Initial load
duke
parents:
diff changeset
68 // Otherwise, sort inputs (commutativity) to help value numbering.
a61af66fc99e Initial load
duke
parents:
diff changeset
69 (in(1)->_idx > in(2)->_idx) ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
70 swap_edges(1, 2);
a61af66fc99e Initial load
duke
parents:
diff changeset
71 const Type *t = t1;
a61af66fc99e Initial load
duke
parents:
diff changeset
72 t1 = t2;
a61af66fc99e Initial load
duke
parents:
diff changeset
73 t2 = t;
a61af66fc99e Initial load
duke
parents:
diff changeset
74 progress = this; // Made progress
a61af66fc99e Initial load
duke
parents:
diff changeset
75 }
a61af66fc99e Initial load
duke
parents:
diff changeset
76 }
a61af66fc99e Initial load
duke
parents:
diff changeset
77
a61af66fc99e Initial load
duke
parents:
diff changeset
78 // If the right input is a constant, and the left input is a product of a
a61af66fc99e Initial load
duke
parents:
diff changeset
79 // constant, flatten the expression tree.
a61af66fc99e Initial load
duke
parents:
diff changeset
80 uint op = Opcode();
a61af66fc99e Initial load
duke
parents:
diff changeset
81 if( t2->singleton() && // Right input is a constant?
a61af66fc99e Initial load
duke
parents:
diff changeset
82 op != Op_MulF && // Float & double cannot reassociate
a61af66fc99e Initial load
duke
parents:
diff changeset
83 op != Op_MulD ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
84 if( t2 == Type::TOP ) return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
85 Node *mul1 = in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
86 #ifdef ASSERT
a61af66fc99e Initial load
duke
parents:
diff changeset
87 // Check for dead loop
a61af66fc99e Initial load
duke
parents:
diff changeset
88 int op1 = mul1->Opcode();
a61af66fc99e Initial load
duke
parents:
diff changeset
89 if( phase->eqv( mul1, this ) || phase->eqv( in(2), this ) ||
a61af66fc99e Initial load
duke
parents:
diff changeset
90 ( op1 == mul_opcode() || op1 == add_opcode() ) &&
a61af66fc99e Initial load
duke
parents:
diff changeset
91 ( phase->eqv( mul1->in(1), this ) || phase->eqv( mul1->in(2), this ) ||
a61af66fc99e Initial load
duke
parents:
diff changeset
92 phase->eqv( mul1->in(1), mul1 ) || phase->eqv( mul1->in(2), mul1 ) ) )
a61af66fc99e Initial load
duke
parents:
diff changeset
93 assert(false, "dead loop in MulNode::Ideal");
a61af66fc99e Initial load
duke
parents:
diff changeset
94 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
95
a61af66fc99e Initial load
duke
parents:
diff changeset
96 if( mul1->Opcode() == mul_opcode() ) { // Left input is a multiply?
a61af66fc99e Initial load
duke
parents:
diff changeset
97 // Mul of a constant?
a61af66fc99e Initial load
duke
parents:
diff changeset
98 const Type *t12 = phase->type( mul1->in(2) );
a61af66fc99e Initial load
duke
parents:
diff changeset
99 if( t12->singleton() && t12 != Type::TOP) { // Left input is an add of a constant?
a61af66fc99e Initial load
duke
parents:
diff changeset
100 // Compute new constant; check for overflow
a61af66fc99e Initial load
duke
parents:
diff changeset
101 const Type *tcon01 = mul1->as_Mul()->mul_ring(t2,t12);
a61af66fc99e Initial load
duke
parents:
diff changeset
102 if( tcon01->singleton() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
103 // The Mul of the flattened expression
a61af66fc99e Initial load
duke
parents:
diff changeset
104 set_req(1, mul1->in(1));
a61af66fc99e Initial load
duke
parents:
diff changeset
105 set_req(2, phase->makecon( tcon01 ));
a61af66fc99e Initial load
duke
parents:
diff changeset
106 t2 = tcon01;
a61af66fc99e Initial load
duke
parents:
diff changeset
107 progress = this; // Made progress
a61af66fc99e Initial load
duke
parents:
diff changeset
108 }
a61af66fc99e Initial load
duke
parents:
diff changeset
109 }
a61af66fc99e Initial load
duke
parents:
diff changeset
110 }
a61af66fc99e Initial load
duke
parents:
diff changeset
111 // If the right input is a constant, and the left input is an add of a
a61af66fc99e Initial load
duke
parents:
diff changeset
112 // constant, flatten the tree: (X+con1)*con0 ==> X*con0 + con1*con0
a61af66fc99e Initial load
duke
parents:
diff changeset
113 const Node *add1 = in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
114 if( add1->Opcode() == add_opcode() ) { // Left input is an add?
a61af66fc99e Initial load
duke
parents:
diff changeset
115 // Add of a constant?
a61af66fc99e Initial load
duke
parents:
diff changeset
116 const Type *t12 = phase->type( add1->in(2) );
a61af66fc99e Initial load
duke
parents:
diff changeset
117 if( t12->singleton() && t12 != Type::TOP ) { // Left input is an add of a constant?
a61af66fc99e Initial load
duke
parents:
diff changeset
118 assert( add1->in(1) != add1, "dead loop in MulNode::Ideal" );
a61af66fc99e Initial load
duke
parents:
diff changeset
119 // Compute new constant; check for overflow
a61af66fc99e Initial load
duke
parents:
diff changeset
120 const Type *tcon01 = mul_ring(t2,t12);
a61af66fc99e Initial load
duke
parents:
diff changeset
121 if( tcon01->singleton() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
122
a61af66fc99e Initial load
duke
parents:
diff changeset
123 // Convert (X+con1)*con0 into X*con0
a61af66fc99e Initial load
duke
parents:
diff changeset
124 Node *mul = clone(); // mul = ()*con0
a61af66fc99e Initial load
duke
parents:
diff changeset
125 mul->set_req(1,add1->in(1)); // mul = X*con0
a61af66fc99e Initial load
duke
parents:
diff changeset
126 mul = phase->transform(mul);
a61af66fc99e Initial load
duke
parents:
diff changeset
127
a61af66fc99e Initial load
duke
parents:
diff changeset
128 Node *add2 = add1->clone();
a61af66fc99e Initial load
duke
parents:
diff changeset
129 add2->set_req(1, mul); // X*con0 + con0*con1
a61af66fc99e Initial load
duke
parents:
diff changeset
130 add2->set_req(2, phase->makecon(tcon01) );
a61af66fc99e Initial load
duke
parents:
diff changeset
131 progress = add2;
a61af66fc99e Initial load
duke
parents:
diff changeset
132 }
a61af66fc99e Initial load
duke
parents:
diff changeset
133 }
a61af66fc99e Initial load
duke
parents:
diff changeset
134 } // End of is left input an add
a61af66fc99e Initial load
duke
parents:
diff changeset
135 } // End of is right input a Mul
a61af66fc99e Initial load
duke
parents:
diff changeset
136
a61af66fc99e Initial load
duke
parents:
diff changeset
137 return progress;
a61af66fc99e Initial load
duke
parents:
diff changeset
138 }
a61af66fc99e Initial load
duke
parents:
diff changeset
139
a61af66fc99e Initial load
duke
parents:
diff changeset
140 //------------------------------Value-----------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
141 const Type *MulNode::Value( PhaseTransform *phase ) const {
a61af66fc99e Initial load
duke
parents:
diff changeset
142 const Type *t1 = phase->type( in(1) );
a61af66fc99e Initial load
duke
parents:
diff changeset
143 const Type *t2 = phase->type( in(2) );
a61af66fc99e Initial load
duke
parents:
diff changeset
144 // Either input is TOP ==> the result is TOP
a61af66fc99e Initial load
duke
parents:
diff changeset
145 if( t1 == Type::TOP ) return Type::TOP;
a61af66fc99e Initial load
duke
parents:
diff changeset
146 if( t2 == Type::TOP ) return Type::TOP;
a61af66fc99e Initial load
duke
parents:
diff changeset
147
a61af66fc99e Initial load
duke
parents:
diff changeset
148 // Either input is ZERO ==> the result is ZERO.
a61af66fc99e Initial load
duke
parents:
diff changeset
149 // Not valid for floats or doubles since +0.0 * -0.0 --> +0.0
a61af66fc99e Initial load
duke
parents:
diff changeset
150 int op = Opcode();
a61af66fc99e Initial load
duke
parents:
diff changeset
151 if( op == Op_MulI || op == Op_AndI || op == Op_MulL || op == Op_AndL ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
152 const Type *zero = add_id(); // The multiplicative zero
a61af66fc99e Initial load
duke
parents:
diff changeset
153 if( t1->higher_equal( zero ) ) return zero;
a61af66fc99e Initial load
duke
parents:
diff changeset
154 if( t2->higher_equal( zero ) ) return zero;
a61af66fc99e Initial load
duke
parents:
diff changeset
155 }
a61af66fc99e Initial load
duke
parents:
diff changeset
156
a61af66fc99e Initial load
duke
parents:
diff changeset
157 // Either input is BOTTOM ==> the result is the local BOTTOM
a61af66fc99e Initial load
duke
parents:
diff changeset
158 if( t1 == Type::BOTTOM || t2 == Type::BOTTOM )
a61af66fc99e Initial load
duke
parents:
diff changeset
159 return bottom_type();
a61af66fc99e Initial load
duke
parents:
diff changeset
160
404
78c058bc5cdc 6717150: improper constant folding of subnormal strictfp multiplications and divides
rasbold
parents: 196
diff changeset
161 #if defined(IA32)
78c058bc5cdc 6717150: improper constant folding of subnormal strictfp multiplications and divides
rasbold
parents: 196
diff changeset
162 // Can't trust native compilers to properly fold strict double
78c058bc5cdc 6717150: improper constant folding of subnormal strictfp multiplications and divides
rasbold
parents: 196
diff changeset
163 // multiplication with round-to-zero on this platform.
78c058bc5cdc 6717150: improper constant folding of subnormal strictfp multiplications and divides
rasbold
parents: 196
diff changeset
164 if (op == Op_MulD && phase->C->method()->is_strict()) {
78c058bc5cdc 6717150: improper constant folding of subnormal strictfp multiplications and divides
rasbold
parents: 196
diff changeset
165 return TypeD::DOUBLE;
78c058bc5cdc 6717150: improper constant folding of subnormal strictfp multiplications and divides
rasbold
parents: 196
diff changeset
166 }
78c058bc5cdc 6717150: improper constant folding of subnormal strictfp multiplications and divides
rasbold
parents: 196
diff changeset
167 #endif
78c058bc5cdc 6717150: improper constant folding of subnormal strictfp multiplications and divides
rasbold
parents: 196
diff changeset
168
0
a61af66fc99e Initial load
duke
parents:
diff changeset
169 return mul_ring(t1,t2); // Local flavor of type multiplication
a61af66fc99e Initial load
duke
parents:
diff changeset
170 }
a61af66fc99e Initial load
duke
parents:
diff changeset
171
a61af66fc99e Initial load
duke
parents:
diff changeset
172
a61af66fc99e Initial load
duke
parents:
diff changeset
173 //=============================================================================
a61af66fc99e Initial load
duke
parents:
diff changeset
174 //------------------------------Ideal------------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
175 // Check for power-of-2 multiply, then try the regular MulNode::Ideal
a61af66fc99e Initial load
duke
parents:
diff changeset
176 Node *MulINode::Ideal(PhaseGVN *phase, bool can_reshape) {
a61af66fc99e Initial load
duke
parents:
diff changeset
177 // Swap constant to right
a61af66fc99e Initial load
duke
parents:
diff changeset
178 jint con;
a61af66fc99e Initial load
duke
parents:
diff changeset
179 if ((con = in(1)->find_int_con(0)) != 0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
180 swap_edges(1, 2);
a61af66fc99e Initial load
duke
parents:
diff changeset
181 // Finish rest of method to use info in 'con'
a61af66fc99e Initial load
duke
parents:
diff changeset
182 } else if ((con = in(2)->find_int_con(0)) == 0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
183 return MulNode::Ideal(phase, can_reshape);
a61af66fc99e Initial load
duke
parents:
diff changeset
184 }
a61af66fc99e Initial load
duke
parents:
diff changeset
185
a61af66fc99e Initial load
duke
parents:
diff changeset
186 // Now we have a constant Node on the right and the constant in con
a61af66fc99e Initial load
duke
parents:
diff changeset
187 if( con == 0 ) return NULL; // By zero is handled by Value call
a61af66fc99e Initial load
duke
parents:
diff changeset
188 if( con == 1 ) return NULL; // By one is handled by Identity call
a61af66fc99e Initial load
duke
parents:
diff changeset
189
a61af66fc99e Initial load
duke
parents:
diff changeset
190 // Check for negative constant; if so negate the final result
a61af66fc99e Initial load
duke
parents:
diff changeset
191 bool sign_flip = false;
a61af66fc99e Initial load
duke
parents:
diff changeset
192 if( con < 0 ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
193 con = -con;
a61af66fc99e Initial load
duke
parents:
diff changeset
194 sign_flip = true;
a61af66fc99e Initial load
duke
parents:
diff changeset
195 }
a61af66fc99e Initial load
duke
parents:
diff changeset
196
a61af66fc99e Initial load
duke
parents:
diff changeset
197 // Get low bit; check for being the only bit
a61af66fc99e Initial load
duke
parents:
diff changeset
198 Node *res = NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
199 jint bit1 = con & -con; // Extract low bit
a61af66fc99e Initial load
duke
parents:
diff changeset
200 if( bit1 == con ) { // Found a power of 2?
a61af66fc99e Initial load
duke
parents:
diff changeset
201 res = new (phase->C, 3) LShiftINode( in(1), phase->intcon(log2_intptr(bit1)) );
a61af66fc99e Initial load
duke
parents:
diff changeset
202 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
203
a61af66fc99e Initial load
duke
parents:
diff changeset
204 // Check for constant with 2 bits set
a61af66fc99e Initial load
duke
parents:
diff changeset
205 jint bit2 = con-bit1;
a61af66fc99e Initial load
duke
parents:
diff changeset
206 bit2 = bit2 & -bit2; // Extract 2nd bit
a61af66fc99e Initial load
duke
parents:
diff changeset
207 if( bit2 + bit1 == con ) { // Found all bits in con?
a61af66fc99e Initial load
duke
parents:
diff changeset
208 Node *n1 = phase->transform( new (phase->C, 3) LShiftINode( in(1), phase->intcon(log2_intptr(bit1)) ) );
a61af66fc99e Initial load
duke
parents:
diff changeset
209 Node *n2 = phase->transform( new (phase->C, 3) LShiftINode( in(1), phase->intcon(log2_intptr(bit2)) ) );
a61af66fc99e Initial load
duke
parents:
diff changeset
210 res = new (phase->C, 3) AddINode( n2, n1 );
a61af66fc99e Initial load
duke
parents:
diff changeset
211
a61af66fc99e Initial load
duke
parents:
diff changeset
212 } else if (is_power_of_2(con+1)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
213 // Sleezy: power-of-2 -1. Next time be generic.
a61af66fc99e Initial load
duke
parents:
diff changeset
214 jint temp = (jint) (con + 1);
a61af66fc99e Initial load
duke
parents:
diff changeset
215 Node *n1 = phase->transform( new (phase->C, 3) LShiftINode( in(1), phase->intcon(log2_intptr(temp)) ) );
a61af66fc99e Initial load
duke
parents:
diff changeset
216 res = new (phase->C, 3) SubINode( n1, in(1) );
a61af66fc99e Initial load
duke
parents:
diff changeset
217 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
218 return MulNode::Ideal(phase, can_reshape);
a61af66fc99e Initial load
duke
parents:
diff changeset
219 }
a61af66fc99e Initial load
duke
parents:
diff changeset
220 }
a61af66fc99e Initial load
duke
parents:
diff changeset
221
a61af66fc99e Initial load
duke
parents:
diff changeset
222 if( sign_flip ) { // Need to negate result?
a61af66fc99e Initial load
duke
parents:
diff changeset
223 res = phase->transform(res);// Transform, before making the zero con
a61af66fc99e Initial load
duke
parents:
diff changeset
224 res = new (phase->C, 3) SubINode(phase->intcon(0),res);
a61af66fc99e Initial load
duke
parents:
diff changeset
225 }
a61af66fc99e Initial load
duke
parents:
diff changeset
226
a61af66fc99e Initial load
duke
parents:
diff changeset
227 return res; // Return final result
a61af66fc99e Initial load
duke
parents:
diff changeset
228 }
a61af66fc99e Initial load
duke
parents:
diff changeset
229
a61af66fc99e Initial load
duke
parents:
diff changeset
230 //------------------------------mul_ring---------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
231 // Compute the product type of two integer ranges into this node.
a61af66fc99e Initial load
duke
parents:
diff changeset
232 const Type *MulINode::mul_ring(const Type *t0, const Type *t1) const {
a61af66fc99e Initial load
duke
parents:
diff changeset
233 const TypeInt *r0 = t0->is_int(); // Handy access
a61af66fc99e Initial load
duke
parents:
diff changeset
234 const TypeInt *r1 = t1->is_int();
a61af66fc99e Initial load
duke
parents:
diff changeset
235
a61af66fc99e Initial load
duke
parents:
diff changeset
236 // Fetch endpoints of all ranges
a61af66fc99e Initial load
duke
parents:
diff changeset
237 int32 lo0 = r0->_lo;
a61af66fc99e Initial load
duke
parents:
diff changeset
238 double a = (double)lo0;
a61af66fc99e Initial load
duke
parents:
diff changeset
239 int32 hi0 = r0->_hi;
a61af66fc99e Initial load
duke
parents:
diff changeset
240 double b = (double)hi0;
a61af66fc99e Initial load
duke
parents:
diff changeset
241 int32 lo1 = r1->_lo;
a61af66fc99e Initial load
duke
parents:
diff changeset
242 double c = (double)lo1;
a61af66fc99e Initial load
duke
parents:
diff changeset
243 int32 hi1 = r1->_hi;
a61af66fc99e Initial load
duke
parents:
diff changeset
244 double d = (double)hi1;
a61af66fc99e Initial load
duke
parents:
diff changeset
245
a61af66fc99e Initial load
duke
parents:
diff changeset
246 // Compute all endpoints & check for overflow
a61af66fc99e Initial load
duke
parents:
diff changeset
247 int32 A = lo0*lo1;
a61af66fc99e Initial load
duke
parents:
diff changeset
248 if( (double)A != a*c ) return TypeInt::INT; // Overflow?
a61af66fc99e Initial load
duke
parents:
diff changeset
249 int32 B = lo0*hi1;
a61af66fc99e Initial load
duke
parents:
diff changeset
250 if( (double)B != a*d ) return TypeInt::INT; // Overflow?
a61af66fc99e Initial load
duke
parents:
diff changeset
251 int32 C = hi0*lo1;
a61af66fc99e Initial load
duke
parents:
diff changeset
252 if( (double)C != b*c ) return TypeInt::INT; // Overflow?
a61af66fc99e Initial load
duke
parents:
diff changeset
253 int32 D = hi0*hi1;
a61af66fc99e Initial load
duke
parents:
diff changeset
254 if( (double)D != b*d ) return TypeInt::INT; // Overflow?
a61af66fc99e Initial load
duke
parents:
diff changeset
255
a61af66fc99e Initial load
duke
parents:
diff changeset
256 if( A < B ) { lo0 = A; hi0 = B; } // Sort range endpoints
a61af66fc99e Initial load
duke
parents:
diff changeset
257 else { lo0 = B; hi0 = A; }
a61af66fc99e Initial load
duke
parents:
diff changeset
258 if( C < D ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
259 if( C < lo0 ) lo0 = C;
a61af66fc99e Initial load
duke
parents:
diff changeset
260 if( D > hi0 ) hi0 = D;
a61af66fc99e Initial load
duke
parents:
diff changeset
261 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
262 if( D < lo0 ) lo0 = D;
a61af66fc99e Initial load
duke
parents:
diff changeset
263 if( C > hi0 ) hi0 = C;
a61af66fc99e Initial load
duke
parents:
diff changeset
264 }
a61af66fc99e Initial load
duke
parents:
diff changeset
265 return TypeInt::make(lo0, hi0, MAX2(r0->_widen,r1->_widen));
a61af66fc99e Initial load
duke
parents:
diff changeset
266 }
a61af66fc99e Initial load
duke
parents:
diff changeset
267
a61af66fc99e Initial load
duke
parents:
diff changeset
268
a61af66fc99e Initial load
duke
parents:
diff changeset
269 //=============================================================================
a61af66fc99e Initial load
duke
parents:
diff changeset
270 //------------------------------Ideal------------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
271 // Check for power-of-2 multiply, then try the regular MulNode::Ideal
a61af66fc99e Initial load
duke
parents:
diff changeset
272 Node *MulLNode::Ideal(PhaseGVN *phase, bool can_reshape) {
a61af66fc99e Initial load
duke
parents:
diff changeset
273 // Swap constant to right
a61af66fc99e Initial load
duke
parents:
diff changeset
274 jlong con;
a61af66fc99e Initial load
duke
parents:
diff changeset
275 if ((con = in(1)->find_long_con(0)) != 0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
276 swap_edges(1, 2);
a61af66fc99e Initial load
duke
parents:
diff changeset
277 // Finish rest of method to use info in 'con'
a61af66fc99e Initial load
duke
parents:
diff changeset
278 } else if ((con = in(2)->find_long_con(0)) == 0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
279 return MulNode::Ideal(phase, can_reshape);
a61af66fc99e Initial load
duke
parents:
diff changeset
280 }
a61af66fc99e Initial load
duke
parents:
diff changeset
281
a61af66fc99e Initial load
duke
parents:
diff changeset
282 // Now we have a constant Node on the right and the constant in con
a61af66fc99e Initial load
duke
parents:
diff changeset
283 if( con == CONST64(0) ) return NULL; // By zero is handled by Value call
a61af66fc99e Initial load
duke
parents:
diff changeset
284 if( con == CONST64(1) ) return NULL; // By one is handled by Identity call
a61af66fc99e Initial load
duke
parents:
diff changeset
285
a61af66fc99e Initial load
duke
parents:
diff changeset
286 // Check for negative constant; if so negate the final result
a61af66fc99e Initial load
duke
parents:
diff changeset
287 bool sign_flip = false;
a61af66fc99e Initial load
duke
parents:
diff changeset
288 if( con < 0 ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
289 con = -con;
a61af66fc99e Initial load
duke
parents:
diff changeset
290 sign_flip = true;
a61af66fc99e Initial load
duke
parents:
diff changeset
291 }
a61af66fc99e Initial load
duke
parents:
diff changeset
292
a61af66fc99e Initial load
duke
parents:
diff changeset
293 // Get low bit; check for being the only bit
a61af66fc99e Initial load
duke
parents:
diff changeset
294 Node *res = NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
295 jlong bit1 = con & -con; // Extract low bit
a61af66fc99e Initial load
duke
parents:
diff changeset
296 if( bit1 == con ) { // Found a power of 2?
a61af66fc99e Initial load
duke
parents:
diff changeset
297 res = new (phase->C, 3) LShiftLNode( in(1), phase->intcon(log2_long(bit1)) );
a61af66fc99e Initial load
duke
parents:
diff changeset
298 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
299
a61af66fc99e Initial load
duke
parents:
diff changeset
300 // Check for constant with 2 bits set
a61af66fc99e Initial load
duke
parents:
diff changeset
301 jlong bit2 = con-bit1;
a61af66fc99e Initial load
duke
parents:
diff changeset
302 bit2 = bit2 & -bit2; // Extract 2nd bit
a61af66fc99e Initial load
duke
parents:
diff changeset
303 if( bit2 + bit1 == con ) { // Found all bits in con?
a61af66fc99e Initial load
duke
parents:
diff changeset
304 Node *n1 = phase->transform( new (phase->C, 3) LShiftLNode( in(1), phase->intcon(log2_long(bit1)) ) );
a61af66fc99e Initial load
duke
parents:
diff changeset
305 Node *n2 = phase->transform( new (phase->C, 3) LShiftLNode( in(1), phase->intcon(log2_long(bit2)) ) );
a61af66fc99e Initial load
duke
parents:
diff changeset
306 res = new (phase->C, 3) AddLNode( n2, n1 );
a61af66fc99e Initial load
duke
parents:
diff changeset
307
a61af66fc99e Initial load
duke
parents:
diff changeset
308 } else if (is_power_of_2_long(con+1)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
309 // Sleezy: power-of-2 -1. Next time be generic.
a61af66fc99e Initial load
duke
parents:
diff changeset
310 jlong temp = (jlong) (con + 1);
a61af66fc99e Initial load
duke
parents:
diff changeset
311 Node *n1 = phase->transform( new (phase->C, 3) LShiftLNode( in(1), phase->intcon(log2_long(temp)) ) );
a61af66fc99e Initial load
duke
parents:
diff changeset
312 res = new (phase->C, 3) SubLNode( n1, in(1) );
a61af66fc99e Initial load
duke
parents:
diff changeset
313 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
314 return MulNode::Ideal(phase, can_reshape);
a61af66fc99e Initial load
duke
parents:
diff changeset
315 }
a61af66fc99e Initial load
duke
parents:
diff changeset
316 }
a61af66fc99e Initial load
duke
parents:
diff changeset
317
a61af66fc99e Initial load
duke
parents:
diff changeset
318 if( sign_flip ) { // Need to negate result?
a61af66fc99e Initial load
duke
parents:
diff changeset
319 res = phase->transform(res);// Transform, before making the zero con
a61af66fc99e Initial load
duke
parents:
diff changeset
320 res = new (phase->C, 3) SubLNode(phase->longcon(0),res);
a61af66fc99e Initial load
duke
parents:
diff changeset
321 }
a61af66fc99e Initial load
duke
parents:
diff changeset
322
a61af66fc99e Initial load
duke
parents:
diff changeset
323 return res; // Return final result
a61af66fc99e Initial load
duke
parents:
diff changeset
324 }
a61af66fc99e Initial load
duke
parents:
diff changeset
325
a61af66fc99e Initial load
duke
parents:
diff changeset
326 //------------------------------mul_ring---------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
327 // Compute the product type of two integer ranges into this node.
a61af66fc99e Initial load
duke
parents:
diff changeset
328 const Type *MulLNode::mul_ring(const Type *t0, const Type *t1) const {
a61af66fc99e Initial load
duke
parents:
diff changeset
329 const TypeLong *r0 = t0->is_long(); // Handy access
a61af66fc99e Initial load
duke
parents:
diff changeset
330 const TypeLong *r1 = t1->is_long();
a61af66fc99e Initial load
duke
parents:
diff changeset
331
a61af66fc99e Initial load
duke
parents:
diff changeset
332 // Fetch endpoints of all ranges
a61af66fc99e Initial load
duke
parents:
diff changeset
333 jlong lo0 = r0->_lo;
a61af66fc99e Initial load
duke
parents:
diff changeset
334 double a = (double)lo0;
a61af66fc99e Initial load
duke
parents:
diff changeset
335 jlong hi0 = r0->_hi;
a61af66fc99e Initial load
duke
parents:
diff changeset
336 double b = (double)hi0;
a61af66fc99e Initial load
duke
parents:
diff changeset
337 jlong lo1 = r1->_lo;
a61af66fc99e Initial load
duke
parents:
diff changeset
338 double c = (double)lo1;
a61af66fc99e Initial load
duke
parents:
diff changeset
339 jlong hi1 = r1->_hi;
a61af66fc99e Initial load
duke
parents:
diff changeset
340 double d = (double)hi1;
a61af66fc99e Initial load
duke
parents:
diff changeset
341
a61af66fc99e Initial load
duke
parents:
diff changeset
342 // Compute all endpoints & check for overflow
a61af66fc99e Initial load
duke
parents:
diff changeset
343 jlong A = lo0*lo1;
a61af66fc99e Initial load
duke
parents:
diff changeset
344 if( (double)A != a*c ) return TypeLong::LONG; // Overflow?
a61af66fc99e Initial load
duke
parents:
diff changeset
345 jlong B = lo0*hi1;
a61af66fc99e Initial load
duke
parents:
diff changeset
346 if( (double)B != a*d ) return TypeLong::LONG; // Overflow?
a61af66fc99e Initial load
duke
parents:
diff changeset
347 jlong C = hi0*lo1;
a61af66fc99e Initial load
duke
parents:
diff changeset
348 if( (double)C != b*c ) return TypeLong::LONG; // Overflow?
a61af66fc99e Initial load
duke
parents:
diff changeset
349 jlong D = hi0*hi1;
a61af66fc99e Initial load
duke
parents:
diff changeset
350 if( (double)D != b*d ) return TypeLong::LONG; // Overflow?
a61af66fc99e Initial load
duke
parents:
diff changeset
351
a61af66fc99e Initial load
duke
parents:
diff changeset
352 if( A < B ) { lo0 = A; hi0 = B; } // Sort range endpoints
a61af66fc99e Initial load
duke
parents:
diff changeset
353 else { lo0 = B; hi0 = A; }
a61af66fc99e Initial load
duke
parents:
diff changeset
354 if( C < D ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
355 if( C < lo0 ) lo0 = C;
a61af66fc99e Initial load
duke
parents:
diff changeset
356 if( D > hi0 ) hi0 = D;
a61af66fc99e Initial load
duke
parents:
diff changeset
357 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
358 if( D < lo0 ) lo0 = D;
a61af66fc99e Initial load
duke
parents:
diff changeset
359 if( C > hi0 ) hi0 = C;
a61af66fc99e Initial load
duke
parents:
diff changeset
360 }
a61af66fc99e Initial load
duke
parents:
diff changeset
361 return TypeLong::make(lo0, hi0, MAX2(r0->_widen,r1->_widen));
a61af66fc99e Initial load
duke
parents:
diff changeset
362 }
a61af66fc99e Initial load
duke
parents:
diff changeset
363
a61af66fc99e Initial load
duke
parents:
diff changeset
364 //=============================================================================
a61af66fc99e Initial load
duke
parents:
diff changeset
365 //------------------------------mul_ring---------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
366 // Compute the product type of two double ranges into this node.
a61af66fc99e Initial load
duke
parents:
diff changeset
367 const Type *MulFNode::mul_ring(const Type *t0, const Type *t1) const {
a61af66fc99e Initial load
duke
parents:
diff changeset
368 if( t0 == Type::FLOAT || t1 == Type::FLOAT ) return Type::FLOAT;
a61af66fc99e Initial load
duke
parents:
diff changeset
369 return TypeF::make( t0->getf() * t1->getf() );
a61af66fc99e Initial load
duke
parents:
diff changeset
370 }
a61af66fc99e Initial load
duke
parents:
diff changeset
371
a61af66fc99e Initial load
duke
parents:
diff changeset
372 //=============================================================================
a61af66fc99e Initial load
duke
parents:
diff changeset
373 //------------------------------mul_ring---------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
374 // Compute the product type of two double ranges into this node.
a61af66fc99e Initial load
duke
parents:
diff changeset
375 const Type *MulDNode::mul_ring(const Type *t0, const Type *t1) const {
a61af66fc99e Initial load
duke
parents:
diff changeset
376 if( t0 == Type::DOUBLE || t1 == Type::DOUBLE ) return Type::DOUBLE;
404
78c058bc5cdc 6717150: improper constant folding of subnormal strictfp multiplications and divides
rasbold
parents: 196
diff changeset
377 // We must be multiplying 2 double constants.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
378 return TypeD::make( t0->getd() * t1->getd() );
a61af66fc99e Initial load
duke
parents:
diff changeset
379 }
a61af66fc99e Initial load
duke
parents:
diff changeset
380
a61af66fc99e Initial load
duke
parents:
diff changeset
381 //=============================================================================
145
f3de1255b035 6603011: RFE: Optimize long division
rasbold
parents: 0
diff changeset
382 //------------------------------Value------------------------------------------
f3de1255b035 6603011: RFE: Optimize long division
rasbold
parents: 0
diff changeset
383 const Type *MulHiLNode::Value( PhaseTransform *phase ) const {
f3de1255b035 6603011: RFE: Optimize long division
rasbold
parents: 0
diff changeset
384 // Either input is TOP ==> the result is TOP
f3de1255b035 6603011: RFE: Optimize long division
rasbold
parents: 0
diff changeset
385 const Type *t1 = phase->type( in(1) );
f3de1255b035 6603011: RFE: Optimize long division
rasbold
parents: 0
diff changeset
386 const Type *t2 = phase->type( in(2) );
f3de1255b035 6603011: RFE: Optimize long division
rasbold
parents: 0
diff changeset
387 if( t1 == Type::TOP ) return Type::TOP;
f3de1255b035 6603011: RFE: Optimize long division
rasbold
parents: 0
diff changeset
388 if( t2 == Type::TOP ) return Type::TOP;
f3de1255b035 6603011: RFE: Optimize long division
rasbold
parents: 0
diff changeset
389
f3de1255b035 6603011: RFE: Optimize long division
rasbold
parents: 0
diff changeset
390 // Either input is BOTTOM ==> the result is the local BOTTOM
f3de1255b035 6603011: RFE: Optimize long division
rasbold
parents: 0
diff changeset
391 const Type *bot = bottom_type();
f3de1255b035 6603011: RFE: Optimize long division
rasbold
parents: 0
diff changeset
392 if( (t1 == bot) || (t2 == bot) ||
f3de1255b035 6603011: RFE: Optimize long division
rasbold
parents: 0
diff changeset
393 (t1 == Type::BOTTOM) || (t2 == Type::BOTTOM) )
f3de1255b035 6603011: RFE: Optimize long division
rasbold
parents: 0
diff changeset
394 return bot;
f3de1255b035 6603011: RFE: Optimize long division
rasbold
parents: 0
diff changeset
395
f3de1255b035 6603011: RFE: Optimize long division
rasbold
parents: 0
diff changeset
396 // It is not worth trying to constant fold this stuff!
f3de1255b035 6603011: RFE: Optimize long division
rasbold
parents: 0
diff changeset
397 return TypeLong::LONG;
f3de1255b035 6603011: RFE: Optimize long division
rasbold
parents: 0
diff changeset
398 }
f3de1255b035 6603011: RFE: Optimize long division
rasbold
parents: 0
diff changeset
399
f3de1255b035 6603011: RFE: Optimize long division
rasbold
parents: 0
diff changeset
400 //=============================================================================
0
a61af66fc99e Initial load
duke
parents:
diff changeset
401 //------------------------------mul_ring---------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
402 // Supplied function returns the product of the inputs IN THE CURRENT RING.
a61af66fc99e Initial load
duke
parents:
diff changeset
403 // For the logical operations the ring's MUL is really a logical AND function.
a61af66fc99e Initial load
duke
parents:
diff changeset
404 // This also type-checks the inputs for sanity. Guaranteed never to
a61af66fc99e Initial load
duke
parents:
diff changeset
405 // be passed a TOP or BOTTOM type, these are filtered out by pre-check.
a61af66fc99e Initial load
duke
parents:
diff changeset
406 const Type *AndINode::mul_ring( const Type *t0, const Type *t1 ) const {
a61af66fc99e Initial load
duke
parents:
diff changeset
407 const TypeInt *r0 = t0->is_int(); // Handy access
a61af66fc99e Initial load
duke
parents:
diff changeset
408 const TypeInt *r1 = t1->is_int();
a61af66fc99e Initial load
duke
parents:
diff changeset
409 int widen = MAX2(r0->_widen,r1->_widen);
a61af66fc99e Initial load
duke
parents:
diff changeset
410
a61af66fc99e Initial load
duke
parents:
diff changeset
411 // If either input is a constant, might be able to trim cases
a61af66fc99e Initial load
duke
parents:
diff changeset
412 if( !r0->is_con() && !r1->is_con() )
a61af66fc99e Initial load
duke
parents:
diff changeset
413 return TypeInt::INT; // No constants to be had
a61af66fc99e Initial load
duke
parents:
diff changeset
414
a61af66fc99e Initial load
duke
parents:
diff changeset
415 // Both constants? Return bits
a61af66fc99e Initial load
duke
parents:
diff changeset
416 if( r0->is_con() && r1->is_con() )
a61af66fc99e Initial load
duke
parents:
diff changeset
417 return TypeInt::make( r0->get_con() & r1->get_con() );
a61af66fc99e Initial load
duke
parents:
diff changeset
418
a61af66fc99e Initial load
duke
parents:
diff changeset
419 if( r0->is_con() && r0->get_con() > 0 )
a61af66fc99e Initial load
duke
parents:
diff changeset
420 return TypeInt::make(0, r0->get_con(), widen);
a61af66fc99e Initial load
duke
parents:
diff changeset
421
a61af66fc99e Initial load
duke
parents:
diff changeset
422 if( r1->is_con() && r1->get_con() > 0 )
a61af66fc99e Initial load
duke
parents:
diff changeset
423 return TypeInt::make(0, r1->get_con(), widen);
a61af66fc99e Initial load
duke
parents:
diff changeset
424
a61af66fc99e Initial load
duke
parents:
diff changeset
425 if( r0 == TypeInt::BOOL || r1 == TypeInt::BOOL ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
426 return TypeInt::BOOL;
a61af66fc99e Initial load
duke
parents:
diff changeset
427 }
a61af66fc99e Initial load
duke
parents:
diff changeset
428
a61af66fc99e Initial load
duke
parents:
diff changeset
429 return TypeInt::INT; // No constants to be had
a61af66fc99e Initial load
duke
parents:
diff changeset
430 }
a61af66fc99e Initial load
duke
parents:
diff changeset
431
a61af66fc99e Initial load
duke
parents:
diff changeset
432 //------------------------------Identity---------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
433 // Masking off the high bits of an unsigned load is not required
a61af66fc99e Initial load
duke
parents:
diff changeset
434 Node *AndINode::Identity( PhaseTransform *phase ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
435
a61af66fc99e Initial load
duke
parents:
diff changeset
436 // x & x => x
a61af66fc99e Initial load
duke
parents:
diff changeset
437 if (phase->eqv(in(1), in(2))) return in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
438
824
18a08a7e16b5 5057225: Remove useless I2L conversions
twisti
parents: 624
diff changeset
439 Node* in1 = in(1);
18a08a7e16b5 5057225: Remove useless I2L conversions
twisti
parents: 624
diff changeset
440 uint op = in1->Opcode();
18a08a7e16b5 5057225: Remove useless I2L conversions
twisti
parents: 624
diff changeset
441 const TypeInt* t2 = phase->type(in(2))->isa_int();
18a08a7e16b5 5057225: Remove useless I2L conversions
twisti
parents: 624
diff changeset
442 if (t2 && t2->is_con()) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
443 int con = t2->get_con();
a61af66fc99e Initial load
duke
parents:
diff changeset
444 // Masking off high bits which are always zero is useless.
a61af66fc99e Initial load
duke
parents:
diff changeset
445 const TypeInt* t1 = phase->type( in(1) )->isa_int();
a61af66fc99e Initial load
duke
parents:
diff changeset
446 if (t1 != NULL && t1->_lo >= 0) {
824
18a08a7e16b5 5057225: Remove useless I2L conversions
twisti
parents: 624
diff changeset
447 jint t1_support = right_n_bits(1 + log2_intptr(t1->_hi));
0
a61af66fc99e Initial load
duke
parents:
diff changeset
448 if ((t1_support & con) == t1_support)
824
18a08a7e16b5 5057225: Remove useless I2L conversions
twisti
parents: 624
diff changeset
449 return in1;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
450 }
a61af66fc99e Initial load
duke
parents:
diff changeset
451 // Masking off the high bits of a unsigned-shift-right is not
a61af66fc99e Initial load
duke
parents:
diff changeset
452 // needed either.
824
18a08a7e16b5 5057225: Remove useless I2L conversions
twisti
parents: 624
diff changeset
453 if (op == Op_URShiftI) {
18a08a7e16b5 5057225: Remove useless I2L conversions
twisti
parents: 624
diff changeset
454 const TypeInt* t12 = phase->type(in1->in(2))->isa_int();
18a08a7e16b5 5057225: Remove useless I2L conversions
twisti
parents: 624
diff changeset
455 if (t12 && t12->is_con()) { // Shift is by a constant
559
7628781568e1 6795362: 32bit server compiler leads to wrong results on solaris-x86
twisti
parents: 558
diff changeset
456 int shift = t12->get_con();
7628781568e1 6795362: 32bit server compiler leads to wrong results on solaris-x86
twisti
parents: 558
diff changeset
457 shift &= BitsPerJavaInteger - 1; // semantics of Java shifts
7628781568e1 6795362: 32bit server compiler leads to wrong results on solaris-x86
twisti
parents: 558
diff changeset
458 int mask = max_juint >> shift;
824
18a08a7e16b5 5057225: Remove useless I2L conversions
twisti
parents: 624
diff changeset
459 if ((mask & con) == mask) // If AND is useless, skip it
18a08a7e16b5 5057225: Remove useless I2L conversions
twisti
parents: 624
diff changeset
460 return in1;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
461 }
a61af66fc99e Initial load
duke
parents:
diff changeset
462 }
a61af66fc99e Initial load
duke
parents:
diff changeset
463 }
a61af66fc99e Initial load
duke
parents:
diff changeset
464 return MulNode::Identity(phase);
a61af66fc99e Initial load
duke
parents:
diff changeset
465 }
a61af66fc99e Initial load
duke
parents:
diff changeset
466
a61af66fc99e Initial load
duke
parents:
diff changeset
467 //------------------------------Ideal------------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
468 Node *AndINode::Ideal(PhaseGVN *phase, bool can_reshape) {
a61af66fc99e Initial load
duke
parents:
diff changeset
469 // Special case constant AND mask
a61af66fc99e Initial load
duke
parents:
diff changeset
470 const TypeInt *t2 = phase->type( in(2) )->isa_int();
a61af66fc99e Initial load
duke
parents:
diff changeset
471 if( !t2 || !t2->is_con() ) return MulNode::Ideal(phase, can_reshape);
a61af66fc99e Initial load
duke
parents:
diff changeset
472 const int mask = t2->get_con();
a61af66fc99e Initial load
duke
parents:
diff changeset
473 Node *load = in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
474 uint lop = load->Opcode();
a61af66fc99e Initial load
duke
parents:
diff changeset
475
a61af66fc99e Initial load
duke
parents:
diff changeset
476 // Masking bits off of a Character? Hi bits are already zero.
558
3b5ac9e7e6ea 6796746: rename LoadC (char) opcode class to LoadUS (unsigned short)
twisti
parents: 404
diff changeset
477 if( lop == Op_LoadUS &&
0
a61af66fc99e Initial load
duke
parents:
diff changeset
478 (mask & 0xFFFF0000) ) // Can we make a smaller mask?
a61af66fc99e Initial load
duke
parents:
diff changeset
479 return new (phase->C, 3) AndINode(load,phase->intcon(mask&0xFFFF));
a61af66fc99e Initial load
duke
parents:
diff changeset
480
a61af66fc99e Initial load
duke
parents:
diff changeset
481 // Masking bits off of a Short? Loading a Character does some masking
824
18a08a7e16b5 5057225: Remove useless I2L conversions
twisti
parents: 624
diff changeset
482 if (lop == Op_LoadS && (mask & 0xFFFF0000) == 0 ) {
558
3b5ac9e7e6ea 6796746: rename LoadC (char) opcode class to LoadUS (unsigned short)
twisti
parents: 404
diff changeset
483 Node *ldus = new (phase->C, 3) LoadUSNode(load->in(MemNode::Control),
824
18a08a7e16b5 5057225: Remove useless I2L conversions
twisti
parents: 624
diff changeset
484 load->in(MemNode::Memory),
18a08a7e16b5 5057225: Remove useless I2L conversions
twisti
parents: 624
diff changeset
485 load->in(MemNode::Address),
18a08a7e16b5 5057225: Remove useless I2L conversions
twisti
parents: 624
diff changeset
486 load->adr_type());
558
3b5ac9e7e6ea 6796746: rename LoadC (char) opcode class to LoadUS (unsigned short)
twisti
parents: 404
diff changeset
487 ldus = phase->transform(ldus);
824
18a08a7e16b5 5057225: Remove useless I2L conversions
twisti
parents: 624
diff changeset
488 return new (phase->C, 3) AndINode(ldus, phase->intcon(mask & 0xFFFF));
0
a61af66fc99e Initial load
duke
parents:
diff changeset
489 }
a61af66fc99e Initial load
duke
parents:
diff changeset
490
824
18a08a7e16b5 5057225: Remove useless I2L conversions
twisti
parents: 624
diff changeset
491 // Masking sign bits off of a Byte? Do an unsigned byte load plus
18a08a7e16b5 5057225: Remove useless I2L conversions
twisti
parents: 624
diff changeset
492 // an and.
624
337400e7a5dd 6797305: Add LoadUB and LoadUI opcode class
twisti
parents: 559
diff changeset
493 if (lop == Op_LoadB && (mask & 0xFFFFFF00) == 0) {
337400e7a5dd 6797305: Add LoadUB and LoadUI opcode class
twisti
parents: 559
diff changeset
494 Node* ldub = new (phase->C, 3) LoadUBNode(load->in(MemNode::Control),
337400e7a5dd 6797305: Add LoadUB and LoadUI opcode class
twisti
parents: 559
diff changeset
495 load->in(MemNode::Memory),
337400e7a5dd 6797305: Add LoadUB and LoadUI opcode class
twisti
parents: 559
diff changeset
496 load->in(MemNode::Address),
337400e7a5dd 6797305: Add LoadUB and LoadUI opcode class
twisti
parents: 559
diff changeset
497 load->adr_type());
337400e7a5dd 6797305: Add LoadUB and LoadUI opcode class
twisti
parents: 559
diff changeset
498 ldub = phase->transform(ldub);
337400e7a5dd 6797305: Add LoadUB and LoadUI opcode class
twisti
parents: 559
diff changeset
499 return new (phase->C, 3) AndINode(ldub, phase->intcon(mask));
0
a61af66fc99e Initial load
duke
parents:
diff changeset
500 }
a61af66fc99e Initial load
duke
parents:
diff changeset
501
a61af66fc99e Initial load
duke
parents:
diff changeset
502 // Masking off sign bits? Dont make them!
a61af66fc99e Initial load
duke
parents:
diff changeset
503 if( lop == Op_RShiftI ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
504 const TypeInt *t12 = phase->type(load->in(2))->isa_int();
a61af66fc99e Initial load
duke
parents:
diff changeset
505 if( t12 && t12->is_con() ) { // Shift is by a constant
a61af66fc99e Initial load
duke
parents:
diff changeset
506 int shift = t12->get_con();
a61af66fc99e Initial load
duke
parents:
diff changeset
507 shift &= BitsPerJavaInteger-1; // semantics of Java shifts
a61af66fc99e Initial load
duke
parents:
diff changeset
508 const int sign_bits_mask = ~right_n_bits(BitsPerJavaInteger - shift);
a61af66fc99e Initial load
duke
parents:
diff changeset
509 // If the AND'ing of the 2 masks has no bits, then only original shifted
a61af66fc99e Initial load
duke
parents:
diff changeset
510 // bits survive. NO sign-extension bits survive the maskings.
a61af66fc99e Initial load
duke
parents:
diff changeset
511 if( (sign_bits_mask & mask) == 0 ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
512 // Use zero-fill shift instead
a61af66fc99e Initial load
duke
parents:
diff changeset
513 Node *zshift = phase->transform(new (phase->C, 3) URShiftINode(load->in(1),load->in(2)));
a61af66fc99e Initial load
duke
parents:
diff changeset
514 return new (phase->C, 3) AndINode( zshift, in(2) );
a61af66fc99e Initial load
duke
parents:
diff changeset
515 }
a61af66fc99e Initial load
duke
parents:
diff changeset
516 }
a61af66fc99e Initial load
duke
parents:
diff changeset
517 }
a61af66fc99e Initial load
duke
parents:
diff changeset
518
a61af66fc99e Initial load
duke
parents:
diff changeset
519 // Check for 'negate/and-1', a pattern emitted when someone asks for
a61af66fc99e Initial load
duke
parents:
diff changeset
520 // 'mod 2'. Negate leaves the low order bit unchanged (think: complement
a61af66fc99e Initial load
duke
parents:
diff changeset
521 // plus 1) and the mask is of the low order bit. Skip the negate.
a61af66fc99e Initial load
duke
parents:
diff changeset
522 if( lop == Op_SubI && mask == 1 && load->in(1) &&
a61af66fc99e Initial load
duke
parents:
diff changeset
523 phase->type(load->in(1)) == TypeInt::ZERO )
a61af66fc99e Initial load
duke
parents:
diff changeset
524 return new (phase->C, 3) AndINode( load->in(2), in(2) );
a61af66fc99e Initial load
duke
parents:
diff changeset
525
a61af66fc99e Initial load
duke
parents:
diff changeset
526 return MulNode::Ideal(phase, can_reshape);
a61af66fc99e Initial load
duke
parents:
diff changeset
527 }
a61af66fc99e Initial load
duke
parents:
diff changeset
528
a61af66fc99e Initial load
duke
parents:
diff changeset
529 //=============================================================================
a61af66fc99e Initial load
duke
parents:
diff changeset
530 //------------------------------mul_ring---------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
531 // Supplied function returns the product of the inputs IN THE CURRENT RING.
a61af66fc99e Initial load
duke
parents:
diff changeset
532 // For the logical operations the ring's MUL is really a logical AND function.
a61af66fc99e Initial load
duke
parents:
diff changeset
533 // This also type-checks the inputs for sanity. Guaranteed never to
a61af66fc99e Initial load
duke
parents:
diff changeset
534 // be passed a TOP or BOTTOM type, these are filtered out by pre-check.
a61af66fc99e Initial load
duke
parents:
diff changeset
535 const Type *AndLNode::mul_ring( const Type *t0, const Type *t1 ) const {
a61af66fc99e Initial load
duke
parents:
diff changeset
536 const TypeLong *r0 = t0->is_long(); // Handy access
a61af66fc99e Initial load
duke
parents:
diff changeset
537 const TypeLong *r1 = t1->is_long();
a61af66fc99e Initial load
duke
parents:
diff changeset
538 int widen = MAX2(r0->_widen,r1->_widen);
a61af66fc99e Initial load
duke
parents:
diff changeset
539
a61af66fc99e Initial load
duke
parents:
diff changeset
540 // If either input is a constant, might be able to trim cases
a61af66fc99e Initial load
duke
parents:
diff changeset
541 if( !r0->is_con() && !r1->is_con() )
a61af66fc99e Initial load
duke
parents:
diff changeset
542 return TypeLong::LONG; // No constants to be had
a61af66fc99e Initial load
duke
parents:
diff changeset
543
a61af66fc99e Initial load
duke
parents:
diff changeset
544 // Both constants? Return bits
a61af66fc99e Initial load
duke
parents:
diff changeset
545 if( r0->is_con() && r1->is_con() )
a61af66fc99e Initial load
duke
parents:
diff changeset
546 return TypeLong::make( r0->get_con() & r1->get_con() );
a61af66fc99e Initial load
duke
parents:
diff changeset
547
a61af66fc99e Initial load
duke
parents:
diff changeset
548 if( r0->is_con() && r0->get_con() > 0 )
a61af66fc99e Initial load
duke
parents:
diff changeset
549 return TypeLong::make(CONST64(0), r0->get_con(), widen);
a61af66fc99e Initial load
duke
parents:
diff changeset
550
a61af66fc99e Initial load
duke
parents:
diff changeset
551 if( r1->is_con() && r1->get_con() > 0 )
a61af66fc99e Initial load
duke
parents:
diff changeset
552 return TypeLong::make(CONST64(0), r1->get_con(), widen);
a61af66fc99e Initial load
duke
parents:
diff changeset
553
a61af66fc99e Initial load
duke
parents:
diff changeset
554 return TypeLong::LONG; // No constants to be had
a61af66fc99e Initial load
duke
parents:
diff changeset
555 }
a61af66fc99e Initial load
duke
parents:
diff changeset
556
a61af66fc99e Initial load
duke
parents:
diff changeset
557 //------------------------------Identity---------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
558 // Masking off the high bits of an unsigned load is not required
a61af66fc99e Initial load
duke
parents:
diff changeset
559 Node *AndLNode::Identity( PhaseTransform *phase ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
560
a61af66fc99e Initial load
duke
parents:
diff changeset
561 // x & x => x
a61af66fc99e Initial load
duke
parents:
diff changeset
562 if (phase->eqv(in(1), in(2))) return in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
563
a61af66fc99e Initial load
duke
parents:
diff changeset
564 Node *usr = in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
565 const TypeLong *t2 = phase->type( in(2) )->isa_long();
a61af66fc99e Initial load
duke
parents:
diff changeset
566 if( t2 && t2->is_con() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
567 jlong con = t2->get_con();
a61af66fc99e Initial load
duke
parents:
diff changeset
568 // Masking off high bits which are always zero is useless.
a61af66fc99e Initial load
duke
parents:
diff changeset
569 const TypeLong* t1 = phase->type( in(1) )->isa_long();
a61af66fc99e Initial load
duke
parents:
diff changeset
570 if (t1 != NULL && t1->_lo >= 0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
571 jlong t1_support = ((jlong)1 << (1 + log2_long(t1->_hi))) - 1;
a61af66fc99e Initial load
duke
parents:
diff changeset
572 if ((t1_support & con) == t1_support)
a61af66fc99e Initial load
duke
parents:
diff changeset
573 return usr;
a61af66fc99e Initial load
duke
parents:
diff changeset
574 }
a61af66fc99e Initial load
duke
parents:
diff changeset
575 uint lop = usr->Opcode();
a61af66fc99e Initial load
duke
parents:
diff changeset
576 // Masking off the high bits of a unsigned-shift-right is not
a61af66fc99e Initial load
duke
parents:
diff changeset
577 // needed either.
a61af66fc99e Initial load
duke
parents:
diff changeset
578 if( lop == Op_URShiftL ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
579 const TypeInt *t12 = phase->type( usr->in(2) )->isa_int();
559
7628781568e1 6795362: 32bit server compiler leads to wrong results on solaris-x86
twisti
parents: 558
diff changeset
580 if( t12 && t12->is_con() ) { // Shift is by a constant
7628781568e1 6795362: 32bit server compiler leads to wrong results on solaris-x86
twisti
parents: 558
diff changeset
581 int shift = t12->get_con();
7628781568e1 6795362: 32bit server compiler leads to wrong results on solaris-x86
twisti
parents: 558
diff changeset
582 shift &= BitsPerJavaLong - 1; // semantics of Java shifts
7628781568e1 6795362: 32bit server compiler leads to wrong results on solaris-x86
twisti
parents: 558
diff changeset
583 jlong mask = max_julong >> shift;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
584 if( (mask&con) == mask ) // If AND is useless, skip it
a61af66fc99e Initial load
duke
parents:
diff changeset
585 return usr;
a61af66fc99e Initial load
duke
parents:
diff changeset
586 }
a61af66fc99e Initial load
duke
parents:
diff changeset
587 }
a61af66fc99e Initial load
duke
parents:
diff changeset
588 }
a61af66fc99e Initial load
duke
parents:
diff changeset
589 return MulNode::Identity(phase);
a61af66fc99e Initial load
duke
parents:
diff changeset
590 }
a61af66fc99e Initial load
duke
parents:
diff changeset
591
a61af66fc99e Initial load
duke
parents:
diff changeset
592 //------------------------------Ideal------------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
593 Node *AndLNode::Ideal(PhaseGVN *phase, bool can_reshape) {
a61af66fc99e Initial load
duke
parents:
diff changeset
594 // Special case constant AND mask
a61af66fc99e Initial load
duke
parents:
diff changeset
595 const TypeLong *t2 = phase->type( in(2) )->isa_long();
a61af66fc99e Initial load
duke
parents:
diff changeset
596 if( !t2 || !t2->is_con() ) return MulNode::Ideal(phase, can_reshape);
a61af66fc99e Initial load
duke
parents:
diff changeset
597 const jlong mask = t2->get_con();
a61af66fc99e Initial load
duke
parents:
diff changeset
598
624
337400e7a5dd 6797305: Add LoadUB and LoadUI opcode class
twisti
parents: 559
diff changeset
599 Node* in1 = in(1);
337400e7a5dd 6797305: Add LoadUB and LoadUI opcode class
twisti
parents: 559
diff changeset
600 uint op = in1->Opcode();
337400e7a5dd 6797305: Add LoadUB and LoadUI opcode class
twisti
parents: 559
diff changeset
601
824
18a08a7e16b5 5057225: Remove useless I2L conversions
twisti
parents: 624
diff changeset
602 // Masking sign bits off of an integer? Do an unsigned integer to
18a08a7e16b5 5057225: Remove useless I2L conversions
twisti
parents: 624
diff changeset
603 // long load.
18a08a7e16b5 5057225: Remove useless I2L conversions
twisti
parents: 624
diff changeset
604 // NOTE: This check must be *before* we try to convert the AndLNode
18a08a7e16b5 5057225: Remove useless I2L conversions
twisti
parents: 624
diff changeset
605 // to an AndINode and commute it with ConvI2LNode because
18a08a7e16b5 5057225: Remove useless I2L conversions
twisti
parents: 624
diff changeset
606 // 0xFFFFFFFFL masks the whole integer and we get a sign extension,
18a08a7e16b5 5057225: Remove useless I2L conversions
twisti
parents: 624
diff changeset
607 // which is wrong.
18a08a7e16b5 5057225: Remove useless I2L conversions
twisti
parents: 624
diff changeset
608 if (op == Op_ConvI2L && in1->in(1)->Opcode() == Op_LoadI && mask == CONST64(0x00000000FFFFFFFF)) {
624
337400e7a5dd 6797305: Add LoadUB and LoadUI opcode class
twisti
parents: 559
diff changeset
609 Node* load = in1->in(1);
337400e7a5dd 6797305: Add LoadUB and LoadUI opcode class
twisti
parents: 559
diff changeset
610 return new (phase->C, 3) LoadUI2LNode(load->in(MemNode::Control),
337400e7a5dd 6797305: Add LoadUB and LoadUI opcode class
twisti
parents: 559
diff changeset
611 load->in(MemNode::Memory),
337400e7a5dd 6797305: Add LoadUB and LoadUI opcode class
twisti
parents: 559
diff changeset
612 load->in(MemNode::Address),
337400e7a5dd 6797305: Add LoadUB and LoadUI opcode class
twisti
parents: 559
diff changeset
613 load->adr_type());
337400e7a5dd 6797305: Add LoadUB and LoadUI opcode class
twisti
parents: 559
diff changeset
614 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
615
824
18a08a7e16b5 5057225: Remove useless I2L conversions
twisti
parents: 624
diff changeset
616 // Are we masking a long that was converted from an int with a mask
897
52898b0c43e9 6863155: Server compiler generates incorrect code (x86, long, bitshift, bitmask)
twisti
parents: 824
diff changeset
617 // that fits in 32-bits? Commute them and use an AndINode. Don't
52898b0c43e9 6863155: Server compiler generates incorrect code (x86, long, bitshift, bitmask)
twisti
parents: 824
diff changeset
618 // convert masks which would cause a sign extension of the integer
52898b0c43e9 6863155: Server compiler generates incorrect code (x86, long, bitshift, bitmask)
twisti
parents: 824
diff changeset
619 // value. This check includes UI2L masks (0x00000000FFFFFFFF) which
52898b0c43e9 6863155: Server compiler generates incorrect code (x86, long, bitshift, bitmask)
twisti
parents: 824
diff changeset
620 // would be optimized away later in Identity.
52898b0c43e9 6863155: Server compiler generates incorrect code (x86, long, bitshift, bitmask)
twisti
parents: 824
diff changeset
621 if (op == Op_ConvI2L && (mask & CONST64(0xFFFFFFFF80000000)) == 0) {
52898b0c43e9 6863155: Server compiler generates incorrect code (x86, long, bitshift, bitmask)
twisti
parents: 824
diff changeset
622 Node* andi = new (phase->C, 3) AndINode(in1->in(1), phase->intcon(mask));
52898b0c43e9 6863155: Server compiler generates incorrect code (x86, long, bitshift, bitmask)
twisti
parents: 824
diff changeset
623 andi = phase->transform(andi);
52898b0c43e9 6863155: Server compiler generates incorrect code (x86, long, bitshift, bitmask)
twisti
parents: 824
diff changeset
624 return new (phase->C, 2) ConvI2LNode(andi);
824
18a08a7e16b5 5057225: Remove useless I2L conversions
twisti
parents: 624
diff changeset
625 }
18a08a7e16b5 5057225: Remove useless I2L conversions
twisti
parents: 624
diff changeset
626
0
a61af66fc99e Initial load
duke
parents:
diff changeset
627 // Masking off sign bits? Dont make them!
624
337400e7a5dd 6797305: Add LoadUB and LoadUI opcode class
twisti
parents: 559
diff changeset
628 if (op == Op_RShiftL) {
824
18a08a7e16b5 5057225: Remove useless I2L conversions
twisti
parents: 624
diff changeset
629 const TypeInt* t12 = phase->type(in1->in(2))->isa_int();
0
a61af66fc99e Initial load
duke
parents:
diff changeset
630 if( t12 && t12->is_con() ) { // Shift is by a constant
a61af66fc99e Initial load
duke
parents:
diff changeset
631 int shift = t12->get_con();
559
7628781568e1 6795362: 32bit server compiler leads to wrong results on solaris-x86
twisti
parents: 558
diff changeset
632 shift &= BitsPerJavaLong - 1; // semantics of Java shifts
7628781568e1 6795362: 32bit server compiler leads to wrong results on solaris-x86
twisti
parents: 558
diff changeset
633 const jlong sign_bits_mask = ~(((jlong)CONST64(1) << (jlong)(BitsPerJavaLong - shift)) -1);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
634 // If the AND'ing of the 2 masks has no bits, then only original shifted
a61af66fc99e Initial load
duke
parents:
diff changeset
635 // bits survive. NO sign-extension bits survive the maskings.
a61af66fc99e Initial load
duke
parents:
diff changeset
636 if( (sign_bits_mask & mask) == 0 ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
637 // Use zero-fill shift instead
624
337400e7a5dd 6797305: Add LoadUB and LoadUI opcode class
twisti
parents: 559
diff changeset
638 Node *zshift = phase->transform(new (phase->C, 3) URShiftLNode(in1->in(1), in1->in(2)));
824
18a08a7e16b5 5057225: Remove useless I2L conversions
twisti
parents: 624
diff changeset
639 return new (phase->C, 3) AndLNode(zshift, in(2));
0
a61af66fc99e Initial load
duke
parents:
diff changeset
640 }
a61af66fc99e Initial load
duke
parents:
diff changeset
641 }
a61af66fc99e Initial load
duke
parents:
diff changeset
642 }
a61af66fc99e Initial load
duke
parents:
diff changeset
643
a61af66fc99e Initial load
duke
parents:
diff changeset
644 return MulNode::Ideal(phase, can_reshape);
a61af66fc99e Initial load
duke
parents:
diff changeset
645 }
a61af66fc99e Initial load
duke
parents:
diff changeset
646
a61af66fc99e Initial load
duke
parents:
diff changeset
647 //=============================================================================
a61af66fc99e Initial load
duke
parents:
diff changeset
648 //------------------------------Identity---------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
649 Node *LShiftINode::Identity( PhaseTransform *phase ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
650 const TypeInt *ti = phase->type( in(2) )->isa_int(); // shift count is an int
a61af66fc99e Initial load
duke
parents:
diff changeset
651 return ( ti && ti->is_con() && ( ti->get_con() & ( BitsPerInt - 1 ) ) == 0 ) ? in(1) : this;
a61af66fc99e Initial load
duke
parents:
diff changeset
652 }
a61af66fc99e Initial load
duke
parents:
diff changeset
653
a61af66fc99e Initial load
duke
parents:
diff changeset
654 //------------------------------Ideal------------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
655 // If the right input is a constant, and the left input is an add of a
a61af66fc99e Initial load
duke
parents:
diff changeset
656 // constant, flatten the tree: (X+con1)<<con0 ==> X<<con0 + con1<<con0
a61af66fc99e Initial load
duke
parents:
diff changeset
657 Node *LShiftINode::Ideal(PhaseGVN *phase, bool can_reshape) {
a61af66fc99e Initial load
duke
parents:
diff changeset
658 const Type *t = phase->type( in(2) );
a61af66fc99e Initial load
duke
parents:
diff changeset
659 if( t == Type::TOP ) return NULL; // Right input is dead
a61af66fc99e Initial load
duke
parents:
diff changeset
660 const TypeInt *t2 = t->isa_int();
a61af66fc99e Initial load
duke
parents:
diff changeset
661 if( !t2 || !t2->is_con() ) return NULL; // Right input is a constant
a61af66fc99e Initial load
duke
parents:
diff changeset
662 const int con = t2->get_con() & ( BitsPerInt - 1 ); // masked shift count
a61af66fc99e Initial load
duke
parents:
diff changeset
663
a61af66fc99e Initial load
duke
parents:
diff changeset
664 if ( con == 0 ) return NULL; // let Identity() handle 0 shift count
a61af66fc99e Initial load
duke
parents:
diff changeset
665
a61af66fc99e Initial load
duke
parents:
diff changeset
666 // Left input is an add of a constant?
a61af66fc99e Initial load
duke
parents:
diff changeset
667 Node *add1 = in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
668 int add1_op = add1->Opcode();
a61af66fc99e Initial load
duke
parents:
diff changeset
669 if( add1_op == Op_AddI ) { // Left input is an add?
a61af66fc99e Initial load
duke
parents:
diff changeset
670 assert( add1 != add1->in(1), "dead loop in LShiftINode::Ideal" );
a61af66fc99e Initial load
duke
parents:
diff changeset
671 const TypeInt *t12 = phase->type(add1->in(2))->isa_int();
a61af66fc99e Initial load
duke
parents:
diff changeset
672 if( t12 && t12->is_con() ){ // Left input is an add of a con?
a61af66fc99e Initial load
duke
parents:
diff changeset
673 // Transform is legal, but check for profit. Avoid breaking 'i2s'
a61af66fc99e Initial load
duke
parents:
diff changeset
674 // and 'i2b' patterns which typically fold into 'StoreC/StoreB'.
a61af66fc99e Initial load
duke
parents:
diff changeset
675 if( con < 16 ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
676 // Compute X << con0
a61af66fc99e Initial load
duke
parents:
diff changeset
677 Node *lsh = phase->transform( new (phase->C, 3) LShiftINode( add1->in(1), in(2) ) );
a61af66fc99e Initial load
duke
parents:
diff changeset
678 // Compute X<<con0 + (con1<<con0)
a61af66fc99e Initial load
duke
parents:
diff changeset
679 return new (phase->C, 3) AddINode( lsh, phase->intcon(t12->get_con() << con));
a61af66fc99e Initial load
duke
parents:
diff changeset
680 }
a61af66fc99e Initial load
duke
parents:
diff changeset
681 }
a61af66fc99e Initial load
duke
parents:
diff changeset
682 }
a61af66fc99e Initial load
duke
parents:
diff changeset
683
a61af66fc99e Initial load
duke
parents:
diff changeset
684 // Check for "(x>>c0)<<c0" which just masks off low bits
a61af66fc99e Initial load
duke
parents:
diff changeset
685 if( (add1_op == Op_RShiftI || add1_op == Op_URShiftI ) &&
a61af66fc99e Initial load
duke
parents:
diff changeset
686 add1->in(2) == in(2) )
a61af66fc99e Initial load
duke
parents:
diff changeset
687 // Convert to "(x & -(1<<c0))"
a61af66fc99e Initial load
duke
parents:
diff changeset
688 return new (phase->C, 3) AndINode(add1->in(1),phase->intcon( -(1<<con)));
a61af66fc99e Initial load
duke
parents:
diff changeset
689
a61af66fc99e Initial load
duke
parents:
diff changeset
690 // Check for "((x>>c0) & Y)<<c0" which just masks off more low bits
a61af66fc99e Initial load
duke
parents:
diff changeset
691 if( add1_op == Op_AndI ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
692 Node *add2 = add1->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
693 int add2_op = add2->Opcode();
a61af66fc99e Initial load
duke
parents:
diff changeset
694 if( (add2_op == Op_RShiftI || add2_op == Op_URShiftI ) &&
a61af66fc99e Initial load
duke
parents:
diff changeset
695 add2->in(2) == in(2) ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
696 // Convert to "(x & (Y<<c0))"
a61af66fc99e Initial load
duke
parents:
diff changeset
697 Node *y_sh = phase->transform( new (phase->C, 3) LShiftINode( add1->in(2), in(2) ) );
a61af66fc99e Initial load
duke
parents:
diff changeset
698 return new (phase->C, 3) AndINode( add2->in(1), y_sh );
a61af66fc99e Initial load
duke
parents:
diff changeset
699 }
a61af66fc99e Initial load
duke
parents:
diff changeset
700 }
a61af66fc99e Initial load
duke
parents:
diff changeset
701
a61af66fc99e Initial load
duke
parents:
diff changeset
702 // Check for ((x & ((1<<(32-c0))-1)) << c0) which ANDs off high bits
a61af66fc99e Initial load
duke
parents:
diff changeset
703 // before shifting them away.
a61af66fc99e Initial load
duke
parents:
diff changeset
704 const jint bits_mask = right_n_bits(BitsPerJavaInteger-con);
a61af66fc99e Initial load
duke
parents:
diff changeset
705 if( add1_op == Op_AndI &&
a61af66fc99e Initial load
duke
parents:
diff changeset
706 phase->type(add1->in(2)) == TypeInt::make( bits_mask ) )
a61af66fc99e Initial load
duke
parents:
diff changeset
707 return new (phase->C, 3) LShiftINode( add1->in(1), in(2) );
a61af66fc99e Initial load
duke
parents:
diff changeset
708
a61af66fc99e Initial load
duke
parents:
diff changeset
709 return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
710 }
a61af66fc99e Initial load
duke
parents:
diff changeset
711
a61af66fc99e Initial load
duke
parents:
diff changeset
712 //------------------------------Value------------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
713 // A LShiftINode shifts its input2 left by input1 amount.
a61af66fc99e Initial load
duke
parents:
diff changeset
714 const Type *LShiftINode::Value( PhaseTransform *phase ) const {
a61af66fc99e Initial load
duke
parents:
diff changeset
715 const Type *t1 = phase->type( in(1) );
a61af66fc99e Initial load
duke
parents:
diff changeset
716 const Type *t2 = phase->type( in(2) );
a61af66fc99e Initial load
duke
parents:
diff changeset
717 // Either input is TOP ==> the result is TOP
a61af66fc99e Initial load
duke
parents:
diff changeset
718 if( t1 == Type::TOP ) return Type::TOP;
a61af66fc99e Initial load
duke
parents:
diff changeset
719 if( t2 == Type::TOP ) return Type::TOP;
a61af66fc99e Initial load
duke
parents:
diff changeset
720
a61af66fc99e Initial load
duke
parents:
diff changeset
721 // Left input is ZERO ==> the result is ZERO.
a61af66fc99e Initial load
duke
parents:
diff changeset
722 if( t1 == TypeInt::ZERO ) return TypeInt::ZERO;
a61af66fc99e Initial load
duke
parents:
diff changeset
723 // Shift by zero does nothing
a61af66fc99e Initial load
duke
parents:
diff changeset
724 if( t2 == TypeInt::ZERO ) return t1;
a61af66fc99e Initial load
duke
parents:
diff changeset
725
a61af66fc99e Initial load
duke
parents:
diff changeset
726 // Either input is BOTTOM ==> the result is BOTTOM
a61af66fc99e Initial load
duke
parents:
diff changeset
727 if( (t1 == TypeInt::INT) || (t2 == TypeInt::INT) ||
a61af66fc99e Initial load
duke
parents:
diff changeset
728 (t1 == Type::BOTTOM) || (t2 == Type::BOTTOM) )
a61af66fc99e Initial load
duke
parents:
diff changeset
729 return TypeInt::INT;
a61af66fc99e Initial load
duke
parents:
diff changeset
730
a61af66fc99e Initial load
duke
parents:
diff changeset
731 const TypeInt *r1 = t1->is_int(); // Handy access
a61af66fc99e Initial load
duke
parents:
diff changeset
732 const TypeInt *r2 = t2->is_int(); // Handy access
a61af66fc99e Initial load
duke
parents:
diff changeset
733
a61af66fc99e Initial load
duke
parents:
diff changeset
734 if (!r2->is_con())
a61af66fc99e Initial load
duke
parents:
diff changeset
735 return TypeInt::INT;
a61af66fc99e Initial load
duke
parents:
diff changeset
736
a61af66fc99e Initial load
duke
parents:
diff changeset
737 uint shift = r2->get_con();
a61af66fc99e Initial load
duke
parents:
diff changeset
738 shift &= BitsPerJavaInteger-1; // semantics of Java shifts
a61af66fc99e Initial load
duke
parents:
diff changeset
739 // Shift by a multiple of 32 does nothing:
a61af66fc99e Initial load
duke
parents:
diff changeset
740 if (shift == 0) return t1;
a61af66fc99e Initial load
duke
parents:
diff changeset
741
a61af66fc99e Initial load
duke
parents:
diff changeset
742 // If the shift is a constant, shift the bounds of the type,
a61af66fc99e Initial load
duke
parents:
diff changeset
743 // unless this could lead to an overflow.
a61af66fc99e Initial load
duke
parents:
diff changeset
744 if (!r1->is_con()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
745 jint lo = r1->_lo, hi = r1->_hi;
a61af66fc99e Initial load
duke
parents:
diff changeset
746 if (((lo << shift) >> shift) == lo &&
a61af66fc99e Initial load
duke
parents:
diff changeset
747 ((hi << shift) >> shift) == hi) {
a61af66fc99e Initial load
duke
parents:
diff changeset
748 // No overflow. The range shifts up cleanly.
a61af66fc99e Initial load
duke
parents:
diff changeset
749 return TypeInt::make((jint)lo << (jint)shift,
a61af66fc99e Initial load
duke
parents:
diff changeset
750 (jint)hi << (jint)shift,
a61af66fc99e Initial load
duke
parents:
diff changeset
751 MAX2(r1->_widen,r2->_widen));
a61af66fc99e Initial load
duke
parents:
diff changeset
752 }
a61af66fc99e Initial load
duke
parents:
diff changeset
753 return TypeInt::INT;
a61af66fc99e Initial load
duke
parents:
diff changeset
754 }
a61af66fc99e Initial load
duke
parents:
diff changeset
755
a61af66fc99e Initial load
duke
parents:
diff changeset
756 return TypeInt::make( (jint)r1->get_con() << (jint)shift );
a61af66fc99e Initial load
duke
parents:
diff changeset
757 }
a61af66fc99e Initial load
duke
parents:
diff changeset
758
a61af66fc99e Initial load
duke
parents:
diff changeset
759 //=============================================================================
a61af66fc99e Initial load
duke
parents:
diff changeset
760 //------------------------------Identity---------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
761 Node *LShiftLNode::Identity( PhaseTransform *phase ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
762 const TypeInt *ti = phase->type( in(2) )->isa_int(); // shift count is an int
a61af66fc99e Initial load
duke
parents:
diff changeset
763 return ( ti && ti->is_con() && ( ti->get_con() & ( BitsPerLong - 1 ) ) == 0 ) ? in(1) : this;
a61af66fc99e Initial load
duke
parents:
diff changeset
764 }
a61af66fc99e Initial load
duke
parents:
diff changeset
765
a61af66fc99e Initial load
duke
parents:
diff changeset
766 //------------------------------Ideal------------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
767 // If the right input is a constant, and the left input is an add of a
a61af66fc99e Initial load
duke
parents:
diff changeset
768 // constant, flatten the tree: (X+con1)<<con0 ==> X<<con0 + con1<<con0
a61af66fc99e Initial load
duke
parents:
diff changeset
769 Node *LShiftLNode::Ideal(PhaseGVN *phase, bool can_reshape) {
a61af66fc99e Initial load
duke
parents:
diff changeset
770 const Type *t = phase->type( in(2) );
a61af66fc99e Initial load
duke
parents:
diff changeset
771 if( t == Type::TOP ) return NULL; // Right input is dead
a61af66fc99e Initial load
duke
parents:
diff changeset
772 const TypeInt *t2 = t->isa_int();
a61af66fc99e Initial load
duke
parents:
diff changeset
773 if( !t2 || !t2->is_con() ) return NULL; // Right input is a constant
a61af66fc99e Initial load
duke
parents:
diff changeset
774 const int con = t2->get_con() & ( BitsPerLong - 1 ); // masked shift count
a61af66fc99e Initial load
duke
parents:
diff changeset
775
a61af66fc99e Initial load
duke
parents:
diff changeset
776 if ( con == 0 ) return NULL; // let Identity() handle 0 shift count
a61af66fc99e Initial load
duke
parents:
diff changeset
777
a61af66fc99e Initial load
duke
parents:
diff changeset
778 // Left input is an add of a constant?
a61af66fc99e Initial load
duke
parents:
diff changeset
779 Node *add1 = in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
780 int add1_op = add1->Opcode();
a61af66fc99e Initial load
duke
parents:
diff changeset
781 if( add1_op == Op_AddL ) { // Left input is an add?
a61af66fc99e Initial load
duke
parents:
diff changeset
782 // Avoid dead data cycles from dead loops
a61af66fc99e Initial load
duke
parents:
diff changeset
783 assert( add1 != add1->in(1), "dead loop in LShiftLNode::Ideal" );
a61af66fc99e Initial load
duke
parents:
diff changeset
784 const TypeLong *t12 = phase->type(add1->in(2))->isa_long();
a61af66fc99e Initial load
duke
parents:
diff changeset
785 if( t12 && t12->is_con() ){ // Left input is an add of a con?
a61af66fc99e Initial load
duke
parents:
diff changeset
786 // Compute X << con0
a61af66fc99e Initial load
duke
parents:
diff changeset
787 Node *lsh = phase->transform( new (phase->C, 3) LShiftLNode( add1->in(1), in(2) ) );
a61af66fc99e Initial load
duke
parents:
diff changeset
788 // Compute X<<con0 + (con1<<con0)
a61af66fc99e Initial load
duke
parents:
diff changeset
789 return new (phase->C, 3) AddLNode( lsh, phase->longcon(t12->get_con() << con));
a61af66fc99e Initial load
duke
parents:
diff changeset
790 }
a61af66fc99e Initial load
duke
parents:
diff changeset
791 }
a61af66fc99e Initial load
duke
parents:
diff changeset
792
a61af66fc99e Initial load
duke
parents:
diff changeset
793 // Check for "(x>>c0)<<c0" which just masks off low bits
a61af66fc99e Initial load
duke
parents:
diff changeset
794 if( (add1_op == Op_RShiftL || add1_op == Op_URShiftL ) &&
a61af66fc99e Initial load
duke
parents:
diff changeset
795 add1->in(2) == in(2) )
a61af66fc99e Initial load
duke
parents:
diff changeset
796 // Convert to "(x & -(1<<c0))"
a61af66fc99e Initial load
duke
parents:
diff changeset
797 return new (phase->C, 3) AndLNode(add1->in(1),phase->longcon( -(CONST64(1)<<con)));
a61af66fc99e Initial load
duke
parents:
diff changeset
798
a61af66fc99e Initial load
duke
parents:
diff changeset
799 // Check for "((x>>c0) & Y)<<c0" which just masks off more low bits
a61af66fc99e Initial load
duke
parents:
diff changeset
800 if( add1_op == Op_AndL ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
801 Node *add2 = add1->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
802 int add2_op = add2->Opcode();
a61af66fc99e Initial load
duke
parents:
diff changeset
803 if( (add2_op == Op_RShiftL || add2_op == Op_URShiftL ) &&
a61af66fc99e Initial load
duke
parents:
diff changeset
804 add2->in(2) == in(2) ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
805 // Convert to "(x & (Y<<c0))"
a61af66fc99e Initial load
duke
parents:
diff changeset
806 Node *y_sh = phase->transform( new (phase->C, 3) LShiftLNode( add1->in(2), in(2) ) );
a61af66fc99e Initial load
duke
parents:
diff changeset
807 return new (phase->C, 3) AndLNode( add2->in(1), y_sh );
a61af66fc99e Initial load
duke
parents:
diff changeset
808 }
a61af66fc99e Initial load
duke
parents:
diff changeset
809 }
a61af66fc99e Initial load
duke
parents:
diff changeset
810
a61af66fc99e Initial load
duke
parents:
diff changeset
811 // Check for ((x & ((CONST64(1)<<(64-c0))-1)) << c0) which ANDs off high bits
a61af66fc99e Initial load
duke
parents:
diff changeset
812 // before shifting them away.
559
7628781568e1 6795362: 32bit server compiler leads to wrong results on solaris-x86
twisti
parents: 558
diff changeset
813 const jlong bits_mask = ((jlong)CONST64(1) << (jlong)(BitsPerJavaLong - con)) - CONST64(1);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
814 if( add1_op == Op_AndL &&
a61af66fc99e Initial load
duke
parents:
diff changeset
815 phase->type(add1->in(2)) == TypeLong::make( bits_mask ) )
a61af66fc99e Initial load
duke
parents:
diff changeset
816 return new (phase->C, 3) LShiftLNode( add1->in(1), in(2) );
a61af66fc99e Initial load
duke
parents:
diff changeset
817
a61af66fc99e Initial load
duke
parents:
diff changeset
818 return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
819 }
a61af66fc99e Initial load
duke
parents:
diff changeset
820
a61af66fc99e Initial load
duke
parents:
diff changeset
821 //------------------------------Value------------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
822 // A LShiftLNode shifts its input2 left by input1 amount.
a61af66fc99e Initial load
duke
parents:
diff changeset
823 const Type *LShiftLNode::Value( PhaseTransform *phase ) const {
a61af66fc99e Initial load
duke
parents:
diff changeset
824 const Type *t1 = phase->type( in(1) );
a61af66fc99e Initial load
duke
parents:
diff changeset
825 const Type *t2 = phase->type( in(2) );
a61af66fc99e Initial load
duke
parents:
diff changeset
826 // Either input is TOP ==> the result is TOP
a61af66fc99e Initial load
duke
parents:
diff changeset
827 if( t1 == Type::TOP ) return Type::TOP;
a61af66fc99e Initial load
duke
parents:
diff changeset
828 if( t2 == Type::TOP ) return Type::TOP;
a61af66fc99e Initial load
duke
parents:
diff changeset
829
a61af66fc99e Initial load
duke
parents:
diff changeset
830 // Left input is ZERO ==> the result is ZERO.
a61af66fc99e Initial load
duke
parents:
diff changeset
831 if( t1 == TypeLong::ZERO ) return TypeLong::ZERO;
a61af66fc99e Initial load
duke
parents:
diff changeset
832 // Shift by zero does nothing
a61af66fc99e Initial load
duke
parents:
diff changeset
833 if( t2 == TypeInt::ZERO ) return t1;
a61af66fc99e Initial load
duke
parents:
diff changeset
834
a61af66fc99e Initial load
duke
parents:
diff changeset
835 // Either input is BOTTOM ==> the result is BOTTOM
a61af66fc99e Initial load
duke
parents:
diff changeset
836 if( (t1 == TypeLong::LONG) || (t2 == TypeInt::INT) ||
a61af66fc99e Initial load
duke
parents:
diff changeset
837 (t1 == Type::BOTTOM) || (t2 == Type::BOTTOM) )
a61af66fc99e Initial load
duke
parents:
diff changeset
838 return TypeLong::LONG;
a61af66fc99e Initial load
duke
parents:
diff changeset
839
a61af66fc99e Initial load
duke
parents:
diff changeset
840 const TypeLong *r1 = t1->is_long(); // Handy access
a61af66fc99e Initial load
duke
parents:
diff changeset
841 const TypeInt *r2 = t2->is_int(); // Handy access
a61af66fc99e Initial load
duke
parents:
diff changeset
842
a61af66fc99e Initial load
duke
parents:
diff changeset
843 if (!r2->is_con())
a61af66fc99e Initial load
duke
parents:
diff changeset
844 return TypeLong::LONG;
a61af66fc99e Initial load
duke
parents:
diff changeset
845
a61af66fc99e Initial load
duke
parents:
diff changeset
846 uint shift = r2->get_con();
559
7628781568e1 6795362: 32bit server compiler leads to wrong results on solaris-x86
twisti
parents: 558
diff changeset
847 shift &= BitsPerJavaLong - 1; // semantics of Java shifts
0
a61af66fc99e Initial load
duke
parents:
diff changeset
848 // Shift by a multiple of 64 does nothing:
a61af66fc99e Initial load
duke
parents:
diff changeset
849 if (shift == 0) return t1;
a61af66fc99e Initial load
duke
parents:
diff changeset
850
a61af66fc99e Initial load
duke
parents:
diff changeset
851 // If the shift is a constant, shift the bounds of the type,
a61af66fc99e Initial load
duke
parents:
diff changeset
852 // unless this could lead to an overflow.
a61af66fc99e Initial load
duke
parents:
diff changeset
853 if (!r1->is_con()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
854 jlong lo = r1->_lo, hi = r1->_hi;
a61af66fc99e Initial load
duke
parents:
diff changeset
855 if (((lo << shift) >> shift) == lo &&
a61af66fc99e Initial load
duke
parents:
diff changeset
856 ((hi << shift) >> shift) == hi) {
a61af66fc99e Initial load
duke
parents:
diff changeset
857 // No overflow. The range shifts up cleanly.
a61af66fc99e Initial load
duke
parents:
diff changeset
858 return TypeLong::make((jlong)lo << (jint)shift,
a61af66fc99e Initial load
duke
parents:
diff changeset
859 (jlong)hi << (jint)shift,
a61af66fc99e Initial load
duke
parents:
diff changeset
860 MAX2(r1->_widen,r2->_widen));
a61af66fc99e Initial load
duke
parents:
diff changeset
861 }
a61af66fc99e Initial load
duke
parents:
diff changeset
862 return TypeLong::LONG;
a61af66fc99e Initial load
duke
parents:
diff changeset
863 }
a61af66fc99e Initial load
duke
parents:
diff changeset
864
a61af66fc99e Initial load
duke
parents:
diff changeset
865 return TypeLong::make( (jlong)r1->get_con() << (jint)shift );
a61af66fc99e Initial load
duke
parents:
diff changeset
866 }
a61af66fc99e Initial load
duke
parents:
diff changeset
867
a61af66fc99e Initial load
duke
parents:
diff changeset
868 //=============================================================================
a61af66fc99e Initial load
duke
parents:
diff changeset
869 //------------------------------Identity---------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
870 Node *RShiftINode::Identity( PhaseTransform *phase ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
871 const TypeInt *t2 = phase->type(in(2))->isa_int();
a61af66fc99e Initial load
duke
parents:
diff changeset
872 if( !t2 ) return this;
a61af66fc99e Initial load
duke
parents:
diff changeset
873 if ( t2->is_con() && ( t2->get_con() & ( BitsPerInt - 1 ) ) == 0 )
a61af66fc99e Initial load
duke
parents:
diff changeset
874 return in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
875
a61af66fc99e Initial load
duke
parents:
diff changeset
876 // Check for useless sign-masking
a61af66fc99e Initial load
duke
parents:
diff changeset
877 if( in(1)->Opcode() == Op_LShiftI &&
a61af66fc99e Initial load
duke
parents:
diff changeset
878 in(1)->req() == 3 &&
a61af66fc99e Initial load
duke
parents:
diff changeset
879 in(1)->in(2) == in(2) &&
a61af66fc99e Initial load
duke
parents:
diff changeset
880 t2->is_con() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
881 uint shift = t2->get_con();
a61af66fc99e Initial load
duke
parents:
diff changeset
882 shift &= BitsPerJavaInteger-1; // semantics of Java shifts
a61af66fc99e Initial load
duke
parents:
diff changeset
883 // Compute masks for which this shifting doesn't change
a61af66fc99e Initial load
duke
parents:
diff changeset
884 int lo = (-1 << (BitsPerJavaInteger - shift-1)); // FFFF8000
a61af66fc99e Initial load
duke
parents:
diff changeset
885 int hi = ~lo; // 00007FFF
a61af66fc99e Initial load
duke
parents:
diff changeset
886 const TypeInt *t11 = phase->type(in(1)->in(1))->isa_int();
a61af66fc99e Initial load
duke
parents:
diff changeset
887 if( !t11 ) return this;
a61af66fc99e Initial load
duke
parents:
diff changeset
888 // Does actual value fit inside of mask?
a61af66fc99e Initial load
duke
parents:
diff changeset
889 if( lo <= t11->_lo && t11->_hi <= hi )
a61af66fc99e Initial load
duke
parents:
diff changeset
890 return in(1)->in(1); // Then shifting is a nop
a61af66fc99e Initial load
duke
parents:
diff changeset
891 }
a61af66fc99e Initial load
duke
parents:
diff changeset
892
a61af66fc99e Initial load
duke
parents:
diff changeset
893 return this;
a61af66fc99e Initial load
duke
parents:
diff changeset
894 }
a61af66fc99e Initial load
duke
parents:
diff changeset
895
a61af66fc99e Initial load
duke
parents:
diff changeset
896 //------------------------------Ideal------------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
897 Node *RShiftINode::Ideal(PhaseGVN *phase, bool can_reshape) {
a61af66fc99e Initial load
duke
parents:
diff changeset
898 // Inputs may be TOP if they are dead.
a61af66fc99e Initial load
duke
parents:
diff changeset
899 const TypeInt *t1 = phase->type( in(1) )->isa_int();
a61af66fc99e Initial load
duke
parents:
diff changeset
900 if( !t1 ) return NULL; // Left input is an integer
a61af66fc99e Initial load
duke
parents:
diff changeset
901 const TypeInt *t2 = phase->type( in(2) )->isa_int();
a61af66fc99e Initial load
duke
parents:
diff changeset
902 if( !t2 || !t2->is_con() ) return NULL; // Right input is a constant
a61af66fc99e Initial load
duke
parents:
diff changeset
903 const TypeInt *t3; // type of in(1).in(2)
a61af66fc99e Initial load
duke
parents:
diff changeset
904 int shift = t2->get_con();
a61af66fc99e Initial load
duke
parents:
diff changeset
905 shift &= BitsPerJavaInteger-1; // semantics of Java shifts
a61af66fc99e Initial load
duke
parents:
diff changeset
906
a61af66fc99e Initial load
duke
parents:
diff changeset
907 if ( shift == 0 ) return NULL; // let Identity() handle 0 shift count
a61af66fc99e Initial load
duke
parents:
diff changeset
908
a61af66fc99e Initial load
duke
parents:
diff changeset
909 // Check for (x & 0xFF000000) >> 24, whose mask can be made smaller.
a61af66fc99e Initial load
duke
parents:
diff changeset
910 // Such expressions arise normally from shift chains like (byte)(x >> 24).
a61af66fc99e Initial load
duke
parents:
diff changeset
911 const Node *mask = in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
912 if( mask->Opcode() == Op_AndI &&
a61af66fc99e Initial load
duke
parents:
diff changeset
913 (t3 = phase->type(mask->in(2))->isa_int()) &&
a61af66fc99e Initial load
duke
parents:
diff changeset
914 t3->is_con() ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
915 Node *x = mask->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
916 jint maskbits = t3->get_con();
a61af66fc99e Initial load
duke
parents:
diff changeset
917 // Convert to "(x >> shift) & (mask >> shift)"
a61af66fc99e Initial load
duke
parents:
diff changeset
918 Node *shr_nomask = phase->transform( new (phase->C, 3) RShiftINode(mask->in(1), in(2)) );
a61af66fc99e Initial load
duke
parents:
diff changeset
919 return new (phase->C, 3) AndINode(shr_nomask, phase->intcon( maskbits >> shift));
a61af66fc99e Initial load
duke
parents:
diff changeset
920 }
a61af66fc99e Initial load
duke
parents:
diff changeset
921
a61af66fc99e Initial load
duke
parents:
diff changeset
922 // Check for "(short[i] <<16)>>16" which simply sign-extends
a61af66fc99e Initial load
duke
parents:
diff changeset
923 const Node *shl = in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
924 if( shl->Opcode() != Op_LShiftI ) return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
925
a61af66fc99e Initial load
duke
parents:
diff changeset
926 if( shift == 16 &&
a61af66fc99e Initial load
duke
parents:
diff changeset
927 (t3 = phase->type(shl->in(2))->isa_int()) &&
a61af66fc99e Initial load
duke
parents:
diff changeset
928 t3->is_con(16) ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
929 Node *ld = shl->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
930 if( ld->Opcode() == Op_LoadS ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
931 // Sign extension is just useless here. Return a RShiftI of zero instead
a61af66fc99e Initial load
duke
parents:
diff changeset
932 // returning 'ld' directly. We cannot return an old Node directly as
a61af66fc99e Initial load
duke
parents:
diff changeset
933 // that is the job of 'Identity' calls and Identity calls only work on
a61af66fc99e Initial load
duke
parents:
diff changeset
934 // direct inputs ('ld' is an extra Node removed from 'this'). The
a61af66fc99e Initial load
duke
parents:
diff changeset
935 // combined optimization requires Identity only return direct inputs.
a61af66fc99e Initial load
duke
parents:
diff changeset
936 set_req(1, ld);
a61af66fc99e Initial load
duke
parents:
diff changeset
937 set_req(2, phase->intcon(0));
a61af66fc99e Initial load
duke
parents:
diff changeset
938 return this;
a61af66fc99e Initial load
duke
parents:
diff changeset
939 }
558
3b5ac9e7e6ea 6796746: rename LoadC (char) opcode class to LoadUS (unsigned short)
twisti
parents: 404
diff changeset
940 else if( ld->Opcode() == Op_LoadUS )
0
a61af66fc99e Initial load
duke
parents:
diff changeset
941 // Replace zero-extension-load with sign-extension-load
a61af66fc99e Initial load
duke
parents:
diff changeset
942 return new (phase->C, 3) LoadSNode( ld->in(MemNode::Control),
a61af66fc99e Initial load
duke
parents:
diff changeset
943 ld->in(MemNode::Memory),
a61af66fc99e Initial load
duke
parents:
diff changeset
944 ld->in(MemNode::Address),
a61af66fc99e Initial load
duke
parents:
diff changeset
945 ld->adr_type());
a61af66fc99e Initial load
duke
parents:
diff changeset
946 }
a61af66fc99e Initial load
duke
parents:
diff changeset
947
a61af66fc99e Initial load
duke
parents:
diff changeset
948 // Check for "(byte[i] <<24)>>24" which simply sign-extends
a61af66fc99e Initial load
duke
parents:
diff changeset
949 if( shift == 24 &&
a61af66fc99e Initial load
duke
parents:
diff changeset
950 (t3 = phase->type(shl->in(2))->isa_int()) &&
a61af66fc99e Initial load
duke
parents:
diff changeset
951 t3->is_con(24) ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
952 Node *ld = shl->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
953 if( ld->Opcode() == Op_LoadB ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
954 // Sign extension is just useless here
a61af66fc99e Initial load
duke
parents:
diff changeset
955 set_req(1, ld);
a61af66fc99e Initial load
duke
parents:
diff changeset
956 set_req(2, phase->intcon(0));
a61af66fc99e Initial load
duke
parents:
diff changeset
957 return this;
a61af66fc99e Initial load
duke
parents:
diff changeset
958 }
a61af66fc99e Initial load
duke
parents:
diff changeset
959 }
a61af66fc99e Initial load
duke
parents:
diff changeset
960
a61af66fc99e Initial load
duke
parents:
diff changeset
961 return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
962 }
a61af66fc99e Initial load
duke
parents:
diff changeset
963
a61af66fc99e Initial load
duke
parents:
diff changeset
964 //------------------------------Value------------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
965 // A RShiftINode shifts its input2 right by input1 amount.
a61af66fc99e Initial load
duke
parents:
diff changeset
966 const Type *RShiftINode::Value( PhaseTransform *phase ) const {
a61af66fc99e Initial load
duke
parents:
diff changeset
967 const Type *t1 = phase->type( in(1) );
a61af66fc99e Initial load
duke
parents:
diff changeset
968 const Type *t2 = phase->type( in(2) );
a61af66fc99e Initial load
duke
parents:
diff changeset
969 // Either input is TOP ==> the result is TOP
a61af66fc99e Initial load
duke
parents:
diff changeset
970 if( t1 == Type::TOP ) return Type::TOP;
a61af66fc99e Initial load
duke
parents:
diff changeset
971 if( t2 == Type::TOP ) return Type::TOP;
a61af66fc99e Initial load
duke
parents:
diff changeset
972
a61af66fc99e Initial load
duke
parents:
diff changeset
973 // Left input is ZERO ==> the result is ZERO.
a61af66fc99e Initial load
duke
parents:
diff changeset
974 if( t1 == TypeInt::ZERO ) return TypeInt::ZERO;
a61af66fc99e Initial load
duke
parents:
diff changeset
975 // Shift by zero does nothing
a61af66fc99e Initial load
duke
parents:
diff changeset
976 if( t2 == TypeInt::ZERO ) return t1;
a61af66fc99e Initial load
duke
parents:
diff changeset
977
a61af66fc99e Initial load
duke
parents:
diff changeset
978 // Either input is BOTTOM ==> the result is BOTTOM
a61af66fc99e Initial load
duke
parents:
diff changeset
979 if (t1 == Type::BOTTOM || t2 == Type::BOTTOM)
a61af66fc99e Initial load
duke
parents:
diff changeset
980 return TypeInt::INT;
a61af66fc99e Initial load
duke
parents:
diff changeset
981
a61af66fc99e Initial load
duke
parents:
diff changeset
982 if (t2 == TypeInt::INT)
a61af66fc99e Initial load
duke
parents:
diff changeset
983 return TypeInt::INT;
a61af66fc99e Initial load
duke
parents:
diff changeset
984
a61af66fc99e Initial load
duke
parents:
diff changeset
985 const TypeInt *r1 = t1->is_int(); // Handy access
a61af66fc99e Initial load
duke
parents:
diff changeset
986 const TypeInt *r2 = t2->is_int(); // Handy access
a61af66fc99e Initial load
duke
parents:
diff changeset
987
a61af66fc99e Initial load
duke
parents:
diff changeset
988 // If the shift is a constant, just shift the bounds of the type.
a61af66fc99e Initial load
duke
parents:
diff changeset
989 // For example, if the shift is 31, we just propagate sign bits.
a61af66fc99e Initial load
duke
parents:
diff changeset
990 if (r2->is_con()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
991 uint shift = r2->get_con();
a61af66fc99e Initial load
duke
parents:
diff changeset
992 shift &= BitsPerJavaInteger-1; // semantics of Java shifts
a61af66fc99e Initial load
duke
parents:
diff changeset
993 // Shift by a multiple of 32 does nothing:
a61af66fc99e Initial load
duke
parents:
diff changeset
994 if (shift == 0) return t1;
a61af66fc99e Initial load
duke
parents:
diff changeset
995 // Calculate reasonably aggressive bounds for the result.
a61af66fc99e Initial load
duke
parents:
diff changeset
996 // This is necessary if we are to correctly type things
a61af66fc99e Initial load
duke
parents:
diff changeset
997 // like (x<<24>>24) == ((byte)x).
a61af66fc99e Initial load
duke
parents:
diff changeset
998 jint lo = (jint)r1->_lo >> (jint)shift;
a61af66fc99e Initial load
duke
parents:
diff changeset
999 jint hi = (jint)r1->_hi >> (jint)shift;
a61af66fc99e Initial load
duke
parents:
diff changeset
1000 assert(lo <= hi, "must have valid bounds");
a61af66fc99e Initial load
duke
parents:
diff changeset
1001 const TypeInt* ti = TypeInt::make(lo, hi, MAX2(r1->_widen,r2->_widen));
a61af66fc99e Initial load
duke
parents:
diff changeset
1002 #ifdef ASSERT
a61af66fc99e Initial load
duke
parents:
diff changeset
1003 // Make sure we get the sign-capture idiom correct.
a61af66fc99e Initial load
duke
parents:
diff changeset
1004 if (shift == BitsPerJavaInteger-1) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1005 if (r1->_lo >= 0) assert(ti == TypeInt::ZERO, ">>31 of + is 0");
a61af66fc99e Initial load
duke
parents:
diff changeset
1006 if (r1->_hi < 0) assert(ti == TypeInt::MINUS_1, ">>31 of - is -1");
a61af66fc99e Initial load
duke
parents:
diff changeset
1007 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1008 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
1009 return ti;
a61af66fc99e Initial load
duke
parents:
diff changeset
1010 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1011
a61af66fc99e Initial load
duke
parents:
diff changeset
1012 if( !r1->is_con() || !r2->is_con() )
a61af66fc99e Initial load
duke
parents:
diff changeset
1013 return TypeInt::INT;
a61af66fc99e Initial load
duke
parents:
diff changeset
1014
a61af66fc99e Initial load
duke
parents:
diff changeset
1015 // Signed shift right
a61af66fc99e Initial load
duke
parents:
diff changeset
1016 return TypeInt::make( r1->get_con() >> (r2->get_con()&31) );
a61af66fc99e Initial load
duke
parents:
diff changeset
1017 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1018
a61af66fc99e Initial load
duke
parents:
diff changeset
1019 //=============================================================================
a61af66fc99e Initial load
duke
parents:
diff changeset
1020 //------------------------------Identity---------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1021 Node *RShiftLNode::Identity( PhaseTransform *phase ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1022 const TypeInt *ti = phase->type( in(2) )->isa_int(); // shift count is an int
a61af66fc99e Initial load
duke
parents:
diff changeset
1023 return ( ti && ti->is_con() && ( ti->get_con() & ( BitsPerLong - 1 ) ) == 0 ) ? in(1) : this;
a61af66fc99e Initial load
duke
parents:
diff changeset
1024 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1025
a61af66fc99e Initial load
duke
parents:
diff changeset
1026 //------------------------------Value------------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1027 // A RShiftLNode shifts its input2 right by input1 amount.
a61af66fc99e Initial load
duke
parents:
diff changeset
1028 const Type *RShiftLNode::Value( PhaseTransform *phase ) const {
a61af66fc99e Initial load
duke
parents:
diff changeset
1029 const Type *t1 = phase->type( in(1) );
a61af66fc99e Initial load
duke
parents:
diff changeset
1030 const Type *t2 = phase->type( in(2) );
a61af66fc99e Initial load
duke
parents:
diff changeset
1031 // Either input is TOP ==> the result is TOP
a61af66fc99e Initial load
duke
parents:
diff changeset
1032 if( t1 == Type::TOP ) return Type::TOP;
a61af66fc99e Initial load
duke
parents:
diff changeset
1033 if( t2 == Type::TOP ) return Type::TOP;
a61af66fc99e Initial load
duke
parents:
diff changeset
1034
a61af66fc99e Initial load
duke
parents:
diff changeset
1035 // Left input is ZERO ==> the result is ZERO.
a61af66fc99e Initial load
duke
parents:
diff changeset
1036 if( t1 == TypeLong::ZERO ) return TypeLong::ZERO;
a61af66fc99e Initial load
duke
parents:
diff changeset
1037 // Shift by zero does nothing
a61af66fc99e Initial load
duke
parents:
diff changeset
1038 if( t2 == TypeInt::ZERO ) return t1;
a61af66fc99e Initial load
duke
parents:
diff changeset
1039
a61af66fc99e Initial load
duke
parents:
diff changeset
1040 // Either input is BOTTOM ==> the result is BOTTOM
a61af66fc99e Initial load
duke
parents:
diff changeset
1041 if (t1 == Type::BOTTOM || t2 == Type::BOTTOM)
a61af66fc99e Initial load
duke
parents:
diff changeset
1042 return TypeLong::LONG;
a61af66fc99e Initial load
duke
parents:
diff changeset
1043
a61af66fc99e Initial load
duke
parents:
diff changeset
1044 if (t2 == TypeInt::INT)
a61af66fc99e Initial load
duke
parents:
diff changeset
1045 return TypeLong::LONG;
a61af66fc99e Initial load
duke
parents:
diff changeset
1046
a61af66fc99e Initial load
duke
parents:
diff changeset
1047 const TypeLong *r1 = t1->is_long(); // Handy access
a61af66fc99e Initial load
duke
parents:
diff changeset
1048 const TypeInt *r2 = t2->is_int (); // Handy access
a61af66fc99e Initial load
duke
parents:
diff changeset
1049
a61af66fc99e Initial load
duke
parents:
diff changeset
1050 // If the shift is a constant, just shift the bounds of the type.
a61af66fc99e Initial load
duke
parents:
diff changeset
1051 // For example, if the shift is 63, we just propagate sign bits.
a61af66fc99e Initial load
duke
parents:
diff changeset
1052 if (r2->is_con()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1053 uint shift = r2->get_con();
a61af66fc99e Initial load
duke
parents:
diff changeset
1054 shift &= (2*BitsPerJavaInteger)-1; // semantics of Java shifts
a61af66fc99e Initial load
duke
parents:
diff changeset
1055 // Shift by a multiple of 64 does nothing:
a61af66fc99e Initial load
duke
parents:
diff changeset
1056 if (shift == 0) return t1;
a61af66fc99e Initial load
duke
parents:
diff changeset
1057 // Calculate reasonably aggressive bounds for the result.
a61af66fc99e Initial load
duke
parents:
diff changeset
1058 // This is necessary if we are to correctly type things
a61af66fc99e Initial load
duke
parents:
diff changeset
1059 // like (x<<24>>24) == ((byte)x).
a61af66fc99e Initial load
duke
parents:
diff changeset
1060 jlong lo = (jlong)r1->_lo >> (jlong)shift;
a61af66fc99e Initial load
duke
parents:
diff changeset
1061 jlong hi = (jlong)r1->_hi >> (jlong)shift;
a61af66fc99e Initial load
duke
parents:
diff changeset
1062 assert(lo <= hi, "must have valid bounds");
a61af66fc99e Initial load
duke
parents:
diff changeset
1063 const TypeLong* tl = TypeLong::make(lo, hi, MAX2(r1->_widen,r2->_widen));
a61af66fc99e Initial load
duke
parents:
diff changeset
1064 #ifdef ASSERT
a61af66fc99e Initial load
duke
parents:
diff changeset
1065 // Make sure we get the sign-capture idiom correct.
a61af66fc99e Initial load
duke
parents:
diff changeset
1066 if (shift == (2*BitsPerJavaInteger)-1) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1067 if (r1->_lo >= 0) assert(tl == TypeLong::ZERO, ">>63 of + is 0");
a61af66fc99e Initial load
duke
parents:
diff changeset
1068 if (r1->_hi < 0) assert(tl == TypeLong::MINUS_1, ">>63 of - is -1");
a61af66fc99e Initial load
duke
parents:
diff changeset
1069 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1070 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
1071 return tl;
a61af66fc99e Initial load
duke
parents:
diff changeset
1072 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1073
a61af66fc99e Initial load
duke
parents:
diff changeset
1074 return TypeLong::LONG; // Give up
a61af66fc99e Initial load
duke
parents:
diff changeset
1075 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1076
a61af66fc99e Initial load
duke
parents:
diff changeset
1077 //=============================================================================
a61af66fc99e Initial load
duke
parents:
diff changeset
1078 //------------------------------Identity---------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1079 Node *URShiftINode::Identity( PhaseTransform *phase ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1080 const TypeInt *ti = phase->type( in(2) )->isa_int();
a61af66fc99e Initial load
duke
parents:
diff changeset
1081 if ( ti && ti->is_con() && ( ti->get_con() & ( BitsPerInt - 1 ) ) == 0 ) return in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1082
a61af66fc99e Initial load
duke
parents:
diff changeset
1083 // Check for "((x << LogBytesPerWord) + (wordSize-1)) >> LogBytesPerWord" which is just "x".
a61af66fc99e Initial load
duke
parents:
diff changeset
1084 // Happens during new-array length computation.
a61af66fc99e Initial load
duke
parents:
diff changeset
1085 // Safe if 'x' is in the range [0..(max_int>>LogBytesPerWord)]
a61af66fc99e Initial load
duke
parents:
diff changeset
1086 Node *add = in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1087 if( add->Opcode() == Op_AddI ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1088 const TypeInt *t2 = phase->type(add->in(2))->isa_int();
a61af66fc99e Initial load
duke
parents:
diff changeset
1089 if( t2 && t2->is_con(wordSize - 1) &&
a61af66fc99e Initial load
duke
parents:
diff changeset
1090 add->in(1)->Opcode() == Op_LShiftI ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1091 // Check that shift_counts are LogBytesPerWord
a61af66fc99e Initial load
duke
parents:
diff changeset
1092 Node *lshift_count = add->in(1)->in(2);
a61af66fc99e Initial load
duke
parents:
diff changeset
1093 const TypeInt *t_lshift_count = phase->type(lshift_count)->isa_int();
a61af66fc99e Initial load
duke
parents:
diff changeset
1094 if( t_lshift_count && t_lshift_count->is_con(LogBytesPerWord) &&
a61af66fc99e Initial load
duke
parents:
diff changeset
1095 t_lshift_count == phase->type(in(2)) ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1096 Node *x = add->in(1)->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1097 const TypeInt *t_x = phase->type(x)->isa_int();
a61af66fc99e Initial load
duke
parents:
diff changeset
1098 if( t_x != NULL && 0 <= t_x->_lo && t_x->_hi <= (max_jint>>LogBytesPerWord) ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1099 return x;
a61af66fc99e Initial load
duke
parents:
diff changeset
1100 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1101 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1102 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1103 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1104
a61af66fc99e Initial load
duke
parents:
diff changeset
1105 return (phase->type(in(2))->higher_equal(TypeInt::ZERO)) ? in(1) : this;
a61af66fc99e Initial load
duke
parents:
diff changeset
1106 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1107
a61af66fc99e Initial load
duke
parents:
diff changeset
1108 //------------------------------Ideal------------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1109 Node *URShiftINode::Ideal(PhaseGVN *phase, bool can_reshape) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1110 const TypeInt *t2 = phase->type( in(2) )->isa_int();
a61af66fc99e Initial load
duke
parents:
diff changeset
1111 if( !t2 || !t2->is_con() ) return NULL; // Right input is a constant
a61af66fc99e Initial load
duke
parents:
diff changeset
1112 const int con = t2->get_con() & 31; // Shift count is always masked
a61af66fc99e Initial load
duke
parents:
diff changeset
1113 if ( con == 0 ) return NULL; // let Identity() handle a 0 shift count
a61af66fc99e Initial load
duke
parents:
diff changeset
1114 // We'll be wanting the right-shift amount as a mask of that many bits
a61af66fc99e Initial load
duke
parents:
diff changeset
1115 const int mask = right_n_bits(BitsPerJavaInteger - con);
a61af66fc99e Initial load
duke
parents:
diff changeset
1116
a61af66fc99e Initial load
duke
parents:
diff changeset
1117 int in1_op = in(1)->Opcode();
a61af66fc99e Initial load
duke
parents:
diff changeset
1118
a61af66fc99e Initial load
duke
parents:
diff changeset
1119 // Check for ((x>>>a)>>>b) and replace with (x>>>(a+b)) when a+b < 32
a61af66fc99e Initial load
duke
parents:
diff changeset
1120 if( in1_op == Op_URShiftI ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1121 const TypeInt *t12 = phase->type( in(1)->in(2) )->isa_int();
a61af66fc99e Initial load
duke
parents:
diff changeset
1122 if( t12 && t12->is_con() ) { // Right input is a constant
a61af66fc99e Initial load
duke
parents:
diff changeset
1123 assert( in(1) != in(1)->in(1), "dead loop in URShiftINode::Ideal" );
a61af66fc99e Initial load
duke
parents:
diff changeset
1124 const int con2 = t12->get_con() & 31; // Shift count is always masked
a61af66fc99e Initial load
duke
parents:
diff changeset
1125 const int con3 = con+con2;
a61af66fc99e Initial load
duke
parents:
diff changeset
1126 if( con3 < 32 ) // Only merge shifts if total is < 32
a61af66fc99e Initial load
duke
parents:
diff changeset
1127 return new (phase->C, 3) URShiftINode( in(1)->in(1), phase->intcon(con3) );
a61af66fc99e Initial load
duke
parents:
diff changeset
1128 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1129 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1130
a61af66fc99e Initial load
duke
parents:
diff changeset
1131 // Check for ((x << z) + Y) >>> z. Replace with x + con>>>z
a61af66fc99e Initial load
duke
parents:
diff changeset
1132 // The idiom for rounding to a power of 2 is "(Q+(2^z-1)) >>> z".
a61af66fc99e Initial load
duke
parents:
diff changeset
1133 // If Q is "X << z" the rounding is useless. Look for patterns like
a61af66fc99e Initial load
duke
parents:
diff changeset
1134 // ((X<<Z) + Y) >>> Z and replace with (X + Y>>>Z) & Z-mask.
a61af66fc99e Initial load
duke
parents:
diff changeset
1135 Node *add = in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1136 if( in1_op == Op_AddI ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1137 Node *lshl = add->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1138 if( lshl->Opcode() == Op_LShiftI &&
a61af66fc99e Initial load
duke
parents:
diff changeset
1139 phase->type(lshl->in(2)) == t2 ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1140 Node *y_z = phase->transform( new (phase->C, 3) URShiftINode(add->in(2),in(2)) );
a61af66fc99e Initial load
duke
parents:
diff changeset
1141 Node *sum = phase->transform( new (phase->C, 3) AddINode( lshl->in(1), y_z ) );
a61af66fc99e Initial load
duke
parents:
diff changeset
1142 return new (phase->C, 3) AndINode( sum, phase->intcon(mask) );
a61af66fc99e Initial load
duke
parents:
diff changeset
1143 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1144 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1145
a61af66fc99e Initial load
duke
parents:
diff changeset
1146 // Check for (x & mask) >>> z. Replace with (x >>> z) & (mask >>> z)
a61af66fc99e Initial load
duke
parents:
diff changeset
1147 // This shortens the mask. Also, if we are extracting a high byte and
a61af66fc99e Initial load
duke
parents:
diff changeset
1148 // storing it to a buffer, the mask will be removed completely.
a61af66fc99e Initial load
duke
parents:
diff changeset
1149 Node *andi = in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1150 if( in1_op == Op_AndI ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1151 const TypeInt *t3 = phase->type( andi->in(2) )->isa_int();
a61af66fc99e Initial load
duke
parents:
diff changeset
1152 if( t3 && t3->is_con() ) { // Right input is a constant
a61af66fc99e Initial load
duke
parents:
diff changeset
1153 jint mask2 = t3->get_con();
a61af66fc99e Initial load
duke
parents:
diff changeset
1154 mask2 >>= con; // *signed* shift downward (high-order zeroes do not help)
a61af66fc99e Initial load
duke
parents:
diff changeset
1155 Node *newshr = phase->transform( new (phase->C, 3) URShiftINode(andi->in(1), in(2)) );
a61af66fc99e Initial load
duke
parents:
diff changeset
1156 return new (phase->C, 3) AndINode(newshr, phase->intcon(mask2));
a61af66fc99e Initial load
duke
parents:
diff changeset
1157 // The negative values are easier to materialize than positive ones.
a61af66fc99e Initial load
duke
parents:
diff changeset
1158 // A typical case from address arithmetic is ((x & ~15) >> 4).
a61af66fc99e Initial load
duke
parents:
diff changeset
1159 // It's better to change that to ((x >> 4) & ~0) versus
a61af66fc99e Initial load
duke
parents:
diff changeset
1160 // ((x >> 4) & 0x0FFFFFFF). The difference is greatest in LP64.
a61af66fc99e Initial load
duke
parents:
diff changeset
1161 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1162 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1163
a61af66fc99e Initial load
duke
parents:
diff changeset
1164 // Check for "(X << z ) >>> z" which simply zero-extends
a61af66fc99e Initial load
duke
parents:
diff changeset
1165 Node *shl = in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1166 if( in1_op == Op_LShiftI &&
a61af66fc99e Initial load
duke
parents:
diff changeset
1167 phase->type(shl->in(2)) == t2 )
a61af66fc99e Initial load
duke
parents:
diff changeset
1168 return new (phase->C, 3) AndINode( shl->in(1), phase->intcon(mask) );
a61af66fc99e Initial load
duke
parents:
diff changeset
1169
a61af66fc99e Initial load
duke
parents:
diff changeset
1170 return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
1171 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1172
a61af66fc99e Initial load
duke
parents:
diff changeset
1173 //------------------------------Value------------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1174 // A URShiftINode shifts its input2 right by input1 amount.
a61af66fc99e Initial load
duke
parents:
diff changeset
1175 const Type *URShiftINode::Value( PhaseTransform *phase ) const {
a61af66fc99e Initial load
duke
parents:
diff changeset
1176 // (This is a near clone of RShiftINode::Value.)
a61af66fc99e Initial load
duke
parents:
diff changeset
1177 const Type *t1 = phase->type( in(1) );
a61af66fc99e Initial load
duke
parents:
diff changeset
1178 const Type *t2 = phase->type( in(2) );
a61af66fc99e Initial load
duke
parents:
diff changeset
1179 // Either input is TOP ==> the result is TOP
a61af66fc99e Initial load
duke
parents:
diff changeset
1180 if( t1 == Type::TOP ) return Type::TOP;
a61af66fc99e Initial load
duke
parents:
diff changeset
1181 if( t2 == Type::TOP ) return Type::TOP;
a61af66fc99e Initial load
duke
parents:
diff changeset
1182
a61af66fc99e Initial load
duke
parents:
diff changeset
1183 // Left input is ZERO ==> the result is ZERO.
a61af66fc99e Initial load
duke
parents:
diff changeset
1184 if( t1 == TypeInt::ZERO ) return TypeInt::ZERO;
a61af66fc99e Initial load
duke
parents:
diff changeset
1185 // Shift by zero does nothing
a61af66fc99e Initial load
duke
parents:
diff changeset
1186 if( t2 == TypeInt::ZERO ) return t1;
a61af66fc99e Initial load
duke
parents:
diff changeset
1187
a61af66fc99e Initial load
duke
parents:
diff changeset
1188 // Either input is BOTTOM ==> the result is BOTTOM
a61af66fc99e Initial load
duke
parents:
diff changeset
1189 if (t1 == Type::BOTTOM || t2 == Type::BOTTOM)
a61af66fc99e Initial load
duke
parents:
diff changeset
1190 return TypeInt::INT;
a61af66fc99e Initial load
duke
parents:
diff changeset
1191
a61af66fc99e Initial load
duke
parents:
diff changeset
1192 if (t2 == TypeInt::INT)
a61af66fc99e Initial load
duke
parents:
diff changeset
1193 return TypeInt::INT;
a61af66fc99e Initial load
duke
parents:
diff changeset
1194
a61af66fc99e Initial load
duke
parents:
diff changeset
1195 const TypeInt *r1 = t1->is_int(); // Handy access
a61af66fc99e Initial load
duke
parents:
diff changeset
1196 const TypeInt *r2 = t2->is_int(); // Handy access
a61af66fc99e Initial load
duke
parents:
diff changeset
1197
a61af66fc99e Initial load
duke
parents:
diff changeset
1198 if (r2->is_con()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1199 uint shift = r2->get_con();
a61af66fc99e Initial load
duke
parents:
diff changeset
1200 shift &= BitsPerJavaInteger-1; // semantics of Java shifts
a61af66fc99e Initial load
duke
parents:
diff changeset
1201 // Shift by a multiple of 32 does nothing:
a61af66fc99e Initial load
duke
parents:
diff changeset
1202 if (shift == 0) return t1;
a61af66fc99e Initial load
duke
parents:
diff changeset
1203 // Calculate reasonably aggressive bounds for the result.
a61af66fc99e Initial load
duke
parents:
diff changeset
1204 jint lo = (juint)r1->_lo >> (juint)shift;
a61af66fc99e Initial load
duke
parents:
diff changeset
1205 jint hi = (juint)r1->_hi >> (juint)shift;
a61af66fc99e Initial load
duke
parents:
diff changeset
1206 if (r1->_hi >= 0 && r1->_lo < 0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1207 // If the type has both negative and positive values,
a61af66fc99e Initial load
duke
parents:
diff changeset
1208 // there are two separate sub-domains to worry about:
a61af66fc99e Initial load
duke
parents:
diff changeset
1209 // The positive half and the negative half.
a61af66fc99e Initial load
duke
parents:
diff changeset
1210 jint neg_lo = lo;
a61af66fc99e Initial load
duke
parents:
diff changeset
1211 jint neg_hi = (juint)-1 >> (juint)shift;
a61af66fc99e Initial load
duke
parents:
diff changeset
1212 jint pos_lo = (juint) 0 >> (juint)shift;
a61af66fc99e Initial load
duke
parents:
diff changeset
1213 jint pos_hi = hi;
a61af66fc99e Initial load
duke
parents:
diff changeset
1214 lo = MIN2(neg_lo, pos_lo); // == 0
a61af66fc99e Initial load
duke
parents:
diff changeset
1215 hi = MAX2(neg_hi, pos_hi); // == -1 >>> shift;
a61af66fc99e Initial load
duke
parents:
diff changeset
1216 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1217 assert(lo <= hi, "must have valid bounds");
a61af66fc99e Initial load
duke
parents:
diff changeset
1218 const TypeInt* ti = TypeInt::make(lo, hi, MAX2(r1->_widen,r2->_widen));
a61af66fc99e Initial load
duke
parents:
diff changeset
1219 #ifdef ASSERT
a61af66fc99e Initial load
duke
parents:
diff changeset
1220 // Make sure we get the sign-capture idiom correct.
a61af66fc99e Initial load
duke
parents:
diff changeset
1221 if (shift == BitsPerJavaInteger-1) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1222 if (r1->_lo >= 0) assert(ti == TypeInt::ZERO, ">>>31 of + is 0");
a61af66fc99e Initial load
duke
parents:
diff changeset
1223 if (r1->_hi < 0) assert(ti == TypeInt::ONE, ">>>31 of - is +1");
a61af66fc99e Initial load
duke
parents:
diff changeset
1224 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1225 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
1226 return ti;
a61af66fc99e Initial load
duke
parents:
diff changeset
1227 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1228
a61af66fc99e Initial load
duke
parents:
diff changeset
1229 //
a61af66fc99e Initial load
duke
parents:
diff changeset
1230 // Do not support shifted oops in info for GC
a61af66fc99e Initial load
duke
parents:
diff changeset
1231 //
a61af66fc99e Initial load
duke
parents:
diff changeset
1232 // else if( t1->base() == Type::InstPtr ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1233 //
a61af66fc99e Initial load
duke
parents:
diff changeset
1234 // const TypeInstPtr *o = t1->is_instptr();
a61af66fc99e Initial load
duke
parents:
diff changeset
1235 // if( t1->singleton() )
a61af66fc99e Initial load
duke
parents:
diff changeset
1236 // return TypeInt::make( ((uint32)o->const_oop() + o->_offset) >> shift );
a61af66fc99e Initial load
duke
parents:
diff changeset
1237 // }
a61af66fc99e Initial load
duke
parents:
diff changeset
1238 // else if( t1->base() == Type::KlassPtr ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1239 // const TypeKlassPtr *o = t1->is_klassptr();
a61af66fc99e Initial load
duke
parents:
diff changeset
1240 // if( t1->singleton() )
a61af66fc99e Initial load
duke
parents:
diff changeset
1241 // return TypeInt::make( ((uint32)o->const_oop() + o->_offset) >> shift );
a61af66fc99e Initial load
duke
parents:
diff changeset
1242 // }
a61af66fc99e Initial load
duke
parents:
diff changeset
1243
a61af66fc99e Initial load
duke
parents:
diff changeset
1244 return TypeInt::INT;
a61af66fc99e Initial load
duke
parents:
diff changeset
1245 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1246
a61af66fc99e Initial load
duke
parents:
diff changeset
1247 //=============================================================================
a61af66fc99e Initial load
duke
parents:
diff changeset
1248 //------------------------------Identity---------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1249 Node *URShiftLNode::Identity( PhaseTransform *phase ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1250 const TypeInt *ti = phase->type( in(2) )->isa_int(); // shift count is an int
a61af66fc99e Initial load
duke
parents:
diff changeset
1251 return ( ti && ti->is_con() && ( ti->get_con() & ( BitsPerLong - 1 ) ) == 0 ) ? in(1) : this;
a61af66fc99e Initial load
duke
parents:
diff changeset
1252 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1253
a61af66fc99e Initial load
duke
parents:
diff changeset
1254 //------------------------------Ideal------------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1255 Node *URShiftLNode::Ideal(PhaseGVN *phase, bool can_reshape) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1256 const TypeInt *t2 = phase->type( in(2) )->isa_int();
a61af66fc99e Initial load
duke
parents:
diff changeset
1257 if( !t2 || !t2->is_con() ) return NULL; // Right input is a constant
a61af66fc99e Initial load
duke
parents:
diff changeset
1258 const int con = t2->get_con() & ( BitsPerLong - 1 ); // Shift count is always masked
a61af66fc99e Initial load
duke
parents:
diff changeset
1259 if ( con == 0 ) return NULL; // let Identity() handle a 0 shift count
a61af66fc99e Initial load
duke
parents:
diff changeset
1260 // note: mask computation below does not work for 0 shift count
a61af66fc99e Initial load
duke
parents:
diff changeset
1261 // We'll be wanting the right-shift amount as a mask of that many bits
559
7628781568e1 6795362: 32bit server compiler leads to wrong results on solaris-x86
twisti
parents: 558
diff changeset
1262 const jlong mask = (((jlong)CONST64(1) << (jlong)(BitsPerJavaLong - con)) -1);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1263
a61af66fc99e Initial load
duke
parents:
diff changeset
1264 // Check for ((x << z) + Y) >>> z. Replace with x + con>>>z
a61af66fc99e Initial load
duke
parents:
diff changeset
1265 // The idiom for rounding to a power of 2 is "(Q+(2^z-1)) >>> z".
a61af66fc99e Initial load
duke
parents:
diff changeset
1266 // If Q is "X << z" the rounding is useless. Look for patterns like
a61af66fc99e Initial load
duke
parents:
diff changeset
1267 // ((X<<Z) + Y) >>> Z and replace with (X + Y>>>Z) & Z-mask.
a61af66fc99e Initial load
duke
parents:
diff changeset
1268 Node *add = in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1269 if( add->Opcode() == Op_AddL ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1270 Node *lshl = add->in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1271 if( lshl->Opcode() == Op_LShiftL &&
a61af66fc99e Initial load
duke
parents:
diff changeset
1272 phase->type(lshl->in(2)) == t2 ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1273 Node *y_z = phase->transform( new (phase->C, 3) URShiftLNode(add->in(2),in(2)) );
a61af66fc99e Initial load
duke
parents:
diff changeset
1274 Node *sum = phase->transform( new (phase->C, 3) AddLNode( lshl->in(1), y_z ) );
a61af66fc99e Initial load
duke
parents:
diff changeset
1275 return new (phase->C, 3) AndLNode( sum, phase->longcon(mask) );
a61af66fc99e Initial load
duke
parents:
diff changeset
1276 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1277 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1278
a61af66fc99e Initial load
duke
parents:
diff changeset
1279 // Check for (x & mask) >>> z. Replace with (x >>> z) & (mask >>> z)
a61af66fc99e Initial load
duke
parents:
diff changeset
1280 // This shortens the mask. Also, if we are extracting a high byte and
a61af66fc99e Initial load
duke
parents:
diff changeset
1281 // storing it to a buffer, the mask will be removed completely.
a61af66fc99e Initial load
duke
parents:
diff changeset
1282 Node *andi = in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1283 if( andi->Opcode() == Op_AndL ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1284 const TypeLong *t3 = phase->type( andi->in(2) )->isa_long();
a61af66fc99e Initial load
duke
parents:
diff changeset
1285 if( t3 && t3->is_con() ) { // Right input is a constant
a61af66fc99e Initial load
duke
parents:
diff changeset
1286 jlong mask2 = t3->get_con();
a61af66fc99e Initial load
duke
parents:
diff changeset
1287 mask2 >>= con; // *signed* shift downward (high-order zeroes do not help)
a61af66fc99e Initial load
duke
parents:
diff changeset
1288 Node *newshr = phase->transform( new (phase->C, 3) URShiftLNode(andi->in(1), in(2)) );
a61af66fc99e Initial load
duke
parents:
diff changeset
1289 return new (phase->C, 3) AndLNode(newshr, phase->longcon(mask2));
a61af66fc99e Initial load
duke
parents:
diff changeset
1290 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1291 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1292
a61af66fc99e Initial load
duke
parents:
diff changeset
1293 // Check for "(X << z ) >>> z" which simply zero-extends
a61af66fc99e Initial load
duke
parents:
diff changeset
1294 Node *shl = in(1);
a61af66fc99e Initial load
duke
parents:
diff changeset
1295 if( shl->Opcode() == Op_LShiftL &&
a61af66fc99e Initial load
duke
parents:
diff changeset
1296 phase->type(shl->in(2)) == t2 )
a61af66fc99e Initial load
duke
parents:
diff changeset
1297 return new (phase->C, 3) AndLNode( shl->in(1), phase->longcon(mask) );
a61af66fc99e Initial load
duke
parents:
diff changeset
1298
a61af66fc99e Initial load
duke
parents:
diff changeset
1299 return NULL;
a61af66fc99e Initial load
duke
parents:
diff changeset
1300 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1301
a61af66fc99e Initial load
duke
parents:
diff changeset
1302 //------------------------------Value------------------------------------------
a61af66fc99e Initial load
duke
parents:
diff changeset
1303 // A URShiftINode shifts its input2 right by input1 amount.
a61af66fc99e Initial load
duke
parents:
diff changeset
1304 const Type *URShiftLNode::Value( PhaseTransform *phase ) const {
a61af66fc99e Initial load
duke
parents:
diff changeset
1305 // (This is a near clone of RShiftLNode::Value.)
a61af66fc99e Initial load
duke
parents:
diff changeset
1306 const Type *t1 = phase->type( in(1) );
a61af66fc99e Initial load
duke
parents:
diff changeset
1307 const Type *t2 = phase->type( in(2) );
a61af66fc99e Initial load
duke
parents:
diff changeset
1308 // Either input is TOP ==> the result is TOP
a61af66fc99e Initial load
duke
parents:
diff changeset
1309 if( t1 == Type::TOP ) return Type::TOP;
a61af66fc99e Initial load
duke
parents:
diff changeset
1310 if( t2 == Type::TOP ) return Type::TOP;
a61af66fc99e Initial load
duke
parents:
diff changeset
1311
a61af66fc99e Initial load
duke
parents:
diff changeset
1312 // Left input is ZERO ==> the result is ZERO.
a61af66fc99e Initial load
duke
parents:
diff changeset
1313 if( t1 == TypeLong::ZERO ) return TypeLong::ZERO;
a61af66fc99e Initial load
duke
parents:
diff changeset
1314 // Shift by zero does nothing
a61af66fc99e Initial load
duke
parents:
diff changeset
1315 if( t2 == TypeInt::ZERO ) return t1;
a61af66fc99e Initial load
duke
parents:
diff changeset
1316
a61af66fc99e Initial load
duke
parents:
diff changeset
1317 // Either input is BOTTOM ==> the result is BOTTOM
a61af66fc99e Initial load
duke
parents:
diff changeset
1318 if (t1 == Type::BOTTOM || t2 == Type::BOTTOM)
a61af66fc99e Initial load
duke
parents:
diff changeset
1319 return TypeLong::LONG;
a61af66fc99e Initial load
duke
parents:
diff changeset
1320
a61af66fc99e Initial load
duke
parents:
diff changeset
1321 if (t2 == TypeInt::INT)
a61af66fc99e Initial load
duke
parents:
diff changeset
1322 return TypeLong::LONG;
a61af66fc99e Initial load
duke
parents:
diff changeset
1323
a61af66fc99e Initial load
duke
parents:
diff changeset
1324 const TypeLong *r1 = t1->is_long(); // Handy access
a61af66fc99e Initial load
duke
parents:
diff changeset
1325 const TypeInt *r2 = t2->is_int (); // Handy access
a61af66fc99e Initial load
duke
parents:
diff changeset
1326
a61af66fc99e Initial load
duke
parents:
diff changeset
1327 if (r2->is_con()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1328 uint shift = r2->get_con();
559
7628781568e1 6795362: 32bit server compiler leads to wrong results on solaris-x86
twisti
parents: 558
diff changeset
1329 shift &= BitsPerJavaLong - 1; // semantics of Java shifts
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1330 // Shift by a multiple of 64 does nothing:
a61af66fc99e Initial load
duke
parents:
diff changeset
1331 if (shift == 0) return t1;
a61af66fc99e Initial load
duke
parents:
diff changeset
1332 // Calculate reasonably aggressive bounds for the result.
a61af66fc99e Initial load
duke
parents:
diff changeset
1333 jlong lo = (julong)r1->_lo >> (juint)shift;
a61af66fc99e Initial load
duke
parents:
diff changeset
1334 jlong hi = (julong)r1->_hi >> (juint)shift;
a61af66fc99e Initial load
duke
parents:
diff changeset
1335 if (r1->_hi >= 0 && r1->_lo < 0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
1336 // If the type has both negative and positive values,
a61af66fc99e Initial load
duke
parents:
diff changeset
1337 // there are two separate sub-domains to worry about:
a61af66fc99e Initial load
duke
parents:
diff changeset
1338 // The positive half and the negative half.
a61af66fc99e Initial load
duke
parents:
diff changeset
1339 jlong neg_lo = lo;
a61af66fc99e Initial load
duke
parents:
diff changeset
1340 jlong neg_hi = (julong)-1 >> (juint)shift;
a61af66fc99e Initial load
duke
parents:
diff changeset
1341 jlong pos_lo = (julong) 0 >> (juint)shift;
a61af66fc99e Initial load
duke
parents:
diff changeset
1342 jlong pos_hi = hi;
a61af66fc99e Initial load
duke
parents:
diff changeset
1343 //lo = MIN2(neg_lo, pos_lo); // == 0
a61af66fc99e Initial load
duke
parents:
diff changeset
1344 lo = neg_lo < pos_lo ? neg_lo : pos_lo;
a61af66fc99e Initial load
duke
parents:
diff changeset
1345 //hi = MAX2(neg_hi, pos_hi); // == -1 >>> shift;
a61af66fc99e Initial load
duke
parents:
diff changeset
1346 hi = neg_hi > pos_hi ? neg_hi : pos_hi;
a61af66fc99e Initial load
duke
parents:
diff changeset
1347 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1348 assert(lo <= hi, "must have valid bounds");
a61af66fc99e Initial load
duke
parents:
diff changeset
1349 const TypeLong* tl = TypeLong::make(lo, hi, MAX2(r1->_widen,r2->_widen));
a61af66fc99e Initial load
duke
parents:
diff changeset
1350 #ifdef ASSERT
a61af66fc99e Initial load
duke
parents:
diff changeset
1351 // Make sure we get the sign-capture idiom correct.
559
7628781568e1 6795362: 32bit server compiler leads to wrong results on solaris-x86
twisti
parents: 558
diff changeset
1352 if (shift == BitsPerJavaLong - 1) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1353 if (r1->_lo >= 0) assert(tl == TypeLong::ZERO, ">>>63 of + is 0");
a61af66fc99e Initial load
duke
parents:
diff changeset
1354 if (r1->_hi < 0) assert(tl == TypeLong::ONE, ">>>63 of - is +1");
a61af66fc99e Initial load
duke
parents:
diff changeset
1355 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1356 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
1357 return tl;
a61af66fc99e Initial load
duke
parents:
diff changeset
1358 }
a61af66fc99e Initial load
duke
parents:
diff changeset
1359
a61af66fc99e Initial load
duke
parents:
diff changeset
1360 return TypeLong::LONG; // Give up
a61af66fc99e Initial load
duke
parents:
diff changeset
1361 }