hotspot/src/share/vm/opto/ifnode.cpp
author thartmann
Tue, 12 Jan 2016 12:55:09 +0100
changeset 35551 36ef3841fb34
parent 34181 663d662fdff0
child 35552 be1bff8945dc
permissions -rw-r--r--
8146629: Make phase->is_IterGVN() accessible from Node::Identity and Node::Value Summary: Change arguments of Node::Identity() and Node::Value() from PhaseTransform* to PhaseGVN*. Reviewed-by: kvn, roland
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     1
/*
32084
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 31132
diff changeset
     2
 * Copyright (c) 2000, 2015, Oracle and/or its affiliates. All rights reserved.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
489c9b5090e2 Initial load
duke
parents:
diff changeset
     4
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
489c9b5090e2 Initial load
duke
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
489c9b5090e2 Initial load
duke
parents:
diff changeset
     7
 * published by the Free Software Foundation.
489c9b5090e2 Initial load
duke
parents:
diff changeset
     8
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
489c9b5090e2 Initial load
duke
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
489c9b5090e2 Initial load
duke
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
489c9b5090e2 Initial load
duke
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
489c9b5090e2 Initial load
duke
parents:
diff changeset
    13
 * accompanied this code).
489c9b5090e2 Initial load
duke
parents:
diff changeset
    14
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
489c9b5090e2 Initial load
duke
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
489c9b5090e2 Initial load
duke
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    18
 *
5547
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 4582
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 4582
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 4582
diff changeset
    21
 * questions.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    22
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
    23
 */
489c9b5090e2 Initial load
duke
parents:
diff changeset
    24
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5901
diff changeset
    25
#include "precompiled.hpp"
34151
8e0bebdbc29e 8140574: C2 must re-execute checks after deoptimizing from merged uncommon traps
thartmann
parents: 32732
diff changeset
    26
#include "ci/ciTypeFlow.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5901
diff changeset
    27
#include "memory/allocation.inline.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5901
diff changeset
    28
#include "opto/addnode.hpp"
30183
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
    29
#include "opto/castnode.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5901
diff changeset
    30
#include "opto/cfgnode.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5901
diff changeset
    31
#include "opto/connode.hpp"
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
    32
#include "opto/loopnode.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5901
diff changeset
    33
#include "opto/phaseX.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5901
diff changeset
    34
#include "opto/runtime.hpp"
30183
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
    35
#include "opto/rootnode.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5901
diff changeset
    36
#include "opto/subnode.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5901
diff changeset
    37
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    38
// Portions of code courtesy of Clifford Click
489c9b5090e2 Initial load
duke
parents:
diff changeset
    39
489c9b5090e2 Initial load
duke
parents:
diff changeset
    40
// Optimization - Graph Style
489c9b5090e2 Initial load
duke
parents:
diff changeset
    41
489c9b5090e2 Initial load
duke
parents:
diff changeset
    42
489c9b5090e2 Initial load
duke
parents:
diff changeset
    43
extern int explicit_null_checks_elided;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    44
489c9b5090e2 Initial load
duke
parents:
diff changeset
    45
//=============================================================================
489c9b5090e2 Initial load
duke
parents:
diff changeset
    46
//------------------------------Value------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
    47
// Return a tuple for whichever arm of the IF is reachable
35551
36ef3841fb34 8146629: Make phase->is_IterGVN() accessible from Node::Identity and Node::Value
thartmann
parents: 34181
diff changeset
    48
