hotspot/src/share/vm/opto/ifnode.cpp
author goetz
Fri, 30 Jan 2015 12:53:56 +0100
changeset 28927 48e4a707e777
parent 28486 b0df113b962e
child 30183 a6588c0a3259
permissions -rw-r--r--
8071996: split_if accesses NULL region of ConstraintCast Reviewed-by: kvn
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     1
/*
22234
da823d78ad65 8029233: Update copyright year to match last edit in jdk8 hotspot repository for 2013
mikael
parents: 21089
diff changeset
     2
 * Copyright (c) 2000, 2013, 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"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5901
diff changeset
    26
#include "memory/allocation.inline.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5901
diff changeset
    27
#include "opto/addnode.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5901
diff changeset
    28
#include "opto/cfgnode.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5901
diff changeset
    29
#include "opto/connode.hpp"
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
    30
#include "opto/loopnode.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5901
diff changeset
    31
#include "opto/phaseX.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5901
diff changeset
    32
#include "opto/runtime.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5901
diff changeset
    33
#include "opto/subnode.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5901
diff changeset
    34
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    35
// Portions of code courtesy of Clifford Click
489c9b5090e2 Initial load
duke
parents:
diff changeset
    36
489c9b5090e2 Initial load
duke
parents:
diff changeset
    37
// Optimization - Graph Style
489c9b5090e2 Initial load
duke
parents:
diff changeset
    38
489c9b5090e2 Initial load
duke
parents:
diff changeset
    39
489c9b5090e2 Initial load
duke
parents:
diff changeset
    40
extern int explicit_null_checks_elided;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    41
489c9b5090e2 Initial load
duke
parents:
diff changeset
    42
//=============================================================================
489c9b5090e2 Initial load
duke
parents:
diff changeset
    43
//------------------------------Value------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
    44
// Return a tuple for whichever arm of the IF is reachable
489c9b5090e2 Initial load
duke
parents:
diff changeset
    45
const Type *IfNode::Value( PhaseTransform *phase ) const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    46
  if( !in(0) ) return Type::TOP;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    47
  if( phase->type(in(0)) == Type::TOP )
489c9b5090e2 Initial load
duke
parents:
diff changeset
    48
    return Type::TOP;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    49
  const Type *t = phase->type(in(1));
489c9b5090e2 Initial load
duke
parents:
diff changeset
    50
  if( t == Type::TOP )          // data is undefined
489c9b5090e2 Initial load
duke
parents:
diff changeset
    51
    return TypeTuple::IFNEITHER; // unreachable altogether
489c9b5090e2 Initial load
duke
parents:
diff changeset
    52
  if( t == TypeInt::ZERO )      // zero, or false
489c9b5090e2 Initial load
duke
parents:
diff changeset
    53
    return TypeTuple::IFFALSE;  // only false branch is reachable
489c9b5090e2 Initial load
duke
parents:
diff changeset
    54
  if( t == TypeInt::ONE )       // 1, or true
489c9b5090e2 Initial load
duke
parents:
diff changeset
    55
    return TypeTuple::IFTRUE;   // only true branch is reachable
489c9b5090e2 Initial load
duke
parents:
diff changeset
    56
  assert( t == TypeInt::BOOL, "expected boolean type" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
    57
489c9b5090e2 Initial load
duke
parents:
diff changeset
    58
  return TypeTuple::IFBOTH;     // No progress
489c9b5090e2 Initial load
duke
parents:
diff changeset
    59
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    60
489c9b5090e2 Initial load
duke
parents:
diff changeset
    61
const RegMask &IfNode::out_RegMask() const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    62
  return RegMask::Empty;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    63
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    64
489c9b5090e2 Initial load
duke
parents:
diff changeset
    65
//------------------------------split_if---------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
    66
// Look for places where we merge constants, then test on the merged value.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    67
// If the IF test will be constant folded on the path with the constant, we
489c9b5090e2 Initial load
duke
parents:
diff changeset
    68
// win by splitting the IF to before the merge point.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    69
static Node* split_if(IfNode *iff, PhaseIterGVN *igvn) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    70
  // I could be a lot more general here, but I'm trying to squeeze this
489c9b5090e2 Initial load
duke
parents:
diff changeset
    71
  // in before the Christmas '98 break so I'm gonna be kinda restrictive
489c9b5090e2 Initial load
duke
parents:
diff changeset
    72
  // on the patterns I accept.  CNC
489c9b5090e2 Initial load
duke
parents:
diff changeset
    73
489c9b5090e2 Initial load
duke
parents:
diff changeset
    74
  // Look for a compare of a constant and a merged value
489c9b5090e2 Initial load
duke
parents:
diff changeset
    75
  Node *i1 = iff->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    76
  if( !i1->is_Bool() ) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    77
  BoolNode *b = i1->as_Bool();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    78
  Node *cmp = b->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    79
  if( !cmp->is_Cmp() ) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    80
  i1 = cmp->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    81
  if( i1 == NULL || !i1->is_Phi() ) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    82
  PhiNode *phi = i1->as_Phi();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    83
  if( phi->is_copy() ) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    84
  Node *con2 = cmp->in(2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    85
  if( !con2->is_Con() ) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    86
  // See that the merge point contains some constants
489c9b5090e2 Initial load
duke
parents:
diff changeset
    87
  Node *con1=NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    88
  uint i4;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    89
  for( i4 = 1; i4 < phi->req(); i4++ ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    90
    con1 = phi->in(i4);
2131
98f9cef66a34 6810672: Comment typos
twisti
parents: 1217
diff changeset
    91
    if( !con1 ) return NULL;    // Do not optimize partially collapsed merges
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    92
    if( con1->is_Con() ) break; // Found a constant
489c9b5090e2 Initial load
duke
parents:
diff changeset
    93
    // Also allow null-vs-not-null checks
489c9b5090e2 Initial load
duke
parents:
diff changeset
    94
    const TypePtr *tp = igvn->type(con1)->isa_ptr();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    95
    if( tp && tp->_ptr == TypePtr::NotNull )
489c9b5090e2 Initial load
duke
parents:
diff changeset
    96
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    97
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    98
  if( i4 >= phi->req() ) return NULL; // Found no constants
489c9b5090e2 Initial load
duke
parents:
diff changeset
    99
489c9b5090e2 Initial load
duke
parents:
diff changeset
   100
  igvn->C->set_has_split_ifs(true); // Has chance for split-if
489c9b5090e2 Initial load
duke
parents:
diff changeset
   101
489c9b5090e2 Initial load
duke
parents:
diff changeset
   102
  // Make sure that the compare can be constant folded away
489c9b5090e2 Initial load
duke
parents:
diff changeset
   103
  Node *cmp2 = cmp->clone();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   104
  cmp2->set_req(1,con1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   105
  cmp2->set_req(2,con2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   106
  const Type *t = cmp2->Value(igvn);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   107
  // This compare is dead, so whack it!
489c9b5090e2 Initial load
duke
parents:
diff changeset
   108
  igvn->remove_dead_node(cmp2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   109
  if( !t->singleton() ) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   110
489c9b5090e2 Initial load
duke
parents:
diff changeset
   111
  // No intervening control, like a simple Call
489c9b5090e2 Initial load
duke
parents:
diff changeset
   112
  Node *r = iff->in(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   113
  if( !r->is_Region() ) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   114
  if( phi->region() != r ) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   115
  // No other users of the cmp/bool
489c9b5090e2 Initial load
duke
parents:
diff changeset
   116
  if (b->outcnt() != 1 || cmp->outcnt() != 1) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   117
    //tty->print_cr("many users of cmp/bool");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   118
    return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   119
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   120
489c9b5090e2 Initial load
duke
parents:
diff changeset
   121
  // Make sure we can determine where all the uses of merged values go
489c9b5090e2 Initial load
duke
parents:
diff changeset
   122
  for (DUIterator_Fast jmax, j = r->fast_outs(jmax); j < jmax; j++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   123
    Node* u = r->fast_out(j);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   124
    if( u == r ) continue;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   125
    if( u == iff ) continue;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   126
    if( u->outcnt() == 0 ) continue; // use is dead & ignorable
489c9b5090e2 Initial load
duke
parents:
diff changeset
   127
    if( !u->is_Phi() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   128
      /*
489c9b5090e2 Initial load
duke
parents:
diff changeset
   129
      if( u->is_Start() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   130
        tty->print_cr("Region has inlined start use");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   131
      } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   132
        tty->print_cr("Region has odd use");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   133
        u->dump(2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   134
      }*/
