Mercurial > hg > truffle
annotate src/share/vm/opto/parse2.cpp @ 20439:f6f9aec27858
8030976: Untaken paths should be more vigorously pruned at highest optimization level
Reviewed-by: roland, vlivanov
| author | rbackman |
|---|---|
| date | Wed, 10 Sep 2014 12:39:11 +0200 |
| parents | 78bbf4d43a14 |
| children | 42460b71ba70 |
| rev | line source |
|---|---|
| 0 | 1 /* |
|
17937
78bbf4d43a14
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
17799
diff
changeset
|
2 * Copyright (c) 1998, 2014, Oracle and/or its affiliates. All rights reserved. |
| 0 | 3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
| 4 * | |
| 5 * This code is free software; you can redistribute it and/or modify it | |
| 6 * under the terms of the GNU General Public License version 2 only, as | |
| 7 * published by the Free Software Foundation. | |
| 8 * | |
| 9 * This code is distributed in the hope that it will be useful, but WITHOUT | |
| 10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or | |
| 11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License | |
| 12 * version 2 for more details (a copy is included in the LICENSE file that | |
| 13 * accompanied this code). | |
| 14 * | |
| 15 * You should have received a copy of the GNU General Public License version | |
| 16 * 2 along with this work; if not, write to the Free Software Foundation, | |
| 17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. | |
| 18 * | |
|
1552
c18cbe5936b8
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
1213
diff
changeset
|
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
|
c18cbe5936b8
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
1213
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:
1213
diff
changeset
|
21 * questions. |
| 0 | 22 * |
| 23 */ | |
| 24 | |
| 1972 | 25 #include "precompiled.hpp" |
| 26 #include "ci/ciMethodData.hpp" | |
| 27 #include "classfile/systemDictionary.hpp" | |
| 28 #include "classfile/vmSymbols.hpp" | |
| 29 #include "compiler/compileLog.hpp" | |
| 30 #include "interpreter/linkResolver.hpp" | |
| 31 #include "memory/universe.inline.hpp" | |
| 32 #include "opto/addnode.hpp" | |
| 33 #include "opto/divnode.hpp" | |
| 34 #include "opto/idealGraphPrinter.hpp" | |
| 35 #include "opto/matcher.hpp" | |
| 36 #include "opto/memnode.hpp" | |
| 37 #include "opto/mulnode.hpp" | |
| 38 #include "opto/parse.hpp" | |
| 39 #include "opto/runtime.hpp" | |
| 40 #include "runtime/deoptimization.hpp" | |
| 41 #include "runtime/sharedRuntime.hpp" | |
| 0 | 42 |
| 43 extern int explicit_null_checks_inserted, | |
| 44 explicit_null_checks_elided; | |
| 45 | |
| 46 //---------------------------------array_load---------------------------------- | |
| 47 void Parse::array_load(BasicType elem_type) { | |
| 48 const Type* elem = Type::TOP; | |
| 49 Node* adr = array_addressing(elem_type, 0, &elem); | |
| 605 | 50 if (stopped()) return; // guaranteed null or range check |
|
7194
beebba0acc11
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
6848
diff
changeset
|
51 dec_sp(2); // Pop array and index |
| 0 | 52 const TypeAryPtr* adr_type = TypeAryPtr::get_array_body_type(elem_type); |
|
14429
2113136690bc
8024921: PPC64 (part 113): Extend Load and Store nodes to know about memory ordering
goetz
parents:
12966
diff
changeset
|
53 Node* ld = make_load(control(), adr, elem, elem_type, adr_type, MemNode::unordered); |
| 0 | 54 push(ld); |
| 55 } | |
| 56 | |
| 57 | |
| 58 //--------------------------------array_store---------------------------------- | |
| 59 void Parse::array_store(BasicType elem_type) { | |
| 60 Node* adr = array_addressing(elem_type, 1); | |
| 605 | 61 if (stopped()) return; // guaranteed null or range check |
| 0 | 62 Node* val = pop(); |
|
7194
beebba0acc11
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
6848
diff
changeset
|
63 dec_sp(2); // Pop array and index |
| 0 | 64 const TypeAryPtr* adr_type = TypeAryPtr::get_array_body_type(elem_type); |
|
14429
2113136690bc
8024921: PPC64 (part 113): Extend Load and Store nodes to know about memory ordering
goetz
parents:
12966
diff
changeset
|
65 store_to_memory(control(), adr, val, elem_type, adr_type, StoreNode::release_if_reference(elem_type)); |
| 0 | 66 } |
| 67 | |
| 68 | |
| 69 //------------------------------array_addressing------------------------------- | |
| 70 // Pull array and index from the stack. Compute pointer-to-element. | |
| 71 Node* Parse::array_addressing(BasicType type, int vals, const Type* *result2) { | |
| 72 Node *idx = peek(0+vals); // Get from stack without popping | |
| 73 Node *ary = peek(1+vals); // in case of exception | |
| 74 | |
| 75 // Null check the array base, with correct stack contents | |
|
7194
beebba0acc11
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
6848
diff
changeset
|
76 ary = null_check(ary, T_ARRAY); |
| 0 | 77 // Compile-time detect of null-exception? |
| 78 if (stopped()) return top(); | |
| 79 | |
| 80 const TypeAryPtr* arytype = _gvn.type(ary)->is_aryptr(); | |
| 81 const TypeInt* sizetype = arytype->size(); | |
| 82 const Type* elemtype = arytype->elem(); | |
| 83 | |
| 84 if (UseUniqueSubclasses && result2 != NULL) { | |
|
221
1e026f8da827
6710487: More than half of JDI Regression tests hang with COOPs in -Xcomp mode
kvn
parents:
129
diff
changeset
|
85 const Type* el = elemtype->make_ptr(); |
|
1e026f8da827
6710487: More than half of JDI Regression tests hang with COOPs in -Xcomp mode
kvn
parents:
129
diff
changeset
|
86 if (el && el->isa_instptr()) { |
|
1e026f8da827
6710487: More than half of JDI Regression tests hang with COOPs in -Xcomp mode
kvn
parents:
129
diff
changeset
|
87 const TypeInstPtr* toop = el->is_instptr(); |
| 0 | 88 if (toop->klass()->as_instance_klass()->unique_concrete_subklass()) { |
| 89 // If we load from "AbstractClass[]" we must see "ConcreteSubClass". | |
| 90 const Type* subklass = Type::get_const_type(toop->klass()); | |
|
17671
de95063c0e34
8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents:
17467
diff
changeset
|
91 elemtype = subklass->join_speculative(el); |
| 0 | 92 } |
| 93 } | |
| 94 } | |
| 95 | |
| 96 // Check for big class initializers with all constant offsets | |
| 97 // feeding into a known-size array. | |
| 98 const TypeInt* idxtype = _gvn.type(idx)->is_int(); | |
| 99 // See if the highest idx value is less than the lowest array bound, | |
| 100 // and if the idx value cannot be negative: | |
| 101 bool need_range_check = true; | |
| 102 if (idxtype->_hi < sizetype->_lo && idxtype->_lo >= 0) { | |
| 103 need_range_check = false; | |
| 104 if (C->log() != NULL) C->log()->elem("observe that='!need_range_check'"); | |
| 105 } | |
| 106 | |
| 8862 | 107 ciKlass * arytype_klass = arytype->klass(); |
| 108 if ((arytype_klass != NULL) && (!arytype_klass->is_loaded())) { | |
| 0 | 109 // Only fails for some -Xcomp runs |
| 110 // The class is unloaded. We have to run this bytecode in the interpreter. | |
| 111 uncommon_trap(Deoptimization::Reason_unloaded, | |
| 112 Deoptimization::Action_reinterpret, | |
| 113 arytype->klass(), "!loaded array"); | |
| 114 return top(); | |
| 115 } | |
| 116 | |
| 117 // Do the range check | |
| 118 if (GenerateRangeChecks && need_range_check) { | |
|
129
c0939256690b
6646019: array subscript expressions become top() with -d64
rasbold
parents:
113
diff
changeset
|
119 Node* tst; |
|
c0939256690b
6646019: array subscript expressions become top() with -d64
rasbold
parents:
113
diff
changeset
|
120 if (sizetype->_hi <= 0) { |
|
366
8261ee795323
6711100: 64bit fastdebug server vm crashes with assert(_base == Int,"Not an Int")
rasbold
parents:
254
diff
changeset
|
121 // The greatest array bound is negative, so we can conclude that we're |
|
129
c0939256690b
6646019: array subscript expressions become top() with -d64
rasbold
parents:
113
diff
changeset
|
122 // compiling unreachable code, but the unsigned compare trick used below |
|
c0939256690b
6646019: array subscript expressions become top() with -d64
rasbold
parents:
113
diff
changeset
|
123 // only works with non-negative lengths. Instead, hack "tst" to be zero so |
|
c0939256690b
6646019: array subscript expressions become top() with -d64
rasbold
parents:
113
diff
changeset
|
124 // the uncommon_trap path will always be taken. |
|
c0939256690b
6646019: array subscript expressions become top() with -d64
rasbold
parents:
113
diff
changeset
|
125 tst = _gvn.intcon(0); |
|
c0939256690b
6646019: array subscript expressions become top() with -d64
rasbold
parents:
113
diff
changeset
|
126 } else { |
|
366
8261ee795323
6711100: 64bit fastdebug server vm crashes with assert(_base == Int,"Not an Int")
rasbold
parents:
254
diff
changeset
|
127 // Range is constant in array-oop, so we can use the original state of mem |
|
8261ee795323
6711100: 64bit fastdebug server vm crashes with assert(_base == Int,"Not an Int")
rasbold
parents:
254
diff
changeset
|
128 Node* len = load_array_length(ary); |
|
8261ee795323
6711100: 64bit fastdebug server vm crashes with assert(_base == Int,"Not an Int")
rasbold
parents:
254
diff
changeset
|
129 |
|
129
c0939256690b
6646019: array subscript expressions become top() with -d64
rasbold
parents:
113
diff
changeset
|
130 // Test length vs index (standard trick using unsigned compare) |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
131 Node* chk = _gvn.transform( new (C) CmpUNode(idx, len) ); |
|
129
c0939256690b
6646019: array subscript expressions become top() with -d64
rasbold
parents:
113
diff
changeset
|
132 BoolTest::mask btest = BoolTest::lt; |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
133 tst = _gvn.transform( new (C) BoolNode(chk, btest) ); |
|
129
c0939256690b
6646019: array subscript expressions become top() with -d64
rasbold
parents:
113
diff
changeset
|
134 } |
| 0 | 135 // Branch to failure if out of bounds |
| 136 { BuildCutout unless(this, tst, PROB_MAX); | |
| 137 if (C->allow_range_check_smearing()) { | |
| 138 // Do not use builtin_throw, since range checks are sometimes | |
| 139 // made more stringent by an optimistic transformation. | |
| 140 // This creates "tentative" range checks at this point, | |
| 141 // which are not guaranteed to throw exceptions. | |
| 142 // See IfNode::Ideal, is_range_check, adjust_check. | |
| 143 uncommon_trap(Deoptimization::Reason_range_check, | |
| 144 Deoptimization::Action_make_not_entrant, | |
| 145 NULL, "range_check"); | |
| 146 } else { | |
| 147 // If we have already recompiled with the range-check-widening | |
| 148 // heroic optimization turned off, then we must really be throwing | |
| 149 // range check exceptions. | |
| 150 builtin_throw(Deoptimization::Reason_range_check, idx); | |
| 151 } | |
| 152 } | |
| 153 } | |
| 154 // Check for always knowing you are throwing a range-check exception | |
| 155 if (stopped()) return top(); | |
| 156 | |
|
366
8261ee795323
6711100: 64bit fastdebug server vm crashes with assert(_base == Int,"Not an Int")
rasbold
parents:
254
diff
changeset
|
157 Node* ptr = array_element_address(ary, idx, type, sizetype); |
| 0 | 158 |
| 159 if (result2 != NULL) *result2 = elemtype; | |
|
366
8261ee795323
6711100: 64bit fastdebug server vm crashes with assert(_base == Int,"Not an Int")
rasbold
parents:
254
diff
changeset
|
160 |
|
8261ee795323
6711100: 64bit fastdebug server vm crashes with assert(_base == Int,"Not an Int")
rasbold
parents:
254
diff
changeset
|
161 assert(ptr != top(), "top should go hand-in-hand with stopped"); |
|
8261ee795323
6711100: 64bit fastdebug server vm crashes with assert(_base == Int,"Not an Int")
rasbold
parents:
254
diff
changeset
|
162 |
| 0 | 163 return ptr; |
| 164 } | |
| 165 | |
| 166 | |
| 167 // returns IfNode | |
| 168 IfNode* Parse::jump_if_fork_int(Node* a, Node* b, BoolTest::mask mask) { | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
169 Node *cmp = _gvn.transform( new (C) CmpINode( a, b)); // two cases: shiftcount > 32 and shiftcount <= 32 |
|
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
170 Node *tst = _gvn.transform( new (C) BoolNode( cmp, mask)); |
| 0 | 171 IfNode *iff = create_and_map_if( control(), tst, ((mask == BoolTest::eq) ? PROB_STATIC_INFREQUENT : PROB_FAIR), COUNT_UNKNOWN ); |
| 172 return iff; | |
| 173 } | |
| 174 | |
| 175 // return Region node | |
| 176 Node* Parse::jump_if_join(Node* iffalse, Node* iftrue) { | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
177 Node *region = new (C) RegionNode(3); // 2 results |
| 0 | 178 record_for_igvn(region); |
| 179 region->init_req(1, iffalse); | |
| 180 region->init_req(2, iftrue ); | |
| 181 _gvn.set_type(region, Type::CONTROL); | |
| 182 region = _gvn.transform(region); | |
| 183 set_control (region); | |
| 184 return region; | |
| 185 } | |
| 186 | |
| 187 | |
| 188 //------------------------------helper for tableswitch------------------------- | |
| 189 void Parse::jump_if_true_fork(IfNode *iff, int dest_bci_if_true, int prof_table_index) { | |
| 190 // True branch, use existing map info | |
| 191 { PreserveJVMState pjvms(this); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
192 Node *iftrue = _gvn.transform( new (C) IfTrueNode (iff) ); |
| 0 | 193 set_control( iftrue ); |
| 194 profile_switch_case(prof_table_index); | |
| 195 merge_new_path(dest_bci_if_true); | |
| 196 } | |
| 197 | |
| 198 // False branch | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
199 Node *iffalse = _gvn.transform( new (C) IfFalseNode(iff) ); |
| 0 | 200 set_control( iffalse ); |
| 201 } | |
| 202 | |
| 203 void Parse::jump_if_false_fork(IfNode *iff, int dest_bci_if_true, int prof_table_index) { | |
| 204 // True branch, use existing map info | |
| 205 { PreserveJVMState pjvms(this); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
206 Node *iffalse = _gvn.transform( new (C) IfFalseNode (iff) ); |
| 0 | 207 set_control( iffalse ); |
| 208 profile_switch_case(prof_table_index); | |
| 209 merge_new_path(dest_bci_if_true); | |
| 210 } | |
| 211 | |
| 212 // False branch | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
213 Node *iftrue = _gvn.transform( new (C) IfTrueNode(iff) ); |
| 0 | 214 set_control( iftrue ); |
| 215 } | |
| 216 | |
| 217 void Parse::jump_if_always_fork(int dest_bci, int prof_table_index) { | |
| 218 // False branch, use existing map and control() | |
| 219 profile_switch_case(prof_table_index); | |
| 220 merge_new_path(dest_bci); | |
| 221 } | |
| 222 | |
| 223 | |
| 224 extern "C" { | |
| 225 static int jint_cmp(const void *i, const void *j) { | |
| 226 int a = *(jint *)i; | |
| 227 int b = *(jint *)j; | |
| 228 return a > b ? 1 : a < b ? -1 : 0; | |
| 229 } | |
| 230 } | |
| 231 | |
| 232 | |
| 233 // Default value for methodData switch indexing. Must be a negative value to avoid | |
| 234 // conflict with any legal switch index. | |
| 235 #define NullTableIndex -1 | |
| 236 | |
| 237 class SwitchRange : public StackObj { | |
| 238 // a range of integers coupled with a bci destination | |
| 239 jint _lo; // inclusive lower limit | |
| 240 jint _hi; // inclusive upper limit | |
| 241 int _dest; | |
| 242 int _table_index; // index into method data table | |
| 243 | |
| 244 public: | |
| 245 jint lo() const { return _lo; } | |
| 246 jint hi() const { return _hi; } | |
| 247 int dest() const { return _dest; } | |
| 248 int table_index() const { return _table_index; } | |
| 249 bool is_singleton() const { return _lo == _hi; } | |
| 250 | |
| 251 void setRange(jint lo, jint hi, int dest, int table_index) { | |
| 252 assert(lo <= hi, "must be a non-empty range"); | |
| 253 _lo = lo, _hi = hi; _dest = dest; _table_index = table_index; | |
| 254 } | |
| 255 bool adjoinRange(jint lo, jint hi, int dest, int table_index) { | |
| 256 assert(lo <= hi, "must be a non-empty range"); | |
| 257 if (lo == _hi+1 && dest == _dest && table_index == _table_index) { | |
| 258 _hi = hi; | |
| 259 return true; | |
| 260 } | |
| 261 return false; | |
| 262 } | |
| 263 | |
| 264 void set (jint value, int dest, int table_index) { | |
| 265 setRange(value, value, dest, table_index); | |
| 266 } | |
| 267 bool adjoin(jint value, int dest, int table_index) { | |
| 268 return adjoinRange(value, value, dest, table_index); | |
| 269 } | |
| 270 | |
|
12866
f478c98e8114
8024774: assert(_con < t->is_tuple()->cnt()) failed: ProjNode::_con must be in range
vlivanov
parents:
10278
diff
changeset
|
271 void print() { |
| 0 | 272 if (is_singleton()) |
| 273 tty->print(" {%d}=>%d", lo(), dest()); | |
| 274 else if (lo() == min_jint) | |
| 275 tty->print(" {..%d}=>%d", hi(), dest()); | |
| 276 else if (hi() == max_jint) | |
| 277 tty->print(" {%d..}=>%d", lo(), dest()); | |
| 278 else | |
| 279 tty->print(" {%d..%d}=>%d", lo(), hi(), dest()); | |
| 280 } | |
| 281 }; | |
| 282 | |
| 283 | |
| 284 //-------------------------------do_tableswitch-------------------------------- | |
| 285 void Parse::do_tableswitch() { | |
| 286 Node* lookup = pop(); | |
| 287 | |
| 288 // Get information about tableswitch | |
| 289 int default_dest = iter().get_dest_table(0); | |
| 290 int lo_index = iter().get_int_table(1); | |
| 291 int hi_index = iter().get_int_table(2); | |
| 292 int len = hi_index - lo_index + 1; | |
| 293 | |
| 294 if (len < 1) { | |
| 295 // If this is a backward branch, add safepoint | |
| 296 maybe_add_safepoint(default_dest); | |
| 297 merge(default_dest); | |
| 298 return; | |
| 299 } | |
| 300 | |
| 301 // generate decision tree, using trichotomy when possible | |
| 302 int rnum = len+2; | |
| 303 bool makes_backward_branch = false; | |
| 304 SwitchRange* ranges = NEW_RESOURCE_ARRAY(SwitchRange, rnum); | |
| 305 int rp = -1; | |
| 306 if (lo_index != min_jint) { | |
| 307 ranges[++rp].setRange(min_jint, lo_index-1, default_dest, NullTableIndex); | |
| 308 } | |
| 309 for (int j = 0; j < len; j++) { | |
| 310 jint match_int = lo_index+j; | |
| 311 int dest = iter().get_dest_table(j+3); | |
| 312 makes_backward_branch |= (dest <= bci()); | |
| 313 int table_index = method_data_update() ? j : NullTableIndex; | |
| 314 if (rp < 0 || !ranges[rp].adjoin(match_int, dest, table_index)) { | |
| 315 ranges[++rp].set(match_int, dest, table_index); | |
| 316 } | |
| 317 } | |
| 318 jint highest = lo_index+(len-1); | |
| 319 assert(ranges[rp].hi() == highest, ""); | |
| 320 if (highest != max_jint | |
| 321 && !ranges[rp].adjoinRange(highest+1, max_jint, default_dest, NullTableIndex)) { | |
| 322 ranges[++rp].setRange(highest+1, max_jint, default_dest, NullTableIndex); | |
| 323 } | |
| 324 assert(rp < len+2, "not too many ranges"); | |
| 325 | |
| 326 // Safepoint in case if backward branch observed | |
| 327 if( makes_backward_branch && UseLoopSafepoints ) | |
| 328 add_safepoint(); | |
| 329 | |
| 330 jump_switch_ranges(lookup, &ranges[0], &ranges[rp]); | |
| 331 } | |
| 332 | |
| 333 | |
| 334 //------------------------------do_lookupswitch-------------------------------- | |
| 335 void Parse::do_lookupswitch() { | |
| 336 Node *lookup = pop(); // lookup value | |
| 337 // Get information about lookupswitch | |
| 338 int default_dest = iter().get_dest_table(0); | |
| 339 int len = iter().get_int_table(1); | |
| 340 | |
| 341 if (len < 1) { // If this is a backward branch, add safepoint | |
| 342 maybe_add_safepoint(default_dest); | |
| 343 merge(default_dest); | |
| 344 return; | |
| 345 } | |
| 346 | |
| 347 // generate decision tree, using trichotomy when possible | |
| 348 jint* table = NEW_RESOURCE_ARRAY(jint, len*2); | |
| 349 { | |
| 350 for( int j = 0; j < len; j++ ) { | |
| 351 table[j+j+0] = iter().get_int_table(2+j+j); | |
| 352 table[j+j+1] = iter().get_dest_table(2+j+j+1); | |
| 353 } | |
| 354 qsort( table, len, 2*sizeof(table[0]), jint_cmp ); | |
| 355 } | |
| 356 | |
| 357 int rnum = len*2+1; | |
| 358 bool makes_backward_branch = false; | |
| 359 SwitchRange* ranges = NEW_RESOURCE_ARRAY(SwitchRange, rnum); | |
| 360 int rp = -1; | |
| 361 for( int j = 0; j < len; j++ ) { | |
| 362 jint match_int = table[j+j+0]; | |
| 363 int dest = table[j+j+1]; | |
| 364 int next_lo = rp < 0 ? min_jint : ranges[rp].hi()+1; | |
| 365 int table_index = method_data_update() ? j : NullTableIndex; | |
| 366 makes_backward_branch |= (dest <= bci()); | |
| 367 if( match_int != next_lo ) { | |
| 368 ranges[++rp].setRange(next_lo, match_int-1, default_dest, NullTableIndex); | |
| 369 } | |
| 370 if( rp < 0 || !ranges[rp].adjoin(match_int, dest, table_index) ) { | |
| 371 ranges[++rp].set(match_int, dest, table_index); | |
| 372 } | |
| 373 } | |
| 374 jint highest = table[2*(len-1)]; | |
| 375 assert(ranges[rp].hi() == highest, ""); | |
| 376 if( highest != max_jint | |
| 377 && !ranges[rp].adjoinRange(highest+1, max_jint, default_dest, NullTableIndex) ) { | |
| 378 ranges[++rp].setRange(highest+1, max_jint, default_dest, NullTableIndex); | |
| 379 } | |
| 380 assert(rp < rnum, "not too many ranges"); | |
| 381 | |
| 382 // Safepoint in case backward branch observed | |
| 383 if( makes_backward_branch && UseLoopSafepoints ) | |
| 384 add_safepoint(); | |
| 385 | |
| 386 jump_switch_ranges(lookup, &ranges[0], &ranges[rp]); | |
| 387 } | |
| 388 | |
| 389 //----------------------------create_jump_tables------------------------------- | |
| 390 bool Parse::create_jump_tables(Node* key_val, SwitchRange* lo, SwitchRange* hi) { | |
| 391 // Are jumptables enabled | |
| 392 if (!UseJumpTables) return false; | |
| 393 | |
| 394 // Are jumptables supported | |
| 395 if (!Matcher::has_match_rule(Op_Jump)) return false; | |
| 396 | |
| 397 // Don't make jump table if profiling | |
| 398 if (method_data_update()) return false; | |
| 399 | |
| 400 // Decide if a guard is needed to lop off big ranges at either (or | |
| 401 // both) end(s) of the input set. We'll call this the default target | |
| 402 // even though we can't be sure that it is the true "default". | |
| 403 | |
| 404 bool needs_guard = false; | |
| 405 int default_dest; | |
| 406 int64 total_outlier_size = 0; | |
| 407 int64 hi_size = ((int64)hi->hi()) - ((int64)hi->lo()) + 1; | |
| 408 int64 lo_size = ((int64)lo->hi()) - ((int64)lo->lo()) + 1; | |
| 409 | |
| 410 if (lo->dest() == hi->dest()) { | |
| 411 total_outlier_size = hi_size + lo_size; | |
| 412 default_dest = lo->dest(); | |
| 413 } else if (lo_size > hi_size) { | |
| 414 total_outlier_size = lo_size; | |
| 415 default_dest = lo->dest(); | |
| 416 } else { | |
| 417 total_outlier_size = hi_size; | |
| 418 default_dest = hi->dest(); | |
| 419 } | |
| 420 | |
| 421 // If a guard test will eliminate very sparse end ranges, then | |
| 422 // it is worth the cost of an extra jump. | |
| 423 if (total_outlier_size > (MaxJumpTableSparseness * 4)) { | |
| 424 needs_guard = true; | |
| 425 if (default_dest == lo->dest()) lo++; | |
| 426 if (default_dest == hi->dest()) hi--; | |
| 427 } | |
| 428 | |
| 429 // Find the total number of cases and ranges | |
| 430 int64 num_cases = ((int64)hi->hi()) - ((int64)lo->lo()) + 1; | |
| 431 int num_range = hi - lo + 1; | |
| 432 | |
| 433 // Don't create table if: too large, too small, or too sparse. | |
| 434 if (num_cases < MinJumpTableSize || num_cases > MaxJumpTableSize) | |
| 435 return false; | |
| 436 if (num_cases > (MaxJumpTableSparseness * num_range)) | |
| 437 return false; | |
| 438 | |
| 439 // Normalize table lookups to zero | |
| 440 int lowval = lo->lo(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
441 key_val = _gvn.transform( new (C) SubINode(key_val, _gvn.intcon(lowval)) ); |
| 0 | 442 |
| 443 // Generate a guard to protect against input keyvals that aren't | |
| 444 // in the switch domain. | |
| 445 if (needs_guard) { | |
| 446 Node* size = _gvn.intcon(num_cases); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
447 Node* cmp = _gvn.transform( new (C) CmpUNode(key_val, size) ); |
|
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
448 Node* tst = _gvn.transform( new (C) BoolNode(cmp, BoolTest::ge) ); |
| 0 | 449 IfNode* iff = create_and_map_if( control(), tst, PROB_FAIR, COUNT_UNKNOWN); |
| 450 jump_if_true_fork(iff, default_dest, NullTableIndex); | |
| 451 } | |
| 452 | |
| 453 // Create an ideal node JumpTable that has projections | |
| 454 // of all possible ranges for a switch statement | |
| 455 // The key_val input must be converted to a pointer offset and scaled. | |
| 456 // Compare Parse::array_addressing above. | |
| 457 #ifdef _LP64 | |
| 458 // Clean the 32-bit int into a real 64-bit offset. | |
| 459 // Otherwise, the jint value 0 might turn into an offset of 0x0800000000. | |
| 460 const TypeLong* lkeytype = TypeLong::make(CONST64(0), num_cases-1, Type::WidenMin); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
461 key_val = _gvn.transform( new (C) ConvI2LNode(key_val, lkeytype) ); |
| 0 | 462 #endif |
| 463 // Shift the value by wordsize so we have an index into the table, rather | |
| 464 // than a switch value | |
| 465 Node *shiftWord = _gvn.MakeConX(wordSize); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
466 key_val = _gvn.transform( new (C) MulXNode( key_val, shiftWord)); |
| 0 | 467 |
| 468 // Create the JumpNode | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
469 Node* jtn = _gvn.transform( new (C) JumpNode(control(), key_val, num_cases) ); |
| 0 | 470 |
| 471 // These are the switch destinations hanging off the jumpnode | |
| 472 int i = 0; | |
| 473 for (SwitchRange* r = lo; r <= hi; r++) { | |
|
12866
f478c98e8114
8024774: assert(_con < t->is_tuple()->cnt()) failed: ProjNode::_con must be in range
vlivanov
parents:
10278
diff
changeset
|
474 for (int64 j = r->lo(); j <= r->hi(); j++, i++) { |
|
f478c98e8114
8024774: assert(_con < t->is_tuple()->cnt()) failed: ProjNode::_con must be in range
vlivanov
parents:
10278
diff
changeset
|
475 Node* input = _gvn.transform(new (C) JumpProjNode(jtn, i, r->dest(), (int)(j - lowval))); |
| 0 | 476 { |
| 477 PreserveJVMState pjvms(this); | |
| 478 set_control(input); | |
| 479 jump_if_always_fork(r->dest(), r->table_index()); | |
| 480 } | |
| 481 } | |
| 482 } | |
| 483 assert(i == num_cases, "miscount of cases"); | |
| 484 stop_and_kill_map(); // no more uses for this JVMS | |
| 485 return true; | |
| 486 } | |
| 487 | |
| 488 //----------------------------jump_switch_ranges------------------------------- | |
| 489 void Parse::jump_switch_ranges(Node* key_val, SwitchRange *lo, SwitchRange *hi, int switch_depth) { | |
| 490 Block* switch_block = block(); | |
| 491 | |
| 492 if (switch_depth == 0) { | |
| 493 // Do special processing for the top-level call. | |
| 494 assert(lo->lo() == min_jint, "initial range must exhaust Type::INT"); | |
| 495 assert(hi->hi() == max_jint, "initial range must exhaust Type::INT"); | |
| 496 | |
| 497 // Decrement pred-numbers for the unique set of nodes. | |
| 498 #ifdef ASSERT | |
| 499 // Ensure that the block's successors are a (duplicate-free) set. | |
| 500 int successors_counted = 0; // block occurrences in [hi..lo] | |
| 501 int unique_successors = switch_block->num_successors(); | |
| 502 for (int i = 0; i < unique_successors; i++) { | |
| 503 Block* target = switch_block->successor_at(i); | |
| 504 | |
| 505 // Check that the set of successors is the same in both places. | |
| 506 int successors_found = 0; | |
| 507 for (SwitchRange* p = lo; p <= hi; p++) { | |
| 508 if (p->dest() == target->start()) successors_found++; | |
| 509 } | |
| 510 assert(successors_found > 0, "successor must be known"); | |
| 511 successors_counted += successors_found; | |
| 512 } | |
| 513 assert(successors_counted == (hi-lo)+1, "no unexpected successors"); | |
| 514 #endif | |
| 515 | |
| 516 // Maybe prune the inputs, based on the type of key_val. | |
| 517 jint min_val = min_jint; | |
| 518 jint max_val = max_jint; | |
| 519 const TypeInt* ti = key_val->bottom_type()->isa_int(); | |
| 520 if (ti != NULL) { | |
| 521 min_val = ti->_lo; | |
| 522 max_val = ti->_hi; | |
| 523 assert(min_val <= max_val, "invalid int type"); | |
| 524 } | |
| 525 while (lo->hi() < min_val) lo++; | |
| 526 if (lo->lo() < min_val) lo->setRange(min_val, lo->hi(), lo->dest(), lo->table_index()); | |
| 527 while (hi->lo() > max_val) hi--; | |
| 528 if (hi->hi() > max_val) hi->setRange(hi->lo(), max_val, hi->dest(), hi->table_index()); | |
| 529 } | |
| 530 | |
| 531 #ifndef PRODUCT | |
| 532 if (switch_depth == 0) { | |
| 533 _max_switch_depth = 0; | |
| 534 _est_switch_depth = log2_intptr((hi-lo+1)-1)+1; | |
| 535 } | |
| 536 #endif | |
| 537 | |
| 538 assert(lo <= hi, "must be a non-empty set of ranges"); | |
| 539 if (lo == hi) { | |
| 540 jump_if_always_fork(lo->dest(), lo->table_index()); | |
| 541 } else { | |
| 542 assert(lo->hi() == (lo+1)->lo()-1, "contiguous ranges"); | |
| 543 assert(hi->lo() == (hi-1)->hi()+1, "contiguous ranges"); | |
| 544 | |
| 545 if (create_jump_tables(key_val, lo, hi)) return; | |
| 546 | |
| 547 int nr = hi - lo + 1; | |
| 548 | |
| 549 SwitchRange* mid = lo + nr/2; | |
| 550 // if there is an easy choice, pivot at a singleton: | |
| 551 if (nr > 3 && !mid->is_singleton() && (mid-1)->is_singleton()) mid--; | |
| 552 | |
| 553 assert(lo < mid && mid <= hi, "good pivot choice"); | |
| 554 assert(nr != 2 || mid == hi, "should pick higher of 2"); | |
| 555 assert(nr != 3 || mid == hi-1, "should pick middle of 3"); | |
| 556 | |
| 557 Node *test_val = _gvn.intcon(mid->lo()); | |
| 558 | |
| 559 if (mid->is_singleton()) { | |
| 560 IfNode *iff_ne = jump_if_fork_int(key_val, test_val, BoolTest::ne); | |
| 561 jump_if_false_fork(iff_ne, mid->dest(), mid->table_index()); | |
| 562 | |
| 563 // Special Case: If there are exactly three ranges, and the high | |
| 564 // and low range each go to the same place, omit the "gt" test, | |
| 565 // since it will not discriminate anything. | |
| 566 bool eq_test_only = (hi == lo+2 && hi->dest() == lo->dest()); | |
| 567 if (eq_test_only) { | |
| 568 assert(mid == hi-1, ""); | |
| 569 } | |
| 570 | |
| 571 // if there is a higher range, test for it and process it: | |
| 572 if (mid < hi && !eq_test_only) { | |
| 573 // two comparisons of same values--should enable 1 test for 2 branches | |
| 574 // Use BoolTest::le instead of BoolTest::gt | |
| 575 IfNode *iff_le = jump_if_fork_int(key_val, test_val, BoolTest::le); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
576 Node *iftrue = _gvn.transform( new (C) IfTrueNode(iff_le) ); |
|
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
577 Node *iffalse = _gvn.transform( new (C) IfFalseNode(iff_le) ); |
| 0 | 578 { PreserveJVMState pjvms(this); |
| 579 set_control(iffalse); | |
| 580 jump_switch_ranges(key_val, mid+1, hi, switch_depth+1); | |
| 581 } | |
| 582 set_control(iftrue); | |
| 583 } | |
| 584 | |
| 585 } else { | |
| 586 // mid is a range, not a singleton, so treat mid..hi as a unit | |
| 587 IfNode *iff_ge = jump_if_fork_int(key_val, test_val, BoolTest::ge); | |
| 588 | |
| 589 // if there is a higher range, test for it and process it: | |
| 590 if (mid == hi) { | |
| 591 jump_if_true_fork(iff_ge, mid->dest(), mid->table_index()); | |
| 592 } else { | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
593 Node *iftrue = _gvn.transform( new (C) IfTrueNode(iff_ge) ); |
|
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
594 Node *iffalse = _gvn.transform( new (C) IfFalseNode(iff_ge) ); |
| 0 | 595 { PreserveJVMState pjvms(this); |
| 596 set_control(iftrue); | |
| 597 jump_switch_ranges(key_val, mid, hi, switch_depth+1); | |
| 598 } | |
| 599 set_control(iffalse); | |
| 600 } | |
| 601 } | |
| 602 | |
| 603 // in any case, process the lower range | |
| 604 jump_switch_ranges(key_val, lo, mid-1, switch_depth+1); | |
| 605 } | |
| 606 | |
| 607 // Decrease pred_count for each successor after all is done. | |
| 608 if (switch_depth == 0) { | |
| 609 int unique_successors = switch_block->num_successors(); | |
| 610 for (int i = 0; i < unique_successors; i++) { | |
| 611 Block* target = switch_block->successor_at(i); | |
| 612 // Throw away the pre-allocated path for each unique successor. | |
| 613 target->next_path_num(); | |
| 614 } | |
| 615 } | |
| 616 | |
| 617 #ifndef PRODUCT | |
| 618 _max_switch_depth = MAX2(switch_depth, _max_switch_depth); | |
| 619 if (TraceOptoParse && Verbose && WizardMode && switch_depth == 0) { | |
| 620 SwitchRange* r; | |
| 621 int nsing = 0; | |
| 622 for( r = lo; r <= hi; r++ ) { | |
| 623 if( r->is_singleton() ) nsing++; | |
| 624 } | |
| 625 tty->print(">>> "); | |
| 626 _method->print_short_name(); | |
| 627 tty->print_cr(" switch decision tree"); | |
| 628 tty->print_cr(" %d ranges (%d singletons), max_depth=%d, est_depth=%d", | |
|
17937
78bbf4d43a14
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
17799
diff
changeset
|
629 (int) (hi-lo+1), nsing, _max_switch_depth, _est_switch_depth); |
| 0 | 630 if (_max_switch_depth > _est_switch_depth) { |
| 631 tty->print_cr("******** BAD SWITCH DEPTH ********"); | |
| 632 } | |
| 633 tty->print(" "); | |
| 634 for( r = lo; r <= hi; r++ ) { | |
|
12866
f478c98e8114
8024774: assert(_con < t->is_tuple()->cnt()) failed: ProjNode::_con must be in range
vlivanov
parents:
10278
diff
changeset
|
635 r->print(); |
| 0 | 636 } |
|
17937
78bbf4d43a14
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
17799
diff
changeset
|
637 tty->cr(); |
| 0 | 638 } |
| 639 #endif | |
| 640 } | |
| 641 | |
| 642 void Parse::modf() { | |
| 643 Node *f2 = pop(); | |
| 644 Node *f1 = pop(); | |
| 645 Node* c = make_runtime_call(RC_LEAF, OptoRuntime::modf_Type(), | |
| 646 CAST_FROM_FN_PTR(address, SharedRuntime::frem), | |
| 647 "frem", NULL, //no memory effects | |
| 648 f1, f2); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
649 Node* res = _gvn.transform(new (C) ProjNode(c, TypeFunc::Parms + 0)); |
| 0 | 650 |
| 651 push(res); | |
| 652 } | |
| 653 | |
| 654 void Parse::modd() { | |
| 655 Node *d2 = pop_pair(); | |
| 656 Node *d1 = pop_pair(); | |
| 657 Node* c = make_runtime_call(RC_LEAF, OptoRuntime::Math_DD_D_Type(), | |
| 658 CAST_FROM_FN_PTR(address, SharedRuntime::drem), | |
| 659 "drem", NULL, //no memory effects | |
| 660 d1, top(), d2, top()); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
661 Node* res_d = _gvn.transform(new (C) ProjNode(c, TypeFunc::Parms + 0)); |
| 0 | 662 |
| 663 #ifdef ASSERT | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
664 Node* res_top = _gvn.transform(new (C) ProjNode(c, TypeFunc::Parms + 1)); |
| 0 | 665 assert(res_top == top(), "second value must be top"); |
| 666 #endif | |
| 667 | |
| 668 push_pair(res_d); | |
| 669 } | |
| 670 | |
| 671 void Parse::l2f() { | |
| 672 Node* f2 = pop(); | |
| 673 Node* f1 = pop(); | |
| 674 Node* c = make_runtime_call(RC_LEAF, OptoRuntime::l2f_Type(), | |
| 675 CAST_FROM_FN_PTR(address, SharedRuntime::l2f), | |
| 676 "l2f", NULL, //no memory effects | |
| 677 f1, f2); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
678 Node* res = _gvn.transform(new (C) ProjNode(c, TypeFunc::Parms + 0)); |
| 0 | 679 |
| 680 push(res); | |
| 681 } | |
| 682 | |
| 683 void Parse::do_irem() { | |
| 684 // Must keep both values on the expression-stack during null-check | |
|
7194
beebba0acc11
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
6848
diff
changeset
|
685 zero_check_int(peek()); |
| 0 | 686 // Compile-time detect of null-exception? |
| 687 if (stopped()) return; | |
| 688 | |
| 689 Node* b = pop(); | |
| 690 Node* a = pop(); | |
| 691 | |
| 692 const Type *t = _gvn.type(b); | |
| 693 if (t != Type::TOP) { | |
| 694 const TypeInt *ti = t->is_int(); | |
| 695 if (ti->is_con()) { | |
| 696 int divisor = ti->get_con(); | |
| 697 // check for positive power of 2 | |
| 698 if (divisor > 0 && | |
| 699 (divisor & ~(divisor-1)) == divisor) { | |
| 700 // yes ! | |
| 701 Node *mask = _gvn.intcon((divisor - 1)); | |
| 702 // Sigh, must handle negative dividends | |
| 703 Node *zero = _gvn.intcon(0); | |
| 704 IfNode *ifff = jump_if_fork_int(a, zero, BoolTest::lt); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
705 Node *iff = _gvn.transform( new (C) IfFalseNode(ifff) ); |
|
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
706 Node *ift = _gvn.transform( new (C) IfTrueNode (ifff) ); |
| 0 | 707 Node *reg = jump_if_join(ift, iff); |
| 708 Node *phi = PhiNode::make(reg, NULL, TypeInt::INT); | |
| 709 // Negative path; negate/and/negate | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
710 Node *neg = _gvn.transform( new (C) SubINode(zero, a) ); |
|
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
711 Node *andn= _gvn.transform( new (C) AndINode(neg, mask) ); |
|
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
712 Node *negn= _gvn.transform( new (C) SubINode(zero, andn) ); |
| 0 | 713 phi->init_req(1, negn); |
| 714 // Fast positive case | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
715 Node *andx = _gvn.transform( new (C) AndINode(a, mask) ); |
| 0 | 716 phi->init_req(2, andx); |
| 717 // Push the merge | |
| 718 push( _gvn.transform(phi) ); | |
| 719 return; | |
| 720 } | |
| 721 } | |
| 722 } | |
| 723 // Default case | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
724 push( _gvn.transform( new (C) ModINode(control(),a,b) ) ); |
| 0 | 725 } |
| 726 | |
| 727 // Handle jsr and jsr_w bytecode | |
| 728 void Parse::do_jsr() { | |
| 729 assert(bc() == Bytecodes::_jsr || bc() == Bytecodes::_jsr_w, "wrong bytecode"); | |
| 730 | |
| 731 // Store information about current state, tagged with new _jsr_bci | |
| 732 int return_bci = iter().next_bci(); | |
| 733 int jsr_bci = (bc() == Bytecodes::_jsr) ? iter().get_dest() : iter().get_far_dest(); | |
| 734 | |
| 735 // Update method data | |
| 736 profile_taken_branch(jsr_bci); | |
| 737 | |
| 738 // The way we do things now, there is only one successor block | |
| 739 // for the jsr, because the target code is cloned by ciTypeFlow. | |
| 740 Block* target = successor_for_bci(jsr_bci); | |
| 741 | |
| 742 // What got pushed? | |
| 743 const Type* ret_addr = target->peek(); | |
| 744 assert(ret_addr->singleton(), "must be a constant (cloned jsr body)"); | |
| 745 | |
| 746 // Effect on jsr on stack | |
| 747 push(_gvn.makecon(ret_addr)); | |
| 748 | |
| 749 // Flow to the jsr. | |
| 750 merge(jsr_bci); | |
| 751 } | |
| 752 | |
| 753 // Handle ret bytecode | |
| 754 void Parse::do_ret() { | |
| 755 // Find to whom we return. | |
| 756 assert(block()->num_successors() == 1, "a ret can only go one place now"); | |
| 757 Block* target = block()->successor_at(0); | |
| 758 assert(!target->is_ready(), "our arrival must be expected"); | |
| 759 profile_ret(target->flow()->start()); | |
| 760 int pnum = target->next_path_num(); | |
| 761 merge_common(target, pnum); | |
| 762 } | |
| 763 | |
| 764 //--------------------------dynamic_branch_prediction-------------------------- | |
| 765 // Try to gather dynamic branch prediction behavior. Return a probability | |
| 766 // of the branch being taken and set the "cnt" field. Returns a -1.0 | |
| 767 // if we need to use static prediction for some reason. | |
| 768 float Parse::dynamic_branch_prediction(float &cnt) { | |
| 769 ResourceMark rm; | |
| 770 | |
| 771 cnt = COUNT_UNKNOWN; | |
| 772 | |
| 773 // Use MethodData information if it is available | |
| 774 // FIXME: free the ProfileData structure | |
| 775 ciMethodData* methodData = method()->method_data(); | |
| 776 if (!methodData->is_mature()) return PROB_UNKNOWN; | |
| 777 ciProfileData* data = methodData->bci_to_data(bci()); | |
| 778 if (!data->is_JumpData()) return PROB_UNKNOWN; | |
| 779 | |
| 780 // get taken and not taken values | |
| 781 int taken = data->as_JumpData()->taken(); | |
| 782 int not_taken = 0; | |
| 783 if (data->is_BranchData()) { | |
| 784 not_taken = data->as_BranchData()->not_taken(); | |
| 785 } | |
| 786 | |
| 787 // scale the counts to be commensurate with invocation counts: | |
| 788 taken = method()->scale_count(taken); | |
| 789 not_taken = method()->scale_count(not_taken); | |
| 790 | |
|
2481
3a808be061ff
6988308: assert((cnt > 0.0f) && (prob > 0.0f)) failed: Bad frequency assignment in if
iveresov
parents:
2426
diff
changeset
|
791 // Give up if too few (or too many, in which case the sum will overflow) counts to be meaningful. |
|
3a808be061ff
6988308: assert((cnt > 0.0f) && (prob > 0.0f)) failed: Bad frequency assignment in if
iveresov
parents:
2426
diff
changeset
|
792 // We also check that individual counters are positive first, overwise the sum can become positive. |
|
3a808be061ff
6988308: assert((cnt > 0.0f) && (prob > 0.0f)) failed: Bad frequency assignment in if
iveresov
parents:
2426
diff
changeset
|
793 if (taken < 0 || not_taken < 0 || taken + not_taken < 40) { |
| 0 | 794 if (C->log() != NULL) { |
| 795 C->log()->elem("branch target_bci='%d' taken='%d' not_taken='%d'", iter().get_dest(), taken, not_taken); | |
| 796 } | |
| 797 return PROB_UNKNOWN; | |
| 798 } | |
| 799 | |
| 800 // Compute frequency that we arrive here | |
|
2481
3a808be061ff
6988308: assert((cnt > 0.0f) && (prob > 0.0f)) failed: Bad frequency assignment in if
iveresov
parents:
2426
diff
changeset
|
801 float sum = taken + not_taken; |
| 0 | 802 // Adjust, if this block is a cloned private block but the |
| 803 // Jump counts are shared. Taken the private counts for | |
| 804 // just this path instead of the shared counts. | |
| 805 if( block()->count() > 0 ) | |
| 806 sum = block()->count(); | |
|
2481
3a808be061ff
6988308: assert((cnt > 0.0f) && (prob > 0.0f)) failed: Bad frequency assignment in if
iveresov
parents:
2426
diff
changeset
|
807 cnt = sum / FreqCountInvocations; |
| 0 | 808 |
| 809 // Pin probability to sane limits | |
| 810 float prob; | |
| 811 if( !taken ) | |
| 812 prob = (0+PROB_MIN) / 2; | |
| 813 else if( !not_taken ) | |
| 814 prob = (1+PROB_MAX) / 2; | |
| 815 else { // Compute probability of true path | |
| 816 prob = (float)taken / (float)(taken + not_taken); | |
| 817 if (prob > PROB_MAX) prob = PROB_MAX; | |
| 818 if (prob < PROB_MIN) prob = PROB_MIN; | |
| 819 } | |
| 820 | |
| 821 assert((cnt > 0.0f) && (prob > 0.0f), | |
| 822 "Bad frequency assignment in if"); | |
| 823 | |
| 824 if (C->log() != NULL) { | |
| 825 const char* prob_str = NULL; | |
| 826 if (prob >= PROB_MAX) prob_str = (prob == PROB_MAX) ? "max" : "always"; | |
| 827 if (prob <= PROB_MIN) prob_str = (prob == PROB_MIN) ? "min" : "never"; | |
| 828 char prob_str_buf[30]; | |
| 829 if (prob_str == NULL) { | |
| 830 sprintf(prob_str_buf, "%g", prob); | |
| 831 prob_str = prob_str_buf; | |
| 832 } | |
| 833 C->log()->elem("branch target_bci='%d' taken='%d' not_taken='%d' cnt='%g' prob='%s'", | |
| 834 iter().get_dest(), taken, not_taken, cnt, prob_str); | |
| 835 } | |
| 836 return prob; | |
| 837 } | |
| 838 | |
| 839 //-----------------------------branch_prediction------------------------------- | |
| 840 float Parse::branch_prediction(float& cnt, | |
| 841 BoolTest::mask btest, | |
| 842 int target_bci) { | |
| 843 float prob = dynamic_branch_prediction(cnt); | |
| 844 // If prob is unknown, switch to static prediction | |
| 845 if (prob != PROB_UNKNOWN) return prob; | |
| 846 | |
| 847 prob = PROB_FAIR; // Set default value | |
| 848 if (btest == BoolTest::eq) // Exactly equal test? | |
| 849 prob = PROB_STATIC_INFREQUENT; // Assume its relatively infrequent | |
| 850 else if (btest == BoolTest::ne) | |
| 851 prob = PROB_STATIC_FREQUENT; // Assume its relatively frequent | |
| 852 | |
| 853 // If this is a conditional test guarding a backwards branch, | |
| 854 // assume its a loop-back edge. Make it a likely taken branch. | |
| 855 if (target_bci < bci()) { | |
| 856 if (is_osr_parse()) { // Could be a hot OSR'd loop; force deopt | |
| 857 // Since it's an OSR, we probably have profile data, but since | |
| 858 // branch_prediction returned PROB_UNKNOWN, the counts are too small. | |
| 859 // Let's make a special check here for completely zero counts. | |
| 860 ciMethodData* methodData = method()->method_data(); | |
| 861 if (!methodData->is_empty()) { | |
| 862 ciProfileData* data = methodData->bci_to_data(bci()); | |
| 863 // Only stop for truly zero counts, which mean an unknown part | |
| 864 // of the OSR-ed method, and we want to deopt to gather more stats. | |
| 865 // If you have ANY counts, then this loop is simply 'cold' relative | |
| 866 // to the OSR loop. | |
| 867 if (data->as_BranchData()->taken() + | |
| 868 data->as_BranchData()->not_taken() == 0 ) { | |
| 869 // This is the only way to return PROB_UNKNOWN: | |
| 870 return PROB_UNKNOWN; | |
| 871 } | |
| 872 } | |
| 873 } | |
| 874 prob = PROB_STATIC_FREQUENT; // Likely to take backwards branch | |
| 875 } | |
| 876 | |
| 877 assert(prob != PROB_UNKNOWN, "must have some guess at this point"); | |
| 878 return prob; | |
| 879 } | |
| 880 | |
| 881 // The magic constants are chosen so as to match the output of | |
| 882 // branch_prediction() when the profile reports a zero taken count. | |
| 883 // It is important to distinguish zero counts unambiguously, because | |
| 884 // some branches (e.g., _213_javac.Assembler.eliminate) validly produce | |
| 885 // very small but nonzero probabilities, which if confused with zero | |
| 886 // counts would keep the program recompiling indefinitely. | |
| 887 bool Parse::seems_never_taken(float prob) { | |
| 888 return prob < PROB_MIN; | |
| 889 } | |
| 890 | |
|
1746
4b29a725c43c
6912064: type profiles need to be exploited more for dynamic language support
jrose
parents:
1602
diff
changeset
|
891 // True if the comparison seems to be the kind that will not change its |
|
4b29a725c43c
6912064: type profiles need to be exploited more for dynamic language support
jrose
parents:
1602
diff
changeset
|
892 // statistics from true to false. See comments in adjust_map_after_if. |
|
4b29a725c43c
6912064: type profiles need to be exploited more for dynamic language support
jrose
parents:
1602
diff
changeset
|
893 // This question is only asked along paths which are already |
|
4b29a725c43c
6912064: type profiles need to be exploited more for dynamic language support
jrose
parents:
1602
diff
changeset
|
894 // classifed as untaken (by seems_never_taken), so really, |
|
4b29a725c43c
6912064: type profiles need to be exploited more for dynamic language support
jrose
parents:
1602
diff
changeset
|
895 // if a path is never taken, its controlling comparison is |
|
4b29a725c43c
6912064: type profiles need to be exploited more for dynamic language support
jrose
parents:
1602
diff
changeset
|
896 // already acting in a stable fashion. If the comparison |
|
4b29a725c43c
6912064: type profiles need to be exploited more for dynamic language support
jrose
parents:
1602
diff
changeset
|
897 // seems stable, we will put an expensive uncommon trap |
|
20439
f6f9aec27858
8030976: Untaken paths should be more vigorously pruned at highest optimization level
rbackman
parents:
17937
diff
changeset
|
898 // on the untaken path. |
|
1746
4b29a725c43c
6912064: type profiles need to be exploited more for dynamic language support
jrose
parents:
1602
diff
changeset
|
899 bool Parse::seems_stable_comparison(BoolTest::mask btest, Node* cmp) { |
|
20439
f6f9aec27858
8030976: Untaken paths should be more vigorously pruned at highest optimization level
rbackman
parents:
17937
diff
changeset
|
900 if (C->too_many_traps(method(), bci(), Deoptimization::Reason_unstable_if)) { |
|
f6f9aec27858
8030976: Untaken paths should be more vigorously pruned at highest optimization level
rbackman
parents:
17937
diff
changeset
|
901 return false; |
|
1746
4b29a725c43c
6912064: type profiles need to be exploited more for dynamic language support
jrose
parents:
1602
diff
changeset
|
902 } |
|
20439
f6f9aec27858
8030976: Untaken paths should be more vigorously pruned at highest optimization level
rbackman
parents:
17937
diff
changeset
|
903 return true; |
|
1746
4b29a725c43c
6912064: type profiles need to be exploited more for dynamic language support
jrose
parents:
1602
diff
changeset
|
904 } |
|
4b29a725c43c
6912064: type profiles need to be exploited more for dynamic language support
jrose
parents:
1602
diff
changeset
|
905 |
|
246
9b66e6287f4a
6707044: uncommon_trap of ifnull bytecode leaves garbage on expression stack
rasbold
parents:
235
diff
changeset
|
906 //-------------------------------repush_if_args-------------------------------- |
|
9b66e6287f4a
6707044: uncommon_trap of ifnull bytecode leaves garbage on expression stack
rasbold
parents:
235
diff
changeset
|
907 // Push arguments of an "if" bytecode back onto the stack by adjusting _sp. |
| 1172 | 908 inline int Parse::repush_if_args() { |
| 0 | 909 #ifndef PRODUCT |
| 910 if (PrintOpto && WizardMode) { | |
| 911 tty->print("defending against excessive implicit null exceptions on %s @%d in ", | |
| 912 Bytecodes::name(iter().cur_bc()), iter().cur_bci()); | |
| 913 method()->print_name(); tty->cr(); | |
| 914 } | |
| 915 #endif | |
| 916 int bc_depth = - Bytecodes::depth(iter().cur_bc()); | |
| 917 assert(bc_depth == 1 || bc_depth == 2, "only two kinds of branches"); | |
| 918 DEBUG_ONLY(sync_jvms()); // argument(n) requires a synced jvms | |
| 919 assert(argument(0) != NULL, "must exist"); | |
| 920 assert(bc_depth == 1 || argument(1) != NULL, "two must exist"); | |
|
7194
beebba0acc11
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
6848
diff
changeset
|
921 inc_sp(bc_depth); |
| 1172 | 922 return bc_depth; |
| 0 | 923 } |
| 924 | |
| 925 //----------------------------------do_ifnull---------------------------------- | |
|
248
18aab3cdd513
6726504: handle do_ifxxx calls in parser more uniformly
rasbold
parents:
246
diff
changeset
|
926 void Parse::do_ifnull(BoolTest::mask btest, Node *c) { |
| 0 | 927 int target_bci = iter().get_dest(); |
| 928 | |
|
17
ff5961f4c095
6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents:
0
diff
changeset
|
929 Block* branch_block = successor_for_bci(target_bci); |
|
ff5961f4c095
6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents:
0
diff
changeset
|
930 Block* next_block = successor_for_bci(iter().next_bci()); |
|
ff5961f4c095
6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents:
0
diff
changeset
|
931 |
| 0 | 932 float cnt; |
| 933 float prob = branch_prediction(cnt, btest, target_bci); | |
| 934 if (prob == PROB_UNKNOWN) { | |
| 935 // (An earlier version of do_ifnull omitted this trap for OSR methods.) | |
| 936 #ifndef PRODUCT | |
| 937 if (PrintOpto && Verbose) | |
|
248
18aab3cdd513
6726504: handle do_ifxxx calls in parser more uniformly
rasbold
parents:
246
diff
changeset
|
938 tty->print_cr("Never-taken edge stops compilation at bci %d",bci()); |
| 0 | 939 #endif |
|
248
18aab3cdd513
6726504: handle do_ifxxx calls in parser more uniformly
rasbold
parents:
246
diff
changeset
|
940 repush_if_args(); // to gather stats on loop |
| 0 | 941 // We need to mark this branch as taken so that if we recompile we will |
| 942 // see that it is possible. In the tiered system the interpreter doesn't | |
| 943 // do profiling and by the time we get to the lower tier from the interpreter | |
| 944 // the path may be cold again. Make sure it doesn't look untaken | |
| 945 profile_taken_branch(target_bci, !ProfileInterpreter); | |
| 946 uncommon_trap(Deoptimization::Reason_unreached, | |
| 947 Deoptimization::Action_reinterpret, | |
| 948 NULL, "cold"); | |
| 10278 | 949 if (C->eliminate_boxing()) { |
|
17
ff5961f4c095
6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents:
0
diff
changeset
|
950 // Mark the successor blocks as parsed |
|
ff5961f4c095
6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents:
0
diff
changeset
|
951 branch_block->next_path_num(); |
|
ff5961f4c095
6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents:
0
diff
changeset
|
952 next_block->next_path_num(); |
|
ff5961f4c095
6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents:
0
diff
changeset
|
953 } |
| 0 | 954 return; |
| 955 } | |
| 956 | |
| 957 explicit_null_checks_inserted++; | |
| 958 | |
| 959 // Generate real control flow | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
960 Node *tst = _gvn.transform( new (C) BoolNode( c, btest ) ); |
| 0 | 961 |
| 962 // Sanity check the probability value | |
| 963 assert(prob > 0.0f,"Bad probability in Parser"); | |
| 964 // Need xform to put node in hash table | |
| 965 IfNode *iff = create_and_xform_if( control(), tst, prob, cnt ); | |
| 966 assert(iff->_prob > 0.0f,"Optimizer made bad probability in parser"); | |
| 967 // True branch | |
| 968 { PreserveJVMState pjvms(this); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
969 Node* iftrue = _gvn.transform( new (C) IfTrueNode (iff) ); |
| 0 | 970 set_control(iftrue); |
| 971 | |
| 972 if (stopped()) { // Path is dead? | |
| 973 explicit_null_checks_elided++; | |
| 10278 | 974 if (C->eliminate_boxing()) { |
|
17
ff5961f4c095
6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents:
0
diff
changeset
|
975 // Mark the successor block as parsed |
|
ff5961f4c095
6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents:
0
diff
changeset
|
976 branch_block->next_path_num(); |
|
ff5961f4c095
6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents:
0
diff
changeset
|
977 } |
| 0 | 978 } else { // Path is live. |
| 979 // Update method data | |
| 980 profile_taken_branch(target_bci); | |
| 981 adjust_map_after_if(btest, c, prob, branch_block, next_block); | |
| 1172 | 982 if (!stopped()) { |
| 0 | 983 merge(target_bci); |
| 1172 | 984 } |
| 0 | 985 } |
| 986 } | |
| 987 | |
| 988 // False branch | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
989 Node* iffalse = _gvn.transform( new (C) IfFalseNode(iff) ); |
| 0 | 990 set_control(iffalse); |
| 991 | |
| 992 if (stopped()) { // Path is dead? | |
| 993 explicit_null_checks_elided++; | |
| 10278 | 994 if (C->eliminate_boxing()) { |
|
17
ff5961f4c095
6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents:
0
diff
changeset
|
995 // Mark the successor block as parsed |
|
ff5961f4c095
6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents:
0
diff
changeset
|
996 next_block->next_path_num(); |
|
ff5961f4c095
6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents:
0
diff
changeset
|
997 } |
| 0 | 998 } else { // Path is live. |
| 999 // Update method data | |
| 1000 profile_not_taken_branch(); | |
| 1001 adjust_map_after_if(BoolTest(btest).negate(), c, 1.0-prob, | |
| 1002 next_block, branch_block); | |
| 1003 } | |
| 1004 } | |
| 1005 | |
| 1006 //------------------------------------do_if------------------------------------ | |
| 1007 void Parse::do_if(BoolTest::mask btest, Node* c) { | |
| 1008 int target_bci = iter().get_dest(); | |
| 1009 | |
|
17
ff5961f4c095
6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents:
0
diff
changeset
|
1010 Block* branch_block = successor_for_bci(target_bci); |
|
ff5961f4c095
6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents:
0
diff
changeset
|
1011 Block* next_block = successor_for_bci(iter().next_bci()); |
|
ff5961f4c095
6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents:
0
diff
changeset
|
1012 |
| 0 | 1013 float cnt; |
| 1014 float prob = branch_prediction(cnt, btest, target_bci); | |
| 1015 float untaken_prob = 1.0 - prob; | |
| 1016 | |
| 1017 if (prob == PROB_UNKNOWN) { | |
| 1018 #ifndef PRODUCT | |
| 1019 if (PrintOpto && Verbose) | |
|
248
18aab3cdd513
6726504: handle do_ifxxx calls in parser more uniformly
rasbold
parents:
246
diff
changeset
|
1020 tty->print_cr("Never-taken edge stops compilation at bci %d",bci()); |
| 0 | 1021 #endif |
| 1022 repush_if_args(); // to gather stats on loop | |
| 1023 // We need to mark this branch as taken so that if we recompile we will | |
| 1024 // see that it is possible. In the tiered system the interpreter doesn't | |
| 1025 // do profiling and by the time we get to the lower tier from the interpreter | |
| 1026 // the path may be cold again. Make sure it doesn't look untaken | |
| 1027 profile_taken_branch(target_bci, !ProfileInterpreter); | |
| 1028 uncommon_trap(Deoptimization::Reason_unreached, | |
| 1029 Deoptimization::Action_reinterpret, | |
| 1030 NULL, "cold"); | |
| 10278 | 1031 if (C->eliminate_boxing()) { |
|
17
ff5961f4c095
6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents:
0
diff
changeset
|
1032 // Mark the successor blocks as parsed |
|
ff5961f4c095
6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents:
0
diff
changeset
|
1033 branch_block->next_path_num(); |
|
ff5961f4c095
6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents:
0
diff
changeset
|
1034 next_block->next_path_num(); |
|
ff5961f4c095
6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents:
0
diff
changeset
|
1035 } |
| 0 | 1036 return; |
| 1037 } | |
| 1038 | |
| 1039 // Sanity check the probability value | |
| 1040 assert(0.0f < prob && prob < 1.0f,"Bad probability in Parser"); | |
| 1041 | |
| 1042 bool taken_if_true = true; | |
| 1043 // Convert BoolTest to canonical form: | |
| 1044 if (!BoolTest(btest).is_canonical()) { | |
| 1045 btest = BoolTest(btest).negate(); | |
| 1046 taken_if_true = false; | |
| 1047 // prob is NOT updated here; it remains the probability of the taken | |
| 1048 // path (as opposed to the prob of the path guarded by an 'IfTrueNode'). | |
| 1049 } | |
| 1050 assert(btest != BoolTest::eq, "!= is the only canonical exact test"); | |
| 1051 | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1052 Node* tst0 = new (C) BoolNode(c, btest); |
| 0 | 1053 Node* tst = _gvn.transform(tst0); |
| 1054 BoolTest::mask taken_btest = BoolTest::illegal; | |
| 1055 BoolTest::mask untaken_btest = BoolTest::illegal; | |
|
37
73970d8c0b27
6671250: In Parse::do_if() old Cmp node 'c' should be replaced with new one after BoolNode transformation
kvn
parents:
17
diff
changeset
|
1056 |
|
73970d8c0b27
6671250: In Parse::do_if() old Cmp node 'c' should be replaced with new one after BoolNode transformation
kvn
parents:
17
diff
changeset
|
1057 if (tst->is_Bool()) { |
|
73970d8c0b27
6671250: In Parse::do_if() old Cmp node 'c' should be replaced with new one after BoolNode transformation
kvn
parents:
17
diff
changeset
|
1058 // Refresh c from the transformed bool node, since it may be |
|
73970d8c0b27
6671250: In Parse::do_if() old Cmp node 'c' should be replaced with new one after BoolNode transformation
kvn
parents:
17
diff
changeset
|
1059 // simpler than the original c. Also re-canonicalize btest. |
|
73970d8c0b27
6671250: In Parse::do_if() old Cmp node 'c' should be replaced with new one after BoolNode transformation
kvn
parents:
17
diff
changeset
|
1060 // This wins when (Bool ne (Conv2B p) 0) => (Bool ne (CmpP p NULL)). |
|
73970d8c0b27
6671250: In Parse::do_if() old Cmp node 'c' should be replaced with new one after BoolNode transformation
kvn
parents:
17
diff
changeset
|
1061 // That can arise from statements like: if (x instanceof C) ... |
|
73970d8c0b27
6671250: In Parse::do_if() old Cmp node 'c' should be replaced with new one after BoolNode transformation
kvn
parents:
17
diff
changeset
|
1062 if (tst != tst0) { |
|
73970d8c0b27
6671250: In Parse::do_if() old Cmp node 'c' should be replaced with new one after BoolNode transformation
kvn
parents:
17
diff
changeset
|
1063 // Canonicalize one more time since transform can change it. |
|
73970d8c0b27
6671250: In Parse::do_if() old Cmp node 'c' should be replaced with new one after BoolNode transformation
kvn
parents:
17
diff
changeset
|
1064 btest = tst->as_Bool()->_test._test; |
|
73970d8c0b27
6671250: In Parse::do_if() old Cmp node 'c' should be replaced with new one after BoolNode transformation
kvn
parents:
17
diff
changeset
|
1065 if (!BoolTest(btest).is_canonical()) { |
|
73970d8c0b27
6671250: In Parse::do_if() old Cmp node 'c' should be replaced with new one after BoolNode transformation
kvn
parents:
17
diff
changeset
|
1066 // Reverse edges one more time... |
|
73970d8c0b27
6671250: In Parse::do_if() old Cmp node 'c' should be replaced with new one after BoolNode transformation
kvn
parents:
17
diff
changeset
|
1067 tst = _gvn.transform( tst->as_Bool()->negate(&_gvn) ); |
|
73970d8c0b27
6671250: In Parse::do_if() old Cmp node 'c' should be replaced with new one after BoolNode transformation
kvn
parents:
17
diff
changeset
|
1068 btest = tst->as_Bool()->_test._test; |
|
73970d8c0b27
6671250: In Parse::do_if() old Cmp node 'c' should be replaced with new one after BoolNode transformation
kvn
parents:
17
diff
changeset
|
1069 assert(BoolTest(btest).is_canonical(), "sanity"); |
|
73970d8c0b27
6671250: In Parse::do_if() old Cmp node 'c' should be replaced with new one after BoolNode transformation
kvn
parents:
17
diff
changeset
|
1070 taken_if_true = !taken_if_true; |
|
73970d8c0b27
6671250: In Parse::do_if() old Cmp node 'c' should be replaced with new one after BoolNode transformation
kvn
parents:
17
diff
changeset
|
1071 } |
|
73970d8c0b27
6671250: In Parse::do_if() old Cmp node 'c' should be replaced with new one after BoolNode transformation
kvn
parents:
17
diff
changeset
|
1072 c = tst->in(1); |
|
73970d8c0b27
6671250: In Parse::do_if() old Cmp node 'c' should be replaced with new one after BoolNode transformation
kvn
parents:
17
diff
changeset
|
1073 } |
|
73970d8c0b27
6671250: In Parse::do_if() old Cmp node 'c' should be replaced with new one after BoolNode transformation
kvn
parents:
17
diff
changeset
|
1074 BoolTest::mask neg_btest = BoolTest(btest).negate(); |
|
73970d8c0b27
6671250: In Parse::do_if() old Cmp node 'c' should be replaced with new one after BoolNode transformation
kvn
parents:
17
diff
changeset
|
1075 taken_btest = taken_if_true ? btest : neg_btest; |
|
73970d8c0b27
6671250: In Parse::do_if() old Cmp node 'c' should be replaced with new one after BoolNode transformation
kvn
parents:
17
diff
changeset
|
1076 untaken_btest = taken_if_true ? neg_btest : btest; |
| 0 | 1077 } |
| 1078 | |
| 1079 // Generate real control flow | |
| 1080 float true_prob = (taken_if_true ? prob : untaken_prob); | |
| 1081 IfNode* iff = create_and_map_if(control(), tst, true_prob, cnt); | |
| 1082 assert(iff->_prob > 0.0f,"Optimizer made bad probability in parser"); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1083 Node* taken_branch = new (C) IfTrueNode(iff); |
|
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1084 Node* untaken_branch = new (C) IfFalseNode(iff); |
| 0 | 1085 if (!taken_if_true) { // Finish conversion to canonical form |
| 1086 Node* tmp = taken_branch; | |
| 1087 taken_branch = untaken_branch; | |
| 1088 untaken_branch = tmp; | |
| 1089 } | |
| 1090 | |
| 1091 // Branch is taken: | |
| 1092 { PreserveJVMState pjvms(this); | |
| 1093 taken_branch = _gvn.transform(taken_branch); | |
| 1094 set_control(taken_branch); | |
| 1095 | |
|
17
ff5961f4c095
6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents:
0
diff
changeset
|
1096 if (stopped()) { |
| 10278 | 1097 if (C->eliminate_boxing()) { |
|
17
ff5961f4c095
6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents:
0
diff
changeset
|
1098 // Mark the successor block as parsed |
|
ff5961f4c095
6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents:
0
diff
changeset
|
1099 branch_block->next_path_num(); |
|
ff5961f4c095
6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents:
0
diff
changeset
|
1100 } |
|
ff5961f4c095
6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents:
0
diff
changeset
|
1101 } else { |
| 0 | 1102 // Update method data |
| 1103 profile_taken_branch(target_bci); | |
| 1104 adjust_map_after_if(taken_btest, c, prob, branch_block, next_block); | |
| 1172 | 1105 if (!stopped()) { |
| 0 | 1106 merge(target_bci); |
| 1172 | 1107 } |
| 0 | 1108 } |
| 1109 } | |
| 1110 | |
| 1111 untaken_branch = _gvn.transform(untaken_branch); | |
| 1112 set_control(untaken_branch); | |
| 1113 | |
| 1114 // Branch not taken. | |
|
17
ff5961f4c095
6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents:
0
diff
changeset
|
1115 if (stopped()) { |
| 10278 | 1116 if (C->eliminate_boxing()) { |
|
17
ff5961f4c095
6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents:
0
diff
changeset
|
1117 // Mark the successor block as parsed |
|
ff5961f4c095
6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents:
0
diff
changeset
|
1118 next_block->next_path_num(); |
|
ff5961f4c095
6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents:
0
diff
changeset
|
1119 } |
|
ff5961f4c095
6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents:
0
diff
changeset
|
1120 } else { |
| 0 | 1121 // Update method data |
| 1122 profile_not_taken_branch(); | |
| 1123 adjust_map_after_if(untaken_btest, c, untaken_prob, | |
| 1124 next_block, branch_block); | |
| 1125 } | |
| 1126 } | |
| 1127 | |
| 1128 //----------------------------adjust_map_after_if------------------------------ | |
| 1129 // Adjust the JVM state to reflect the result of taking this path. | |
| 1130 // Basically, it means inspecting the CmpNode controlling this | |
| 1131 // branch, seeing how it constrains a tested value, and then | |
| 1132 // deciding if it's worth our while to encode this constraint | |
| 1133 // as graph nodes in the current abstract interpretation map. | |
| 1134 void Parse::adjust_map_after_if(BoolTest::mask btest, Node* c, float prob, | |
| 1135 Block* path, Block* other_path) { | |
| 1136 if (stopped() || !c->is_Cmp() || btest == BoolTest::illegal) | |
| 1137 return; // nothing to do | |
| 1138 | |
| 1139 bool is_fallthrough = (path == successor_for_bci(iter().next_bci())); | |
| 1140 | |
|
1746
4b29a725c43c
6912064: type profiles need to be exploited more for dynamic language support
jrose
parents:
1602
diff
changeset
|
1141 if (seems_never_taken(prob) && seems_stable_comparison(btest, c)) { |
| 0 | 1142 repush_if_args(); |
|
20439
f6f9aec27858
8030976: Untaken paths should be more vigorously pruned at highest optimization level
rbackman
parents:
17937
diff
changeset
|
1143 uncommon_trap(Deoptimization::Reason_unstable_if, |
| 0 | 1144 Deoptimization::Action_reinterpret, |
| 1145 NULL, | |
| 1146 (is_fallthrough ? "taken always" : "taken never")); | |
| 1147 return; | |
| 1148 } | |
| 1149 | |
| 1150 Node* val = c->in(1); | |
| 1151 Node* con = c->in(2); | |
| 1152 const Type* tcon = _gvn.type(con); | |
| 1153 const Type* tval = _gvn.type(val); | |
| 1154 bool have_con = tcon->singleton(); | |
| 1155 if (tval->singleton()) { | |
| 1156 if (!have_con) { | |
| 1157 // Swap, so constant is in con. | |
| 1158 con = val; | |
| 1159 tcon = tval; | |
| 1160 val = c->in(2); | |
| 1161 tval = _gvn.type(val); | |
| 1162 btest = BoolTest(btest).commute(); | |
| 1163 have_con = true; | |
| 1164 } else { | |
| 1165 // Do we have two constants? Then leave well enough alone. | |
| 1166 have_con = false; | |
| 1167 } | |
| 1168 } | |
| 1169 if (!have_con) // remaining adjustments need a con | |
| 1170 return; | |
| 1171 | |
|
6131
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1172 sharpen_type_after_if(btest, con, tcon, val, tval); |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1173 } |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1174 |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1175 |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1176 static Node* extract_obj_from_klass_load(PhaseGVN* gvn, Node* n) { |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1177 Node* ldk; |
|
6848
8e47bac5643a
7054512: Compress class pointers after perm gen removal
roland
parents:
6804
diff
changeset
|
1178 if (n->is_DecodeNKlass()) { |
|
6131
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1179 if (n->in(1)->Opcode() != Op_LoadNKlass) { |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1180 return NULL; |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1181 } else { |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1182 ldk = n->in(1); |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1183 } |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1184 } else if (n->Opcode() != Op_LoadKlass) { |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1185 return NULL; |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1186 } else { |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1187 ldk = n; |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1188 } |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1189 assert(ldk != NULL && ldk->is_Load(), "should have found a LoadKlass or LoadNKlass node"); |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1190 |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1191 Node* adr = ldk->in(MemNode::Address); |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1192 intptr_t off = 0; |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1193 Node* obj = AddPNode::Ideal_base_and_offset(adr, gvn, off); |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1194 if (obj == NULL || off != oopDesc::klass_offset_in_bytes()) // loading oopDesc::_klass? |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1195 return NULL; |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1196 const TypePtr* tp = gvn->type(obj)->is_ptr(); |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1197 if (tp == NULL || !(tp->isa_instptr() || tp->isa_aryptr())) // is obj a Java object ptr? |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1198 return NULL; |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1199 |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1200 return obj; |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1201 } |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1202 |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1203 void Parse::sharpen_type_after_if(BoolTest::mask btest, |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1204 Node* con, const Type* tcon, |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1205 Node* val, const Type* tval) { |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1206 // Look for opportunities to sharpen the type of a node |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1207 // whose klass is compared with a constant klass. |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1208 if (btest == BoolTest::eq && tcon->isa_klassptr()) { |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1209 Node* obj = extract_obj_from_klass_load(&_gvn, val); |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1210 const TypeOopPtr* con_type = tcon->isa_klassptr()->as_instance_type(); |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1211 if (obj != NULL && (con_type->isa_instptr() || con_type->isa_aryptr())) { |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1212 // Found: |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1213 // Bool(CmpP(LoadKlass(obj._klass), ConP(Foo.klass)), [eq]) |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1214 // or the narrowOop equivalent. |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1215 const Type* obj_type = _gvn.type(obj); |
|
17671
de95063c0e34
8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents:
17467
diff
changeset
|
1216 const TypeOopPtr* tboth = obj_type->join_speculative(con_type)->isa_oopptr(); |
|
6206
70862d781d01
7180769: assert(tboth->klass_is_exact()) failed: klass should be exact
kvn
parents:
6131
diff
changeset
|
1217 if (tboth != NULL && tboth->klass_is_exact() && tboth != obj_type && |
|
70862d781d01
7180769: assert(tboth->klass_is_exact()) failed: klass should be exact
kvn
parents:
6131
diff
changeset
|
1218 tboth->higher_equal(obj_type)) { |
|
6131
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1219 // obj has to be of the exact type Foo if the CmpP succeeds. |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1220 int obj_in_map = map()->find_edge(obj); |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1221 JVMState* jvms = this->jvms(); |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1222 if (obj_in_map >= 0 && |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1223 (jvms->is_loc(obj_in_map) || jvms->is_stk(obj_in_map))) { |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1224 TypeNode* ccast = new (C) CheckCastPPNode(control(), obj, tboth); |
|
6131
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1225 const Type* tcc = ccast->as_Type()->type(); |
|
17671
de95063c0e34
8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents:
17467
diff
changeset
|
1226 assert(tcc != obj_type && tcc->higher_equal_speculative(obj_type), "must improve"); |
|
6131
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1227 // Delay transform() call to allow recovery of pre-cast value |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1228 // at the control merge. |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1229 _gvn.set_type_bottom(ccast); |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1230 record_for_igvn(ccast); |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1231 // Here's the payoff. |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1232 replace_in_map(obj, ccast); |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1233 } |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1234 } |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1235 } |
|
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1236 } |
| 0 | 1237 |
| 1238 int val_in_map = map()->find_edge(val); | |
| 1239 if (val_in_map < 0) return; // replace_in_map would be useless | |
| 1240 { | |
| 1241 JVMState* jvms = this->jvms(); | |
| 1242 if (!(jvms->is_loc(val_in_map) || | |
| 1243 jvms->is_stk(val_in_map))) | |
| 1244 return; // again, it would be useless | |
| 1245 } | |
| 1246 | |
| 1247 // Check for a comparison to a constant, and "know" that the compared | |
| 1248 // value is constrained on this path. | |
| 1249 assert(tcon->singleton(), ""); | |
| 1250 ConstraintCastNode* ccast = NULL; | |
| 1251 Node* cast = NULL; | |
| 1252 | |
| 1253 switch (btest) { | |
| 1254 case BoolTest::eq: // Constant test? | |
| 1255 { | |
|
17671
de95063c0e34
8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents:
17467
diff
changeset
|
1256 const Type* tboth = tcon->join_speculative(tval); |
| 0 | 1257 if (tboth == tval) break; // Nothing to gain. |
| 1258 if (tcon->isa_int()) { | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1259 ccast = new (C) CastIINode(val, tboth); |
| 0 | 1260 } else if (tcon == TypePtr::NULL_PTR) { |
| 1261 // Cast to null, but keep the pointer identity temporarily live. | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1262 ccast = new (C) CastPPNode(val, tboth); |
| 0 | 1263 } else { |
| 1264 const TypeF* tf = tcon->isa_float_constant(); | |
| 1265 const TypeD* td = tcon->isa_double_constant(); | |
| 1266 // Exclude tests vs float/double 0 as these could be | |
| 1267 // either +0 or -0. Just because you are equal to +0 | |
| 1268 // doesn't mean you ARE +0! | |
|
6131
8f6ce6f1049b
7170463: C2 should recognize "obj.getClass() == A.class" code pattern
kvn
parents:
3899
diff
changeset
|
1269 // Note, following code also replaces Long and Oop values. |
| 0 | 1270 if ((!tf || tf->_f != 0.0) && |
| 1271 (!td || td->_d != 0.0)) | |
| 1272 cast = con; // Replace non-constant val by con. | |
| 1273 } | |
| 1274 } | |
| 1275 break; | |
| 1276 | |
| 1277 case BoolTest::ne: | |
| 1278 if (tcon == TypePtr::NULL_PTR) { | |
| 1279 cast = cast_not_null(val, false); | |
| 1280 } | |
| 1281 break; | |
| 1282 | |
| 1283 default: | |
| 1284 // (At this point we could record int range types with CastII.) | |
| 1285 break; | |
| 1286 } | |
| 1287 | |
| 1288 if (ccast != NULL) { | |
| 1289 const Type* tcc = ccast->as_Type()->type(); | |
|
17671
de95063c0e34
8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents:
17467
diff
changeset
|
1290 assert(tcc != tval && tcc->higher_equal_speculative(tval), "must improve"); |
| 0 | 1291 // Delay transform() call to allow recovery of pre-cast value |
| 1292 // at the control merge. | |
| 1293 ccast->set_req(0, control()); | |
| 1294 _gvn.set_type_bottom(ccast); | |
| 1295 record_for_igvn(ccast); | |
| 1296 cast = ccast; | |
| 1297 } | |
| 1298 | |
| 1299 if (cast != NULL) { // Here's the payoff. | |
| 1300 replace_in_map(val, cast); | |
| 1301 } | |
| 1302 } | |
| 1303 | |
| 12966 | 1304 /** |
| 1305 * Use speculative type to optimize CmpP node: if comparison is | |
| 1306 * against the low level class, cast the object to the speculative | |
| 1307 * type if any. CmpP should then go away. | |
| 1308 * | |
| 1309 * @param c expected CmpP node | |
| 1310 * @return result of CmpP on object casted to speculative type | |
| 1311 * | |
| 1312 */ | |
| 1313 Node* Parse::optimize_cmp_with_klass(Node* c) { | |
| 1314 // If this is transformed by the _gvn to a comparison with the low | |
| 1315 // level klass then we may be able to use speculation | |
| 1316 if (c->Opcode() == Op_CmpP && | |
| 1317 (c->in(1)->Opcode() == Op_LoadKlass || c->in(1)->Opcode() == Op_DecodeNKlass) && | |
| 1318 c->in(2)->is_Con()) { | |
| 1319 Node* load_klass = NULL; | |
| 1320 Node* decode = NULL; | |
| 1321 if (c->in(1)->Opcode() == Op_DecodeNKlass) { | |
| 1322 decode = c->in(1); | |
| 1323 load_klass = c->in(1)->in(1); | |
| 1324 } else { | |
| 1325 load_klass = c->in(1); | |
| 1326 } | |
| 1327 if (load_klass->in(2)->is_AddP()) { | |
| 1328 Node* addp = load_klass->in(2); | |
| 1329 Node* obj = addp->in(AddPNode::Address); | |
| 1330 const TypeOopPtr* obj_type = _gvn.type(obj)->is_oopptr(); | |
| 1331 if (obj_type->speculative_type() != NULL) { | |
| 1332 ciKlass* k = obj_type->speculative_type(); | |
| 1333 inc_sp(2); | |
| 1334 obj = maybe_cast_profiled_obj(obj, k); | |
| 1335 dec_sp(2); | |
| 1336 // Make the CmpP use the casted obj | |
| 1337 addp = basic_plus_adr(obj, addp->in(AddPNode::Offset)); | |
| 1338 load_klass = load_klass->clone(); | |
| 1339 load_klass->set_req(2, addp); | |
| 1340 load_klass = _gvn.transform(load_klass); | |
| 1341 if (decode != NULL) { | |
| 1342 decode = decode->clone(); | |
| 1343 decode->set_req(1, load_klass); | |
| 1344 load_klass = _gvn.transform(decode); | |
| 1345 } | |
| 1346 c = c->clone(); | |
| 1347 c->set_req(1, load_klass); | |
| 1348 c = _gvn.transform(c); | |
| 1349 } | |
| 1350 } | |
| 1351 } | |
| 1352 return c; | |
| 1353 } | |
| 0 | 1354 |
| 1355 //------------------------------do_one_bytecode-------------------------------- | |
| 1356 // Parse this bytecode, and alter the Parsers JVM->Node mapping | |
| 1357 void Parse::do_one_bytecode() { | |
| 1358 Node *a, *b, *c, *d; // Handy temps | |
| 1359 BoolTest::mask btest; | |
| 1360 int i; | |
| 1361 | |
| 1362 assert(!has_exceptions(), "bytecode entry state must be clear of throws"); | |
| 1363 | |
| 1364 if (C->check_node_count(NodeLimitFudgeFactor * 5, | |
| 1365 "out of nodes parsing method")) { | |
| 1366 return; | |
| 1367 } | |
| 1368 | |
| 1369 #ifdef ASSERT | |
| 1370 // for setting breakpoints | |
| 1371 if (TraceOptoParse) { | |
| 1372 tty->print(" @"); | |
| 1373 dump_bci(bci()); | |
| 8862 | 1374 tty->cr(); |
| 0 | 1375 } |
| 1376 #endif | |
| 1377 | |
| 1378 switch (bc()) { | |
| 1379 case Bytecodes::_nop: | |
| 1380 // do nothing | |
| 1381 break; | |
| 1382 case Bytecodes::_lconst_0: | |
| 1383 push_pair(longcon(0)); | |
| 1384 break; | |
| 1385 | |
| 1386 case Bytecodes::_lconst_1: | |
| 1387 push_pair(longcon(1)); | |
| 1388 break; | |
| 1389 | |
| 1390 case Bytecodes::_fconst_0: | |
| 1391 push(zerocon(T_FLOAT)); | |
| 1392 break; | |
| 1393 | |
| 1394 case Bytecodes::_fconst_1: | |
| 1395 push(makecon(TypeF::ONE)); | |
| 1396 break; | |
| 1397 | |
| 1398 case Bytecodes::_fconst_2: | |
| 1399 push(makecon(TypeF::make(2.0f))); | |
| 1400 break; | |
| 1401 | |
| 1402 case Bytecodes::_dconst_0: | |
| 1403 push_pair(zerocon(T_DOUBLE)); | |
| 1404 break; | |
| 1405 | |
| 1406 case Bytecodes::_dconst_1: | |
| 1407 push_pair(makecon(TypeD::ONE)); | |
| 1408 break; | |
| 1409 | |
| 1410 case Bytecodes::_iconst_m1:push(intcon(-1)); break; | |
| 1411 case Bytecodes::_iconst_0: push(intcon( 0)); break; | |
| 1412 case Bytecodes::_iconst_1: push(intcon( 1)); break; | |
| 1413 case Bytecodes::_iconst_2: push(intcon( 2)); break; | |
| 1414 case Bytecodes::_iconst_3: push(intcon( 3)); break; | |
| 1415 case Bytecodes::_iconst_4: push(intcon( 4)); break; | |
| 1416 case Bytecodes::_iconst_5: push(intcon( 5)); break; | |
| 1565 | 1417 case Bytecodes::_bipush: push(intcon(iter().get_constant_u1())); break; |
| 1418 case Bytecodes::_sipush: push(intcon(iter().get_constant_u2())); break; | |
| 0 | 1419 case Bytecodes::_aconst_null: push(null()); break; |
| 1420 case Bytecodes::_ldc: | |
| 1421 case Bytecodes::_ldc_w: | |
| 1422 case Bytecodes::_ldc2_w: | |
| 1423 // If the constant is unresolved, run this BC once in the interpreter. | |
| 1602 | 1424 { |
| 0 | 1425 ciConstant constant = iter().get_constant(); |
| 1602 | 1426 if (constant.basic_type() == T_OBJECT && |
| 1427 !constant.as_object()->is_loaded()) { | |
| 1428 int index = iter().get_constant_pool_index(); | |
| 1429 constantTag tag = iter().get_constant_pool_tag(index); | |
| 1430 uncommon_trap(Deoptimization::make_trap_request | |
| 1431 (Deoptimization::Reason_unloaded, | |
| 1432 Deoptimization::Action_reinterpret, | |
| 1433 index), | |
| 1434 NULL, tag.internal_name()); | |
| 1435 break; | |
| 0 | 1436 } |
|
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6206
diff
changeset
|
1437 assert(constant.basic_type() != T_OBJECT || constant.as_object()->is_instance(), |
| 1602 | 1438 "must be java_mirror of klass"); |
|
989
148e5441d916
6863023: need non-perm oops in code cache for JSR 292
jrose
parents:
844
diff
changeset
|
1439 bool pushed = push_constant(constant, true); |
|
148e5441d916
6863023: need non-perm oops in code cache for JSR 292
jrose
parents:
844
diff
changeset
|
1440 guarantee(pushed, "must be possible to push this constant"); |
| 0 | 1441 } |
| 1442 | |
| 1443 break; | |
| 1444 | |
| 1445 case Bytecodes::_aload_0: | |
| 1446 push( local(0) ); | |
| 1447 break; | |
| 1448 case Bytecodes::_aload_1: | |
| 1449 push( local(1) ); | |
| 1450 break; | |
| 1451 case Bytecodes::_aload_2: | |
| 1452 push( local(2) ); | |
| 1453 break; | |
| 1454 case Bytecodes::_aload_3: | |
| 1455 push( local(3) ); | |
| 1456 break; | |
| 1457 case Bytecodes::_aload: | |
| 1458 push( local(iter().get_index()) ); | |
| 1459 break; | |
| 1460 | |
| 1461 case Bytecodes::_fload_0: | |
| 1462 case Bytecodes::_iload_0: | |
| 1463 push( local(0) ); | |
| 1464 break; | |
| 1465 case Bytecodes::_fload_1: | |
| 1466 case Bytecodes::_iload_1: | |
| 1467 push( local(1) ); | |
| 1468 break; | |
| 1469 case Bytecodes::_fload_2: | |
| 1470 case Bytecodes::_iload_2: | |
| 1471 push( local(2) ); | |
| 1472 break; | |
| 1473 case Bytecodes::_fload_3: | |
| 1474 case Bytecodes::_iload_3: | |
| 1475 push( local(3) ); | |
| 1476 break; | |
| 1477 case Bytecodes::_fload: | |
| 1478 case Bytecodes::_iload: | |
| 1479 push( local(iter().get_index()) ); | |
| 1480 break; | |
| 1481 case Bytecodes::_lload_0: | |
| 1482 push_pair_local( 0 ); | |
| 1483 break; | |
| 1484 case Bytecodes::_lload_1: | |
| 1485 push_pair_local( 1 ); | |
| 1486 break; | |
| 1487 case Bytecodes::_lload_2: | |
| 1488 push_pair_local( 2 ); | |
| 1489 break; | |
| 1490 case Bytecodes::_lload_3: | |
| 1491 push_pair_local( 3 ); | |
| 1492 break; | |
| 1493 case Bytecodes::_lload: | |
| 1494 push_pair_local( iter().get_index() ); | |
| 1495 break; | |
| 1496 | |
| 1497 case Bytecodes::_dload_0: | |
| 1498 push_pair_local(0); | |
| 1499 break; | |
| 1500 case Bytecodes::_dload_1: | |
| 1501 push_pair_local(1); | |
| 1502 break; | |
| 1503 case Bytecodes::_dload_2: | |
| 1504 push_pair_local(2); | |
| 1505 break; | |
| 1506 case Bytecodes::_dload_3: | |
| 1507 push_pair_local(3); | |
| 1508 break; | |
| 1509 case Bytecodes::_dload: | |
| 1510 push_pair_local(iter().get_index()); | |
| 1511 break; | |
| 1512 case Bytecodes::_fstore_0: | |
| 1513 case Bytecodes::_istore_0: | |
| 1514 case Bytecodes::_astore_0: | |
| 1515 set_local( 0, pop() ); | |
| 1516 break; | |
| 1517 case Bytecodes::_fstore_1: | |
| 1518 case Bytecodes::_istore_1: | |
| 1519 case Bytecodes::_astore_1: | |
| 1520 set_local( 1, pop() ); | |
| 1521 break; | |
| 1522 case Bytecodes::_fstore_2: | |
| 1523 case Bytecodes::_istore_2: | |
| 1524 case Bytecodes::_astore_2: | |
| 1525 set_local( 2, pop() ); | |
| 1526 break; | |
| 1527 case Bytecodes::_fstore_3: | |
| 1528 case Bytecodes::_istore_3: | |
| 1529 case Bytecodes::_astore_3: | |
| 1530 set_local( 3, pop() ); | |
| 1531 break; | |
| 1532 case Bytecodes::_fstore: | |
| 1533 case Bytecodes::_istore: | |
| 1534 case Bytecodes::_astore: | |
| 1535 set_local( iter().get_index(), pop() ); | |
| 1536 break; | |
| 1537 // long stores | |
| 1538 case Bytecodes::_lstore_0: | |
| 1539 set_pair_local( 0, pop_pair() ); | |
| 1540 break; | |
| 1541 case Bytecodes::_lstore_1: | |
| 1542 set_pair_local( 1, pop_pair() ); | |
| 1543 break; | |
| 1544 case Bytecodes::_lstore_2: | |
| 1545 set_pair_local( 2, pop_pair() ); | |
| 1546 break; | |
| 1547 case Bytecodes::_lstore_3: | |
| 1548 set_pair_local( 3, pop_pair() ); | |
| 1549 break; | |
| 1550 case Bytecodes::_lstore: | |
| 1551 set_pair_local( iter().get_index(), pop_pair() ); | |
| 1552 break; | |
| 1553 | |
| 1554 // double stores | |
| 1555 case Bytecodes::_dstore_0: | |
| 1556 set_pair_local( 0, dstore_rounding(pop_pair()) ); | |
| 1557 break; | |
| 1558 case Bytecodes::_dstore_1: | |
| 1559 set_pair_local( 1, dstore_rounding(pop_pair()) ); | |
| 1560 break; | |
| 1561 case Bytecodes::_dstore_2: | |
| 1562 set_pair_local( 2, dstore_rounding(pop_pair()) ); | |
| 1563 break; | |
| 1564 case Bytecodes::_dstore_3: | |
| 1565 set_pair_local( 3, dstore_rounding(pop_pair()) ); | |
| 1566 break; | |
| 1567 case Bytecodes::_dstore: | |
| 1568 set_pair_local( iter().get_index(), dstore_rounding(pop_pair()) ); | |
| 1569 break; | |
| 1570 | |
|
7194
beebba0acc11
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
6848
diff
changeset
|
1571 case Bytecodes::_pop: dec_sp(1); break; |
|
beebba0acc11
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
6848
diff
changeset
|
1572 case Bytecodes::_pop2: dec_sp(2); break; |
| 0 | 1573 case Bytecodes::_swap: |
| 1574 a = pop(); | |
| 1575 b = pop(); | |
| 1576 push(a); | |
| 1577 push(b); | |
| 1578 break; | |
| 1579 case Bytecodes::_dup: | |
| 1580 a = pop(); | |
| 1581 push(a); | |
| 1582 push(a); | |
| 1583 break; | |
| 1584 case Bytecodes::_dup_x1: | |
| 1585 a = pop(); | |
| 1586 b = pop(); | |
| 1587 push( a ); | |
| 1588 push( b ); | |
| 1589 push( a ); | |
| 1590 break; | |
| 1591 case Bytecodes::_dup_x2: | |
| 1592 a = pop(); | |
| 1593 b = pop(); | |
| 1594 c = pop(); | |
| 1595 push( a ); | |
| 1596 push( c ); | |
| 1597 push( b ); | |
| 1598 push( a ); | |
| 1599 break; | |
| 1600 case Bytecodes::_dup2: | |
| 1601 a = pop(); | |
| 1602 b = pop(); | |
| 1603 push( b ); | |
| 1604 push( a ); | |
| 1605 push( b ); | |
| 1606 push( a ); | |
| 1607 break; | |
| 1608 | |
| 1609 case Bytecodes::_dup2_x1: | |
| 1610 // before: .. c, b, a | |
| 1611 // after: .. b, a, c, b, a | |
| 1612 // not tested | |
| 1613 a = pop(); | |
| 1614 b = pop(); | |
| 1615 c = pop(); | |
| 1616 push( b ); | |
| 1617 push( a ); | |
| 1618 push( c ); | |
| 1619 push( b ); | |
| 1620 push( a ); | |
| 1621 break; | |
| 1622 case Bytecodes::_dup2_x2: | |
| 1623 // before: .. d, c, b, a | |
| 1624 // after: .. b, a, d, c, b, a | |
| 1625 // not tested | |
| 1626 a = pop(); | |
| 1627 b = pop(); | |
| 1628 c = pop(); | |
| 1629 d = pop(); | |
| 1630 push( b ); | |
| 1631 push( a ); | |
| 1632 push( d ); | |
| 1633 push( c ); | |
| 1634 push( b ); | |
| 1635 push( a ); | |
| 1636 break; | |
| 1637 | |
| 1638 case Bytecodes::_arraylength: { | |
| 1639 // Must do null-check with value on expression stack | |
|
7194
beebba0acc11
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
6848
diff
changeset
|
1640 Node *ary = null_check(peek(), T_ARRAY); |
| 0 | 1641 // Compile-time detect of null-exception? |
| 1642 if (stopped()) return; | |
| 1643 a = pop(); | |
| 1644 push(load_array_length(a)); | |
| 1645 break; | |
| 1646 } | |
| 1647 | |
| 1648 case Bytecodes::_baload: array_load(T_BYTE); break; | |
| 1649 case Bytecodes::_caload: array_load(T_CHAR); break; | |
| 1650 case Bytecodes::_iaload: array_load(T_INT); break; | |
| 1651 case Bytecodes::_saload: array_load(T_SHORT); break; | |
| 1652 case Bytecodes::_faload: array_load(T_FLOAT); break; | |
| 1653 case Bytecodes::_aaload: array_load(T_OBJECT); break; | |
| 1654 case Bytecodes::_laload: { | |
| 1655 a = array_addressing(T_LONG, 0); | |
| 605 | 1656 if (stopped()) return; // guaranteed null or range check |
|
7194
beebba0acc11
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
6848
diff
changeset
|
1657 dec_sp(2); // Pop array and index |
|
14429
2113136690bc
8024921: PPC64 (part 113): Extend Load and Store nodes to know about memory ordering
goetz
parents:
12966
diff
changeset
|
1658 push_pair(make_load(control(), a, TypeLong::LONG, T_LONG, TypeAryPtr::LONGS, MemNode::unordered)); |
| 0 | 1659 break; |
| 1660 } | |
| 1661 case Bytecodes::_daload: { | |
| 1662 a = array_addressing(T_DOUBLE, 0); | |
| 605 | 1663 if (stopped()) return; // guaranteed null or range check |
|
7194
beebba0acc11
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
6848
diff
changeset
|
1664 dec_sp(2); // Pop array and index |
|
14429
2113136690bc
8024921: PPC64 (part 113): Extend Load and Store nodes to know about memory ordering
goetz
parents:
12966
diff
changeset
|
1665 push_pair(make_load(control(), a, Type::DOUBLE, T_DOUBLE, TypeAryPtr::DOUBLES, MemNode::unordered)); |
| 0 | 1666 break; |
| 1667 } | |
| 1668 case Bytecodes::_bastore: array_store(T_BYTE); break; | |
| 1669 case Bytecodes::_castore: array_store(T_CHAR); break; | |
| 1670 case Bytecodes::_iastore: array_store(T_INT); break; | |
| 1671 case Bytecodes::_sastore: array_store(T_SHORT); break; | |
| 1672 case Bytecodes::_fastore: array_store(T_FLOAT); break; | |
| 1673 case Bytecodes::_aastore: { | |
| 1674 d = array_addressing(T_OBJECT, 1); | |
| 605 | 1675 if (stopped()) return; // guaranteed null or range check |
| 0 | 1676 array_store_check(); |
| 1677 c = pop(); // Oop to store | |
| 1678 b = pop(); // index (already used) | |
| 1679 a = pop(); // the array itself | |
|
827
bf3489cc0aa0
6856025: assert(_base >= OopPtr && _base <= KlassPtr,"Not a Java pointer")
never
parents:
825
diff
changeset
|
1680 const TypeOopPtr* elemtype = _gvn.type(a)->is_aryptr()->elem()->make_oopptr(); |
| 0 | 1681 const TypeAryPtr* adr_type = TypeAryPtr::OOPS; |
|
14429
2113136690bc
8024921: PPC64 (part 113): Extend Load and Store nodes to know about memory ordering
goetz
parents:
12966
diff
changeset
|
1682 Node* store = store_oop_to_array(control(), a, d, adr_type, c, elemtype, T_OBJECT, MemNode::release); |
| 0 | 1683 break; |
| 1684 } | |
| 1685 case Bytecodes::_lastore: { | |
| 1686 a = array_addressing(T_LONG, 2); | |
| 605 | 1687 if (stopped()) return; // guaranteed null or range check |
| 0 | 1688 c = pop_pair(); |
|
7194
beebba0acc11
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
6848
diff
changeset
|
1689 dec_sp(2); // Pop array and index |
|
14429
2113136690bc
8024921: PPC64 (part 113): Extend Load and Store nodes to know about memory ordering
goetz
parents:
12966
diff
changeset
|
1690 store_to_memory(control(), a, c, T_LONG, TypeAryPtr::LONGS, MemNode::unordered); |
| 0 | 1691 break; |
| 1692 } | |
| 1693 case Bytecodes::_dastore: { | |
| 1694 a = array_addressing(T_DOUBLE, 2); | |
| 605 | 1695 if (stopped()) return; // guaranteed null or range check |
| 0 | 1696 c = pop_pair(); |
|
7194
beebba0acc11
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
6848
diff
changeset
|
1697 dec_sp(2); // Pop array and index |
| 0 | 1698 c = dstore_rounding(c); |
|
14429
2113136690bc
8024921: PPC64 (part 113): Extend Load and Store nodes to know about memory ordering
goetz
parents:
12966
diff
changeset
|
1699 store_to_memory(control(), a, c, T_DOUBLE, TypeAryPtr::DOUBLES, MemNode::unordered); |
| 0 | 1700 break; |
| 1701 } | |
| 1702 case Bytecodes::_getfield: | |
| 1703 do_getfield(); | |
| 1704 break; | |
| 1705 | |
| 1706 case Bytecodes::_getstatic: | |
| 1707 do_getstatic(); | |
| 1708 break; | |
| 1709 | |
| 1710 case Bytecodes::_putfield: | |
| 1711 do_putfield(); | |
| 1712 break; | |
| 1713 | |
| 1714 case Bytecodes::_putstatic: | |
| 1715 do_putstatic(); | |
| 1716 break; | |
| 1717 | |
| 1718 case Bytecodes::_irem: | |
| 1719 do_irem(); | |
| 1720 break; | |
| 1721 case Bytecodes::_idiv: | |
| 1722 // Must keep both values on the expression-stack during null-check | |
|
7194
beebba0acc11
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
6848
diff
changeset
|
1723 zero_check_int(peek()); |
| 0 | 1724 // Compile-time detect of null-exception? |
| 1725 if (stopped()) return; | |
| 1726 b = pop(); | |
| 1727 a = pop(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1728 push( _gvn.transform( new (C) DivINode(control(),a,b) ) ); |
| 0 | 1729 break; |
| 1730 case Bytecodes::_imul: | |
| 1731 b = pop(); a = pop(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1732 push( _gvn.transform( new (C) MulINode(a,b) ) ); |
| 0 | 1733 break; |
| 1734 case Bytecodes::_iadd: | |
| 1735 b = pop(); a = pop(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1736 push( _gvn.transform( new (C) AddINode(a,b) ) ); |
| 0 | 1737 break; |
| 1738 case Bytecodes::_ineg: | |
| 1739 a = pop(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1740 push( _gvn.transform( new (C) SubINode(_gvn.intcon(0),a)) ); |
| 0 | 1741 break; |
| 1742 case Bytecodes::_isub: | |
| 1743 b = pop(); a = pop(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1744 push( _gvn.transform( new (C) SubINode(a,b) ) ); |
| 0 | 1745 break; |
| 1746 case Bytecodes::_iand: | |
| 1747 b = pop(); a = pop(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1748 push( _gvn.transform( new (C) AndINode(a,b) ) ); |
| 0 | 1749 break; |
| 1750 case Bytecodes::_ior: | |
| 1751 b = pop(); a = pop(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1752 push( _gvn.transform( new (C) OrINode(a,b) ) ); |
| 0 | 1753 break; |
| 1754 case Bytecodes::_ixor: | |
| 1755 b = pop(); a = pop(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1756 push( _gvn.transform( new (C) XorINode(a,b) ) ); |
| 0 | 1757 break; |
| 1758 case Bytecodes::_ishl: | |
| 1759 b = pop(); a = pop(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1760 push( _gvn.transform( new (C) LShiftINode(a,b) ) ); |
| 0 | 1761 break; |
| 1762 case Bytecodes::_ishr: | |
| 1763 b = pop(); a = pop(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1764 push( _gvn.transform( new (C) RShiftINode(a,b) ) ); |
| 0 | 1765 break; |
| 1766 case Bytecodes::_iushr: | |
| 1767 b = pop(); a = pop(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1768 push( _gvn.transform( new (C) URShiftINode(a,b) ) ); |
| 0 | 1769 break; |
| 1770 | |
| 1771 case Bytecodes::_fneg: | |
| 1772 a = pop(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1773 b = _gvn.transform(new (C) NegFNode (a)); |
| 0 | 1774 push(b); |
| 1775 break; | |
| 1776 | |
| 1777 case Bytecodes::_fsub: | |
| 1778 b = pop(); | |
| 1779 a = pop(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1780 c = _gvn.transform( new (C) SubFNode(a,b) ); |
| 0 | 1781 d = precision_rounding(c); |
| 1782 push( d ); | |
| 1783 break; | |
| 1784 | |
| 1785 case Bytecodes::_fadd: | |
| 1786 b = pop(); | |
| 1787 a = pop(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1788 c = _gvn.transform( new (C) AddFNode(a,b) ); |
| 0 | 1789 d = precision_rounding(c); |
| 1790 push( d ); | |
| 1791 break; | |
| 1792 | |
| 1793 case Bytecodes::_fmul: | |
| 1794 b = pop(); | |
| 1795 a = pop(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1796 c = _gvn.transform( new (C) MulFNode(a,b) ); |
| 0 | 1797 d = precision_rounding(c); |
| 1798 push( d ); | |
| 1799 break; | |
| 1800 | |
| 1801 case Bytecodes::_fdiv: | |
| 1802 b = pop(); | |
| 1803 a = pop(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1804 c = _gvn.transform( new (C) DivFNode(0,a,b) ); |
| 0 | 1805 d = precision_rounding(c); |
| 1806 push( d ); | |
| 1807 break; | |
| 1808 | |
| 1809 case Bytecodes::_frem: | |
| 1810 if (Matcher::has_match_rule(Op_ModF)) { | |
| 1811 // Generate a ModF node. | |
| 1812 b = pop(); | |
| 1813 a = pop(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1814 c = _gvn.transform( new (C) ModFNode(0,a,b) ); |
| 0 | 1815 d = precision_rounding(c); |
| 1816 push( d ); | |
| 1817 } | |
| 1818 else { | |
| 1819 // Generate a call. | |
| 1820 modf(); | |
| 1821 } | |
| 1822 break; | |
| 1823 | |
| 1824 case Bytecodes::_fcmpl: | |
| 1825 b = pop(); | |
| 1826 a = pop(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1827 c = _gvn.transform( new (C) CmpF3Node( a, b)); |
| 0 | 1828 push(c); |
| 1829 break; | |
| 1830 case Bytecodes::_fcmpg: | |
| 1831 b = pop(); | |
| 1832 a = pop(); | |
| 1833 | |
| 1834 // Same as fcmpl but need to flip the unordered case. Swap the inputs, | |
| 1835 // which negates the result sign except for unordered. Flip the unordered | |
| 1836 // as well by using CmpF3 which implements unordered-lesser instead of | |
| 1837 // unordered-greater semantics. Finally, commute the result bits. Result | |
| 1838 // is same as using a CmpF3Greater except we did it with CmpF3 alone. | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1839 c = _gvn.transform( new (C) CmpF3Node( b, a)); |
|
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1840 c = _gvn.transform( new (C) SubINode(_gvn.intcon(0),c) ); |
| 0 | 1841 push(c); |
| 1842 break; | |
| 1843 | |
| 1844 case Bytecodes::_f2i: | |
| 1845 a = pop(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1846 push(_gvn.transform(new (C) ConvF2INode(a))); |
| 0 | 1847 break; |
| 1848 | |
| 1849 case Bytecodes::_d2i: | |
| 1850 a = pop_pair(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1851 b = _gvn.transform(new (C) ConvD2INode(a)); |
| 0 | 1852 push( b ); |
| 1853 break; | |
| 1854 | |
| 1855 case Bytecodes::_f2d: | |
| 1856 a = pop(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1857 b = _gvn.transform( new (C) ConvF2DNode(a)); |
| 0 | 1858 push_pair( b ); |
| 1859 break; | |
| 1860 | |
| 1861 case Bytecodes::_d2f: | |
| 1862 a = pop_pair(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1863 b = _gvn.transform( new (C) ConvD2FNode(a)); |
| 0 | 1864 // This breaks _227_mtrt (speed & correctness) and _222_mpegaudio (speed) |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1865 //b = _gvn.transform(new (C) RoundFloatNode(0, b) ); |
| 0 | 1866 push( b ); |
| 1867 break; | |
| 1868 | |
| 1869 case Bytecodes::_l2f: | |
| 1870 if (Matcher::convL2FSupported()) { | |
| 1871 a = pop_pair(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1872 b = _gvn.transform( new (C) ConvL2FNode(a)); |
| 0 | 1873 // For i486.ad, FILD doesn't restrict precision to 24 or 53 bits. |
| 1874 // Rather than storing the result into an FP register then pushing | |
| 1875 // out to memory to round, the machine instruction that implements | |
| 1876 // ConvL2D is responsible for rounding. | |
| 1877 // c = precision_rounding(b); | |
| 1878 c = _gvn.transform(b); | |
| 1879 push(c); | |
| 1880 } else { | |
| 1881 l2f(); | |
| 1882 } | |
| 1883 break; | |
| 1884 | |
| 1885 case Bytecodes::_l2d: | |
| 1886 a = pop_pair(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1887 b = _gvn.transform( new (C) ConvL2DNode(a)); |
| 0 | 1888 // For i486.ad, rounding is always necessary (see _l2f above). |
| 1889 // c = dprecision_rounding(b); | |
| 1890 c = _gvn.transform(b); | |
| 1891 push_pair(c); | |
| 1892 break; | |
| 1893 | |
| 1894 case Bytecodes::_f2l: | |
| 1895 a = pop(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1896 b = _gvn.transform( new (C) ConvF2LNode(a)); |
| 0 | 1897 push_pair(b); |
| 1898 break; | |
| 1899 | |
| 1900 case Bytecodes::_d2l: | |
| 1901 a = pop_pair(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1902 b = _gvn.transform( new (C) ConvD2LNode(a)); |
| 0 | 1903 push_pair(b); |
| 1904 break; | |
| 1905 | |
| 1906 case Bytecodes::_dsub: | |
| 1907 b = pop_pair(); | |
| 1908 a = pop_pair(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1909 c = _gvn.transform( new (C) SubDNode(a,b) ); |
| 0 | 1910 d = dprecision_rounding(c); |
| 1911 push_pair( d ); | |
| 1912 break; | |
| 1913 | |
| 1914 case Bytecodes::_dadd: | |
| 1915 b = pop_pair(); | |
| 1916 a = pop_pair(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1917 c = _gvn.transform( new (C) AddDNode(a,b) ); |
| 0 | 1918 d = dprecision_rounding(c); |
| 1919 push_pair( d ); | |
| 1920 break; | |
| 1921 | |
| 1922 case Bytecodes::_dmul: | |
| 1923 b = pop_pair(); | |
| 1924 a = pop_pair(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1925 c = _gvn.transform( new (C) MulDNode(a,b) ); |
| 0 | 1926 d = dprecision_rounding(c); |
| 1927 push_pair( d ); | |
| 1928 break; | |
| 1929 | |
| 1930 case Bytecodes::_ddiv: | |
| 1931 b = pop_pair(); | |
| 1932 a = pop_pair(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1933 c = _gvn.transform( new (C) DivDNode(0,a,b) ); |
| 0 | 1934 d = dprecision_rounding(c); |
| 1935 push_pair( d ); | |
| 1936 break; | |
| 1937 | |
| 1938 case Bytecodes::_dneg: | |
| 1939 a = pop_pair(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1940 b = _gvn.transform(new (C) NegDNode (a)); |
| 0 | 1941 push_pair(b); |
| 1942 break; | |
| 1943 | |
| 1944 case Bytecodes::_drem: | |
| 1945 if (Matcher::has_match_rule(Op_ModD)) { | |
| 1946 // Generate a ModD node. | |
| 1947 b = pop_pair(); | |
| 1948 a = pop_pair(); | |
| 1949 // a % b | |
| 1950 | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1951 c = _gvn.transform( new (C) ModDNode(0,a,b) ); |
| 0 | 1952 d = dprecision_rounding(c); |
| 1953 push_pair( d ); | |
| 1954 } | |
| 1955 else { | |
| 1956 // Generate a call. | |
| 1957 modd(); | |
| 1958 } | |
| 1959 break; | |
| 1960 | |
| 1961 case Bytecodes::_dcmpl: | |
| 1962 b = pop_pair(); | |
| 1963 a = pop_pair(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1964 c = _gvn.transform( new (C) CmpD3Node( a, b)); |
| 0 | 1965 push(c); |
| 1966 break; | |
| 1967 | |
| 1968 case Bytecodes::_dcmpg: | |
| 1969 b = pop_pair(); | |
| 1970 a = pop_pair(); | |
| 1971 // Same as dcmpl but need to flip the unordered case. | |
| 1972 // Commute the inputs, which negates the result sign except for unordered. | |
| 1973 // Flip the unordered as well by using CmpD3 which implements | |
| 1974 // unordered-lesser instead of unordered-greater semantics. | |
| 1975 // Finally, negate the result bits. Result is same as using a | |
| 1976 // CmpD3Greater except we did it with CmpD3 alone. | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1977 c = _gvn.transform( new (C) CmpD3Node( b, a)); |
|
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1978 c = _gvn.transform( new (C) SubINode(_gvn.intcon(0),c) ); |
| 0 | 1979 push(c); |
| 1980 break; | |
| 1981 | |
| 1982 | |
| 1983 // Note for longs -> lo word is on TOS, hi word is on TOS - 1 | |
| 1984 case Bytecodes::_land: | |
| 1985 b = pop_pair(); | |
| 1986 a = pop_pair(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1987 c = _gvn.transform( new (C) AndLNode(a,b) ); |
| 0 | 1988 push_pair(c); |
| 1989 break; | |
| 1990 case Bytecodes::_lor: | |
| 1991 b = pop_pair(); | |
| 1992 a = pop_pair(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1993 c = _gvn.transform( new (C) OrLNode(a,b) ); |
| 0 | 1994 push_pair(c); |
| 1995 break; | |
| 1996 case Bytecodes::_lxor: | |
| 1997 b = pop_pair(); | |
| 1998 a = pop_pair(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
1999 c = _gvn.transform( new (C) XorLNode(a,b) ); |
| 0 | 2000 push_pair(c); |
| 2001 break; | |
| 2002 | |
| 2003 case Bytecodes::_lshl: | |
| 2004 b = pop(); // the shift count | |
| 2005 a = pop_pair(); // value to be shifted | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
2006 c = _gvn.transform( new (C) LShiftLNode(a,b) ); |
| 0 | 2007 push_pair(c); |
| 2008 break; | |
| 2009 case Bytecodes::_lshr: | |
| 2010 b = pop(); // the shift count | |
| 2011 a = pop_pair(); // value to be shifted | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
2012 c = _gvn.transform( new (C) RShiftLNode(a,b) ); |
| 0 | 2013 push_pair(c); |
| 2014 break; | |
| 2015 case Bytecodes::_lushr: | |
| 2016 b = pop(); // the shift count | |
| 2017 a = pop_pair(); // value to be shifted | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
2018 c = _gvn.transform( new (C) URShiftLNode(a,b) ); |
| 0 | 2019 push_pair(c); |
| 2020 break; | |
| 2021 case Bytecodes::_lmul: | |
| 2022 b = pop_pair(); | |
| 2023 a = pop_pair(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
2024 c = _gvn.transform( new (C) MulLNode(a,b) ); |
| 0 | 2025 push_pair(c); |
| 2026 break; | |
| 2027 | |
| 2028 case Bytecodes::_lrem: | |
| 2029 // Must keep both values on the expression-stack during null-check | |
| 2030 assert(peek(0) == top(), "long word order"); | |
|
7194
beebba0acc11
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
6848
diff
changeset
|
2031 zero_check_long(peek(1)); |
| 0 | 2032 // Compile-time detect of null-exception? |
| 2033 if (stopped()) return; | |
| 2034 b = pop_pair(); | |
| 2035 a = pop_pair(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
2036 c = _gvn.transform( new (C) ModLNode(control(),a,b) ); |
| 0 | 2037 push_pair(c); |
| 2038 break; | |
| 2039 | |
| 2040 case Bytecodes::_ldiv: | |
| 2041 // Must keep both values on the expression-stack during null-check | |
| 2042 assert(peek(0) == top(), "long word order"); | |
|
7194
beebba0acc11
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
6848
diff
changeset
|
2043 zero_check_long(peek(1)); |
| 0 | 2044 // Compile-time detect of null-exception? |
| 2045 if (stopped()) return; | |
| 2046 b = pop_pair(); | |
| 2047 a = pop_pair(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
2048 c = _gvn.transform( new (C) DivLNode(control(),a,b) ); |
| 0 | 2049 push_pair(c); |
| 2050 break; | |
| 2051 | |
| 2052 case Bytecodes::_ladd: | |
| 2053 b = pop_pair(); | |
| 2054 a = pop_pair(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
2055 c = _gvn.transform( new (C) AddLNode(a,b) ); |
| 0 | 2056 push_pair(c); |
| 2057 break; | |
| 2058 case Bytecodes::_lsub: | |
| 2059 b = pop_pair(); | |
| 2060 a = pop_pair(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
2061 c = _gvn.transform( new (C) SubLNode(a,b) ); |
| 0 | 2062 push_pair(c); |
| 2063 break; | |
| 2064 case Bytecodes::_lcmp: | |
| 2065 // Safepoints are now inserted _before_ branches. The long-compare | |
| 2066 // bytecode painfully produces a 3-way value (-1,0,+1) which requires a | |
| 2067 // slew of control flow. These are usually followed by a CmpI vs zero and | |
| 2068 // a branch; this pattern then optimizes to the obvious long-compare and | |
| 2069 // branch. However, if the branch is backwards there's a Safepoint | |
| 2070 // inserted. The inserted Safepoint captures the JVM state at the | |
| 2071 // pre-branch point, i.e. it captures the 3-way value. Thus if a | |
| 2072 // long-compare is used to control a loop the debug info will force | |
| 2073 // computation of the 3-way value, even though the generated code uses a | |
| 2074 // long-compare and branch. We try to rectify the situation by inserting | |
| 2075 // a SafePoint here and have it dominate and kill the safepoint added at a | |
| 2076 // following backwards branch. At this point the JVM state merely holds 2 | |
| 2077 // longs but not the 3-way value. | |
| 2078 if( UseLoopSafepoints ) { | |
| 2079 switch( iter().next_bc() ) { | |
| 2080 case Bytecodes::_ifgt: | |
| 2081 case Bytecodes::_iflt: | |
| 2082 case Bytecodes::_ifge: | |
| 2083 case Bytecodes::_ifle: | |
| 2084 case Bytecodes::_ifne: | |
| 2085 case Bytecodes::_ifeq: | |
| 2086 // If this is a backwards branch in the bytecodes, add Safepoint | |
| 2087 maybe_add_safepoint(iter().next_get_dest()); | |
| 2088 } | |
| 2089 } | |
| 2090 b = pop_pair(); | |
| 2091 a = pop_pair(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
2092 c = _gvn.transform( new (C) CmpL3Node( a, b )); |
| 0 | 2093 push(c); |
| 2094 break; | |
| 2095 | |
| 2096 case Bytecodes::_lneg: | |
| 2097 a = pop_pair(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
2098 b = _gvn.transform( new (C) SubLNode(longcon(0),a)); |
| 0 | 2099 push_pair(b); |
| 2100 break; | |
| 2101 case Bytecodes::_l2i: | |
| 2102 a = pop_pair(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
2103 push( _gvn.transform( new (C) ConvL2INode(a))); |
| 0 | 2104 break; |
| 2105 case Bytecodes::_i2l: | |
| 2106 a = pop(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
2107 b = _gvn.transform( new (C) ConvI2LNode(a)); |
| 0 | 2108 push_pair(b); |
| 2109 break; | |
| 2110 case Bytecodes::_i2b: | |
| 2111 // Sign extend | |
| 2112 a = pop(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
2113 a = _gvn.transform( new (C) LShiftINode(a,_gvn.intcon(24)) ); |
|
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
2114 a = _gvn.transform( new (C) RShiftINode(a,_gvn.intcon(24)) ); |
| 0 | 2115 push( a ); |
| 2116 break; | |
| 2117 case Bytecodes::_i2s: | |
| 2118 a = pop(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
2119 a = _gvn.transform( new (C) LShiftINode(a,_gvn.intcon(16)) ); |
|
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
2120 a = _gvn.transform( new (C) RShiftINode(a,_gvn.intcon(16)) ); |
| 0 | 2121 push( a ); |
| 2122 break; | |
| 2123 case Bytecodes::_i2c: | |
| 2124 a = pop(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
2125 push( _gvn.transform( new (C) AndINode(a,_gvn.intcon(0xFFFF)) ) ); |
| 0 | 2126 break; |
| 2127 | |
| 2128 case Bytecodes::_i2f: | |
| 2129 a = pop(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
2130 b = _gvn.transform( new (C) ConvI2FNode(a) ) ; |
| 0 | 2131 c = precision_rounding(b); |
| 2132 push (b); | |
| 2133 break; | |
| 2134 | |
| 2135 case Bytecodes::_i2d: | |
| 2136 a = pop(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
2137 b = _gvn.transform( new (C) ConvI2DNode(a)); |
| 0 | 2138 push_pair(b); |
| 2139 break; | |
| 2140 | |
| 2141 case Bytecodes::_iinc: // Increment local | |
| 2142 i = iter().get_index(); // Get local index | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
2143 set_local( i, _gvn.transform( new (C) AddINode( _gvn.intcon(iter().get_iinc_con()), local(i) ) ) ); |
| 0 | 2144 break; |
| 2145 | |
| 2146 // Exit points of synchronized methods must have an unlock node | |
| 2147 case Bytecodes::_return: | |
| 2148 return_current(NULL); | |
| 2149 break; | |
| 2150 | |
| 2151 case Bytecodes::_ireturn: | |
| 2152 case Bytecodes::_areturn: | |
| 2153 case Bytecodes::_freturn: | |
| 2154 return_current(pop()); | |
| 2155 break; | |
| 2156 case Bytecodes::_lreturn: | |
| 2157 return_current(pop_pair()); | |
| 2158 break; | |
| 2159 case Bytecodes::_dreturn: | |
| 2160 return_current(pop_pair()); | |
| 2161 break; | |
| 2162 | |
| 2163 case Bytecodes::_athrow: | |
| 2164 // null exception oop throws NULL pointer exception | |
|
7194
beebba0acc11
7172640: C2: instrinsic implementations in LibraryCallKit should use argument() instead of pop()
twisti
parents:
6848
diff
changeset
|
2165 null_check(peek()); |
| 0 | 2166 if (stopped()) return; |
| 2167 // Hook the thrown exception directly to subsequent handlers. | |
| 2168 if (BailoutToInterpreterForThrows) { | |
| 2169 // Keep method interpreted from now on. | |
| 2170 uncommon_trap(Deoptimization::Reason_unhandled, | |
| 2171 Deoptimization::Action_make_not_compilable); | |
| 2172 return; | |
| 2173 } | |
|
1213
6deeaebad47a
6902182: 4/4 Starting with jdwp agent should not incur performance penalty
dcubed
parents:
1172
diff
changeset
|
2174 if (env()->jvmti_can_post_on_exceptions()) { |
|
6deeaebad47a
6902182: 4/4 Starting with jdwp agent should not incur performance penalty
dcubed
parents:
1172
diff
changeset
|
2175 // check if we must post exception events, take uncommon trap if so (with must_throw = false) |
|
6deeaebad47a
6902182: 4/4 Starting with jdwp agent should not incur performance penalty
dcubed
parents:
1172
diff
changeset
|
2176 uncommon_trap_if_should_post_on_exceptions(Deoptimization::Reason_unhandled, false); |
|
6deeaebad47a
6902182: 4/4 Starting with jdwp agent should not incur performance penalty
dcubed
parents:
1172
diff
changeset
|
2177 } |
|
6deeaebad47a
6902182: 4/4 Starting with jdwp agent should not incur performance penalty
dcubed
parents:
1172
diff
changeset
|
2178 // Here if either can_post_on_exceptions or should_post_on_exceptions is false |
| 0 | 2179 add_exception_state(make_exception_state(peek())); |
| 2180 break; | |
| 2181 | |
| 2182 case Bytecodes::_goto: // fall through | |
| 2183 case Bytecodes::_goto_w: { | |
| 2184 int target_bci = (bc() == Bytecodes::_goto) ? iter().get_dest() : iter().get_far_dest(); | |
| 2185 | |
| 2186 // If this is a backwards branch in the bytecodes, add Safepoint | |
| 2187 maybe_add_safepoint(target_bci); | |
| 2188 | |
| 2189 // Update method data | |
| 2190 profile_taken_branch(target_bci); | |
| 2191 | |
| 2192 // Merge the current control into the target basic block | |
| 2193 merge(target_bci); | |
| 2194 | |
| 2195 // See if we can get some profile data and hand it off to the next block | |
| 2196 Block *target_block = block()->successor_for_bci(target_bci); | |
| 2197 if (target_block->pred_count() != 1) break; | |
| 2198 ciMethodData* methodData = method()->method_data(); | |
| 2199 if (!methodData->is_mature()) break; | |
| 2200 ciProfileData* data = methodData->bci_to_data(bci()); | |
| 2201 assert( data->is_JumpData(), "" ); | |
| 2202 int taken = ((ciJumpData*)data)->taken(); | |
| 2203 taken = method()->scale_count(taken); | |
| 2204 target_block->set_count(taken); | |
| 2205 break; | |
| 2206 } | |
| 2207 | |
|
248
18aab3cdd513
6726504: handle do_ifxxx calls in parser more uniformly
rasbold
parents:
246
diff
changeset
|
2208 case Bytecodes::_ifnull: btest = BoolTest::eq; goto handle_if_null; |
|
18aab3cdd513
6726504: handle do_ifxxx calls in parser more uniformly
rasbold
parents:
246
diff
changeset
|
2209 case Bytecodes::_ifnonnull: btest = BoolTest::ne; goto handle_if_null; |
|
18aab3cdd513
6726504: handle do_ifxxx calls in parser more uniformly
rasbold
parents:
246
diff
changeset
|
2210 handle_if_null: |
|
254
3e333d6f35dd
6730192: expression stack wrong at deoptimization point
rasbold
parents:
248
diff
changeset
|
2211 // If this is a backwards branch in the bytecodes, add Safepoint |
|
3e333d6f35dd
6730192: expression stack wrong at deoptimization point
rasbold
parents:
248
diff
changeset
|
2212 maybe_add_safepoint(iter().get_dest()); |
|
248
18aab3cdd513
6726504: handle do_ifxxx calls in parser more uniformly
rasbold
parents:
246
diff
changeset
|
2213 a = null(); |
|
18aab3cdd513
6726504: handle do_ifxxx calls in parser more uniformly
rasbold
parents:
246
diff
changeset
|
2214 b = pop(); |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
2215 c = _gvn.transform( new (C) CmpPNode(b, a) ); |
|
248
18aab3cdd513
6726504: handle do_ifxxx calls in parser more uniformly
rasbold
parents:
246
diff
changeset
|
2216 do_ifnull(btest, c); |
| 0 | 2217 break; |
| 2218 | |
| 2219 case Bytecodes::_if_acmpeq: btest = BoolTest::eq; goto handle_if_acmp; | |
| 2220 case Bytecodes::_if_acmpne: btest = BoolTest::ne; goto handle_if_acmp; | |
| 2221 handle_if_acmp: | |
|
254
3e333d6f35dd
6730192: expression stack wrong at deoptimization point
rasbold
parents:
248
diff
changeset
|
2222 // If this is a backwards branch in the bytecodes, add Safepoint |
|
3e333d6f35dd
6730192: expression stack wrong at deoptimization point
rasbold
parents:
248
diff
changeset
|
2223 maybe_add_safepoint(iter().get_dest()); |
| 0 | 2224 a = pop(); |
| 2225 b = pop(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
2226 c = _gvn.transform( new (C) CmpPNode(b, a) ); |
| 12966 | 2227 c = optimize_cmp_with_klass(c); |
| 0 | 2228 do_if(btest, c); |
| 2229 break; | |
| 2230 | |
| 2231 case Bytecodes::_ifeq: btest = BoolTest::eq; goto handle_ifxx; | |
| 2232 case Bytecodes::_ifne: btest = BoolTest::ne; goto handle_ifxx; | |
| 2233 case Bytecodes::_iflt: btest = BoolTest::lt; goto handle_ifxx; | |
| 2234 case Bytecodes::_ifle: btest = BoolTest::le; goto handle_ifxx; | |
| 2235 case Bytecodes::_ifgt: btest = BoolTest::gt; goto handle_ifxx; | |
| 2236 case Bytecodes::_ifge: btest = BoolTest::ge; goto handle_ifxx; | |
| 2237 handle_ifxx: | |
|
254
3e333d6f35dd
6730192: expression stack wrong at deoptimization point
rasbold
parents:
248
diff
changeset
|
2238 // If this is a backwards branch in the bytecodes, add Safepoint |
|
3e333d6f35dd
6730192: expression stack wrong at deoptimization point
rasbold
parents:
248
diff
changeset
|
2239 maybe_add_safepoint(iter().get_dest()); |
| 0 | 2240 a = _gvn.intcon(0); |
| 2241 b = pop(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
2242 c = _gvn.transform( new (C) CmpINode(b, a) ); |
| 0 | 2243 do_if(btest, c); |
| 2244 break; | |
| 2245 | |
| 2246 case Bytecodes::_if_icmpeq: btest = BoolTest::eq; goto handle_if_icmp; | |
| 2247 case Bytecodes::_if_icmpne: btest = BoolTest::ne; goto handle_if_icmp; | |
| 2248 case Bytecodes::_if_icmplt: btest = BoolTest::lt; goto handle_if_icmp; | |
| 2249 case Bytecodes::_if_icmple: btest = BoolTest::le; goto handle_if_icmp; | |
| 2250 case Bytecodes::_if_icmpgt: btest = BoolTest::gt; goto handle_if_icmp; | |
| 2251 case Bytecodes::_if_icmpge: btest = BoolTest::ge; goto handle_if_icmp; | |
| 2252 handle_if_icmp: | |
|
254
3e333d6f35dd
6730192: expression stack wrong at deoptimization point
rasbold
parents:
248
diff
changeset
|
2253 // If this is a backwards branch in the bytecodes, add Safepoint |
|
3e333d6f35dd
6730192: expression stack wrong at deoptimization point
rasbold
parents:
248
diff
changeset
|
2254 maybe_add_safepoint(iter().get_dest()); |
| 0 | 2255 a = pop(); |
| 2256 b = pop(); | |
|
6804
e626685e9f6c
7193318: C2: remove number of inputs requirement from Node's new operator
kvn
parents:
6725
diff
changeset
|
2257 c = _gvn.transform( new (C) CmpINode( b, a ) ); |
| 0 | 2258 do_if(btest, c); |
| 2259 break; | |
| 2260 | |
| 2261 case Bytecodes::_tableswitch: | |
| 2262 do_tableswitch(); | |
| 2263 break; | |
| 2264 | |
| 2265 case Bytecodes::_lookupswitch: | |
| 2266 do_lookupswitch(); | |
| 2267 break; | |
| 2268 | |
| 2269 case Bytecodes::_invokestatic: | |
|
726
be93aad57795
6655646: dynamic languages need dynamically linked call sites
jrose
parents:
605
diff
changeset
|
2270 case Bytecodes::_invokedynamic: |
| 0 | 2271 case Bytecodes::_invokespecial: |
| 2272 case Bytecodes::_invokevirtual: | |
| 2273 case Bytecodes::_invokeinterface: | |
| 2274 do_call(); | |
| 2275 break; | |
| 2276 case Bytecodes::_checkcast: | |
| 2277 do_checkcast(); | |
| 2278 break; | |
| 2279 case Bytecodes::_instanceof: | |
| 2280 do_instanceof(); | |
| 2281 break; | |
| 2282 case Bytecodes::_anewarray: | |
| 2283 do_anewarray(); | |
| 2284 break; | |
| 2285 case Bytecodes::_newarray: | |
| 2286 do_newarray((BasicType)iter().get_index()); | |
| 2287 break; | |
| 2288 case Bytecodes::_multianewarray: | |
| 2289 do_multianewarray(); | |
| 2290 break; | |
| 2291 case Bytecodes::_new: | |
| 2292 do_new(); | |
| 2293 break; | |
| 2294 | |
| 2295 case Bytecodes::_jsr: | |
| 2296 case Bytecodes::_jsr_w: | |
| 2297 do_jsr(); | |
| 2298 break; | |
| 2299 | |
| 2300 case Bytecodes::_ret: | |
| 2301 do_ret(); | |
| 2302 break; | |
| 2303 | |
| 2304 | |
| 2305 case Bytecodes::_monitorenter: | |
| 2306 do_monitor_enter(); | |
| 2307 break; | |
| 2308 | |
| 2309 case Bytecodes::_monitorexit: | |
| 2310 do_monitor_exit(); | |
| 2311 break; | |
| 2312 | |
| 2313 case Bytecodes::_breakpoint: | |
| 2314 // Breakpoint set concurrently to compile | |
| 2315 // %%% use an uncommon trap? | |
| 2316 C->record_failure("breakpoint in method"); | |
| 2317 return; | |
| 2318 | |
| 2319 default: | |
| 2320 #ifndef PRODUCT | |
| 2321 map()->dump(99); | |
| 2322 #endif | |
| 2323 tty->print("\nUnhandled bytecode %s\n", Bytecodes::name(bc()) ); | |
| 2324 ShouldNotReachHere(); | |
| 2325 } | |
| 2326 | |
| 2327 #ifndef PRODUCT | |
| 2328 IdealGraphPrinter *printer = IdealGraphPrinter::printer(); | |
| 2329 if(printer) { | |
| 2330 char buffer[256]; | |
| 2331 sprintf(buffer, "Bytecode %d: %s", bci(), Bytecodes::name(bc())); | |
| 2332 bool old = printer->traverse_outs(); | |
| 2333 printer->set_traverse_outs(true); | |
| 222 | 2334 printer->print_method(C, buffer, 4); |
| 0 | 2335 printer->set_traverse_outs(old); |
| 2336 } | |
| 2337 #endif | |
| 2338 } |