const Type* IfNode::Value(PhaseGVN* phase) const {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    49
  if( !in(0) ) return Type::TOP;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    50
  if( phase->type(in(0)) == Type::TOP )
489c9b5090e2 Initial load
duke
parents:
diff changeset
    51
    return Type::TOP;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    52
  const Type *t = phase->type(in(1));
489c9b5090e2 Initial load
duke
parents:
diff changeset
    53
  if( t == Type::TOP )          // data is undefined
489c9b5090e2 Initial load
duke
parents:
diff changeset
    54
    return TypeTuple::IFNEITHER; // unreachable altogether
489c9b5090e2 Initial load
duke
parents:
diff changeset
    55
  if( t == TypeInt::ZERO )      // zero, or false
489c9b5090e2 Initial load
duke
parents:
diff changeset
    56
    return TypeTuple::IFFALSE;  // only false branch is reachable
489c9b5090e2 Initial load
duke
parents:
diff changeset
    57
  if( t == TypeInt::ONE )       // 1, or true
489c9b5090e2 Initial load
duke
parents:
diff changeset
    58
    return TypeTuple::IFTRUE;   // only true branch is reachable
489c9b5090e2 Initial load
duke
parents:
diff changeset
    59
  assert( t == TypeInt::BOOL, "expected boolean type" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
    60
489c9b5090e2 Initial load
duke
parents:
diff changeset
    61
  return TypeTuple::IFBOTH;     // No progress
489c9b5090e2 Initial load
duke
parents:
diff changeset
    62
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    63
489c9b5090e2 Initial load
duke
parents:
diff changeset
    64
const RegMask &IfNode::out_RegMask() const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    65
  return RegMask::Empty;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    66
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    67
489c9b5090e2 Initial load
duke
parents:
diff changeset
    68
//------------------------------split_if---------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
    69
// Look for places where we merge constants, then test on the merged value.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    70
// If the IF test will be constant folded on the path with the constant, we
489c9b5090e2 Initial load
duke
parents:
diff changeset
    71
// win by splitting the IF to before the merge point.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    72
static Node* split_if(IfNode *iff, PhaseIterGVN *igvn) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    73
  // I could be a lot more general here, but I'm trying to squeeze this
489c9b5090e2 Initial load
duke
parents:
diff changeset
    74
  // in before the Christmas '98 break so I'm gonna be kinda restrictive
489c9b5090e2 Initial load
duke
parents:
diff changeset
    75
  // on the patterns I accept.  CNC
489c9b5090e2 Initial load
duke
parents:
diff changeset
    76
489c9b5090e2 Initial load
duke
parents:
diff changeset
    77
  // Look for a compare of a constant and a merged value
489c9b5090e2 Initial load
duke
parents:
diff changeset
    78
  Node *i1 = iff->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    79
  if( !i1->is_Bool() ) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    80
  BoolNode *b = i1->as_Bool();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    81
  Node *cmp = b->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    82
  if( !cmp->is_Cmp() ) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    83
  i1 = cmp->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    84
  if( i1 == NULL || !i1->is_Phi() ) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    85
  PhiNode *phi = i1->as_Phi();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    86
  if( phi->is_copy() ) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    87
  Node *con2 = cmp->in(2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    88
  if( !con2->is_Con() ) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    89
  // See that the merge point contains some constants
489c9b5090e2 Initial load
duke
parents:
diff changeset
    90
  Node *con1=NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    91
  uint i4;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    92
  for( i4 = 1; i4 < phi->req(); i4++ ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    93
    con1 = phi->in(i4);
2131
98f9cef66a34 6810672: Comment typos
twisti
parents: 1217
diff changeset
    94
    if( !con1 ) return NULL;    // Do not optimize partially collapsed merges
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    95
    if( con1->is_Con() ) break; // Found a constant
489c9b5090e2 Initial load
duke
parents:
diff changeset
    96
    // Also allow null-vs-not-null checks
489c9b5090e2 Initial load
duke
parents:
diff changeset
    97
    const TypePtr *tp = igvn->type(con1)->isa_ptr();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    98
    if( tp && tp->_ptr == TypePtr::NotNull )
489c9b5090e2 Initial load
duke
parents:
diff changeset
    99
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   100
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   101
  if( i4 >= phi->req() ) return NULL; // Found no constants
489c9b5090e2 Initial load
duke
parents:
diff changeset
   102
489c9b5090e2 Initial load
duke
parents:
diff changeset
   103
  igvn->C->set_has_split_ifs(true); // Has chance for split-if
489c9b5090e2 Initial load
duke
parents:
diff changeset
   104
489c9b5090e2 Initial load
duke
parents:
diff changeset
   105
  // Make sure that the compare can be constant folded away
489c9b5090e2 Initial load
duke
parents:
diff changeset
   106
  Node *cmp2 = cmp->clone();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   107
  cmp2->set_req(1,con1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   108
  cmp2->set_req(2,con2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   109
  const Type *t = cmp2->Value(igvn);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   110
  // This compare is dead, so whack it!
489c9b5090e2 Initial load
duke
parents:
diff changeset
   111
  igvn->remove_dead_node(cmp2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   112
  if( !t->singleton() ) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   113
489c9b5090e2 Initial load
duke
parents:
diff changeset
   114
  // No intervening control, like a simple Call
489c9b5090e2 Initial load
duke
parents:
diff changeset
   115
  Node *r = iff->in(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   116
  if( !r->is_Region() ) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   117
  if( phi->region() != r ) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   118
  // No other users of the cmp/bool
489c9b5090e2 Initial load
duke
parents:
diff changeset
   119
  if (b->outcnt() != 1 || cmp->outcnt() != 1) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   120
    //tty->print_cr("many users of cmp/bool");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   121
    return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   122
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   123
489c9b5090e2 Initial load
duke
parents:
diff changeset
   124
  // Make sure we can determine where all the uses of merged values go
489c9b5090e2 Initial load
duke
parents:
diff changeset
   125
  for (DUIterator_Fast jmax, j = r->fast_outs(jmax); j < jmax; j++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   126
    Node* u = r->fast_out(j);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   127
    if( u == r ) continue;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   128
    if( u == iff ) continue;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   129
    if( u->outcnt() == 0 ) continue; // use is dead & ignorable
489c9b5090e2 Initial load
duke
parents:
diff changeset
   130
    if( !u->is_Phi() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   131
      /*
489c9b5090e2 Initial load
duke
parents:
diff changeset
   132
      if( u->is_Start() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   133
        tty->print_cr("Region has inlined start use");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   134
      } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   135
        tty->print_cr("Region has odd use");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   136
        u->dump(2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   137
      }*/
489c9b5090e2 Initial load
duke
parents:
diff changeset
   138
      return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   139
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   140
    if( u != phi ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   141
      // CNC - do not allow any other merged value
489c9b5090e2 Initial load
duke
parents:
diff changeset
   142
      //tty->print_cr("Merging another value");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   143
      //u->dump(2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   144
      return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   145
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   146
    // Make sure we can account for all Phi uses
489c9b5090e2 Initial load
duke
parents:
diff changeset
   147
    for (DUIterator_Fast kmax, k = u->fast_outs(kmax); k < kmax; k++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   148
      Node* v = u->fast_out(k); // User of the phi
489c9b5090e2 Initial load
duke
parents:
diff changeset
   149
      // CNC - Allow only really simple patterns.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   150
      // In particular I disallow AddP of the Phi, a fairly common pattern
28927
48e4a707e777 8071996: split_if accesses NULL region of ConstraintCast
goetz
parents: 28486
diff changeset
   151
      if (v == cmp) continue;  // The compare is OK
48e4a707e777 8071996: split_if accesses NULL region of ConstraintCast
goetz
parents: 28486
diff changeset
   152
      if (v->is_ConstraintCast()) {
48e4a707e777 8071996: split_if accesses NULL region of ConstraintCast
goetz
parents: 28486
diff changeset
   153
        // If the cast is derived from data flow edges, it may not have a control edge.
48e4a707e777 8071996: split_if accesses NULL region of ConstraintCast
goetz
parents: 28486
diff changeset
   154
        // If so, it should be safe to split. But follow-up code can not deal with
48e4a707e777 8071996: split_if accesses NULL region of ConstraintCast
goetz
parents: 28486
diff changeset
   155
        // this (l. 359). So skip.
48e4a707e777 8071996: split_if accesses NULL region of ConstraintCast
goetz
parents: 28486
diff changeset
   156
        if (v->in(0) == NULL) {
48e4a707e777 8071996: split_if accesses NULL region of ConstraintCast
goetz
parents: 28486
diff changeset
   157
          return NULL;
48e4a707e777 8071996: split_if accesses NULL region of ConstraintCast
goetz
parents: 28486
diff changeset
   158
        }
48e4a707e777 8071996: split_if accesses NULL region of ConstraintCast
goetz
parents: 28486
diff changeset
   159
        if (v->in(0)->in(0) == iff) {
48e4a707e777 8071996: split_if accesses NULL region of ConstraintCast
goetz
parents: 28486
diff changeset
   160
          continue;               // CastPP/II of the IfNode is OK
48e4a707e777 8071996: split_if accesses NULL region of ConstraintCast
goetz
parents: 28486
diff changeset
   161
        }
48e4a707e777 8071996: split_if accesses NULL region of ConstraintCast
goetz
parents: 28486
diff changeset
   162
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   163
      // Disabled following code because I cannot tell if exactly one
489c9b5090e2 Initial load
duke
parents:
diff changeset
   164
      // path dominates without a real dominator check. CNC 9/9/1999
489c9b5090e2 Initial load
duke
parents:
diff changeset
   165
      //uint vop = v->Opcode();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   166
      //if( vop == Op_Phi ) {     // Phi from another merge point might be OK
489c9b5090e2 Initial load
duke
parents:
diff changeset
   167
      //  Node *r = v->in(0);     // Get controlling point
489c9b5090e2 Initial load
duke
parents:
diff changeset
   168
      //  if( !r ) return NULL;   // Degraded to a copy
489c9b5090e2 Initial load
duke
parents:
diff changeset
   169
      //  // Find exactly one path in (either True or False doms, but not IFF)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   170
      //  int cnt = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   171
      //  for( uint i = 1; i < r->req(); i++ )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   172
      //    if( r->in(i) && r->in(i)->in(0) == iff )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   173
      //      cnt++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   174
      //  if( cnt == 1 ) continue; // Exactly one of True or False guards Phi
489c9b5090e2 Initial load
duke
parents:
diff changeset
   175
      //}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   176
      if( !v->is_Call() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   177
        /*
489c9b5090e2 Initial load
duke
parents:
diff changeset
   178
        if( v->Opcode() == Op_AddP ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   179
          tty->print_cr("Phi has AddP use");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   180
        } else if( v->Opcode() == Op_CastPP ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   181
          tty->print_cr("Phi has CastPP use");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   182
        } else if( v->Opcode() == Op_CastII ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   183
          tty->print_cr("Phi has CastII use");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   184
        } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   185
          tty->print_cr("Phi has use I cant be bothered with");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   186
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   187
        */
489c9b5090e2 Initial load
duke
parents:
diff changeset
   188
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   189
      return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   190
489c9b5090e2 Initial load
duke
parents:
diff changeset
   191
      /* CNC - Cut out all the fancy acceptance tests
489c9b5090e2 Initial load
duke
parents:
diff changeset
   192
      // Can we clone this use when doing the transformation?
489c9b5090e2 Initial load
duke
parents:
diff changeset
   193
      // If all uses are from Phis at this merge or constants, then YES.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   194
      if( !v->in(0) && v != cmp ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   195
        tty->print_cr("Phi has free-floating use");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   196
        v->dump(2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   197
        return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   198
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   199
      for( uint l = 1; l < v->req(); l++ ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   200
        if( (!v->in(l)->is_Phi() || v->in(l)->in(0) != r) &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
   201
            !v->in(l)->is_Con() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   202
          tty->print_cr("Phi has use");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   203
          v->dump(2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   204
          return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   205
        } // End of if Phi-use input is neither Phi nor Constant
489c9b5090e2 Initial load
duke
parents:
diff changeset
   206
      } // End of for all inputs to Phi-use
489c9b5090e2 Initial load
duke
parents:
diff changeset
   207
      */
489c9b5090e2 Initial load
duke
parents:
diff changeset
   208
    } // End of for all uses of Phi
489c9b5090e2 Initial load
duke
parents:
diff changeset
   209
  } // End of for all uses of Region
489c9b5090e2 Initial load
duke
parents:
diff changeset
   210
489c9b5090e2 Initial load
duke
parents:
diff changeset
   211
  // Only do this if the IF node is in a sane state
489c9b5090e2 Initial load
duke
parents:
diff changeset
   212
  if (iff->outcnt() != 2)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   213
    return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   214
489c9b5090e2 Initial load
duke
parents:
diff changeset
   215
  // Got a hit!  Do the Mondo Hack!
489c9b5090e2 Initial load
duke
parents:
diff changeset
   216
  //
489c9b5090e2 Initial load
duke
parents:
diff changeset
   217
  //ABC  a1c   def   ghi            B     1     e     h   A C   a c   d f   g i
489c9b5090e2 Initial load
duke
parents:
diff changeset
   218
  // R - Phi - Phi - Phi            Rc - Phi - Phi - Phi   Rx - Phi - Phi - Phi
489c9b5090e2 Initial load
duke
parents:
diff changeset
   219
  //     cmp - 2                         cmp - 2               cmp - 2
489c9b5090e2 Initial load
duke
parents:
diff changeset
   220
  //       bool                            bool_c                bool_x
489c9b5090e2 Initial load
duke
parents:
diff changeset
   221
  //       if                               if_c                  if_x
489c9b5090e2 Initial load
duke
parents:
diff changeset
   222
  //      T  F                              T  F                  T  F
489c9b5090e2 Initial load
duke
parents:
diff changeset
   223
  // ..s..    ..t ..                   ..s..    ..t..        ..s..    ..t..
489c9b5090e2 Initial load
duke
parents:
diff changeset
   224
  //
2131
98f9cef66a34 6810672: Comment typos
twisti
parents: 1217
diff changeset
   225
  // Split the paths coming into the merge point into 2 separate groups of
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   226
  // merges.  On the left will be all the paths feeding constants into the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   227
  // Cmp's Phi.  On the right will be the remaining paths.  The Cmp's Phi
489c9b5090e2 Initial load
duke
parents:
diff changeset
   228
  // will fold up into a constant; this will let the Cmp fold up as well as
489c9b5090e2 Initial load
duke
parents:
diff changeset
   229
  // all the control flow.  Below the original IF we have 2 control
489c9b5090e2 Initial load
duke
parents:
diff changeset
   230
  // dependent regions, 's' and 't'.  Now we will merge the two paths
489c9b5090e2 Initial load
duke
parents:
diff changeset
   231
  // just prior to 's' and 't' from the two IFs.  At least 1 path (and quite
489c9b5090e2 Initial load
duke
parents:
diff changeset
   232
  // likely 2 or more) will promptly constant fold away.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   233
  PhaseGVN *phase = igvn;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   234
489c9b5090e2 Initial load
duke
parents:
diff changeset
   235
  // Make a region merging constants and a region merging the rest
489c9b5090e2 Initial load
duke
parents:
diff changeset
   236
  uint req_c = 0;
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   237
  Node* predicate_proj = NULL;
30309
da3efc8ed2cb 8078426: mb/jvm/compiler/InterfaceCalls/testAC2 - assert(predicate_proj == 0L) failed: only one predicate entry expected
roland
parents: 30183
diff changeset
   238
  int nb_predicate_proj = 0;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   239
  for (uint ii = 1; ii < r->req(); ii++) {
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   240
    if (phi->in(ii) == con1) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   241
      req_c++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   242
    }
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   243
    Node* proj = PhaseIdealLoop::find_predicate(r->in(ii));
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   244
    if (proj != NULL) {
30309
da3efc8ed2cb 8078426: mb/jvm/compiler/InterfaceCalls/testAC2 - assert(predicate_proj == 0L) failed: only one predicate entry expected
roland
parents: 30183
diff changeset
   245
      nb_predicate_proj++;
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   246
      predicate_proj = proj;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   247
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   248
  }
30309
da3efc8ed2cb 8078426: mb/jvm/compiler/InterfaceCalls/testAC2 - assert(predicate_proj == 0L) failed: only one predicate entry expected
roland
parents: 30183
diff changeset
   249
  if (nb_predicate_proj > 1) {
da3efc8ed2cb 8078426: mb/jvm/compiler/InterfaceCalls/testAC2 - assert(predicate_proj == 0L) failed: only one predicate entry expected
roland
parents: 30183
diff changeset
   250
    // Can happen in case of loop unswitching and when the loop is
da3efc8ed2cb 8078426: mb/jvm/compiler/InterfaceCalls/testAC2 - assert(predicate_proj == 0L) failed: only one predicate entry expected
roland
parents: 30183
diff changeset
   251
    // optimized out: it's not a loop anymore so we don't care about
da3efc8ed2cb 8078426: mb/jvm/compiler/InterfaceCalls/testAC2 - assert(predicate_proj == 0L) failed: only one predicate entry expected
roland
parents: 30183
diff changeset
   252
    // predicates.
da3efc8ed2cb 8078426: mb/jvm/compiler/InterfaceCalls/testAC2 - assert(predicate_proj == 0L) failed: only one predicate entry expected
roland
parents: 30183
diff changeset
   253
    assert(!r->is_Loop(), "this must not be a loop anymore");
da3efc8ed2cb 8078426: mb/jvm/compiler/InterfaceCalls/testAC2 - assert(predicate_proj == 0L) failed: only one predicate entry expected
roland
parents: 30183
diff changeset
   254
    predicate_proj = NULL;
da3efc8ed2cb 8078426: mb/jvm/compiler/InterfaceCalls/testAC2 - assert(predicate_proj == 0L) failed: only one predicate entry expected
roland
parents: 30183
diff changeset
   255
  }
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   256
  Node* predicate_c = NULL;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   257
  Node* predicate_x = NULL;
9446
748a37b25d10 5091921: Sign flip issues in loop optimizer
kvn
parents: 9101
diff changeset
   258
  bool counted_loop = r->is_CountedLoop();
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   259
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
   260
  Node *region_c = new RegionNode(req_c + 1);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   261
  Node *phi_c    = con1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   262
  uint  len      = r->req();
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
   263
  Node *region_x = new RegionNode(len - req_c);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   264
  Node *phi_x    = PhiNode::make_blank(region_x, phi);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   265
  for (uint i = 1, i_c = 1, i_x = 1; i < len; i++) {
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   266
    if (phi->in(i) == con1) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   267
      region_c->init_req( i_c++, r  ->in(i) );
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   268
      if (r->in(i) == predicate_proj)
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   269
        predicate_c = predicate_proj;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   270
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   271
      region_x->init_req( i_x,   r  ->in(i) );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   272
      phi_x   ->init_req( i_x++, phi->in(i) );
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   273
      if (r->in(i) == predicate_proj)
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   274
        predicate_x = predicate_proj;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   275
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   276
  }
10258
10c77b8c8d3e 7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents: 10253
diff changeset
   277
  if (predicate_c != NULL && (req_c > 1)) {
10c77b8c8d3e 7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents: 10253
diff changeset
   278
    assert(predicate_x == NULL, "only one predicate entry expected");
10c77b8c8d3e 7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents: 10253
diff changeset
   279
    predicate_c = NULL; // Do not clone predicate below merge point
10c77b8c8d3e 7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents: 10253
diff changeset
   280
  }
10c77b8c8d3e 7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents: 10253
diff changeset
   281
  if (predicate_x != NULL && ((len - req_c) > 2)) {
10c77b8c8d3e 7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents: 10253
diff changeset
   282
    assert(predicate_c == NULL, "only one predicate entry expected");
10c77b8c8d3e 7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents: 10253
diff changeset
   283
    predicate_x = NULL; // Do not clone predicate below merge point
10c77b8c8d3e 7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents: 10253
diff changeset
   284
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   285
489c9b5090e2 Initial load
duke
parents:
diff changeset
   286
  // Register the new RegionNodes but do not transform them.  Cannot
2131
98f9cef66a34 6810672: Comment typos
twisti
parents: 1217
diff changeset
   287
  // transform until the entire Region/Phi conglomerate has been hacked
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   288
  // as a single huge transform.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   289
  igvn->register_new_node_with_optimizer( region_c );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   290
  igvn->register_new_node_with_optimizer( region_x );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   291
  // Prevent the untimely death of phi_x.  Currently he has no uses.  He is
489c9b5090e2 Initial load
duke
parents:
diff changeset
   292
  // about to get one.  If this only use goes away, then phi_x will look dead.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   293
  // However, he will be picking up some more uses down below.
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
   294
  Node *hook = new Node(4);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   295
  hook->init_req(0, phi_x);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   296
  hook->init_req(1, phi_c);
4016
9c11d6d519f7 6889300: assert(i != k || is_new || i->outcnt() > 0, "don't return dead nodes")
kvn
parents: 3268
diff changeset
   297
  phi_x = phase->transform( phi_x );
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   298
489c9b5090e2 Initial load
duke
parents:
diff changeset
   299
  // Make the compare
489c9b5090e2 Initial load
duke
parents:
diff changeset
   300
  Node *cmp_c = phase->makecon(t);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   301
  Node *cmp_x = cmp->clone();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   302
  cmp_x->set_req(1,phi_x);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   303
  cmp_x->set_req(2,con2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   304
  cmp_x = phase->transform(cmp_x);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   305
  // Make the bool
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
   306
  Node *b_c = phase->transform(new BoolNode(cmp_c,b->_test._test));
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
   307
  Node *b_x = phase->transform(new BoolNode(cmp_x,b->_test._test));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   308
  // Make the IfNode
34164
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
   309
  IfNode* iff_c = iff->clone()->as_If();
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
   310
  iff_c->set_req(0, region_c);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
   311
  iff_c->set_req(1, b_c);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   312
  igvn->set_type_bottom(iff_c);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   313
  igvn->_worklist.push(iff_c);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   314
  hook->init_req(2, iff_c);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   315
34164
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
   316
  IfNode* iff_x = iff->clone()->as_If();
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
   317
  iff_x->set_req(0, region_x);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
   318
  iff_x->set_req(1, b_x);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   319
  igvn->set_type_bottom(iff_x);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   320
  igvn->_worklist.push(iff_x);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   321
  hook->init_req(3, iff_x);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   322
489c9b5090e2 Initial load
duke
parents:
diff changeset
   323
  // Make the true/false arms
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
   324
  Node *iff_c_t = phase->transform(new IfTrueNode (iff_c));
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
   325
  Node *iff_c_f = phase->transform(new IfFalseNode(iff_c));
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   326
  if (predicate_c != NULL) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   327
    assert(predicate_x == NULL, "only one predicate entry expected");
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   328
    // Clone loop predicates to each path
9446
748a37b25d10 5091921: Sign flip issues in loop optimizer
kvn
parents: 9101
diff changeset
   329
    iff_c_t = igvn->clone_loop_predicates(predicate_c, iff_c_t, !counted_loop);
748a37b25d10 5091921: Sign flip issues in loop optimizer
kvn
parents: 9101
diff changeset
   330
    iff_c_f = igvn->clone_loop_predicates(predicate_c, iff_c_f, !counted_loop);
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   331
  }
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
   332
  Node *iff_x_t = phase->transform(new IfTrueNode (iff_x));
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
   333
  Node *iff_x_f = phase->transform(new IfFalseNode(iff_x));
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   334
  if (predicate_x != NULL) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   335
    assert(predicate_c == NULL, "only one predicate entry expected");
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   336
    // Clone loop predicates to each path
9446
748a37b25d10 5091921: Sign flip issues in loop optimizer
kvn
parents: 9101
diff changeset
   337
    iff_x_t = igvn->clone_loop_predicates(predicate_x, iff_x_t, !counted_loop);
748a37b25d10 5091921: Sign flip issues in loop optimizer
kvn
parents: 9101
diff changeset
   338
    iff_x_f = igvn->clone_loop_predicates(predicate_x, iff_x_f, !counted_loop);
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   339
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   340
489c9b5090e2 Initial load
duke
parents:
diff changeset
   341
  // Merge the TRUE paths
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
   342
  Node *region_s = new RegionNode(3);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   343
  igvn->_worklist.push(region_s);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   344
  region_s->init_req(1, iff_c_t);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   345
  region_s->init_req(2, iff_x_t);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   346
  igvn->register_new_node_with_optimizer( region_s );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   347
489c9b5090e2 Initial load
duke
parents:
diff changeset
   348
  // Merge the FALSE paths
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
   349
  Node *region_f = new RegionNode(3);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   350
  igvn->_worklist.push(region_f);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   351
  region_f->init_req(1, iff_c_f);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   352
  region_f->init_req(2, iff_x_f);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   353
  igvn->register_new_node_with_optimizer( region_f );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   354
489c9b5090e2 Initial load
duke
parents:
diff changeset
   355
  igvn->hash_delete(cmp);// Remove soon-to-be-dead node from hash table.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   356
  cmp->set_req(1,NULL);  // Whack the inputs to cmp because it will be dead
489c9b5090e2 Initial load
duke
parents:
diff changeset
   357
  cmp->set_req(2,NULL);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   358
  // Check for all uses of the Phi and give them a new home.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   359
  // The 'cmp' got cloned, but CastPP/IIs need to be moved.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   360
  Node *phi_s = NULL;     // do not construct unless needed
489c9b5090e2 Initial load
duke
parents:
diff changeset
   361
  Node *phi_f = NULL;     // do not construct unless needed
489c9b5090e2 Initial load
duke
parents:
diff changeset
   362
  for (DUIterator_Last i2min, i2 = phi->last_outs(i2min); i2 >= i2min; --i2) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   363
    Node* v = phi->last_out(i2);// User of the phi
12958
009b6c9586d8 7173340: C2: code cleanup: use PhaseIterGVN::replace_edge(Node*, int, Node*) where applicable
kvn
parents: 10258
diff changeset
   364
    igvn->rehash_node_delayed(v); // Have to fixup other Phi users
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   365
    uint vop = v->Opcode();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   366
    Node *proj = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   367
    if( vop == Op_Phi ) {       // Remote merge point
489c9b5090e2 Initial load
duke
parents:
diff changeset
   368
      Node *r = v->in(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   369
      for (uint i3 = 1; i3 < r->req(); i3++)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   370
        if (r->in(i3) && r->in(i3)->in(0) == iff) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   371
          proj = r->in(i3);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   372
          break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   373
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   374
    } else if( v->is_ConstraintCast() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   375
      proj = v->in(0);          // Controlling projection
489c9b5090e2 Initial load
duke
parents:
diff changeset
   376
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   377
      assert( 0, "do not know how to handle this guy" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   378
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   379
489c9b5090e2 Initial load
duke
parents:
diff changeset
   380
    Node *proj_path_data, *proj_path_ctrl;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   381
    if( proj->Opcode() == Op_IfTrue ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   382
      if( phi_s == NULL ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   383
        // Only construct phi_s if needed, otherwise provides
489c9b5090e2 Initial load
duke
parents:
diff changeset
   384
        // interfering use.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   385
        phi_s = PhiNode::make_blank(region_s,phi);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   386
        phi_s->init_req( 1, phi_c );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   387
        phi_s->init_req( 2, phi_x );
4016
9c11d6d519f7 6889300: assert(i != k || is_new || i->outcnt() > 0, "don't return dead nodes")
kvn
parents: 3268
diff changeset
   388
        hook->add_req(phi_s);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   389
        phi_s = phase->transform(phi_s);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   390
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   391
      proj_path_data = phi_s;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   392
      proj_path_ctrl = region_s;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   393
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   394
      if( phi_f == NULL ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   395
        // Only construct phi_f if needed, otherwise provides
489c9b5090e2 Initial load
duke
parents:
diff changeset
   396
        // interfering use.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   397
        phi_f = PhiNode::make_blank(region_f,phi);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   398
        phi_f->init_req( 1, phi_c );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   399
        phi_f->init_req( 2, phi_x );
4016
9c11d6d519f7 6889300: assert(i != k || is_new || i->outcnt() > 0, "don't return dead nodes")
kvn
parents: 3268
diff changeset
   400
        hook->add_req(phi_f);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   401
        phi_f = phase->transform(phi_f);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   402
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   403
      proj_path_data = phi_f;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   404
      proj_path_ctrl = region_f;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   405
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   406
489c9b5090e2 Initial load
duke
parents:
diff changeset
   407
    // Fixup 'v' for for the split
489c9b5090e2 Initial load
duke
parents:
diff changeset
   408
    if( vop == Op_Phi ) {       // Remote merge point
489c9b5090e2 Initial load
duke
parents:
diff changeset
   409
      uint i;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   410
      for( i = 1; i < v->req(); i++ )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   411
        if( v->in(i) == phi )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   412
          break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   413
      v->set_req(i, proj_path_data );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   414
    } else if( v->is_ConstraintCast() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   415
      v->set_req(0, proj_path_ctrl );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   416
      v->set_req(1, proj_path_data );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   417
    } else
489c9b5090e2 Initial load
duke
parents:
diff changeset
   418
      ShouldNotReachHere();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   419
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   420
489c9b5090e2 Initial load
duke
parents:
diff changeset
   421
  // Now replace the original iff's True/False with region_s/region_t.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   422
  // This makes the original iff go dead.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   423
  for (DUIterator_Last i3min, i3 = iff->last_outs(i3min); i3 >= i3min; --i3) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   424
    Node* p = iff->last_out(i3);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   425
    assert( p->Opcode() == Op_IfTrue || p->Opcode() == Op_IfFalse, "" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   426
    Node *u = (p->Opcode() == Op_IfTrue) ? region_s : region_f;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   427
    // Replace p with u
489c9b5090e2 Initial load
duke
parents:
diff changeset
   428
    igvn->add_users_to_worklist(p);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   429
    for (DUIterator_Last lmin, l = p->last_outs(lmin); l >= lmin;) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   430
      Node* x = p->last_out(l);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   431
      igvn->hash_delete(x);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   432
      uint uses_found = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   433
      for( uint j = 0; j < x->req(); j++ ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   434
        if( x->in(j) == p ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   435
          x->set_req(j, u);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   436
          uses_found++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   437
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   438
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   439
      l -= uses_found;    // we deleted 1 or more copies of this edge
489c9b5090e2 Initial load
duke
parents:
diff changeset
   440
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   441
    igvn->remove_dead_node(p);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   442
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   443
489c9b5090e2 Initial load
duke
parents:
diff changeset
   444
  // Force the original merge dead
489c9b5090e2 Initial load
duke
parents:
diff changeset
   445
  igvn->hash_delete(r);
3268
f034e0c86895 6851742: (EA) allocation elimination doesn't work with UseG1GC
kvn
parents: 2131
diff changeset
   446
  // First, remove region's dead users.
f034e0c86895 6851742: (EA) allocation elimination doesn't work with UseG1GC
kvn
parents: 2131
diff changeset
   447
  for (DUIterator_Last lmin, l = r->last_outs(lmin); l >= lmin;) {
f034e0c86895 6851742: (EA) allocation elimination doesn't work with UseG1GC
kvn
parents: 2131
diff changeset
   448
    Node* u = r->last_out(l);
f034e0c86895 6851742: (EA) allocation elimination doesn't work with UseG1GC
kvn
parents: 2131
diff changeset
   449
    if( u == r ) {
f034e0c86895 6851742: (EA) allocation elimination doesn't work with UseG1GC
kvn
parents: 2131
diff changeset
   450
      r->set_req(0, NULL);
f034e0c86895 6851742: (EA) allocation elimination doesn't work with UseG1GC
kvn
parents: 2131
diff changeset
   451
    } else {
f034e0c86895 6851742: (EA) allocation elimination doesn't work with UseG1GC
kvn
parents: 2131
diff changeset
   452
      assert(u->outcnt() == 0, "only dead users");
f034e0c86895 6851742: (EA) allocation elimination doesn't work with UseG1GC
kvn
parents: 2131
diff changeset
   453
      igvn->remove_dead_node(u);
f034e0c86895 6851742: (EA) allocation elimination doesn't work with UseG1GC
kvn
parents: 2131
diff changeset
   454
    }
f034e0c86895 6851742: (EA) allocation elimination doesn't work with UseG1GC
kvn
parents: 2131
diff changeset
   455
    l -= 1;
f034e0c86895 6851742: (EA) allocation elimination doesn't work with UseG1GC
kvn
parents: 2131
diff changeset
   456
  }
f034e0c86895 6851742: (EA) allocation elimination doesn't work with UseG1GC
kvn
parents: 2131
diff changeset
   457
  igvn->remove_dead_node(r);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   458
489c9b5090e2 Initial load
duke
parents:
diff changeset
   459
  // Now remove the bogus extra edges used to keep things alive
489c9b5090e2 Initial load
duke
parents:
diff changeset
   460
  igvn->remove_dead_node( hook );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   461
489c9b5090e2 Initial load
duke
parents:
diff changeset
   462
  // Must return either the original node (now dead) or a new node
489c9b5090e2 Initial load
duke
parents:
diff changeset
   463
  // (Do not return a top here, since that would break the uniqueness of top.)
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
   464
  return new ConINode(TypeInt::ZERO);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   465
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   466
30183
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   467
// if this IfNode follows a range check pattern return the projection
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   468
// for the failed path
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   469
ProjNode* IfNode::range_check_trap_proj(int& flip_test, Node*& l, Node*& r) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   470
  Node* b = in(1);
30183
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   471
  if (b == NULL || !b->is_Bool())  return NULL;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   472
  BoolNode* bn = b->as_Bool();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   473
  Node* cmp = bn->in(1);
30183
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   474
  if (cmp == NULL)  return NULL;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   475
  if (cmp->Opcode() != Op_CmpU)  return NULL;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   476
30183
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   477
  l = cmp->in(1);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   478
  r = cmp->in(2);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   479
  flip_test = 1;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   480
  if (bn->_test._test == BoolTest::le) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   481
    l = cmp->in(2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   482
    r = cmp->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   483
    flip_test = 2;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   484
  } else if (bn->_test._test != BoolTest::lt) {
30183
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   485
    return NULL;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   486
  }
30183
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   487
  if (l->is_top())  return NULL;   // Top input means dead test
34180
f0ec91019db2 8042997: Make intrinsic some or all check index/range methods
roland
parents: 34164
diff changeset
   488
  if (r->Opcode() != Op_LoadRange && !is_RangeCheck())  return NULL;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   489
489c9b5090e2 Initial load
duke
parents:
diff changeset
   490
  // We have recognized one of these forms:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   491
  //  Flip 1:  If (Bool[<] CmpU(l, LoadRange)) ...
489c9b5090e2 Initial load
duke
parents:
diff changeset
   492
  //  Flip 2:  If (Bool[<=] CmpU(LoadRange, l)) ...
489c9b5090e2 Initial load
duke
parents:
diff changeset
   493
30183
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   494
  ProjNode* iftrap = proj_out(flip_test == 2 ? true : false);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   495
  return iftrap;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   496
}
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   497
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   498
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   499
//------------------------------is_range_check---------------------------------
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   500
// Return 0 if not a range check.  Return 1 if a range check and set index and
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   501
// offset.  Return 2 if we had to negate the test.  Index is NULL if the check
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   502
// is versus a constant.
34164
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
   503
int RangeCheckNode::is_range_check(Node* &range, Node* &index, jint &offset) {
30183
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   504
  int flip_test = 0;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   505
  Node* l = NULL;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   506
  Node* r = NULL;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   507
  ProjNode* iftrap = range_check_trap_proj(flip_test, l, r);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   508
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   509
  if (iftrap == NULL) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   510
    return 0;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   511
  }
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   512
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   513
  // Make sure it's a real range check by requiring an uncommon trap
489c9b5090e2 Initial load
duke
parents:
diff changeset
   514
  // along the OOB path.  Otherwise, it's possible that the user wrote
489c9b5090e2 Initial load
duke
parents:
diff changeset
   515
  // something which optimized to look like a range check but behaves
489c9b5090e2 Initial load
duke
parents:
diff changeset
   516
  // in some other way.
30183
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   517
  if (iftrap->is_uncommon_trap_proj(Deoptimization::Reason_range_check) == NULL) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   518
    return 0;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   519
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   520
489c9b5090e2 Initial load
duke
parents:
diff changeset
   521
  // Look for index+offset form
489c9b5090e2 Initial load
duke
parents:
diff changeset
   522
  Node* ind = l;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   523
  jint  off = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   524
  if (l->is_top()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   525
    return 0;
26173
4275dfc46177 8054883: Segmentation error while running program
iveresov
parents: 24923
diff changeset
   526
  } else if (l->Opcode() == Op_AddI) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   527
    if ((off = l->in(1)->find_int_con(0)) != 0) {
34180
f0ec91019db2 8042997: Make intrinsic some or all check index/range methods
roland
parents: 34164
diff changeset
   528
      ind = l->in(2)->uncast();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   529
    } else if ((off = l->in(2)->find_int_con(0)) != 0) {
34180
f0ec91019db2 8042997: Make intrinsic some or all check index/range methods
roland
parents: 34164
diff changeset
   530
      ind = l->in(1)->uncast();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   531
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   532
  } else if ((off = l->find_int_con(-1)) >= 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   533
    // constant offset with no variable index
489c9b5090e2 Initial load
duke
parents:
diff changeset
   534
    ind = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   535
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   536
    // variable index with no constant offset (or dead negative index)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   537
    off = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   538
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   539
489c9b5090e2 Initial load
duke
parents:
diff changeset
   540
  // Return all the values:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   541
  index  = ind;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   542
  offset = off;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   543
  range  = r;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   544
  return flip_test;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   545
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   546
489c9b5090e2 Initial load
duke
parents:
diff changeset
   547
//------------------------------adjust_check-----------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   548
// Adjust (widen) a prior range check
489c9b5090e2 Initial load
duke
parents:
diff changeset
   549
static void adjust_check(Node* proj, Node* range, Node* index,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   550
                         int flip, jint off_lo, PhaseIterGVN* igvn) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   551
  PhaseGVN *gvn = igvn;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   552
  // Break apart the old check
489c9b5090e2 Initial load
duke
parents:
diff changeset
   553
  Node *iff = proj->in(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   554
  Node *bol = iff->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   555
  if( bol->is_top() ) return;   // In case a partially dead range check appears
489c9b5090e2 Initial load
duke
parents:
diff changeset
   556
  // bail (or bomb[ASSERT/DEBUG]) if NOT projection-->IfNode-->BoolNode
489c9b5090e2 Initial load
duke
parents:
diff changeset
   557
  DEBUG_ONLY( if( !bol->is_Bool() ) { proj->dump(3); fatal("Expect projection-->IfNode-->BoolNode"); } )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   558
  if( !bol->is_Bool() ) return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   559
489c9b5090e2 Initial load
duke
parents:
diff changeset
   560
  Node *cmp = bol->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   561
  // Compute a new check
489c9b5090e2 Initial load
duke
parents:
diff changeset
   562
  Node *new_add = gvn->intcon(off_lo);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   563
  if( index ) {
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
   564
    new_add = off_lo ? gvn->transform(new AddINode( index, new_add )) : index;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   565
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   566
  Node *new_cmp = (flip == 1)
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
   567
    ? new CmpUNode( new_add, range )
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
   568
    : new CmpUNode( range, new_add );
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   569
  new_cmp = gvn->transform(new_cmp);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   570
  // See if no need to adjust the existing check
489c9b5090e2 Initial load
duke
parents:
diff changeset
   571
  if( new_cmp == cmp ) return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   572
  // Else, adjust existing check
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
   573
  Node *new_bol = gvn->transform( new BoolNode( new_cmp, bol->as_Bool()->_test._test ) );
12958
009b6c9586d8 7173340: C2: code cleanup: use PhaseIterGVN::replace_edge(Node*, int, Node*) where applicable
kvn
parents: 10258
diff changeset
   574
  igvn->rehash_node_delayed( iff );
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   575
  iff->set_req_X( 1, new_bol, igvn );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   576
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   577
489c9b5090e2 Initial load
duke
parents:
diff changeset
   578
//------------------------------up_one_dom-------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   579
// Walk up the dominator tree one step.  Return NULL at root or true
489c9b5090e2 Initial load
duke
parents:
diff changeset
   580
// complex merges.  Skips through small diamonds.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   581
Node* IfNode::up_one_dom(Node *curr, bool linear_only) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   582
  Node *dom = curr->in(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   583
  if( !dom )                    // Found a Region degraded to a copy?
489c9b5090e2 Initial load
duke
parents:
diff changeset
   584
    return curr->nonnull_req(); // Skip thru it
489c9b5090e2 Initial load
duke
parents:
diff changeset
   585
489c9b5090e2 Initial load
duke
parents:
diff changeset
   586
  if( curr != dom )             // Normal walk up one step?
489c9b5090e2 Initial load
duke
parents:
diff changeset
   587
    return dom;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   588
489c9b5090e2 Initial load
duke
parents:
diff changeset
   589
  // Use linear_only if we are still parsing, since we cannot
489c9b5090e2 Initial load
duke
parents:
diff changeset
   590
  // trust the regions to be fully filled in.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   591
  if (linear_only)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   592
    return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   593
4582
1a6662d11385 6915110: IfNode::up_one_dom moves beyond RootNode bug in src/share/vm/opto/ifnode.cpp
kvn
parents: 4020
diff changeset
   594
  if( dom->is_Root() )
1a6662d11385 6915110: IfNode::up_one_dom moves beyond RootNode bug in src/share/vm/opto/ifnode.cpp
kvn
parents: 4020
diff changeset
   595
    return NULL;
1a6662d11385 6915110: IfNode::up_one_dom moves beyond RootNode bug in src/share/vm/opto/ifnode.cpp
kvn
parents: 4020
diff changeset
   596
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   597
  // Else hit a Region.  Check for a loop header
489c9b5090e2 Initial load
duke
parents:
diff changeset
   598
  if( dom->is_Loop() )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   599
    return dom->in(1);          // Skip up thru loops
489c9b5090e2 Initial load
duke
parents:
diff changeset
   600
489c9b5090e2 Initial load
duke
parents:
diff changeset
   601
  // Check for small diamonds
489c9b5090e2 Initial load
duke
parents:
diff changeset
   602
  Node *din1, *din2, *din3, *din4;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   603
  if( dom->req() == 3 &&        // 2-path merge point
489c9b5090e2 Initial load
duke
parents:
diff changeset
   604
      (din1 = dom ->in(1)) &&   // Left  path exists
489c9b5090e2 Initial load
duke
parents:
diff changeset
   605
      (din2 = dom ->in(2)) &&   // Right path exists
489c9b5090e2 Initial load
duke
parents:
diff changeset
   606
      (din3 = din1->in(0)) &&   // Left  path up one
489c9b5090e2 Initial load
duke
parents:
diff changeset
   607
      (din4 = din2->in(0)) ) {  // Right path up one
489c9b5090e2 Initial load
duke
parents:
diff changeset
   608
    if( din3->is_Call() &&      // Handle a slow-path call on either arm
489c9b5090e2 Initial load
duke
parents:
diff changeset
   609
        (din3 = din3->in(0)) )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   610
      din3 = din3->in(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   611
    if( din4->is_Call() &&      // Handle a slow-path call on either arm
489c9b5090e2 Initial load
duke
parents:
diff changeset
   612
        (din4 = din4->in(0)) )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   613
      din4 = din4->in(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   614
    if( din3 == din4 && din3->is_If() )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   615
      return din3;              // Skip around diamonds
489c9b5090e2 Initial load
duke
parents:
diff changeset
   616
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   617
489c9b5090e2 Initial load
duke
parents:
diff changeset
   618
  // Give up the search at true merges
489c9b5090e2 Initial load
duke
parents:
diff changeset
   619
  return NULL;                  // Dead loop?  Or hit root?
489c9b5090e2 Initial load
duke
parents:
diff changeset
   620
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   621
190
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   622
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   623
//------------------------------filtered_int_type--------------------------------
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   624
// Return a possibly more restrictive type for val based on condition control flow for an if
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   625
const TypeInt* IfNode::filtered_int_type(PhaseGVN* gvn, Node *val, Node* if_proj) {
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   626
  assert(if_proj &&
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   627
         (if_proj->Opcode() == Op_IfTrue || if_proj->Opcode() == Op_IfFalse), "expecting an if projection");
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   628
  if (if_proj->in(0) && if_proj->in(0)->is_If()) {
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   629
    IfNode* iff = if_proj->in(0)->as_If();
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   630
    if (iff->in(1) && iff->in(1)->is_Bool()) {
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   631
      BoolNode* bol = iff->in(1)->as_Bool();
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   632
      if (bol->in(1) && bol->in(1)->is_Cmp()) {
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   633
        const CmpNode* cmp  = bol->in(1)->as_Cmp();
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   634
        if (cmp->in(1) == val) {
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   635
          const TypeInt* cmp2_t = gvn->type(cmp->in(2))->isa_int();
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   636
          if (cmp2_t != NULL) {
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   637
            jint lo = cmp2_t->_lo;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   638
            jint hi = cmp2_t->_hi;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   639
            BoolTest::mask msk = if_proj->Opcode() == Op_IfTrue ? bol->_test._test : bol->_test.negate();
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   640
            switch (msk) {
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   641
            case BoolTest::ne:
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   642
              // Can't refine type
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   643
              return NULL;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   644
            case BoolTest::eq:
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   645
              return cmp2_t;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   646
            case BoolTest::lt:
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   647
              lo = TypeInt::INT->_lo;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   648
              if (hi - 1 < hi) {
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   649
                hi = hi - 1;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   650
              }
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   651
              break;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   652
            case BoolTest::le:
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   653
              lo = TypeInt::INT->_lo;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   654
              break;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   655
            case BoolTest::gt:
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   656
              if (lo + 1 > lo) {
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   657
                lo = lo + 1;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   658
              }
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   659
              hi = TypeInt::INT->_hi;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   660
              break;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   661
            case BoolTest::ge:
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   662
              // lo unchanged
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   663
              hi = TypeInt::INT->_hi;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   664
              break;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   665
            }
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   666
            const TypeInt* rtn_t = TypeInt::make(lo, hi, cmp2_t->_widen);
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   667
            return rtn_t;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   668
          }
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   669
        }
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   670
      }
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   671
    }
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   672
  }
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   673
  return NULL;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   674
}
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   675
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   676
//------------------------------fold_compares----------------------------
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   677
// See if a pair of CmpIs can be converted into a CmpU.  In some cases
2131
98f9cef66a34 6810672: Comment typos
twisti
parents: 1217
diff changeset
   678
// the direction of this if is determined by the preceding if so it
30183
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   679
// can be eliminate entirely.
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   680
//
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   681
// Given an if testing (CmpI n v) check for an immediately control
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   682
// dependent if that is testing (CmpI n v2) and has one projection
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   683
// leading to this if and the other projection leading to a region
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   684
// that merges one of this ifs control projections.
190
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   685
//
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   686
//                   If
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   687
//                  / |
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   688
//                 /  |
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   689
//                /   |
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   690
//              If    |
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   691
//              /\    |
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   692
//             /  \   |
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   693
//            /    \  |
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   694
//           /    Region
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   695
//
30183
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   696
// Or given an if testing (CmpI n v) check for a dominating if that is
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   697
// testing (CmpI n v2), both having one projection leading to an
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   698
// uncommon trap. Allow Another independent guard in between to cover
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   699
// an explicit range check:
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   700
// if (index < 0 || index >= array.length) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   701
// which may need a null check to guard the LoadRange
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   702
//
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   703
//                   If
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   704
//                  / \
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   705
//                 /   \
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   706
//                /     \
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   707
//              If      unc
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   708
//              /\
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   709
//             /  \
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   710
//            /    \
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   711
//           /      unc
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   712
//
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   713
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   714
// Is the comparison for this If suitable for folding?
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   715
bool IfNode::cmpi_folds(PhaseIterGVN* igvn) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   716
  return in(1) != NULL &&
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   717
    in(1)->is_Bool() &&
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   718
    in(1)->in(1) != NULL &&
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   719
    in(1)->in(1)->Opcode() == Op_CmpI &&
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   720
    in(1)->in(1)->in(2) != NULL &&
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   721
    in(1)->in(1)->in(2) != igvn->C->top() &&
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   722
    (in(1)->as_Bool()->_test.is_less() ||
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   723
     in(1)->as_Bool()->_test.is_greater());
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   724
}
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   725
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   726
// Is a dominating control suitable for folding with this if?
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   727
bool IfNode::is_ctrl_folds(Node* ctrl, PhaseIterGVN* igvn) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   728
  return ctrl != NULL &&
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   729
    ctrl->is_Proj() &&
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   730
    ctrl->in(0) != NULL &&
34164
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
   731
    ctrl->in(0)->Opcode() == Op_If &&
30183
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   732
    ctrl->in(0)->outcnt() == 2 &&
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   733
    ctrl->in(0)->as_If()->cmpi_folds(igvn) &&
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   734
    // Must compare same value
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   735
    ctrl->in(0)->in(1)->in(1)->in(1) != NULL &&
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   736
    ctrl->in(0)->in(1)->in(1)->in(1) == in(1)->in(1)->in(1);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   737
}
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   738
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   739
// Do this If and the dominating If share a region?
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   740
bool IfNode::has_shared_region(ProjNode* proj, ProjNode*& success, ProjNode*& fail) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   741
  ProjNode* otherproj = proj->other_if_proj();
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   742
  Node* otherproj_ctrl_use = otherproj->unique_ctrl_out();
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   743
  RegionNode* region = (otherproj_ctrl_use != NULL && otherproj_ctrl_use->is_Region()) ? otherproj_ctrl_use->as_Region() : NULL;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   744
  success = NULL;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   745
  fail = NULL;
190
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   746
30183
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   747
  if (otherproj->outcnt() == 1 && region != NULL && !region->has_phi()) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   748
    for (int i = 0; i < 2; i++) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   749
      ProjNode* proj = proj_out(i);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   750
      if (success == NULL && proj->outcnt() == 1 && proj->unique_out() == region) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   751
        success = proj;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   752
      } else if (fail == NULL) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   753
        fail = proj;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   754
      } else {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   755
        success = fail = NULL;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   756
      }
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   757
    }
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   758
  }
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   759
  return success != NULL && fail != NULL;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   760
}
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   761
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   762
// Return projection that leads to an uncommon trap if any
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   763
ProjNode* IfNode::uncommon_trap_proj(CallStaticJavaNode*& call) const {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   764
  for (int i = 0; i < 2; i++) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   765
    call = proj_out(i)->is_uncommon_trap_proj(Deoptimization::Reason_none);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   766
    if (call != NULL) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   767
      return proj_out(i);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   768
    }
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   769
  }
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   770
  return NULL;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   771
}
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   772
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   773
// Do this If and the dominating If both branch out to an uncommon trap
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   774
bool IfNode::has_only_uncommon_traps(ProjNode* proj, ProjNode*& success, ProjNode*& fail, PhaseIterGVN* igvn) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   775
  ProjNode* otherproj = proj->other_if_proj();
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   776
  CallStaticJavaNode* dom_unc = otherproj->is_uncommon_trap_proj(Deoptimization::Reason_none);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   777
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   778
  if (otherproj->outcnt() == 1 && dom_unc != NULL) {
34151
8e0bebdbc29e 8140574: C2 must re-execute checks after deoptimizing from merged uncommon traps
thartmann
parents: 32732
diff changeset
   779
    // We need to re-execute the folded Ifs after deoptimization from the merged traps
8e0bebdbc29e 8140574: C2 must re-execute checks after deoptimizing from merged uncommon traps
thartmann
parents: 32732
diff changeset
   780
    if (!dom_unc->jvms()->should_reexecute()) {
8e0bebdbc29e 8140574: C2 must re-execute checks after deoptimizing from merged uncommon traps
thartmann
parents: 32732
diff changeset
   781
      return false;
8e0bebdbc29e 8140574: C2 must re-execute checks after deoptimizing from merged uncommon traps
thartmann
parents: 32732
diff changeset
   782
    }
8e0bebdbc29e 8140574: C2 must re-execute checks after deoptimizing from merged uncommon traps
thartmann
parents: 32732
diff changeset
   783
30183
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   784
    CallStaticJavaNode* unc = NULL;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   785
    ProjNode* unc_proj = uncommon_trap_proj(unc);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   786
    if (unc_proj != NULL && unc_proj->outcnt() == 1) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   787
      if (dom_unc == unc) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   788
        // Allow the uncommon trap to be shared through a region
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   789
        RegionNode* r = unc->in(0)->as_Region();
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   790
        if (r->outcnt() != 2 || r->req() != 3 || r->find_edge(otherproj) == -1 || r->find_edge(unc_proj) == -1) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   791
          return false;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   792
        }
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   793
        assert(r->has_phi() == NULL, "simple region shouldn't have a phi");
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   794
      } else if (dom_unc->in(0) != otherproj || unc->in(0) != unc_proj) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   795
        return false;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   796
      }
34151
8e0bebdbc29e 8140574: C2 must re-execute checks after deoptimizing from merged uncommon traps
thartmann
parents: 32732
diff changeset
   797
8e0bebdbc29e 8140574: C2 must re-execute checks after deoptimizing from merged uncommon traps
thartmann
parents: 32732
diff changeset
   798
      // Different methods and methods containing jsrs are not supported.
8e0bebdbc29e 8140574: C2 must re-execute checks after deoptimizing from merged uncommon traps
thartmann
parents: 32732
diff changeset
   799
      ciMethod* method = unc->jvms()->method();
8e0bebdbc29e 8140574: C2 must re-execute checks after deoptimizing from merged uncommon traps
thartmann
parents: 32732
diff changeset
   800
      ciMethod* dom_method = dom_unc->jvms()->method();
8e0bebdbc29e 8140574: C2 must re-execute checks after deoptimizing from merged uncommon traps
thartmann
parents: 32732
diff changeset
   801
      if (method != dom_method || method->has_jsrs()) {
8e0bebdbc29e 8140574: C2 must re-execute checks after deoptimizing from merged uncommon traps
thartmann
parents: 32732
diff changeset
   802
        return false;
8e0bebdbc29e 8140574: C2 must re-execute checks after deoptimizing from merged uncommon traps
thartmann
parents: 32732
diff changeset
   803
      }
8e0bebdbc29e 8140574: C2 must re-execute checks after deoptimizing from merged uncommon traps
thartmann
parents: 32732
diff changeset
   804
      // Check that both traps are in the same activation of the method (instead
8e0bebdbc29e 8140574: C2 must re-execute checks after deoptimizing from merged uncommon traps
thartmann
parents: 32732
diff changeset
   805
      // of two activations being inlined through different call sites) by verifying
8e0bebdbc29e 8140574: C2 must re-execute checks after deoptimizing from merged uncommon traps
thartmann
parents: 32732
diff changeset
   806
      // that the call stacks are equal for both JVMStates.
8e0bebdbc29e 8140574: C2 must re-execute checks after deoptimizing from merged uncommon traps
thartmann
parents: 32732
diff changeset
   807
      JVMState* dom_caller = dom_unc->jvms()->caller();
8e0bebdbc29e 8140574: C2 must re-execute checks after deoptimizing from merged uncommon traps
thartmann
parents: 32732
diff changeset
   808
      JVMState* caller = unc->jvms()->caller();
34177
e19e45b065c6 8142500: missing null checks in IfNode::has_only_uncommon_traps
thartmann
parents: 34164
diff changeset
   809
      if ((dom_caller == NULL) != (caller == NULL)) {
e19e45b065c6 8142500: missing null checks in IfNode::has_only_uncommon_traps
thartmann
parents: 34164
diff changeset
   810
        // The current method must either be inlined into both dom_caller and
e19e45b065c6 8142500: missing null checks in IfNode::has_only_uncommon_traps
thartmann
parents: 34164
diff changeset
   811
        // caller or must not be inlined at all (top method). Bail out otherwise.
e19e45b065c6 8142500: missing null checks in IfNode::has_only_uncommon_traps
thartmann
parents: 34164
diff changeset
   812
        return false;
e19e45b065c6 8142500: missing null checks in IfNode::has_only_uncommon_traps
thartmann
parents: 34164
diff changeset
   813
      } else if (dom_caller != NULL && !dom_caller->same_calls_as(caller)) {
34151
8e0bebdbc29e 8140574: C2 must re-execute checks after deoptimizing from merged uncommon traps
thartmann
parents: 32732
diff changeset
   814
        return false;
8e0bebdbc29e 8140574: C2 must re-execute checks after deoptimizing from merged uncommon traps
thartmann
parents: 32732
diff changeset
   815
      }
8e0bebdbc29e 8140574: C2 must re-execute checks after deoptimizing from merged uncommon traps
thartmann
parents: 32732
diff changeset
   816
      // Check that the bci of the dominating uncommon trap dominates the bci
8e0bebdbc29e 8140574: C2 must re-execute checks after deoptimizing from merged uncommon traps
thartmann
parents: 32732
diff changeset
   817
      // of the dominated uncommon trap. Otherwise we may not re-execute
8e0bebdbc29e 8140574: C2 must re-execute checks after deoptimizing from merged uncommon traps
thartmann
parents: 32732
diff changeset
   818
      // the dominated check after deoptimization from the merged uncommon trap.
8e0bebdbc29e 8140574: C2 must re-execute checks after deoptimizing from merged uncommon traps
thartmann
parents: 32732
diff changeset
   819
      ciTypeFlow* flow = dom_method->get_flow_analysis();
8e0bebdbc29e 8140574: C2 must re-execute checks after deoptimizing from merged uncommon traps
thartmann
parents: 32732
diff changeset
   820
      int bci = unc->jvms()->bci();
8e0bebdbc29e 8140574: C2 must re-execute checks after deoptimizing from merged uncommon traps
thartmann
parents: 32732
diff changeset
   821
      int dom_bci = dom_unc->jvms()->bci();
8e0bebdbc29e 8140574: C2 must re-execute checks after deoptimizing from merged uncommon traps
thartmann
parents: 32732
diff changeset
   822
      if (!flow->is_dominated_by(bci, dom_bci)) {
8e0bebdbc29e 8140574: C2 must re-execute checks after deoptimizing from merged uncommon traps
thartmann
parents: 32732
diff changeset
   823
        return false;
8e0bebdbc29e 8140574: C2 must re-execute checks after deoptimizing from merged uncommon traps
thartmann
parents: 32732
diff changeset
   824
      }
8e0bebdbc29e 8140574: C2 must re-execute checks after deoptimizing from merged uncommon traps
thartmann
parents: 32732
diff changeset
   825
30183
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   826
      // See merge_uncommon_traps: the reason of the uncommon trap
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   827
      // will be changed and the state of the dominating If will be
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   828
      // used. Checked that we didn't apply this transformation in a
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   829
      // previous compilation and it didn't cause too many traps
34151
8e0bebdbc29e 8140574: C2 must re-execute checks after deoptimizing from merged uncommon traps
thartmann
parents: 32732
diff changeset
   830
      if (!igvn->C->too_many_traps(dom_method, dom_bci, Deoptimization::Reason_unstable_fused_if) &&
8e0bebdbc29e 8140574: C2 must re-execute checks after deoptimizing from merged uncommon traps
thartmann
parents: 32732
diff changeset
   831
          !igvn->C->too_many_traps(dom_method, dom_bci, Deoptimization::Reason_range_check)) {
30183
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   832
        success = unc_proj;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   833
        fail = unc_proj->other_if_proj();
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   834
        return true;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   835
      }
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   836
    }
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   837
  }
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   838
  return false;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   839
}
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   840
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   841
// Check that the 2 CmpI can be folded into as single CmpU and proceed with the folding
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   842
bool IfNode::fold_compares_helper(ProjNode* proj, ProjNode* success, ProjNode* fail, PhaseIterGVN* igvn) {
190
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   843
  Node* this_cmp = in(1)->in(1);
30183
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   844
  BoolNode* this_bool = in(1)->as_Bool();
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   845
  IfNode* dom_iff = proj->in(0)->as_If();
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   846
  BoolNode* dom_bool = dom_iff->in(1)->as_Bool();
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   847
  Node* lo = dom_iff->in(1)->in(1)->in(2);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   848
  Node* hi = this_cmp->in(2);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   849
  Node* n = this_cmp->in(1);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   850
  ProjNode* otherproj = proj->other_if_proj();
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   851
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   852
  const TypeInt* lo_type = IfNode::filtered_int_type(igvn, n, otherproj);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   853
  const TypeInt* hi_type = IfNode::filtered_int_type(igvn, n, success);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   854
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   855
  BoolTest::mask lo_test = dom_bool->_test._test;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   856
  BoolTest::mask hi_test = this_bool->_test._test;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   857
  BoolTest::mask cond = hi_test;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   858
31132
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   859
  // convert:
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   860
  //
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   861
  //          dom_bool = x {<,<=,>,>=} a
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   862
  //                           / \
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   863
  //     proj = {True,False}  /   \ otherproj = {False,True}
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   864
  //                         /
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   865
  //        this_bool = x {<,<=} b
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   866
  //                       / \
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   867
  //  fail = {True,False} /   \ success = {False,True}
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   868
  //                     /
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   869
  //
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   870
  // (Second test guaranteed canonicalized, first one may not have
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   871
  // been canonicalized yet)
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   872
  //
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   873
  // into:
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   874
  //
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   875
  // cond = (x - lo) {<u,<=u,>u,>=u} adjusted_lim
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   876
  //                       / \
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   877
  //                 fail /   \ success
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   878
  //                     /
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   879
  //
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   880
30183
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   881
  // Figure out which of the two tests sets the upper bound and which
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   882
  // sets the lower bound if any.
31132
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   883
  Node* adjusted_lim = NULL;
30183
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   884
  if (hi_type->_lo > lo_type->_hi && hi_type->_hi == max_jint && lo_type->_lo == min_jint) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   885
    assert((dom_bool->_test.is_less() && !proj->_con) ||
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   886
           (dom_bool->_test.is_greater() && proj->_con), "incorrect test");
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   887
    // this test was canonicalized
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   888
    assert(this_bool->_test.is_less() && fail->_con, "incorrect test");
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   889
31132
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   890
    // this_bool = <
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   891
    //   dom_bool = >= (proj = True) or dom_bool = < (proj = False)
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   892
    //     x in [a, b[ on the fail (= True) projection, b > a-1 (because of hi_type->_lo > lo_type->_hi test above):
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   893
    //     lo = a, hi = b, adjusted_lim = b-a, cond = <u
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   894
    //   dom_bool = > (proj = True) or dom_bool = <= (proj = False)
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   895
    //     x in ]a, b[ on the fail (= True) projection, b > a:
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   896
    //     lo = a+1, hi = b, adjusted_lim = b-a-1, cond = <u
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   897
    // this_bool = <=
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   898
    //   dom_bool = >= (proj = True) or dom_bool = < (proj = False)
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   899
    //     x in [a, b] on the fail (= True) projection, b+1 > a-1:
32732
9f4cf1523072 8135069: C2 replaces range checks by unsigned comparison with -1
roland
parents: 32084
diff changeset
   900
    //     lo = a, hi = b, adjusted_lim = b-a+1, cond = <u
9f4cf1523072 8135069: C2 replaces range checks by unsigned comparison with -1
roland
parents: 32084
diff changeset
   901
    //     lo = a, hi = b, adjusted_lim = b-a, cond = <=u doesn't work because b = a - 1 is possible, then b-a = -1
31132
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   902
    //   dom_bool = > (proj = True) or dom_bool = <= (proj = False)
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   903
    //     x in ]a, b] on the fail (= True) projection b+1 > a:
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   904
    //     lo = a+1, hi = b, adjusted_lim = b-a, cond = <u
32732
9f4cf1523072 8135069: C2 replaces range checks by unsigned comparison with -1
roland
parents: 32084
diff changeset
   905
    //     lo = a+1, hi = b, adjusted_lim = b-a-1, cond = <=u doesn't work because a = b is possible, then b-a-1 = -1
31132
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   906
32732
9f4cf1523072 8135069: C2 replaces range checks by unsigned comparison with -1
roland
parents: 32084
diff changeset
   907
    if (hi_test == BoolTest::lt) {
9f4cf1523072 8135069: C2 replaces range checks by unsigned comparison with -1
roland
parents: 32084
diff changeset
   908
      if (lo_test == BoolTest::gt || lo_test == BoolTest::le) {
9f4cf1523072 8135069: C2 replaces range checks by unsigned comparison with -1
roland
parents: 32084
diff changeset
   909
        lo = igvn->transform(new AddINode(lo, igvn->intcon(1)));
9f4cf1523072 8135069: C2 replaces range checks by unsigned comparison with -1
roland
parents: 32084
diff changeset
   910
      }
9f4cf1523072 8135069: C2 replaces range checks by unsigned comparison with -1
roland
parents: 32084
diff changeset
   911
    } else {
9f4cf1523072 8135069: C2 replaces range checks by unsigned comparison with -1
roland
parents: 32084
diff changeset
   912
      assert(hi_test == BoolTest::le, "bad test");
9f4cf1523072 8135069: C2 replaces range checks by unsigned comparison with -1
roland
parents: 32084
diff changeset
   913
      if (lo_test == BoolTest::ge || lo_test == BoolTest::lt) {
31132
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   914
        adjusted_lim = igvn->transform(new SubINode(hi, lo));
32732
9f4cf1523072 8135069: C2 replaces range checks by unsigned comparison with -1
roland
parents: 32084
diff changeset
   915
        adjusted_lim = igvn->transform(new AddINode(adjusted_lim, igvn->intcon(1)));
9f4cf1523072 8135069: C2 replaces range checks by unsigned comparison with -1
roland
parents: 32084
diff changeset
   916
        cond = BoolTest::lt;
9f4cf1523072 8135069: C2 replaces range checks by unsigned comparison with -1
roland
parents: 32084
diff changeset
   917
      } else {
9f4cf1523072 8135069: C2 replaces range checks by unsigned comparison with -1
roland
parents: 32084
diff changeset
   918
        assert(lo_test == BoolTest::gt || lo_test == BoolTest::le, "bad test");
9f4cf1523072 8135069: C2 replaces range checks by unsigned comparison with -1
roland
parents: 32084
diff changeset
   919
        adjusted_lim = igvn->transform(new SubINode(hi, lo));
9f4cf1523072 8135069: C2 replaces range checks by unsigned comparison with -1
roland
parents: 32084
diff changeset
   920
        lo = igvn->transform(new AddINode(lo, igvn->intcon(1)));
31132
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   921
        cond = BoolTest::lt;
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   922
      }
30183
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   923
    }
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   924
  } else if (lo_type->_lo > hi_type->_hi && lo_type->_hi == max_jint && hi_type->_lo == min_jint) {
31132
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   925
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   926
    // this_bool = <
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   927
    //   dom_bool = < (proj = True) or dom_bool = >= (proj = False)
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   928
    //     x in [b, a[ on the fail (= False) projection, a > b-1 (because of lo_type->_lo > hi_type->_hi above):
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   929
    //     lo = b, hi = a, adjusted_lim = a-b, cond = >=u
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   930
    //   dom_bool = <= (proj = True) or dom_bool = > (proj = False)
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   931
    //     x in [b, a] on the fail (= False) projection, a+1 > b-1:
32732
9f4cf1523072 8135069: C2 replaces range checks by unsigned comparison with -1
roland
parents: 32084
diff changeset
   932
    //     lo = b, hi = a, adjusted_lim = a-b+1, cond = >=u
9f4cf1523072 8135069: C2 replaces range checks by unsigned comparison with -1
roland
parents: 32084
diff changeset
   933
    //     lo = b, hi = a, adjusted_lim = a-b, cond = >u doesn't work because a = b - 1 is possible, then b-a = -1
31132
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   934
    // this_bool = <=
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   935
    //   dom_bool = < (proj = True) or dom_bool = >= (proj = False)
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   936
    //     x in ]b, a[ on the fail (= False) projection, a > b:
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   937
    //     lo = b+1, hi = a, adjusted_lim = a-b-1, cond = >=u
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   938
    //   dom_bool = <= (proj = True) or dom_bool = > (proj = False)
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   939
    //     x in ]b, a] on the fail (= False) projection, a+1 > b:
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   940
    //     lo = b+1, hi = a, adjusted_lim = a-b, cond = >=u
32732
9f4cf1523072 8135069: C2 replaces range checks by unsigned comparison with -1
roland
parents: 32084
diff changeset
   941
    //     lo = b+1, hi = a, adjusted_lim = a-b-1, cond = >u doesn't work because a = b is possible, then b-a-1 = -1
31132
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   942
30183
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   943
    swap(lo, hi);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   944
    swap(lo_type, hi_type);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   945
    swap(lo_test, hi_test);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   946
30631
483c444f36f4 8078436: java/util/stream/boottest/java/util/stream/UnorderedTest.java crashed with an assert in ifnode.cpp
roland
parents: 30629
diff changeset
   947
    assert((dom_bool->_test.is_less() && proj->_con) ||
483c444f36f4 8078436: java/util/stream/boottest/java/util/stream/UnorderedTest.java crashed with an assert in ifnode.cpp
roland
parents: 30629
diff changeset
   948
           (dom_bool->_test.is_greater() && !proj->_con), "incorrect test");
30183
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   949
    // this test was canonicalized
30631
483c444f36f4 8078436: java/util/stream/boottest/java/util/stream/UnorderedTest.java crashed with an assert in ifnode.cpp
roland
parents: 30629
diff changeset
   950
    assert(this_bool->_test.is_less() && !fail->_con, "incorrect test");
30183
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   951
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   952
    cond = (hi_test == BoolTest::le || hi_test == BoolTest::gt) ? BoolTest::gt : BoolTest::ge;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   953
32732
9f4cf1523072 8135069: C2 replaces range checks by unsigned comparison with -1
roland
parents: 32084
diff changeset
   954
    if (lo_test == BoolTest::lt) {
9f4cf1523072 8135069: C2 replaces range checks by unsigned comparison with -1
roland
parents: 32084
diff changeset
   955
      if (hi_test == BoolTest::lt || hi_test == BoolTest::ge) {
9f4cf1523072 8135069: C2 replaces range checks by unsigned comparison with -1
roland
parents: 32084
diff changeset
   956
        cond = BoolTest::ge;
9f4cf1523072 8135069: C2 replaces range checks by unsigned comparison with -1
roland
parents: 32084
diff changeset
   957
      } else {
9f4cf1523072 8135069: C2 replaces range checks by unsigned comparison with -1
roland
parents: 32084
diff changeset
   958
        assert(hi_test == BoolTest::le || hi_test == BoolTest::gt, "bad test");
31132
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   959
        adjusted_lim = igvn->transform(new SubINode(hi, lo));
32732
9f4cf1523072 8135069: C2 replaces range checks by unsigned comparison with -1
roland
parents: 32084
diff changeset
   960
        adjusted_lim = igvn->transform(new AddINode(adjusted_lim, igvn->intcon(1)));
31132
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   961
        cond = BoolTest::ge;
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   962
      }
32732
9f4cf1523072 8135069: C2 replaces range checks by unsigned comparison with -1
roland
parents: 32084
diff changeset
   963
    } else if (lo_test == BoolTest::le) {
9f4cf1523072 8135069: C2 replaces range checks by unsigned comparison with -1
roland
parents: 32084
diff changeset
   964
      if (hi_test == BoolTest::lt || hi_test == BoolTest::ge) {
9f4cf1523072 8135069: C2 replaces range checks by unsigned comparison with -1
roland
parents: 32084
diff changeset
   965
        lo = igvn->transform(new AddINode(lo, igvn->intcon(1)));
9f4cf1523072 8135069: C2 replaces range checks by unsigned comparison with -1
roland
parents: 32084
diff changeset
   966
        cond = BoolTest::ge;
9f4cf1523072 8135069: C2 replaces range checks by unsigned comparison with -1
roland
parents: 32084
diff changeset
   967
      } else {
9f4cf1523072 8135069: C2 replaces range checks by unsigned comparison with -1
roland
parents: 32084
diff changeset
   968
        assert(hi_test == BoolTest::le || hi_test == BoolTest::gt, "bad test");
9f4cf1523072 8135069: C2 replaces range checks by unsigned comparison with -1
roland
parents: 32084
diff changeset
   969
        adjusted_lim = igvn->transform(new SubINode(hi, lo));
9f4cf1523072 8135069: C2 replaces range checks by unsigned comparison with -1
roland
parents: 32084
diff changeset
   970
        lo = igvn->transform(new AddINode(lo, igvn->intcon(1)));
9f4cf1523072 8135069: C2 replaces range checks by unsigned comparison with -1
roland
parents: 32084
diff changeset
   971
        cond = BoolTest::ge;
9f4cf1523072 8135069: C2 replaces range checks by unsigned comparison with -1
roland
parents: 32084
diff changeset
   972
      }
30183
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   973
    }
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   974
  } else {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   975
    const TypeInt* failtype  = filtered_int_type(igvn, n, proj);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   976
    if (failtype != NULL) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   977
      const TypeInt* type2 = filtered_int_type(igvn, n, fail);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   978
      if (type2 != NULL) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   979
        failtype = failtype->join(type2)->is_int();
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   980
        if (failtype->_lo > failtype->_hi) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   981
          // previous if determines the result of this if so
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   982
          // replace Bool with constant
34164
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
   983
          igvn->_worklist.push(in(1));
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
   984
          igvn->replace_input_of(this, 1, igvn->intcon(success->_con));
30183
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   985
          return true;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   986
        }
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   987
      }
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   988
    }
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   989
    lo = NULL;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   990
    hi = NULL;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   991
  }
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   992
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   993
  if (lo && hi) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   994
    // Merge the two compares into a single unsigned compare by building (CmpU (n - lo) (hi - lo))
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   995
    Node* adjusted_val = igvn->transform(new SubINode(n,  lo));
31132
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   996
    if (adjusted_lim == NULL) {
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   997
      adjusted_lim = igvn->transform(new SubINode(hi, lo));
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
   998
    }
30183
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
   999
    Node* newcmp = igvn->transform(new CmpUNode(adjusted_val, adjusted_lim));
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1000
    Node* newbool = igvn->transform(new BoolNode(newcmp, cond));
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1001
31132
328ac96a30d6 8081823: C2 performs unsigned comparison against -1
roland
parents: 30631
diff changeset
  1002
    igvn->replace_input_of(dom_iff, 1, igvn->intcon(proj->_con));
34164
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1003
    igvn->_worklist.push(in(1));
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1004
    igvn->replace_input_of(this, 1, newbool);
30183
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1005
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1006
    return true;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1007
  }
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1008
  return false;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1009
}
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1010
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1011
// Merge the branches that trap for this If and the dominating If into
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1012
// a single region that branches to the uncommon trap for the
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1013
// dominating If
34164
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1014
Node* IfNode::merge_uncommon_traps(ProjNode* proj, ProjNode* success, ProjNode* fail, PhaseIterGVN* igvn) {
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1015
  Node* res = this;
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1016
  assert(success->in(0) == this, "bad projection");
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1017
30183
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1018
  ProjNode* otherproj = proj->other_if_proj();
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1019
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1020
  CallStaticJavaNode* unc = success->is_uncommon_trap_proj(Deoptimization::Reason_none);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1021
  CallStaticJavaNode* dom_unc = otherproj->is_uncommon_trap_proj(Deoptimization::Reason_none);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1022
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1023
  if (unc != dom_unc) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1024
    Node* r = new RegionNode(3);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1025
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1026
    r->set_req(1, otherproj);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1027
    r->set_req(2, success);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1028
    r = igvn->transform(r);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1029
    assert(r->is_Region(), "can't go away");
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1030
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1031
    // Make both If trap at the state of the first If: once the CmpI
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1032
    // nodes are merged, if we trap we don't know which of the CmpI
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1033
    // nodes would have caused the trap so we have to restart
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1034
    // execution at the first one
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1035
    igvn->replace_input_of(dom_unc, 0, r);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1036
    igvn->replace_input_of(unc, 0, igvn->C->top());
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1037
  }
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1038
  int trap_request = dom_unc->uncommon_trap_request();
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1039
  Deoptimization::DeoptReason reason = Deoptimization::trap_request_reason(trap_request);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1040
  Deoptimization::DeoptAction action = Deoptimization::trap_request_action(trap_request);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1041
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1042
  int flip_test = 0;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1043
  Node* l = NULL;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1044
  Node* r = NULL;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1045
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1046
  if (success->in(0)->as_If()->range_check_trap_proj(flip_test, l, r) != NULL) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1047
    // If this looks like a range check, change the trap to
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1048
    // Reason_range_check so the compiler recognizes it as a range
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1049
    // check and applies the corresponding optimizations
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1050
    trap_request = Deoptimization::make_trap_request(Deoptimization::Reason_range_check, action);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1051
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1052
    improve_address_types(l, r, fail, igvn);
34164
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1053
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1054
    res = igvn->transform(new RangeCheckNode(in(0), in(1), _prob, _fcnt));
30183
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1055
  } else if (unc != dom_unc) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1056
    // If we trap we won't know what CmpI would have caused the trap
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1057
    // so use a special trap reason to mark this pair of CmpI nodes as
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1058
    // bad candidate for folding. On recompilation we won't fold them
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1059
    // and we may trap again but this time we'll know what branch
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1060
    // traps
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1061
    trap_request = Deoptimization::make_trap_request(Deoptimization::Reason_unstable_fused_if, action);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1062
  }
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1063
  igvn->replace_input_of(dom_unc, TypeFunc::Parms, igvn->intcon(trap_request));
34164
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1064
  return res;
30183
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1065
}
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1066
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1067
// If we are turning 2 CmpI nodes into a CmpU that follows the pattern
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1068
// of a rangecheck on index i, on 64 bit the compares may be followed
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1069
// by memory accesses using i as index. In that case, the CmpU tells
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1070
// us something about the values taken by i that can help the compiler
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1071
// (see Compile::conv_I2X_index())
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1072
void IfNode::improve_address_types(Node* l, Node* r, ProjNode* fail, PhaseIterGVN* igvn) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1073
#ifdef _LP64
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1074
  ResourceMark rm;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1075
  Node_Stack stack(2);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1076
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1077
  assert(r->Opcode() == Op_LoadRange, "unexpected range check");
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1078
  const TypeInt* array_size = igvn->type(r)->is_int();
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1079
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1080
  stack.push(l, 0);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1081
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1082
  while(stack.size() > 0) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1083
    Node* n = stack.node();
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1084
    uint start = stack.index();
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1085
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1086
    uint i = start;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1087
    for (; i < n->outcnt(); i++) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1088
      Node* use = n->raw_out(i);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1089
      if (stack.size() == 1) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1090
        if (use->Opcode() == Op_ConvI2L) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1091
          const TypeLong* bounds = use->as_Type()->type()->is_long();
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1092
          if (bounds->_lo <= array_size->_lo && bounds->_hi >= array_size->_hi &&
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1093
              (bounds->_lo != array_size->_lo || bounds->_hi != array_size->_hi)) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1094
            stack.set_index(i+1);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1095
            stack.push(use, 0);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1096
            break;
190
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
  1097
          }
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
  1098
        }
30183
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1099
      } else if (use->is_Mem()) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1100
        Node* ctrl = use->in(0);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1101
        for (int i = 0; i < 10 && ctrl != NULL && ctrl != fail; i++) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1102
          ctrl = up_one_dom(ctrl);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1103
        }
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1104
        if (ctrl == fail) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1105
          Node* init_n = stack.node_at(1);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1106
          assert(init_n->Opcode() == Op_ConvI2L, "unexpected first node");
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1107
          Node* new_n = igvn->C->conv_I2X_index(igvn, l, array_size);
190
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
  1108
30629
b6e5ad2f18d5 8076188: Optimize arraycopy out for non escaping destination
roland
parents: 30309
diff changeset
  1109
          // The type of the ConvI2L may be widen and so the new
b6e5ad2f18d5 8076188: Optimize arraycopy out for non escaping destination
roland
parents: 30309
diff changeset
  1110
          // ConvI2L may not be better than an existing ConvI2L
b6e5ad2f18d5 8076188: Optimize arraycopy out for non escaping destination
roland
parents: 30309
diff changeset
  1111
          if (new_n != init_n) {
b6e5ad2f18d5 8076188: Optimize arraycopy out for non escaping destination
roland
parents: 30309
diff changeset
  1112
            for (uint j = 2; j < stack.size(); j++) {
b6e5ad2f18d5 8076188: Optimize arraycopy out for non escaping destination
roland
parents: 30309
diff changeset
  1113
              Node* n = stack.node_at(j);
b6e5ad2f18d5 8076188: Optimize arraycopy out for non escaping destination
roland
parents: 30309
diff changeset
  1114
              Node* clone = n->clone();
b6e5ad2f18d5 8076188: Optimize arraycopy out for non escaping destination
roland
parents: 30309
diff changeset
  1115
              int rep = clone->replace_edge(init_n, new_n);
b6e5ad2f18d5 8076188: Optimize arraycopy out for non escaping destination
roland
parents: 30309
diff changeset
  1116
              assert(rep > 0, "can't find expected node?");
b6e5ad2f18d5 8076188: Optimize arraycopy out for non escaping destination
roland
parents: 30309
diff changeset
  1117
              clone = igvn->transform(clone);
b6e5ad2f18d5 8076188: Optimize arraycopy out for non escaping destination
roland
parents: 30309
diff changeset
  1118
              init_n = n;
b6e5ad2f18d5 8076188: Optimize arraycopy out for non escaping destination
roland
parents: 30309
diff changeset
  1119
              new_n = clone;
b6e5ad2f18d5 8076188: Optimize arraycopy out for non escaping destination
roland
parents: 30309
diff changeset
  1120
            }
b6e5ad2f18d5 8076188: Optimize arraycopy out for non escaping destination
roland
parents: 30309
diff changeset
  1121
            igvn->hash_delete(use);
b6e5ad2f18d5 8076188: Optimize arraycopy out for non escaping destination
roland
parents: 30309
diff changeset
  1122
            int rep = use->replace_edge(init_n, new_n);
30183
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1123
            assert(rep > 0, "can't find expected node?");
30629
b6e5ad2f18d5 8076188: Optimize arraycopy out for non escaping destination
roland
parents: 30309
diff changeset
  1124
            igvn->transform(use);
b6e5ad2f18d5 8076188: Optimize arraycopy out for non escaping destination
roland
parents: 30309
diff changeset
  1125
            if (init_n->outcnt() == 0) {
b6e5ad2f18d5 8076188: Optimize arraycopy out for non escaping destination
roland
parents: 30309
diff changeset
  1126
              igvn->_worklist.push(init_n);
b6e5ad2f18d5 8076188: Optimize arraycopy out for non escaping destination
roland
parents: 30309
diff changeset
  1127
            }
190
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
  1128
          }
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
  1129
        }
30183
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1130
      } else if (use->in(0) == NULL && (igvn->type(use)->isa_long() ||
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1131
                                        igvn->type(use)->isa_ptr())) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1132
        stack.set_index(i+1);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1133
        stack.push(use, 0);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1134
        break;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1135
      }
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1136
    }
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1137
    if (i == n->outcnt()) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1138
      stack.pop();
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1139
    }
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1140
  }
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1141
#endif
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1142
}
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1143
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1144
bool IfNode::is_cmp_with_loadrange(ProjNode* proj) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1145
  if (in(1) != NULL &&
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1146
      in(1)->in(1) != NULL &&
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1147
      in(1)->in(1)->in(2) != NULL) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1148
    Node* other = in(1)->in(1)->in(2);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1149
    if (other->Opcode() == Op_LoadRange &&
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1150
        ((other->in(0) != NULL && other->in(0) == proj) ||
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1151
         (other->in(0) == NULL &&
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1152
          other->in(2) != NULL &&
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1153
          other->in(2)->is_AddP() &&
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1154
          other->in(2)->in(1) != NULL &&
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1155
          other->in(2)->in(1)->Opcode() == Op_CastPP &&
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1156
          other->in(2)->in(1)->in(0) == proj))) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1157
      return true;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1158
    }
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1159
  }
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1160
  return false;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1161
}
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1162
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1163
bool IfNode::is_null_check(ProjNode* proj, PhaseIterGVN* igvn) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1164
  Node* other = in(1)->in(1)->in(2);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1165
  if (other->in(MemNode::Address) != NULL &&
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1166
      proj->in(0)->in(1) != NULL &&
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1167
      proj->in(0)->in(1)->is_Bool() &&
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1168
      proj->in(0)->in(1)->in(1) != NULL &&
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1169
      proj->in(0)->in(1)->in(1)->Opcode() == Op_CmpP &&
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1170
      proj->in(0)->in(1)->in(1)->in(2) != NULL &&
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1171
      proj->in(0)->in(1)->in(1)->in(1) == other->in(MemNode::Address)->in(AddPNode::Address)->uncast() &&
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1172
      igvn->type(proj->in(0)->in(1)->in(1)->in(2)) == TypePtr::NULL_PTR) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1173
    return true;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1174
  }
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1175
  return false;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1176
}
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1177
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1178
// Check that the If that is in between the 2 integer comparisons has
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1179
// no side effect
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1180
bool IfNode::is_side_effect_free_test(ProjNode* proj, PhaseIterGVN* igvn) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1181
  if (proj != NULL &&
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1182
      proj->is_uncommon_trap_if_pattern(Deoptimization::Reason_none) &&
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1183
      proj->outcnt() <= 2) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1184
    if (proj->outcnt() == 1 ||
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1185
        // Allow simple null check from LoadRange
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1186
        (is_cmp_with_loadrange(proj) && is_null_check(proj, igvn))) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1187
      CallStaticJavaNode* unc = proj->is_uncommon_trap_if_pattern(Deoptimization::Reason_none);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1188
      CallStaticJavaNode* dom_unc = proj->in(0)->in(0)->as_Proj()->is_uncommon_trap_if_pattern(Deoptimization::Reason_none);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1189
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1190
      // reroute_side_effect_free_unc changes the state of this
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1191
      // uncommon trap to restart execution at the previous
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1192
      // CmpI. Check that this change in a previous compilation didn't
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1193
      // cause too many traps.
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1194
      int trap_request = unc->uncommon_trap_request();
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1195
      Deoptimization::DeoptReason reason = Deoptimization::trap_request_reason(trap_request);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1196
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1197
      if (igvn->C->too_many_traps(dom_unc->jvms()->method(), dom_unc->jvms()->bci(), reason)) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1198
        return false;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1199
      }
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1200
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1201
      return true;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1202
    }
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1203
  }
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1204
  return false;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1205
}
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1206
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1207
// Make the If between the 2 integer comparisons trap at the state of
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1208
// the first If: the last CmpI is the one replaced by a CmpU and the
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1209
// first CmpI is eliminated, so the test between the 2 CmpI nodes
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1210
// won't be guarded by the first CmpI anymore. It can trap in cases
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1211
// where the first CmpI would have prevented it from executing: on a
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1212
// trap, we need to restart execution at the state of the first CmpI
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1213
void IfNode::reroute_side_effect_free_unc(ProjNode* proj, ProjNode* dom_proj, PhaseIterGVN* igvn) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1214
  CallStaticJavaNode* dom_unc = dom_proj->is_uncommon_trap_if_pattern(Deoptimization::Reason_none);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1215
  ProjNode* otherproj = proj->other_if_proj();
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1216
  CallStaticJavaNode* unc = proj->is_uncommon_trap_if_pattern(Deoptimization::Reason_none);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1217
  Node* call_proj = dom_unc->unique_ctrl_out();
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1218
  Node* halt = call_proj->unique_ctrl_out();
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1219
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1220
  Node* new_unc = dom_unc->clone();
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1221
  call_proj = call_proj->clone();
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1222
  halt = halt->clone();
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1223
  Node* c = otherproj->clone();
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1224
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1225
  c = igvn->transform(c);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1226
  new_unc->set_req(TypeFunc::Parms, unc->in(TypeFunc::Parms));
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1227
  new_unc->set_req(0, c);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1228
  new_unc = igvn->transform(new_unc);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1229
  call_proj->set_req(0, new_unc);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1230
  call_proj = igvn->transform(call_proj);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1231
  halt->set_req(0, call_proj);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1232
  halt = igvn->transform(halt);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1233
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1234
  igvn->replace_node(otherproj, igvn->C->top());
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1235
  igvn->C->root()->add_req(halt);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1236
}
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1237
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1238
Node* IfNode::fold_compares(PhaseIterGVN* igvn) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1239
  if (Opcode() != Op_If) return NULL;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1240
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1241
  if (cmpi_folds(igvn)) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1242
    Node* ctrl = in(0);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1243
    if (is_ctrl_folds(ctrl, igvn) &&
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1244
        ctrl->outcnt() == 1) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1245
      // A integer comparison immediately dominated by another integer
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1246
      // comparison
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1247
      ProjNode* success = NULL;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1248
      ProjNode* fail = NULL;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1249
      ProjNode* dom_cmp = ctrl->as_Proj();
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1250
      if (has_shared_region(dom_cmp, success, fail) &&
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1251
          // Next call modifies graph so must be last
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1252
          fold_compares_helper(dom_cmp, success, fail, igvn)) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1253
        return this;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1254
      }
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1255
      if (has_only_uncommon_traps(dom_cmp, success, fail, igvn) &&
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1256
          // Next call modifies graph so must be last
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1257
          fold_compares_helper(dom_cmp, success, fail, igvn)) {
34164
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1258
        return merge_uncommon_traps(dom_cmp, success, fail, igvn);
30183
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1259
      }
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1260
      return NULL;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1261
    } else if (ctrl->in(0) != NULL &&
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1262
               ctrl->in(0)->in(0) != NULL) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1263
      ProjNode* success = NULL;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1264
      ProjNode* fail = NULL;
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1265
      Node* dom = ctrl->in(0)->in(0);
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1266
      ProjNode* dom_cmp = dom->isa_Proj();
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1267
      ProjNode* other_cmp = ctrl->isa_Proj();
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1268
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1269
      // Check if it's an integer comparison dominated by another
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1270
      // integer comparison with another test in between
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1271
      if (is_ctrl_folds(dom, igvn) &&
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1272
          has_only_uncommon_traps(dom_cmp, success, fail, igvn) &&
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1273
          is_side_effect_free_test(other_cmp, igvn) &&
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1274
          // Next call modifies graph so must be last
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1275
          fold_compares_helper(dom_cmp, success, fail, igvn)) {
a6588c0a3259 8073480: C2 should optimize explicit range checks
roland
parents: 28927
diff changeset
  1276
        reroute_side_effect_free_unc(other_cmp, dom_cmp, igvn);
34164
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1277
        return merge_uncommon_traps(dom_cmp, success, fail, igvn);
190
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
  1278
      }
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
  1279
    }
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
  1280
  }
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
  1281
  return NULL;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
  1282
}
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
  1283
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1284
//------------------------------remove_useless_bool----------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1285
// Check for people making a useless boolean: things like
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1286
// if( (x < y ? true : false) ) { ... }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1287
// Replace with if( x < y ) { ... }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1288
static Node *remove_useless_bool(IfNode *iff, PhaseGVN *phase) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1289
  Node *i1 = iff->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1290
  if( !i1->is_Bool() ) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1291
  BoolNode *bol = i1->as_Bool();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1292
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1293
  Node *cmp = bol->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1294
  if( cmp->Opcode() != Op_CmpI ) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1295
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1296
  // Must be comparing against a bool
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1297
  const Type *cmp2_t = phase->type( cmp->in(2) );
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1298
  if( cmp2_t != TypeInt::ZERO &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1299
      cmp2_t != TypeInt::ONE )
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1300
    return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1301
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1302
  // Find a prior merge point merging the boolean
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1303
  i1 = cmp->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1304
  if( !i1->is_Phi() ) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1305
  PhiNode *phi = i1->as_Phi();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1306
  if( phase->type( phi ) != TypeInt::BOOL )
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1307
    return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1308
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1309
  // Check for diamond pattern
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1310
  int true_path = phi->is_diamond_phi();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1311
  if( true_path == 0 ) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1312