489c9b5090e2 Initial load
duke
parents:
diff changeset
   135
      return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   136
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   137
    if( u != phi ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   138
      // CNC - do not allow any other merged value
489c9b5090e2 Initial load
duke
parents:
diff changeset
   139
      //tty->print_cr("Merging another value");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   140
      //u->dump(2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   141
      return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   142
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   143
    // Make sure we can account for all Phi uses
489c9b5090e2 Initial load
duke
parents:
diff changeset
   144
    for (DUIterator_Fast kmax, k = u->fast_outs(kmax); k < kmax; k++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   145
      Node* v = u->fast_out(k); // User of the phi
489c9b5090e2 Initial load
duke
parents:
diff changeset
   146
      // CNC - Allow only really simple patterns.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   147
      // 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
   148
      if (v == cmp) continue;  // The compare is OK
48e4a707e777 8071996: split_if accesses NULL region of ConstraintCast
goetz
parents: 28486
diff changeset
   149
      if (v->is_ConstraintCast()) {
48e4a707e777 8071996: split_if accesses NULL region of ConstraintCast
goetz
parents: 28486
diff changeset
   150
        // 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
   151
        // 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
   152
        // this (l. 359). So skip.
48e4a707e777 8071996: split_if accesses NULL region of ConstraintCast
goetz
parents: 28486
diff changeset
   153
        if (v->in(0) == NULL) {
48e4a707e777 8071996: split_if accesses NULL region of ConstraintCast
goetz
parents: 28486
diff changeset
   154
          return NULL;
48e4a707e777 8071996: split_if accesses NULL region of ConstraintCast
goetz
parents: 28486
diff changeset
   155
        }
48e4a707e777 8071996: split_if accesses NULL region of ConstraintCast
goetz
parents: 28486
diff changeset
   156
        if (v->in(0)->in(0) == iff) {
48e4a707e777 8071996: split_if accesses NULL region of ConstraintCast
goetz
parents: 28486
diff changeset
   157
          continue;               // CastPP/II of the IfNode is OK
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
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   160
      // Disabled following code because I cannot tell if exactly one
489c9b5090e2 Initial load
duke
parents:
diff changeset
   161
      // path dominates without a real dominator check. CNC 9/9/1999
489c9b5090e2 Initial load
duke
parents:
diff changeset
   162
      //uint vop = v->Opcode();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   163
      //if( vop == Op_Phi ) {     // Phi from another merge point might be OK
489c9b5090e2 Initial load
duke
parents:
diff changeset
   164
      //  Node *r = v->in(0);     // Get controlling point
489c9b5090e2 Initial load
duke
parents:
diff changeset
   165
      //  if( !r ) return NULL;   // Degraded to a copy
489c9b5090e2 Initial load
duke
parents:
diff changeset
   166
      //  // Find exactly one path in (either True or False doms, but not IFF)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   167
      //  int cnt = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   168
      //  for( uint i = 1; i < r->req(); i++ )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   169
      //    if( r->in(i) && r->in(i)->in(0) == iff )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   170
      //      cnt++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   171
      //  if( cnt == 1 ) continue; // Exactly one of True or False guards Phi
489c9b5090e2 Initial load
duke
parents:
diff changeset
   172
      //}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   173
      if( !v->is_Call() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   174
        /*
489c9b5090e2 Initial load
duke
parents:
diff changeset
   175
        if( v->Opcode() == Op_AddP ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   176
          tty->print_cr("Phi has AddP use");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   177
        } else if( v->Opcode() == Op_CastPP ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   178
          tty->print_cr("Phi has CastPP use");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   179
        } else if( v->Opcode() == Op_CastII ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   180
          tty->print_cr("Phi has CastII use");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   181
        } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   182
          tty->print_cr("Phi has use I cant be bothered with");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   183
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   184
        */
489c9b5090e2 Initial load
duke
parents:
diff changeset
   185
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   186
      return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   187
489c9b5090e2 Initial load
duke
parents:
diff changeset
   188
      /* CNC - Cut out all the fancy acceptance tests
489c9b5090e2 Initial load
duke
parents:
diff changeset
   189
      // Can we clone this use when doing the transformation?
489c9b5090e2 Initial load
duke
parents:
diff changeset
   190
      // If all uses are from Phis at this merge or constants, then YES.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   191
      if( !v->in(0) && v != cmp ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   192
        tty->print_cr("Phi has free-floating use");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   193
        v->dump(2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   194
        return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   195
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   196
      for( uint l = 1; l < v->req(); l++ ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   197
        if( (!v->in(l)->is_Phi() || v->in(l)->in(0) != r) &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
   198
            !v->in(l)->is_Con() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   199
          tty->print_cr("Phi has use");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   200
          v->dump(2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   201
          return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   202
        } // End of if Phi-use input is neither Phi nor Constant
489c9b5090e2 Initial load
duke
parents:
diff changeset
   203
      } // End of for all inputs to Phi-use
489c9b5090e2 Initial load
duke
parents:
diff changeset
   204
      */
489c9b5090e2 Initial load
duke
parents:
diff changeset
   205
    } // End of for all uses of Phi
489c9b5090e2 Initial load
duke
parents:
diff changeset
   206
  } // End of for all uses of Region
489c9b5090e2 Initial load
duke
parents:
diff changeset
   207
489c9b5090e2 Initial load
duke
parents:
diff changeset
   208
  // Only do this if the IF node is in a sane state
489c9b5090e2 Initial load
duke
parents:
diff changeset
   209
  if (iff->outcnt() != 2)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   210
    return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   211
489c9b5090e2 Initial load
duke
parents:
diff changeset
   212
  // Got a hit!  Do the Mondo Hack!
489c9b5090e2 Initial load
duke
parents:
diff changeset
   213
  //
489c9b5090e2 Initial load
duke
parents:
diff changeset
   214
  //ABC  a1c   def   ghi            B     1     e     h   A C   a c   d f   g i
489c9b5090e2 Initial load
duke
parents:
diff changeset
   215
  // R - Phi - Phi - Phi            Rc - Phi - Phi - Phi   Rx - Phi - Phi - Phi
489c9b5090e2 Initial load
duke
parents:
diff changeset
   216
  //     cmp - 2                         cmp - 2               cmp - 2
489c9b5090e2 Initial load
duke
parents:
diff changeset
   217
  //       bool                            bool_c                bool_x
489c9b5090e2 Initial load
duke
parents:
diff changeset
   218
  //       if                               if_c                  if_x
489c9b5090e2 Initial load
duke
parents:
diff changeset
   219
  //      T  F                              T  F                  T  F
489c9b5090e2 Initial load
duke
parents:
diff changeset
   220
  // ..s..    ..t ..                   ..s..    ..t..        ..s..    ..t..
489c9b5090e2 Initial load
duke
parents:
diff changeset
   221
  //
2131
98f9cef66a34 6810672: Comment typos
twisti
parents: 1217
diff changeset
   222
  // Split the paths coming into the merge point into 2 separate groups of
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   223
  // merges.  On the left will be all the paths feeding constants into the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   224
  // Cmp's Phi.  On the right will be the remaining paths.  The Cmp's Phi
489c9b5090e2 Initial load
duke
parents:
diff changeset
   225
  // will fold up into a constant; this will let the Cmp fold up as well as
489c9b5090e2 Initial load
duke
parents:
diff changeset
   226
  // all the control flow.  Below the original IF we have 2 control
489c9b5090e2 Initial load
duke
parents:
diff changeset
   227
  // dependent regions, 's' and 't'.  Now we will merge the two paths
489c9b5090e2 Initial load
duke
parents:
diff changeset
   228
  // just prior to 's' and 't' from the two IFs.  At least 1 path (and quite
489c9b5090e2 Initial load
duke
parents:
diff changeset
   229
  // likely 2 or more) will promptly constant fold away.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   230
  PhaseGVN *phase = igvn;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   231
489c9b5090e2 Initial load
duke
parents:
diff changeset
   232
  // Make a region merging constants and a region merging the rest
489c9b5090e2 Initial load
duke
parents:
diff changeset
   233
  uint req_c = 0;
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   234
  Node* predicate_proj = NULL;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   235
  for (uint ii = 1; ii < r->req(); ii++) {
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   236
    if (phi->in(ii) == con1) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   237
      req_c++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   238
    }
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   239
    Node* proj = PhaseIdealLoop::find_predicate(r->in(ii));
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   240
    if (proj != NULL) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   241
      assert(predicate_proj == NULL, "only one predicate entry expected");
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   242
      predicate_proj = proj;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   243
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   244
  }
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   245
  Node* predicate_c = NULL;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   246
  Node* predicate_x = NULL;
9446
748a37b25d10 5091921: Sign flip issues in loop optimizer
kvn
parents: 9101
diff changeset
   247
  bool counted_loop = r->is_CountedLoop();
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   248
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
   249
  Node *region_c = new RegionNode(req_c + 1);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   250
  Node *phi_c    = con1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   251
  uint  len      = r->req();
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
   252
  Node *region_x = new RegionNode(len - req_c);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   253
  Node *phi_x    = PhiNode::make_blank(region_x, phi);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   254
  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
   255
    if (phi->in(i) == con1) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   256
      region_c->init_req( i_c++, r  ->in(i) );
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   257
      if (r->in(i) == predicate_proj)
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   258
        predicate_c = predicate_proj;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   259
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   260
      region_x->init_req( i_x,   r  ->in(i) );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   261
      phi_x   ->init_req( i_x++, phi->in(i) );
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   262
      if (r->in(i) == predicate_proj)
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   263
        predicate_x = predicate_proj;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   264
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   265
  }
10258
10c77b8c8d3e 7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents: 10253
diff changeset
   266
  if (predicate_c != NULL && (req_c > 1)) {
10c77b8c8d3e 7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents: 10253
diff changeset
   267
    assert(predicate_x == NULL, "only one predicate entry expected");
10c77b8c8d3e 7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents: 10253
diff changeset
   268
    predicate_c = NULL; // Do not clone predicate below merge point
10c77b8c8d3e 7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents: 10253
diff changeset
   269
  }
10c77b8c8d3e 7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents: 10253
diff changeset
   270
  if (predicate_x != NULL && ((len - req_c) > 2)) {
10c77b8c8d3e 7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents: 10253
diff changeset
   271
    assert(predicate_c == NULL, "only one predicate entry expected");
10c77b8c8d3e 7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents: 10253
diff changeset
   272
    predicate_x = NULL; // Do not clone predicate below merge point
10c77b8c8d3e 7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents: 10253
diff changeset
   273
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   274
489c9b5090e2 Initial load
duke
parents:
diff changeset
   275
  // Register the new RegionNodes but do not transform them.  Cannot
2131
98f9cef66a34 6810672: Comment typos
twisti
parents: 1217
diff changeset
   276
  // transform until the entire Region/Phi conglomerate has been hacked
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   277
  // as a single huge transform.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   278
  igvn->register_new_node_with_optimizer( region_c );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   279
  igvn->register_new_node_with_optimizer( region_x );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   280
  // Prevent the untimely death of phi_x.  Currently he has no uses.  He is
489c9b5090e2 Initial load
duke
parents:
diff changeset
   281
  // about to get one.  If this only use goes away, then phi_x will look dead.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   282
  // 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
   283
  Node *hook = new Node(4);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   284
  hook->init_req(0, phi_x);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   285
  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
   286
  phi_x = phase->transform( phi_x );
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   287
489c9b5090e2 Initial load
duke
parents:
diff changeset
   288
  // Make the compare
489c9b5090e2 Initial load
duke
parents:
diff changeset
   289
  Node *cmp_c = phase->makecon(t);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   290
  Node *cmp_x = cmp->clone();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   291
  cmp_x->set_req(1,phi_x);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   292
  cmp_x->set_req(2,con2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   293
  cmp_x = phase->transform(cmp_x);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   294
  // Make the bool
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
   295
  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
   296
  Node *b_x = phase->transform(new BoolNode(cmp_x,b->_test._test));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   297
  // Make the IfNode
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
   298
  IfNode *iff_c = new IfNode(region_c,b_c,iff->_prob,iff->_fcnt);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   299
  igvn->set_type_bottom(iff_c);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   300
  igvn->_worklist.push(iff_c);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   301
  hook->init_req(2, iff_c);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   302
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
   303
  IfNode *iff_x = new IfNode(region_x,b_x,iff->_prob, iff->_fcnt);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   304
  igvn->set_type_bottom(iff_x);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   305
  igvn->_worklist.push(iff_x);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   306
  hook->init_req(3, iff_x);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   307
489c9b5090e2 Initial load
duke
parents:
diff changeset
   308
  // Make the true/false arms
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
   309
  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
   310
  Node *iff_c_f = phase->transform(new IfFalseNode(iff_c));
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   311
  if (predicate_c != NULL) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   312
    assert(predicate_x == NULL, "only one predicate entry expected");
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   313
    // Clone loop predicates to each path
9446
748a37b25d10 5091921: Sign flip issues in loop optimizer
kvn
parents: 9101
diff changeset
   314
    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
   315
    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
   316
  }
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
   317
  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
   318
  Node *iff_x_f = phase->transform(new IfFalseNode(iff_x));
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   319
  if (predicate_x != NULL) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   320
    assert(predicate_c == NULL, "only one predicate entry expected");
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
   321
    // Clone loop predicates to each path
9446
748a37b25d10 5091921: Sign flip issues in loop optimizer
kvn
parents: 9101
diff changeset
   322
    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
   323
    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
   324
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   325
489c9b5090e2 Initial load
duke
parents:
diff changeset
   326
  // Merge the TRUE paths
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
   327
  Node *region_s = new RegionNode(3);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   328
  igvn->_worklist.push(region_s);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   329
  region_s->init_req(1, iff_c_t);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   330
  region_s->init_req(2, iff_x_t);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   331
  igvn->register_new_node_with_optimizer( region_s );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   332
489c9b5090e2 Initial load
duke
parents:
diff changeset
   333
  // Merge the FALSE paths
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
   334
  Node *region_f = new RegionNode(3);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   335
  igvn->_worklist.push(region_f);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   336
  region_f->init_req(1, iff_c_f);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   337
  region_f->init_req(2, iff_x_f);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   338
  igvn->register_new_node_with_optimizer( region_f );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   339
489c9b5090e2 Initial load
duke
parents:
diff changeset
   340
  igvn->hash_delete(cmp);// Remove soon-to-be-dead node from hash table.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   341
  cmp->set_req(1,NULL);  // Whack the inputs to cmp because it will be dead
489c9b5090e2 Initial load
duke
parents:
diff changeset
   342
  cmp->set_req(2,NULL);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   343
  // Check for all uses of the Phi and give them a new home.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   344
  // The 'cmp' got cloned, but CastPP/IIs need to be moved.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   345
  Node *phi_s = NULL;     // do not construct unless needed
489c9b5090e2 Initial load
duke
parents:
diff changeset
   346
  Node *phi_f = NULL;     // do not construct unless needed
489c9b5090e2 Initial load
duke
parents:
diff changeset
   347
  for (DUIterator_Last i2min, i2 = phi->last_outs(i2min); i2 >= i2min; --i2) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   348
    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
   349
    igvn->rehash_node_delayed(v); // Have to fixup other Phi users
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   350
    uint vop = v->Opcode();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   351
    Node *proj = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   352
    if( vop == Op_Phi ) {       // Remote merge point
489c9b5090e2 Initial load
duke
parents:
diff changeset
   353
      Node *r = v->in(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   354
      for (uint i3 = 1; i3 < r->req(); i3++)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   355
        if (r->in(i3) && r->in(i3)->in(0) == iff) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   356
          proj = r->in(i3);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   357
          break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   358
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   359
    } else if( v->is_ConstraintCast() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   360
      proj = v->in(0);          // Controlling projection
489c9b5090e2 Initial load
duke
parents:
diff changeset
   361
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   362
      assert( 0, "do not know how to handle this guy" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   363
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   364
489c9b5090e2 Initial load
duke
parents:
diff changeset
   365
    Node *proj_path_data, *proj_path_ctrl;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   366
    if( proj->Opcode() == Op_IfTrue ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   367
      if( phi_s == NULL ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   368
        // Only construct phi_s if needed, otherwise provides
489c9b5090e2 Initial load
duke
parents:
diff changeset
   369
        // interfering use.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   370
        phi_s = PhiNode::make_blank(region_s,phi);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   371
        phi_s->init_req( 1, phi_c );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   372
        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
   373
        hook->add_req(phi_s);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   374
        phi_s = phase->transform(phi_s);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   375
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   376
      proj_path_data = phi_s;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   377
      proj_path_ctrl = region_s;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   378
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   379
      if( phi_f == NULL ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   380
        // Only construct phi_f if needed, otherwise provides
489c9b5090e2 Initial load
duke
parents:
diff changeset
   381
        // interfering use.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   382
        phi_f = PhiNode::make_blank(region_f,phi);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   383
        phi_f->init_req( 1, phi_c );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   384
        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
   385
        hook->add_req(phi_f);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   386
        phi_f = phase->transform(phi_f);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   387
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   388
      proj_path_data = phi_f;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   389
      proj_path_ctrl = region_f;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   390
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   391
489c9b5090e2 Initial load
duke
parents:
diff changeset
   392
    // Fixup 'v' for for the split
489c9b5090e2 Initial load
duke
parents:
diff changeset
   393
    if( vop == Op_Phi ) {       // Remote merge point
489c9b5090e2 Initial load
duke
parents:
diff changeset
   394
      uint i;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   395
      for( i = 1; i < v->req(); i++ )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   396
        if( v->in(i) == phi )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   397
          break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   398
      v->set_req(i, proj_path_data );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   399
    } else if( v->is_ConstraintCast() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   400
      v->set_req(0, proj_path_ctrl );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   401
      v->set_req(1, proj_path_data );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   402
    } else
489c9b5090e2 Initial load
duke
parents:
diff changeset
   403
      ShouldNotReachHere();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   404
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   405
489c9b5090e2 Initial load
duke
parents:
diff changeset
   406
  // Now replace the original iff's True/False with region_s/region_t.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   407
  // This makes the original iff go dead.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   408
  for (DUIterator_Last i3min, i3 = iff->last_outs(i3min); i3 >= i3min; --i3) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   409
    Node* p = iff->last_out(i3);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   410
    assert( p->Opcode() == Op_IfTrue || p->Opcode() == Op_IfFalse, "" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   411
    Node *u = (p->Opcode() == Op_IfTrue) ? region_s : region_f;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   412
    // Replace p with u
489c9b5090e2 Initial load
duke
parents:
diff changeset
   413
    igvn->add_users_to_worklist(p);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   414
    for (DUIterator_Last lmin, l = p->last_outs(lmin); l >= lmin;) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   415
      Node* x = p->last_out(l);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   416
      igvn->hash_delete(x);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   417
      uint uses_found = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   418
      for( uint j = 0; j < x->req(); j++ ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   419
        if( x->in(j) == p ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   420
          x->set_req(j, u);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   421
          uses_found++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   422
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   423
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   424
      l -= uses_found;    // we deleted 1 or more copies of this edge
489c9b5090e2 Initial load
duke
parents:
diff changeset
   425
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   426
    igvn->remove_dead_node(p);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   427
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   428
489c9b5090e2 Initial load
duke
parents:
diff changeset
   429
  // Force the original merge dead
489c9b5090e2 Initial load
duke
parents:
diff changeset
   430
  igvn->hash_delete(r);
3268
f034e0c86895 6851742: (EA) allocation elimination doesn't work with UseG1GC
kvn
parents: 2131
diff changeset
   431
  // First, remove region's dead users.
f034e0c86895 6851742: (EA) allocation elimination doesn't work with UseG1GC
kvn
parents: 2131
diff changeset
   432
  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
   433
    Node* u = r->last_out(l);
f034e0c86895 6851742: (EA) allocation elimination doesn't work with UseG1GC
kvn
parents: 2131
diff changeset
   434
    if( u == r ) {
f034e0c86895 6851742: (EA) allocation elimination doesn't work with UseG1GC
kvn
parents: 2131
diff changeset
   435
      r->set_req(0, NULL);
f034e0c86895 6851742: (EA) allocation elimination doesn't work with UseG1GC
kvn
parents: 2131
diff changeset
   436
    } else {
f034e0c86895 6851742: (EA) allocation elimination doesn't work with UseG1GC
kvn
parents: 2131
diff changeset
   437
      assert(u->outcnt() == 0, "only dead users");
f034e0c86895 6851742: (EA) allocation elimination doesn't work with UseG1GC
kvn
parents: 2131
diff changeset
   438
      igvn->remove_dead_node(u);
f034e0c86895 6851742: (EA) allocation elimination doesn't work with UseG1GC
kvn
parents: 2131
diff changeset
   439
    }
f034e0c86895 6851742: (EA) allocation elimination doesn't work with UseG1GC
kvn
parents: 2131
diff changeset
   440
    l -= 1;
f034e0c86895 6851742: (EA) allocation elimination doesn't work with UseG1GC
kvn
parents: 2131
diff changeset
   441
  }
f034e0c86895 6851742: (EA) allocation elimination doesn't work with UseG1GC
kvn
parents: 2131
diff changeset
   442
  igvn->remove_dead_node(r);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   443
489c9b5090e2 Initial load
duke
parents:
diff changeset
   444
  // Now remove the bogus extra edges used to keep things alive
489c9b5090e2 Initial load
duke
parents:
diff changeset
   445
  igvn->remove_dead_node( hook );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   446
489c9b5090e2 Initial load
duke
parents:
diff changeset
   447
  // Must return either the original node (now dead) or a new node
489c9b5090e2 Initial load
duke
parents:
diff changeset
   448
  // (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
   449
  return new ConINode(TypeInt::ZERO);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   450
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   451
489c9b5090e2 Initial load
duke
parents:
diff changeset
   452
//------------------------------is_range_check---------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   453
// Return 0 if not a range check.  Return 1 if a range check and set index and
489c9b5090e2 Initial load
duke
parents:
diff changeset
   454
// offset.  Return 2 if we had to negate the test.  Index is NULL if the check
489c9b5090e2 Initial load
duke
parents:
diff changeset
   455
// is versus a constant.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   456
int IfNode::is_range_check(Node* &range, Node* &index, jint &offset) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   457
  Node* b = in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   458
  if (b == NULL || !b->is_Bool())  return 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   459
  BoolNode* bn = b->as_Bool();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   460
  Node* cmp = bn->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   461
  if (cmp == NULL)  return 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   462
  if (cmp->Opcode() != Op_CmpU)  return 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   463
489c9b5090e2 Initial load
duke
parents:
diff changeset
   464
  Node* l = cmp->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   465
  Node* r = cmp->in(2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   466
  int flip_test = 1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   467
  if (bn->_test._test == BoolTest::le) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   468
    l = cmp->in(2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   469
    r = cmp->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   470
    flip_test = 2;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   471
  } else if (bn->_test._test != BoolTest::lt) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   472
    return 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   473
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   474
  if (l->is_top())  return 0;   // Top input means dead test
489c9b5090e2 Initial load
duke
parents:
diff changeset
   475
  if (r->Opcode() != Op_LoadRange)  return 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   476
489c9b5090e2 Initial load
duke
parents:
diff changeset
   477
  // We have recognized one of these forms:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   478
  //  Flip 1:  If (Bool[<] CmpU(l, LoadRange)) ...
489c9b5090e2 Initial load
duke
parents:
diff changeset
   479
  //  Flip 2:  If (Bool[<=] CmpU(LoadRange, l)) ...
489c9b5090e2 Initial load
duke
parents:
diff changeset
   480
489c9b5090e2 Initial load
duke
parents:
diff changeset
   481
  // Make sure it's a real range check by requiring an uncommon trap
489c9b5090e2 Initial load
duke
parents:
diff changeset
   482
  // along the OOB path.  Otherwise, it's possible that the user wrote
489c9b5090e2 Initial load
duke
parents:
diff changeset
   483
  // something which optimized to look like a range check but behaves
489c9b5090e2 Initial load
duke
parents:
diff changeset
   484
  // in some other way.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   485
  Node* iftrap = proj_out(flip_test == 2 ? true : false);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   486
  bool found_trap = false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   487
  if (iftrap != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   488
    Node* u = iftrap->unique_ctrl_out();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   489
    if (u != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   490
      // It could be a merge point (Region) for uncommon trap.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   491
      if (u->is_Region()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   492
        Node* c = u->unique_ctrl_out();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   493
        if (c != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   494
          iftrap = u;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   495
          u = c;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   496
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   497
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   498
      if (u->in(0) == iftrap && u->is_CallStaticJava()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   499
        int req = u->as_CallStaticJava()->uncommon_trap_request();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   500
        if (Deoptimization::trap_request_reason(req) ==
489c9b5090e2 Initial load
duke
parents:
diff changeset
   501
            Deoptimization::Reason_range_check) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   502
          found_trap = true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   503
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   504
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   505
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   506
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   507
  if (!found_trap)  return 0;   // sorry, no cigar
489c9b5090e2 Initial load
duke
parents:
diff changeset
   508
489c9b5090e2 Initial load
duke
parents:
diff changeset
   509
  // Look for index+offset form
489c9b5090e2 Initial load
duke
parents:
diff changeset
   510
  Node* ind = l;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   511
  jint  off = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   512
  if (l->is_top()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   513
    return 0;
26173
4275dfc46177 8054883: Segmentation error while running program
iveresov
parents: 24923
diff changeset
   514
  } else if (l->Opcode() == Op_AddI) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   515
    if ((off = l->in(1)->find_int_con(0)) != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   516
      ind = l->in(2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   517
    } else if ((off = l->in(2)->find_int_con(0)) != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   518
      ind = l->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   519
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   520
  } else if ((off = l->find_int_con(-1)) >= 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   521
    // constant offset with no variable index
489c9b5090e2 Initial load
duke
parents:
diff changeset
   522
    ind = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   523
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   524
    // variable index with no constant offset (or dead negative index)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   525
    off = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   526
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   527
489c9b5090e2 Initial load
duke
parents:
diff changeset
   528
  // Return all the values:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   529
  index  = ind;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   530
  offset = off;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   531
  range  = r;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   532
  return flip_test;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   533
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   534
489c9b5090e2 Initial load
duke
parents:
diff changeset
   535
//------------------------------adjust_check-----------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   536
// Adjust (widen) a prior range check
489c9b5090e2 Initial load
duke
parents:
diff changeset
   537
static void adjust_check(Node* proj, Node* range, Node* index,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   538
                         int flip, jint off_lo, PhaseIterGVN* igvn) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   539
  PhaseGVN *gvn = igvn;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   540
  // Break apart the old check
489c9b5090e2 Initial load
duke
parents:
diff changeset
   541
  Node *iff = proj->in(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   542
  Node *bol = iff->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   543
  if( bol->is_top() ) return;   // In case a partially dead range check appears
489c9b5090e2 Initial load
duke
parents:
diff changeset
   544
  // bail (or bomb[ASSERT/DEBUG]) if NOT projection-->IfNode-->BoolNode
489c9b5090e2 Initial load
duke
parents:
diff changeset
   545
  DEBUG_ONLY( if( !bol->is_Bool() ) { proj->dump(3); fatal("Expect projection-->IfNode-->BoolNode"); } )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   546
  if( !bol->is_Bool() ) return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   547
489c9b5090e2 Initial load
duke
parents:
diff changeset
   548
  Node *cmp = bol->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   549
  // Compute a new check
489c9b5090e2 Initial load
duke
parents:
diff changeset
   550
  Node *new_add = gvn->intcon(off_lo);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   551
  if( index ) {
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
   552
    new_add = off_lo ? gvn->transform(new AddINode( index, new_add )) : index;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   553
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   554
  Node *new_cmp = (flip == 1)
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
   555
    ? new CmpUNode( new_add, range )
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
   556
    : new CmpUNode( range, new_add );
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   557
  new_cmp = gvn->transform(new_cmp);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   558
  // See if no need to adjust the existing check
489c9b5090e2 Initial load
duke
parents:
diff changeset
   559
  if( new_cmp == cmp ) return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   560
  // Else, adjust existing check
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
   561
  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
   562
  igvn->rehash_node_delayed( iff );
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   563
  iff->set_req_X( 1, new_bol, igvn );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   564
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   565
489c9b5090e2 Initial load
duke
parents:
diff changeset
   566
//------------------------------up_one_dom-------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   567
// Walk up the dominator tree one step.  Return NULL at root or true
489c9b5090e2 Initial load
duke
parents:
diff changeset
   568
// complex merges.  Skips through small diamonds.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   569
Node* IfNode::up_one_dom(Node *curr, bool linear_only) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   570
  Node *dom = curr->in(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   571
  if( !dom )                    // Found a Region degraded to a copy?
489c9b5090e2 Initial load
duke
parents:
diff changeset
   572
    return curr->nonnull_req(); // Skip thru it
489c9b5090e2 Initial load
duke
parents:
diff changeset
   573
489c9b5090e2 Initial load
duke
parents:
diff changeset
   574
  if( curr != dom )             // Normal walk up one step?
489c9b5090e2 Initial load
duke
parents:
diff changeset
   575
    return dom;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   576
489c9b5090e2 Initial load
duke
parents:
diff changeset
   577
  // Use linear_only if we are still parsing, since we cannot
489c9b5090e2 Initial load
duke
parents:
diff changeset
   578
  // trust the regions to be fully filled in.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   579
  if (linear_only)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   580
    return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   581
4582
1a6662d11385 6915110: IfNode::up_one_dom moves beyond RootNode bug in src/share/vm/opto/ifnode.cpp
kvn
parents: 4020
diff changeset
   582
  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
   583
    return NULL;
1a6662d11385 6915110: IfNode::up_one_dom moves beyond RootNode bug in src/share/vm/opto/ifnode.cpp
kvn
parents: 4020
diff changeset
   584
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   585
  // Else hit a Region.  Check for a loop header
489c9b5090e2 Initial load
duke
parents:
diff changeset
   586
  if( dom->is_Loop() )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   587
    return dom->in(1);          // Skip up thru loops
489c9b5090e2 Initial load
duke
parents:
diff changeset
   588
489c9b5090e2 Initial load
duke
parents:
diff changeset
   589
  // Check for small diamonds
489c9b5090e2 Initial load
duke
parents:
diff changeset
   590
  Node *din1, *din2, *din3, *din4;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   591
  if( dom->req() == 3 &&        // 2-path merge point
489c9b5090e2 Initial load
duke
parents:
diff changeset
   592
      (din1 = dom ->in(1)) &&   // Left  path exists
489c9b5090e2 Initial load
duke
parents:
diff changeset
   593
      (din2 = dom ->in(2)) &&   // Right path exists
489c9b5090e2 Initial load
duke
parents:
diff changeset
   594
      (din3 = din1->in(0)) &&   // Left  path up one
489c9b5090e2 Initial load
duke
parents:
diff changeset
   595
      (din4 = din2->in(0)) ) {  // Right path up one
489c9b5090e2 Initial load
duke
parents:
diff changeset
   596
    if( din3->is_Call() &&      // Handle a slow-path call on either arm
489c9b5090e2 Initial load
duke
parents:
diff changeset
   597
        (din3 = din3->in(0)) )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   598
      din3 = din3->in(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   599
    if( din4->is_Call() &&      // Handle a slow-path call on either arm
489c9b5090e2 Initial load
duke
parents:
diff changeset
   600
        (din4 = din4->in(0)) )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   601
      din4 = din4->in(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   602
    if( din3 == din4 && din3->is_If() )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   603
      return din3;              // Skip around diamonds
489c9b5090e2 Initial load
duke
parents:
diff changeset
   604
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   605
489c9b5090e2 Initial load
duke
parents:
diff changeset
   606
  // Give up the search at true merges
489c9b5090e2 Initial load
duke
parents:
diff changeset
   607
  return NULL;                  // Dead loop?  Or hit root?
489c9b5090e2 Initial load
duke
parents:
diff changeset
   608
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   609
190
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   610
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   611
//------------------------------filtered_int_type--------------------------------
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   612
// 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
   613
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
   614
  assert(if_proj &&
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   615
         (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
   616
  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
   617
    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
   618
    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
   619
      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
   620
      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
   621
        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
   622
        if (cmp->in(1) == val) {
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   623
          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
   624
          if (cmp2_t != NULL) {
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   625
            jint lo = cmp2_t->_lo;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   626
            jint hi = cmp2_t->_hi;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   627
            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
   628
            switch (msk) {
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   629
            case BoolTest::ne:
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   630
              // Can't refine type
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   631
              return NULL;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   632
            case BoolTest::eq:
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   633
              return cmp2_t;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   634
            case BoolTest::lt:
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   635
              lo = TypeInt::INT->_lo;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   636
              if (hi - 1 < hi) {
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   637
                hi = hi - 1;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   638
              }
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   639
              break;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   640
            case BoolTest::le:
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   641
              lo = TypeInt::INT->_lo;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   642
              break;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   643
            case BoolTest::gt:
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   644
              if (lo + 1 > lo) {
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   645
                lo = lo + 1;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   646
              }
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   647
              hi = TypeInt::INT->_hi;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   648
              break;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   649
            case BoolTest::ge:
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   650
              // lo unchanged
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   651
              hi = TypeInt::INT->_hi;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   652
              break;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   653
            }
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   654
            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
   655
            return rtn_t;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   656
          }
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   657
        }
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
    }
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   660
  }
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   661
  return NULL;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   662
}
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   663
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   664
//------------------------------fold_compares----------------------------
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   665
// 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
   666
// the direction of this if is determined by the preceding if so it
190
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   667
// can be eliminate entirely.  Given an if testing (CmpI n c) check
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   668
// for an immediately control dependent if that is testing (CmpI n c2)
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   669
// and has one projection leading to this if and the other projection
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   670
// leading to a region that merges one of this ifs control
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   671
// projections.
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
//                   If
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
//                /   |
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   677
//              If    |
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   678
//              /\    |
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   679
//             /  \   |
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   680
//            /    \  |
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   681
//           /    Region
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   682
//
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   683
Node* IfNode::fold_compares(PhaseGVN* phase) {
24479
20eb4c752fb5 8042786: Proper fix for 8032566
kvn
parents: 22911
diff changeset
   684
  if (Opcode() != Op_If) return NULL;
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
  Node* this_cmp = in(1)->in(1);
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   687
  if (this_cmp != NULL && this_cmp->Opcode() == Op_CmpI &&
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   688
      this_cmp->in(2)->is_Con() && this_cmp->in(2) != phase->C->top()) {
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   689
    Node* ctrl = in(0);
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   690
    BoolNode* this_bool = in(1)->as_Bool();
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   691
    Node* n = this_cmp->in(1);
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   692
    int hi = this_cmp->in(2)->get_int();
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   693
    if (ctrl != NULL && ctrl->is_Proj() && ctrl->outcnt() == 1 &&
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   694
        ctrl->in(0)->is_If() &&
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   695
        ctrl->in(0)->outcnt() == 2 &&
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   696
        ctrl->in(0)->in(1)->is_Bool() &&
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   697
        ctrl->in(0)->in(1)->in(1)->Opcode() == Op_CmpI &&
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   698
        ctrl->in(0)->in(1)->in(1)->in(2)->is_Con() &&
20701
ef9996662fd5 8020750: Node::get_int: guarantee(t != NULL) failed: must be con
twisti
parents: 20289
diff changeset
   699
        ctrl->in(0)->in(1)->in(1)->in(2) != phase->C->top() &&
190
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   700
        ctrl->in(0)->in(1)->in(1)->in(1) == n) {
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   701
      IfNode* dom_iff = ctrl->in(0)->as_If();
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   702
      Node* otherproj = dom_iff->proj_out(!ctrl->as_Proj()->_con);
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   703
      if (otherproj->outcnt() == 1 && otherproj->unique_out()->is_Region() &&
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   704
          this_bool->_test._test != BoolTest::ne && this_bool->_test._test != BoolTest::eq) {
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   705
        // Identify which proj goes to the region and which continues on
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   706
        RegionNode* region = otherproj->unique_out()->as_Region();
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   707
        Node* success = NULL;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   708
        Node* fail = NULL;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   709
        for (int i = 0; i < 2; i++) {
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   710
          Node* proj = proj_out(i);
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   711
          if (success == NULL && proj->outcnt() == 1 && proj->unique_out() == region) {
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   712
            success = proj;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   713
          } else if (fail == NULL) {
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   714
            fail = proj;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   715
          } else {
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   716
            success = fail = NULL;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   717
          }
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   718
        }
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   719
        if (success != NULL && fail != NULL && !region->has_phi()) {
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   720
          int lo = dom_iff->in(1)->in(1)->in(2)->get_int();
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   721
          BoolNode* dom_bool = dom_iff->in(1)->as_Bool();
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   722
          Node* dom_cmp =  dom_bool->in(1);
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   723
          const TypeInt* failtype  = filtered_int_type(phase, n, ctrl);
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   724
          if (failtype != NULL) {
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   725
            const TypeInt* type2 = filtered_int_type(phase, n, fail);
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   726
            if (type2 != NULL) {
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   727
              failtype = failtype->join(type2)->is_int();
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   728
            } else {
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   729
              failtype = NULL;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   730
            }
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   731
          }
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   732
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   733
          if (failtype != NULL &&
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   734
              dom_bool->_test._test != BoolTest::ne && dom_bool->_test._test != BoolTest::eq) {
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   735
            int bound = failtype->_hi - failtype->_lo + 1;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   736
            if (failtype->_hi != max_jint && failtype->_lo != min_jint && bound > 1) {
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   737
              // Merge the two compares into a single unsigned compare by building  (CmpU (n - lo) hi)
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   738
              BoolTest::mask cond = fail->as_Proj()->_con ? BoolTest::lt : BoolTest::ge;
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
   739
              Node* adjusted = phase->transform(new SubINode(n, phase->intcon(failtype->_lo)));
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
   740
              Node* newcmp = phase->transform(new CmpUNode(adjusted, phase->intcon(bound)));
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
   741
              Node* newbool = phase->transform(new BoolNode(newcmp, cond));
12958
009b6c9586d8 7173340: C2: code cleanup: use PhaseIterGVN::replace_edge(Node*, int, Node*) where applicable
kvn
parents: 10258
diff changeset
   742
              phase->is_IterGVN()->replace_input_of(dom_iff, 1, phase->intcon(ctrl->as_Proj()->_con));
190
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   743
              phase->hash_delete(this);
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   744
              set_req(1, newbool);
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   745
              return this;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   746
            }
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   747
            if (failtype->_lo > failtype->_hi) {
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   748
              // previous if determines the result of this if so
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   749
              // replace Bool with constant
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   750
              phase->hash_delete(this);
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   751
              set_req(1, phase->intcon(success->as_Proj()->_con));
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   752
              return this;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   753
            }
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   754
          }
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   755
        }
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   756
      }
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   757
    }
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   758
  }
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   759
  return NULL;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   760
}
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
   761
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   762
//------------------------------remove_useless_bool----------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   763
// Check for people making a useless boolean: things like
489c9b5090e2 Initial load
duke
parents:
diff changeset
   764
// if( (x < y ? true : false) ) { ... }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   765
// Replace with if( x < y ) { ... }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   766
static Node *remove_useless_bool(IfNode *iff, PhaseGVN *phase) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   767
  Node *i1 = iff->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   768
  if( !i1->is_Bool() ) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   769
  BoolNode *bol = i1->as_Bool();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   770
489c9b5090e2 Initial load
duke
parents:
diff changeset
   771
  Node *cmp = bol->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   772
  if( cmp->Opcode() != Op_CmpI ) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   773
489c9b5090e2 Initial load
duke
parents:
diff changeset
   774
  // Must be comparing against a bool
489c9b5090e2 Initial load
duke
parents:
diff changeset
   775
  const Type *cmp2_t = phase->type( cmp->in(2) );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   776
  if( cmp2_t != TypeInt::ZERO &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
   777
      cmp2_t != TypeInt::ONE )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   778
    return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   779
489c9b5090e2 Initial load
duke
parents:
diff changeset
   780
  // Find a prior merge point merging the boolean
489c9b5090e2 Initial load
duke
parents:
diff changeset
   781
  i1 = cmp->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   782
  if( !i1->is_Phi() ) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   783
  PhiNode *phi = i1->as_Phi();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   784
  if( phase->type( phi ) != TypeInt::BOOL )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   785
    return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   786
489c9b5090e2 Initial load
duke
parents:
diff changeset
   787
  // Check for diamond pattern
489c9b5090e2 Initial load
duke
parents:
diff changeset
   788
  int true_path = phi->is_diamond_phi();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   789
  if( true_path == 0 ) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   790
958
4c4709e8b7ee 6712835: Server compiler fails with assertion (loop_count < K,"infinite loop in PhaseIterGVN::transform")
never
parents: 190
diff changeset
   791
  // 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
   792
  // 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
   793
  // an illegal cycle.
4c4709e8b7ee 6712835: Server compiler fails with assertion (loop_count < K,"infinite loop in PhaseIterGVN::transform")
never
parents: 190
diff changeset
   794
  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
   795
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   796
  // phi->region->if_proj->ifnode->bool->cmp
489c9b5090e2 Initial load
duke
parents:
diff changeset
   797
  BoolNode *bol2 = phi->in(0)->in(1)->in(0)->in(1)->as_Bool();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   798
489c9b5090e2 Initial load
duke
parents:
diff changeset
   799
  // Now get the 'sense' of the test correct so we can plug in
489c9b5090e2 Initial load
duke
parents:
diff changeset
   800
  // either iff2->in(1) or its complement.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   801
  int flip = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   802
  if( bol->_test._test == BoolTest::ne ) flip = 1-flip;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   803
  else if( bol->_test._test != BoolTest::eq ) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   804
  if( cmp2_t == TypeInt::ZERO ) flip = 1-flip;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   805
489c9b5090e2 Initial load
duke
parents:
diff changeset
   806
  const Type *phi1_t = phase->type( phi->in(1) );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   807
  const Type *phi2_t = phase->type( phi->in(2) );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   808
  // Check for Phi(0,1) and flip
489c9b5090e2 Initial load
duke
parents:
diff changeset
   809
  if( phi1_t == TypeInt::ZERO ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   810
    if( phi2_t != TypeInt::ONE ) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   811
    flip = 1-flip;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   812
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   813
    // Check for Phi(1,0)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   814
    if( phi1_t != TypeInt::ONE  ) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   815
    if( phi2_t != TypeInt::ZERO ) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   816
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   817
  if( true_path == 2 ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   818
    flip = 1-flip;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   819
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   820
489c9b5090e2 Initial load
duke
parents:
diff changeset
   821
  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
   822
  assert(new_bol != iff->in(1), "must make progress");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   823
  iff->set_req(1, new_bol);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   824
  // Intervening diamond probably goes dead
489c9b5090e2 Initial load
duke
parents:
diff changeset
   825
  phase->C->set_major_progress();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   826
  return iff;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   827
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   828
489c9b5090e2 Initial load
duke
parents:
diff changeset
   829
static IfNode* idealize_test(PhaseGVN* phase, IfNode* iff);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   830
28044
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   831
struct RangeCheck {
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   832
  Node* ctl;
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   833
  jint off;
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   834
};
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   835
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   836
//------------------------------Ideal------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   837
// Return a node which is more "ideal" than the current node.  Strip out
489c9b5090e2 Initial load
duke
parents:
diff changeset
   838
// control copies
489c9b5090e2 Initial load
duke
parents:
diff changeset
   839
Node *IfNode::Ideal(PhaseGVN *phase, bool can_reshape) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   840
  if (remove_dead_region(phase, can_reshape))  return this;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   841
  // No Def-Use info?
489c9b5090e2 Initial load
duke
parents:
diff changeset
   842
  if (!can_reshape)  return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   843
  PhaseIterGVN *igvn = phase->is_IterGVN();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   844
489c9b5090e2 Initial load
duke
parents:
diff changeset
   845
  // Don't bother trying to transform a dead if
489c9b5090e2 Initial load
duke
parents:
diff changeset
   846
  if (in(0)->is_top())  return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   847
  // Don't bother trying to transform an if with a dead test
489c9b5090e2 Initial load
duke
parents:
diff changeset
   848
  if (in(1)->is_top())  return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   849
  // Another variation of a dead test
489c9b5090e2 Initial load
duke
parents:
diff changeset
   850
  if (in(1)->is_Con())  return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   851
  // Another variation of a dead if
489c9b5090e2 Initial load
duke
parents:
diff changeset
   852
  if (outcnt() < 2)  return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   853
489c9b5090e2 Initial load
duke
parents:
diff changeset
   854
  // Canonicalize the test.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   855
  Node* idt_if = idealize_test(phase, this);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   856
  if (idt_if != NULL)  return idt_if;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   857
489c9b5090e2 Initial load
duke
parents:
diff changeset
   858
  // Try to split the IF
489c9b5090e2 Initial load
duke
parents:
diff changeset
   859
  Node *s = split_if(this, igvn);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   860
  if (s != NULL)  return s;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   861
489c9b5090e2 Initial load
duke
parents:
diff changeset
   862
  // Check for people making a useless boolean: things like
489c9b5090e2 Initial load
duke
parents:
diff changeset
   863
  // if( (x < y ? true : false) ) { ... }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   864
  // Replace with if( x < y ) { ... }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   865
  Node *bol2 = remove_useless_bool(this, phase);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   866
  if( bol2 ) return bol2;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   867
489c9b5090e2 Initial load
duke
parents:
diff changeset
   868
  // Setup to scan up the CFG looking for a dominating test
489c9b5090e2 Initial load
duke
parents:
diff changeset
   869
  Node *dom = in(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   870
  Node *prev_dom = this;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   871
489c9b5090e2 Initial load
duke
parents:
diff changeset
   872
  // Check for range-check vs other kinds of tests
489c9b5090e2 Initial load
duke
parents:
diff changeset
   873
  Node *index1, *range1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   874
  jint offset1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   875
  int flip1 = is_range_check(range1, index1, offset1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   876
  if( flip1 ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   877
    // Try to remove extra range checks.  All 'up_one_dom' gives up at merges
489c9b5090e2 Initial load
duke
parents:
diff changeset
   878
    // so all checks we inspect post-dominate the top-most check we find.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   879
    // 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
   880
    // 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
   881
    // We 'expand' the top 3 range checks to include all post-dominating
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   882
    // checks.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   883
28044
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   884
    // The top 3 range checks seen
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   885
    const int NRC =3;
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   886
    RangeCheck prev_checks[NRC];
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   887
    int nb_checks = 0;
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   888
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   889
    // Low and high offsets seen so far
489c9b5090e2 Initial load
duke
parents:
diff changeset
   890
    jint off_lo = offset1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   891
    jint off_hi = offset1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   892
28044
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   893
    bool found_immediate_dominator = false;
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   894
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   895
    // 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
   896
    for (int dist = 0; dist < 999; dist++) { // Range-Check scan limit
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   897
      if (dom->Opcode() == Op_If &&  // Not same opcode?
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   898
          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
   899
        if (dom == this) return NULL; // dead loop
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   900
        // See if this is a range check
489c9b5090e2 Initial load
duke
parents:
diff changeset
   901
        Node *index2, *range2;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   902
        jint offset2;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   903
        int flip2 = dom->as_If()->is_range_check(range2, index2, offset2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   904
        // See if this is a _matching_ range check, checking against
489c9b5090e2 Initial load
duke
parents:
diff changeset
   905
        // the same array bounds.
28044
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   906
        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
   907
            dom->outcnt() == 2) {
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   908
          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
   909
            // 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
   910
            // 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
   911
            // 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
   912
            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
   913
            found_immediate_dominator = true;
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   914
            break;
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   915
          }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   916
          // Gather expanded bounds
489c9b5090e2 Initial load
duke
parents:
diff changeset
   917
          off_lo = MIN2(off_lo,offset2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   918
          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
   919
          // Record top NRC range checks
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   920
          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
   921
          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
   922
          nb_checks++;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   923
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   924
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   925
      prev_dom = dom;
28044
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   926
      dom = up_one_dom(dom);
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   927
      if (!dom) break;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   928
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   929
28044
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   930
    if (!found_immediate_dominator) {
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   931
      // 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
   932
      // 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
   933
      // 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
   934
      // 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
   935
      if (!phase->C->allow_range_check_smearing())  return NULL;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   936
28044
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   937
      // 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
   938
      if (nb_checks == 0) {
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   939
        return NULL;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   940
      }
28044
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   941
      // 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
   942
      // 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
   943
      int chk0 = (nb_checks - 1) % NRC;
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   944
      if (index1) {
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   945
        if (nb_checks == 1) {
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   946
          return NULL;
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   947
        } else {
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   948
          // 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
   949
          // 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
   950
          // range of constants.
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   951
          RangeCheck rc0 = prev_checks[chk0];
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   952
          int chk1 = (nb_checks - 2) % NRC;
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   953
          RangeCheck rc1 = prev_checks[chk1];
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   954
          if (rc0.off == off_lo) {
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   955
            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
   956
            prev_dom = rc1.ctl;
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   957
          } else if (rc0.off == off_hi) {
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   958
            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
   959
            prev_dom = rc1.ctl;
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   960
          } else {
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   961
            // 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
   962
            // 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
   963
            // 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
   964
            // 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
   965
            // bounds.
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   966
            if (nb_checks == 2) {
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   967
              return NULL;
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   968
            }
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   969
            int chk2 = (nb_checks - 3) % NRC;
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   970
            RangeCheck rc2 = prev_checks[chk2];
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   971
            // 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
   972
            // 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
   973
            if (rc1.off <= rc0.off) {
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   974
              // 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
   975
              // -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
   976
              // 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
   977
              // -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
   978
              // 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
   979
              // -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
   980
              // 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
   981
              // -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
   982
              // 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
   983
              // -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
   984
              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
   985
              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
   986
            } else {
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
   987
              // 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
   988
              // -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
   989
              // 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
   990
              // -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
   991
              // 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
   992
              // -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
   993
              // 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
   994
              // -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
   995
              // 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
   996
              // -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
   997
              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
   998
              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
   999
            }
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1000
            prev_dom = rc2.ctl;
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1001
          }
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1002
        }
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1003
      } else {
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1004
        RangeCheck rc0 = prev_checks[chk0];
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1005
        // '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
  1006
        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
  1007
        // 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
  1008
        prev_dom = rc0.ctl;
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1009
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1010
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1011
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1012
  } else {                      // Scan for an equivalent test
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1013
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1014
    Node *cmp;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1015
    int dist = 0;               // Cutoff limit for search
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1016
    int op = Opcode();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1017
    if( op == Op_If &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1018
        (cmp=in(1)->in(1))->Opcode() == Op_CmpP ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1019
      if( cmp->in(2) != NULL && // make sure cmp is not already dead
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1020
          cmp->in(2)->bottom_type() == TypePtr::NULL_PTR ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1021
        dist = 64;              // Limit for null-pointer scans
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1022
      } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1023
        dist = 4;               // Do not bother for random pointer tests
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1024
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1025
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1026
      dist = 4;                 // Limit for random junky scans
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1027
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1028
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1029
    // Normal equivalent-test check.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1030
    if( !dom ) return NULL;     // Dead loop?
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1031
190
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
  1032
    Node* result = fold_compares(phase);
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
  1033
    if (result != NULL) {
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
  1034
      return result;
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
  1035
    }
e9a0a9dcd4f6 6395208: Elide autoboxing for calls to HashMap.get(int) and HashMap.get(long)
never
parents: 1
diff changeset
  1036
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1037
    // Search up the dominator tree for an If with an identical test
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1038
    while( dom->Opcode() != op    ||  // Not same opcode?
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1039
           dom->in(1)    != in(1) ||  // Not same input 1?
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1040
           (req() == 3 && dom->in(2) != in(2)) || // Not same input 2?
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1041
           prev_dom->in(0) != dom ) { // One path of test does not dominate?
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1042
      if( dist < 0 ) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1043
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1044
      dist--;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1045
      prev_dom = dom;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1046
      dom = up_one_dom( dom );
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1047
      if( !dom ) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1048
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1049
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1050
    // Check that we did not follow a loop back to ourselves
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1051
    if( this == dom )
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1052
      return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1053
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1054
    if( dist > 2 )              // Add to count of NULL checks elided
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1055
      explicit_null_checks_elided++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1056
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1057
  } // End of Else scan for an equivalent test
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1058
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1059
  // Hit!  Remove this IF
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1060
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1061
  if( TraceIterativeGVN ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1062
    tty->print("   Removing IfNode: "); this->dump();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1063
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1064
  if( VerifyOpto && !phase->allow_progress() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1065
    // Found an equivalent dominating test,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1066
    // we can not guarantee reaching a fix-point for these during iterativeGVN
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1067
    // since intervening nodes may not change.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1068
    return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1069
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1070
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1071
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1072
  // Replace dominated IfNode
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1073
  dominated_by( prev_dom, igvn );
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1074
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1075
  // Must return either the original node (now dead) or a new node
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1076
  // (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
  1077
  return new ConINode(TypeInt::ZERO);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1078
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1079
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1080
//------------------------------dominated_by-----------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1081
void IfNode::dominated_by( Node *prev_dom, PhaseIterGVN *igvn ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1082
  igvn->hash_delete(this);      // Remove self to prevent spurious V-N
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1083
  Node *idom = in(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1084
  // Need opcode to decide which way 'this' test goes
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1085
  int prev_op = prev_dom->Opcode();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1086
  Node *top = igvn->C->top(); // Shortcut to top
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1087
10253
35b975b1e8f3 7070134: Hotspot crashes with sigsegv from PorterStemmer
kvn
parents: 9449
diff changeset
  1088
  // Loop predicates may have depending checks which should not
35b975b1e8f3 7070134: Hotspot crashes with sigsegv from PorterStemmer
kvn
parents: 9449
diff changeset
  1089
  // be skipped. For example, range check predicate has two checks
35b975b1e8f3 7070134: Hotspot crashes with sigsegv from PorterStemmer
kvn
parents: 9449
diff changeset
  1090
  // for lower and upper bounds.
35b975b1e8f3 7070134: Hotspot crashes with sigsegv from PorterStemmer
kvn
parents: 9449
diff changeset
  1091
  ProjNode* unc_proj = proj_out(1 - prev_dom->as_Proj()->_con)->as_Proj();
21089
e1986ff6fe2e 8024069: replace_in_map() should operate on parent maps
roland
parents: 20701
diff changeset
  1092
  if (unc_proj->is_uncommon_trap_proj(Deoptimization::Reason_predicate))
28044
ede40159fd3b 8066103: C2's range check smearing allows out of bound array accesses
roland
parents: 26173
diff changeset
  1093
   prev_dom = idom;
10253
35b975b1e8f3 7070134: Hotspot crashes with sigsegv from PorterStemmer
kvn
parents: 9449
diff changeset
  1094
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1095
  // Now walk the current IfNode's projections.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1096
  // Loop ends when 'this' has no more uses.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1097
  for (DUIterator_Last imin, i = last_outs(imin); i >= imin; --i) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1098
    Node *ifp = last_out(i);     // Get IfTrue/IfFalse
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1099
    igvn->add_users_to_worklist(ifp);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1100
    // Check which projection it is and set target.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1101
    // Data-target is either the dominating projection of the same type
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1102
    // or TOP if the dominating projection is of opposite type.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1103
    // Data-target will be used as the new control edge for the non-CFG
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1104
    // nodes like Casts and Loads.
10253
35b975b1e8f3 7070134: Hotspot crashes with sigsegv from PorterStemmer
kvn
parents: 9449
diff changeset
  1105
    Node *data_target = (ifp->Opcode() == prev_op) ? prev_dom : top;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1106
    // Control-target is just the If's immediate dominator or TOP.
10253
35b975b1e8f3 7070134: Hotspot crashes with sigsegv from PorterStemmer
kvn
parents: 9449
diff changeset
  1107
    Node *ctrl_target = (ifp->Opcode() == prev_op) ?     idom : top;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1108
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1109
    // For each child of an IfTrue/IfFalse projection, reroute.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1110
    // Loop ends when projection has no more uses.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1111
    for (DUIterator_Last jmin, j = ifp->last_outs(jmin); j >= jmin; --j) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1112
      Node* s = ifp->last_out(j);   // Get child of IfTrue/IfFalse
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1113
      if( !s->depends_only_on_test() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1114
        // Find the control input matching this def-use edge.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1115
        // For Regions it may not be in slot 0.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1116
        uint l;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1117
        for( l = 0; s->in(l) != ifp; l++ ) { }
12958
009b6c9586d8 7173340: C2: code cleanup: use PhaseIterGVN::replace_edge(Node*, int, Node*) where applicable
kvn
parents: 10258
diff changeset
  1118
        igvn->replace_input_of(s, l, ctrl_target);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1119
      } else {                      // Else, for control producers,
12958
009b6c9586d8 7173340: C2: code cleanup: use PhaseIterGVN::replace_edge(Node*, int, Node*) where applicable
kvn
parents: 10258
diff changeset
  1120
        igvn->replace_input_of(s, 0, data_target); // Move child to data-target
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1121
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1122
    } // End for each child of a projection
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1123
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1124
    igvn->remove_dead_node(ifp);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1125
  } // End for each IfTrue/IfFalse child of If
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1126
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1127
  // Kill the IfNode
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1128
  igvn->remove_dead_node(this);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1129
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1130
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1131
//------------------------------Identity---------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1132
// If the test is constant & we match, then we are the input Control
28486
b0df113b962e 8027626: assert(Opcode() != Op_If || outcnt() == 2) failed: bad if #1
roland
parents: 28044
diff changeset
  1133
Node *IfProjNode::Identity(PhaseTransform *phase) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1134
  // Can only optimize if cannot go the other way
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1135
  const TypeTuple *t = phase->type(in(0))->is_tuple();
28486
b0df113b962e 8027626: assert(Opcode() != Op_If || outcnt() == 2) failed: bad if #1
roland
parents: 28044
diff changeset
  1136
  if (t == TypeTuple::IFNEITHER ||
b0df113b962e 8027626: assert(Opcode() != Op_If || outcnt() == 2) failed: bad if #1
roland
parents: 28044
diff changeset
  1137
      // kill dead branch first otherwise the IfNode's control will
b0df113b962e 8027626: assert(Opcode() != Op_If || outcnt() == 2) failed: bad if #1
roland
parents: 28044
diff changeset
  1138
      // have 2 control uses (the IfNode that doesn't go away because
b0df113b962e 8027626: assert(Opcode() != Op_If || outcnt() == 2) failed: bad if #1
roland
parents: 28044
diff changeset
  1139
      // it still has uses and this branch of the
b0df113b962e 8027626: assert(Opcode() != Op_If || outcnt() == 2) failed: bad if #1
roland
parents: 28044
diff changeset
  1140
      // If). Node::has_special_unique_user() will cause this node to
b0df113b962e 8027626: assert(Opcode() != Op_If || outcnt() == 2) failed: bad if #1
roland
parents: 28044
diff changeset
  1141
      // be reprocessed once the dead branch is killed.
b0df113b962e 8027626: assert(Opcode() != Op_If || outcnt() == 2) failed: bad if #1
roland
parents: 28044
diff changeset
  1142
      (always_taken(t) && in(0)->outcnt() == 1)) {
b0df113b962e 8027626: assert(Opcode() != Op_If || outcnt() == 2) failed: bad if #1
roland
parents: 28044
diff changeset
  1143
    // IfNode control
b0df113b962e 8027626: assert(Opcode() != Op_If || outcnt() == 2) failed: bad if #1
roland
parents: 28044
diff changeset
  1144
    return in(0)->in(0);
b0df113b962e 8027626: assert(Opcode() != Op_If || outcnt() == 2) failed: bad if #1
roland
parents: 28044
diff changeset
  1145
  }
b0df113b962e 8027626: assert(Opcode() != Op_If || outcnt() == 2) failed: bad if #1
roland
parents: 28044
diff changeset
  1146
  // no progress
b0df113b962e 8027626: assert(Opcode() != Op_If || outcnt() == 2) failed: bad if #1
roland
parents: 28044
diff changeset
  1147
  return this;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1148
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1149
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1150
//------------------------------dump_spec--------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1151
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1152
void IfNode::dump_spec(outputStream *st) const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1153
  st->print("P=%f, C=%f",_prob,_fcnt);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1154
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1155
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1156
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1157
//------------------------------idealize_test----------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1158
// Try to canonicalize tests better.  Peek at the Cmp/Bool/If sequence and
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1159
// come up with a canonical sequence.  Bools getting 'eq', 'gt' and 'ge' forms
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1160
// converted to 'ne', 'le' and 'lt' forms.  IfTrue/IfFalse get swapped as
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1161
// needed.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1162
static IfNode* idealize_test(PhaseGVN* phase, IfNode* iff) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1163
  assert(iff->in(0) != NULL, "If must be live");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1164
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1165
  if (iff->outcnt() != 2)  return NULL; // Malformed projections.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1166
  Node* old_if_f = iff->proj_out(false);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1167
  Node* old_if_t = iff->proj_out(true);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1168
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1169
  // CountedLoopEnds want the back-control test to be TRUE, irregardless of
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1170
  // whether they are testing a 'gt' or 'lt' condition.  The 'gt' condition
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1171
  // happens in count-down loops
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1172
  if (iff->is_CountedLoopEnd())  return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1173
  if (!iff->in(1)->is_Bool())  return NULL; // Happens for partially optimized IF tests
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1174
  BoolNode *b = iff->in(1)->as_Bool();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1175
  BoolTest bt = b->_test;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1176
  // Test already in good order?
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1177
  if( bt.is_canonical() )
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1178
    return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1179
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1180
  // Flip test to be canonical.  Requires flipping the IfFalse/IfTrue and
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1181
  // cloning the IfNode.
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
  1182
  Node* new_b = phase->transform( new BoolNode(b->in(1), bt.negate()) );
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1183
  if( !new_b->is_Bool() ) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1184
  b = new_b->as_Bool();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1185
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1186
  PhaseIterGVN *igvn = phase->is_IterGVN();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1187
  assert( igvn, "Test is not canonical in parser?" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1188
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1189
  // The IF node never really changes, but it needs to be cloned
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
  1190
  iff = new IfNode( iff->in(0), b, 1.0-iff->_prob, iff->_fcnt);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1191
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1192
  Node *prior = igvn->hash_find_insert(iff);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1193
  if( prior ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1194
    igvn->remove_dead_node(iff);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1195
    iff = (IfNode*)prior;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1196
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1197
    // Cannot call transform on it just yet
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1198
    igvn->set_type_bottom(iff);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1199
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1200
  igvn->_worklist.push(iff);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1201
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1202
  // Now handle projections.  Cloning not required.
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
  1203
  Node* new_if_f = (Node*)(new IfFalseNode( iff ));
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
  1204
  Node* new_if_t = (Node*)(new IfTrueNode ( iff ));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1205
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1206
  igvn->register_new_node_with_optimizer(new_if_f);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1207
  igvn->register_new_node_with_optimizer(new_if_t);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1208
  // Flip test, so flip trailing control
5901
c046f8e9c52b 6677629: PhaseIterGVN::subsume_node() should call hash_delete() and add_users_to_worklist()
kvn
parents: 5547
diff changeset
  1209
  igvn->replace_node(old_if_f, new_if_t);
c046f8e9c52b 6677629: PhaseIterGVN::subsume_node() should call hash_delete() and add_users_to_worklist()
kvn
parents: 5547
diff changeset
  1210
  igvn->replace_node(old_if_t, new_if_f);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1211
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1212
  // Progress
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1213
  return iff;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1214
}