958
4c4709e8b7ee 6712835: Server compiler fails with assertion (loop_count < K,"infinite loop in PhaseIterGVN::transform")
never
parents: 190
diff changeset
  1313
  // Make sure that iff and the control of the phi are different. This
4c4709e8b7ee 6712835: Server compiler fails with assertion (loop_count < K,"infinite loop in PhaseIterGVN::transform")
never
parents: 190
diff changeset
  1314
  // should really only happen for dead control flow since it requires
4c4709e8b7ee 6712835: Server compiler fails with assertion (loop_count < K,"infinite loop in PhaseIterGVN::transform")
never
parents: 190
diff changeset
  1315
  // an illegal cycle.
4c4709e8b7ee 6712835: Server compiler fails with assertion (loop_count < K,"infinite loop in PhaseIterGVN::transform")
never
parents: 190
diff changeset
  1316
  if (phi->in(0)->in(1)->in(0) == iff) return NULL;
4c4709e8b7ee 6712835: Server compiler fails with assertion (loop_count < K,"infinite loop in PhaseIterGVN::transform")
never
parents: 190
diff changeset
  1317
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1318
  // phi->region->if_proj->ifnode->bool->cmp
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1319
  BoolNode *bol2 = phi->in(0)->in(1)->in(0)->in(1)->as_Bool();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1320
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1321
  // Now get the 'sense' of the test correct so we can plug in
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1322
  // either iff2->in(1) or its complement.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1323
  int flip = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1324
  if( bol->_test._test == BoolTest::ne ) flip = 1-flip;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1325
  else if( bol->_test._test != BoolTest::eq ) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1326
  if( cmp2_t == TypeInt::ZERO ) flip = 1-flip;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1327
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1328
  const Type *phi1_t = phase->type( phi->in(1) );
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1329
  const Type *phi2_t = phase->type( phi->in(2) );
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1330
  // Check for Phi(0,1) and flip
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1331
  if( phi1_t == TypeInt::ZERO ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1332
    if( phi2_t != TypeInt::ONE ) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1333
    flip = 1-flip;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1334
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1335
    // Check for Phi(1,0)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1336
    if( phi1_t != TypeInt::ONE  ) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1337
    if( phi2_t != TypeInt::ZERO ) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1338
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1339
  if( true_path == 2 ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1340
    flip = 1-flip;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1341
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1342
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1343
  Node* new_bol = (flip ? phase->transform( bol2->negate(phase) ) : bol2);
958
4c4709e8b7ee 6712835: Server compiler fails with assertion (loop_count < K,"infinite loop in PhaseIterGVN::transform")
never
parents: 190
diff changeset
  1344
  assert(new_bol != iff->in(1), "must make progress");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1345
  iff->set_req(1, new_bol);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1346
  // Intervening diamond probably goes dead
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1347
  phase->C->set_major_progress();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1348
  return iff;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1349
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1350
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1351
static IfNode* idealize_test(PhaseGVN* phase, IfNode* iff);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1352
28044
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1353
struct RangeCheck {
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1354
  Node* ctl;
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1355
  jint off;
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1356
};
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1357
34164
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1358
Node* IfNode::Ideal_common(PhaseGVN *phase, bool can_reshape) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1359
  if (remove_dead_region(phase, can_reshape))  return this;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1360
  // No Def-Use info?
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1361
  if (!can_reshape)  return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1362
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1363
  // Don't bother trying to transform a dead if
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1364
  if (in(0)->is_top())  return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1365
  // Don't bother trying to transform an if with a dead test
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1366
  if (in(1)->is_top())  return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1367
  // Another variation of a dead test
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1368
  if (in(1)->is_Con())  return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1369
  // Another variation of a dead if
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1370
  if (outcnt() < 2)  return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1371
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1372
  // Canonicalize the test.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1373
  Node* idt_if = idealize_test(phase, this);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1374
  if (idt_if != NULL)  return idt_if;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1375
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1376
  // Try to split the IF
34164
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1377
  PhaseIterGVN *igvn = phase->is_IterGVN();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1378
  Node *s = split_if(this, igvn);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1379
  if (s != NULL)  return s;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1380
34164
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1381
  return NodeSentinel;
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1382
}
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1383
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1384
//------------------------------Ideal------------------------------------------
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1385
// Return a node which is more "ideal" than the current node.  Strip out
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1386
// control copies
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1387
Node* IfNode::Ideal(PhaseGVN *phase, bool can_reshape) {
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1388
  Node* res = Ideal_common(phase, can_reshape);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1389
  if (res != NodeSentinel) {
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1390
    return res;
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1391
  }
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1392
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1393
  // Check for people making a useless boolean: things like
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1394
  // if( (x < y ? true : false) ) { ... }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1395
  // Replace with if( x < y ) { ... }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1396
  Node *bol2 = remove_useless_bool(this, phase);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1397
  if( bol2 ) return bol2;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1398
34164
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1399
  if (in(0) == NULL) return NULL;     // Dead loop?
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1400
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1401
  PhaseIterGVN *igvn = phase->is_IterGVN();
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1402
  Node* result = fold_compares(igvn);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1403
  if (result != NULL) {
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1404
    return result;
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1405
  }
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1406
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1407
  // Scan for an equivalent test
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1408
  Node *cmp;
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1409
  int dist = 0;               // Cutoff limit for search
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1410
  int op = Opcode();
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1411
  if( op == Op_If &&
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1412
      (cmp=in(1)->in(1))->Opcode() == Op_CmpP ) {
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1413
    if( cmp->in(2) != NULL && // make sure cmp is not already dead
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1414
        cmp->in(2)->bottom_type() == TypePtr::NULL_PTR ) {
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1415
      dist = 64;              // Limit for null-pointer scans
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1416
    } else {
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1417
      dist = 4;               // Do not bother for random pointer tests
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1418
    }
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1419
  } else {
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1420
    dist = 4;                 // Limit for random junky scans
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1421
  }
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1422
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1423
  Node* prev_dom = search_identical(dist);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1424
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1425
  if (prev_dom == NULL) {
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1426
    return NULL;
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1427
  }
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1428
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1429
  // Replace dominated IfNode
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1430
  return dominated_by(prev_dom, igvn);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1431
}
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1432
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1433
//------------------------------dominated_by-----------------------------------
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1434
Node* IfNode::dominated_by(Node* prev_dom, PhaseIterGVN *igvn) {
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1435
#ifndef PRODUCT
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1436
  if (TraceIterativeGVN) {
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1437
    tty->print("   Removing IfNode: "); this->dump();
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1438
  }
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1439
  if (VerifyOpto && !igvn->allow_progress()) {
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1440
    // Found an equivalent dominating test,
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1441
    // we can not guarantee reaching a fix-point for these during iterativeGVN
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1442
    // since intervening nodes may not change.
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1443
    return NULL;
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1444
  }
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1445
#endif
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1446
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1447
  igvn->hash_delete(this);      // Remove self to prevent spurious V-N
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1448
  Node *idom = in(0);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1449
  // Need opcode to decide which way 'this' test goes
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1450
  int prev_op = prev_dom->Opcode();
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1451
  Node *top = igvn->C->top(); // Shortcut to top
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1452
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1453
  // Loop predicates may have depending checks which should not
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1454
  // be skipped. For example, range check predicate has two checks
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1455
  // for lower and upper bounds.
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1456
  ProjNode* unc_proj = proj_out(1 - prev_dom->as_Proj()->_con)->as_Proj();
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1457
  if (unc_proj->is_uncommon_trap_proj(Deoptimization::Reason_predicate) != NULL)
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1458
   prev_dom = idom;
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1459
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1460
  // Now walk the current IfNode's projections.
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1461
  // Loop ends when 'this' has no more uses.
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1462
  for (DUIterator_Last imin, i = last_outs(imin); i >= imin; --i) {
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1463
    Node *ifp = last_out(i);     // Get IfTrue/IfFalse
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1464
    igvn->add_users_to_worklist(ifp);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1465
    // Check which projection it is and set target.
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1466
    // Data-target is either the dominating projection of the same type
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1467
    // or TOP if the dominating projection is of opposite type.
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1468
    // Data-target will be used as the new control edge for the non-CFG
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1469
    // nodes like Casts and Loads.
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1470
    Node *data_target = (ifp->Opcode() == prev_op) ? prev_dom : top;
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1471
    // Control-target is just the If's immediate dominator or TOP.
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1472
    Node *ctrl_target = (ifp->Opcode() == prev_op) ?     idom : top;
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1473
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1474
    // For each child of an IfTrue/IfFalse projection, reroute.
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1475
    // Loop ends when projection has no more uses.
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1476
    for (DUIterator_Last jmin, j = ifp->last_outs(jmin); j >= jmin; --j) {
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1477
      Node* s = ifp->last_out(j);   // Get child of IfTrue/IfFalse
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1478
      if( !s->depends_only_on_test() ) {
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1479
        // Find the control input matching this def-use edge.
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1480
        // For Regions it may not be in slot 0.
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1481
        uint l;
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1482
        for( l = 0; s->in(l) != ifp; l++ ) { }
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1483
        igvn->replace_input_of(s, l, ctrl_target);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1484
      } else {                      // Else, for control producers,
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1485
        igvn->replace_input_of(s, 0, data_target); // Move child to data-target
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1486
      }
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1487
    } // End for each child of a projection
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1488
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1489
    igvn->remove_dead_node(ifp);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1490
  } // End for each IfTrue/IfFalse child of If
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1491
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1492
  // Kill the IfNode
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1493
  igvn->remove_dead_node(this);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1494
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1495
  // Must return either the original node (now dead) or a new node
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1496
  // (Do not return a top here, since that would break the uniqueness of top.)
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1497
  return new ConINode(TypeInt::ZERO);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1498
}
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1499
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1500
Node* IfNode::search_identical(int dist) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1501
  // Setup to scan up the CFG looking for a dominating test
34164
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1502
  Node* dom = in(0);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1503
  Node* prev_dom = this;
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1504
  int op = Opcode();
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1505
  // Search up the dominator tree for an If with an identical test
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1506
  while( dom->Opcode() != op    ||  // Not same opcode?
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1507
         dom->in(1)    != in(1) ||  // Not same input 1?
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1508
         (req() == 3 && dom->in(2) != in(2)) || // Not same input 2?
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1509
         prev_dom->in(0) != dom ) { // One path of test does not dominate?
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1510
    if( dist < 0 ) return NULL;
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1511
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1512
    dist--;
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1513
    prev_dom = dom;
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1514
    dom = up_one_dom( dom );
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1515
    if( !dom ) return NULL;
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1516
  }
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1517
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1518
  // Check that we did not follow a loop back to ourselves
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1519
  if( this == dom )
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1520
    return NULL;
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1521
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1522
  if( dist > 2 )              // Add to count of NULL checks elided
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1523
    explicit_null_checks_elided++;
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1524
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1525
  return prev_dom;
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1526
}
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1527
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1528
//------------------------------Identity---------------------------------------
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1529
// If the test is constant & we match, then we are the input Control
35551
36ef3841fb34 8146629: Make phase->is_IterGVN() accessible from Node::Identity and Node::Value
thartmann
parents: 34181
diff changeset
  1530
Node* IfProjNode::Identity(PhaseGVN* phase) {
34164
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1531
  // Can only optimize if cannot go the other way
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1532
  const TypeTuple *t = phase->type(in(0))->is_tuple();
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1533
  if (t == TypeTuple::IFNEITHER ||
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1534
      // kill dead branch first otherwise the IfNode's control will
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1535
      // have 2 control uses (the IfNode that doesn't go away because
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1536
      // it still has uses and this branch of the
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1537
      // If). Node::has_special_unique_user() will cause this node to
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1538
      // be reprocessed once the dead branch is killed.
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1539
      (always_taken(t) && in(0)->outcnt() == 1)) {
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1540
    // IfNode control
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1541
    return in(0)->in(0);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1542
  }
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1543
  // no progress
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1544
  return this;
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1545
}
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1546
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1547
#ifndef PRODUCT
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1548
//-------------------------------related---------------------------------------
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1549
// An IfProjNode's related node set consists of its input (an IfNode) including
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1550
// the IfNode's condition, plus all of its outputs at level 1. In compact mode,
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1551
// the restrictions for IfNode apply (see IfNode::rel).
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1552
void IfProjNode::related(GrowableArray<Node*> *in_rel, GrowableArray<Node*> *out_rel, bool compact) const {
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1553
  Node* ifNode = this->in(0);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1554
  in_rel->append(ifNode);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1555
  if (compact) {
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1556
    ifNode->collect_nodes(in_rel, 3, false, true);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1557
  } else {
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1558
    ifNode->collect_nodes_in_all_data(in_rel, false);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1559
  }
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1560
  this->collect_nodes(out_rel, -1, false, false);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1561
}
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1562
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1563
//------------------------------dump_spec--------------------------------------
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1564
void IfNode::dump_spec(outputStream *st) const {
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1565
  st->print("P=%f, C=%f",_prob,_fcnt);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1566
}
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1567
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1568
//-------------------------------related---------------------------------------
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1569
// For an IfNode, the set of related output nodes is just the output nodes till
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1570
// depth 2, i.e, the IfTrue/IfFalse projection nodes plus the nodes they refer.
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1571
// The related input nodes contain no control nodes, but all data nodes
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1572
// pertaining to the condition. In compact mode, the input nodes are collected
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1573
// up to a depth of 3.
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1574
void IfNode::related(GrowableArray <Node *> *in_rel, GrowableArray <Node *> *out_rel, bool compact) const {
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1575
  if (compact) {
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1576
    this->collect_nodes(in_rel, 3, false, true);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1577
  } else {
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1578
    this->collect_nodes_in_all_data(in_rel, false);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1579
  }
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1580
  this->collect_nodes(out_rel, -2, false, false);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1581
}
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1582
#endif
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1583
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1584
//------------------------------idealize_test----------------------------------
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1585
// Try to canonicalize tests better.  Peek at the Cmp/Bool/If sequence and
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1586
// come up with a canonical sequence.  Bools getting 'eq', 'gt' and 'ge' forms
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1587
// converted to 'ne', 'le' and 'lt' forms.  IfTrue/IfFalse get swapped as
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1588
// needed.
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1589
static IfNode* idealize_test(PhaseGVN* phase, IfNode* iff) {
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1590
  assert(iff->in(0) != NULL, "If must be live");
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1591
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1592
  if (iff->outcnt() != 2)  return NULL; // Malformed projections.
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1593
  Node* old_if_f = iff->proj_out(false);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1594
  Node* old_if_t = iff->proj_out(true);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1595
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1596
  // CountedLoopEnds want the back-control test to be TRUE, irregardless of
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1597
  // whether they are testing a 'gt' or 'lt' condition.  The 'gt' condition
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1598
  // happens in count-down loops
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1599
  if (iff->is_CountedLoopEnd())  return NULL;
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1600
  if (!iff->in(1)->is_Bool())  return NULL; // Happens for partially optimized IF tests
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1601
  BoolNode *b = iff->in(1)->as_Bool();
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1602
  BoolTest bt = b->_test;
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1603
  // Test already in good order?
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1604
  if( bt.is_canonical() )
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1605
    return NULL;
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1606
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1607
  // Flip test to be canonical.  Requires flipping the IfFalse/IfTrue and
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1608
  // cloning the IfNode.
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1609
  Node* new_b = phase->transform( new BoolNode(b->in(1), bt.negate()) );
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1610
  if( !new_b->is_Bool() ) return NULL;
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1611
  b = new_b->as_Bool();
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1612
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1613
  PhaseIterGVN *igvn = phase->is_IterGVN();
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1614
  assert( igvn, "Test is not canonical in parser?" );
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1615
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1616
  // The IF node never really changes, but it needs to be cloned
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1617
  iff = iff->clone()->as_If();
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1618
  iff->set_req(1, b);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1619
  iff->_prob = 1.0-iff->_prob;
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1620
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1621
  Node *prior = igvn->hash_find_insert(iff);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1622
  if( prior ) {
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1623
    igvn->remove_dead_node(iff);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1624
    iff = (IfNode*)prior;
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1625
  } else {
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1626
    // Cannot call transform on it just yet
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1627
    igvn->set_type_bottom(iff);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1628
  }
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1629
  igvn->_worklist.push(iff);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1630
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1631
  // Now handle projections.  Cloning not required.
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1632
  Node* new_if_f = (Node*)(new IfFalseNode( iff ));
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1633
  Node* new_if_t = (Node*)(new IfTrueNode ( iff ));
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1634
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1635
  igvn->register_new_node_with_optimizer(new_if_f);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1636
  igvn->register_new_node_with_optimizer(new_if_t);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1637
  // Flip test, so flip trailing control
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1638
  igvn->replace_node(old_if_f, new_if_t);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1639
  igvn->replace_node(old_if_t, new_if_f);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1640
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1641
  // Progress
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1642
  return iff;
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1643
}
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1644
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1645
Node* RangeCheckNode::Ideal(PhaseGVN *phase, bool can_reshape) {
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1646
  Node* res = Ideal_common(phase, can_reshape);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1647
  if (res != NodeSentinel) {
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1648
    return res;
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1649
  }
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1650
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1651
  PhaseIterGVN *igvn = phase->is_IterGVN();
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1652
  // Setup to scan up the CFG looking for a dominating test
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1653
  Node* prev_dom = this;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1654
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1655
  // Check for range-check vs other kinds of tests
34164
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1656
  Node* index1;
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1657
  Node* range1;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1658
  jint offset1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1659
  int flip1 = is_range_check(range1, index1, offset1);
34164
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1660
  if (flip1) {
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1661
    Node* dom = in(0);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1662
    // Try to remove extra range checks.  All 'up_one_dom' gives up at merges
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1663
    // so all checks we inspect post-dominate the top-most check we find.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1664
    // If we are going to fail the current check and we reach the top check
2131
98f9cef66a34 6810672: Comment typos
twisti
parents: 1217
diff changeset
  1665
    // then we are guaranteed to fail, so just start interpreting there.
28044
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1666
    // We 'expand' the top 3 range checks to include all post-dominating
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1667
    // checks.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1668
28044
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1669
    // The top 3 range checks seen
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1670
    const int NRC =3;
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1671
    RangeCheck prev_checks[NRC];
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1672
    int nb_checks = 0;
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1673
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1674
    // Low and high offsets seen so far
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1675
    jint off_lo = offset1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1676
    jint off_hi = offset1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1677
28044
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1678
    bool found_immediate_dominator = false;
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1679
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1680
    // Scan for the top checks and collect range of offsets
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1681
    for (int dist = 0; dist < 999; dist++) { // Range-Check scan limit
34164
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1682
      if (dom->Opcode() == Op_RangeCheck &&  // Not same opcode?
28044
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1683
          prev_dom->in(0) == dom) { // One path of test does dominate?
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1684
        if (dom == this) return NULL; // dead loop
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1685
        // See if this is a range check
34164
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1686
        Node* index2;
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1687
        Node* range2;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1688
        jint offset2;
34164
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1689
        int flip2 = dom->as_RangeCheck()->is_range_check(range2, index2, offset2);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1690
        // See if this is a _matching_ range check, checking against
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1691
        // the same array bounds.
28044
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1692
        if (flip2 == flip1 && range2 == range1 && index2 == index1 &&
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1693
            dom->outcnt() == 2) {
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1694
          if (nb_checks == 0 && dom->in(1) == in(1)) {
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1695
            // Found an immediately dominating test at the same offset.
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1696
            // This kind of back-to-back test can be eliminated locally,
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1697
            // and there is no need to search further for dominating tests.
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1698
            assert(offset2 == offset1, "Same test but different offsets");
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1699
            found_immediate_dominator = true;
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1700
            break;
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1701
          }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1702
          // Gather expanded bounds
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1703
          off_lo = MIN2(off_lo,offset2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1704
          off_hi = MAX2(off_hi,offset2);
28044
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1705
          // Record top NRC range checks
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1706
          prev_checks[nb_checks%NRC].ctl = prev_dom;
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1707
          prev_checks[nb_checks%NRC].off = offset2;
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1708
          nb_checks++;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1709
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1710
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1711
      prev_dom = dom;
28044
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1712
      dom = up_one_dom(dom);
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1713
      if (!dom) break;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1714
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1715
28044
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1716
    if (!found_immediate_dominator) {
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1717
      // Attempt to widen the dominating range check to cover some later
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1718
      // ones.  Since range checks "fail" by uncommon-trapping to the
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1719
      // interpreter, widening a check can make us speculatively enter
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1720
      // the interpreter.  If we see range-check deopt's, do not widen!
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1721
      if (!phase->C->allow_range_check_smearing())  return NULL;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1722
28044
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1723
      // Didn't find prior covering check, so cannot remove anything.
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1724
      if (nb_checks == 0) {
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1725
        return NULL;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1726
      }
28044
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1727
      // Constant indices only need to check the upper bound.
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1728
      // Non-constant indices must check both low and high.
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1729
      int chk0 = (nb_checks - 1) % NRC;
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1730
      if (index1) {
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1731
        if (nb_checks == 1) {
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1732
          return NULL;
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1733
        } else {
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1734
          // If the top range check's constant is the min or max of
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1735
          // all constants we widen the next one to cover the whole
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1736
          // range of constants.
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1737
          RangeCheck rc0 = prev_checks[chk0];
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1738
          int chk1 = (nb_checks - 2) % NRC;
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1739
          RangeCheck rc1 = prev_checks[chk1];
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1740
          if (rc0.off == off_lo) {
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1741
            adjust_check(rc1.ctl, range1, index1, flip1, off_hi, igvn);
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1742
            prev_dom = rc1.ctl;
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1743
          } else if (rc0.off == off_hi) {
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1744
            adjust_check(rc1.ctl, range1, index1, flip1, off_lo, igvn);
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1745
            prev_dom = rc1.ctl;
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1746
          } else {
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1747
            // If the top test's constant is not the min or max of all
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1748
            // constants, we need 3 range checks. We must leave the
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1749
            // top test unchanged because widening it would allow the
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1750
            // accesses it protects to successfully read/write out of
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1751
            // bounds.
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1752
            if (nb_checks == 2) {
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1753
              return NULL;
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1754
            }
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1755
            int chk2 = (nb_checks - 3) % NRC;
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1756
            RangeCheck rc2 = prev_checks[chk2];
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1757
            // The top range check a+i covers interval: -a <= i < length-a
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1758
            // The second range check b+i covers interval: -b <= i < length-b
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1759
            if (rc1.off <= rc0.off) {
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1760
              // if b <= a, we change the second range check to:
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1761
              // -min_of_all_constants <= i < length-min_of_all_constants
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1762
              // Together top and second range checks now cover:
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1763
              // -min_of_all_constants <= i < length-a
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1764
              // which is more restrictive than -b <= i < length-b:
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1765
              // -b <= -min_of_all_constants <= i < length-a <= length-b
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1766
              // The third check is then changed to:
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1767
              // -max_of_all_constants <= i < length-max_of_all_constants
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1768
              // so 2nd and 3rd checks restrict allowed values of i to:
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1769
              // -min_of_all_constants <= i < length-max_of_all_constants
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1770
              adjust_check(rc1.ctl, range1, index1, flip1, off_lo, igvn);
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1771
              adjust_check(rc2.ctl, range1, index1, flip1, off_hi, igvn);
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1772
            } else {
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1773
              // if b > a, we change the second range check to:
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1774
              // -max_of_all_constants <= i < length-max_of_all_constants
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1775
              // Together top and second range checks now cover:
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1776
              // -a <= i < length-max_of_all_constants
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1777
              // which is more restrictive than -b <= i < length-b:
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1778
              // -b < -a <= i < length-max_of_all_constants <= length-b
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1779
              // The third check is then changed to:
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1780
              // -max_of_all_constants <= i < length-max_of_all_constants
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1781
              // so 2nd and 3rd checks restrict allowed values of i to:
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1782
              // -min_of_all_constants <= i < length-max_of_all_constants
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1783
              adjust_check(rc1.ctl, range1, index1, flip1, off_hi, igvn);
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1784
              adjust_check(rc2.ctl, range1, index1, flip1, off_lo, igvn);
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1785
            }
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1786
            prev_dom = rc2.ctl;
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1787
          }
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1788
        }
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1789
      } else {
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1790
        RangeCheck rc0 = prev_checks[chk0];
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1791
        // 'Widen' the offset of the 1st and only covering check
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1792
        adjust_check(rc0.ctl, range1, index1, flip1, off_hi, igvn);
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1793
        // Test is now covered by prior checks, dominate it out
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1794
        prev_dom = rc0.ctl;
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1795
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1796
    }
34164
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1797
  } else {
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1798
    prev_dom = search_identical(4);
190
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
  1799
34164
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1800
    if (prev_dom == NULL) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1801
      return NULL;
34164
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1802
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1803
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1804
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1805
  // Replace dominated IfNode
34164
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 34151
diff changeset
  1806
  return dominated_by(prev_dom, igvn);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1807
}