hotspot/src/share/vm/opto/cfgnode.cpp
author zmajo
Mon, 18 Apr 2016 11:22:51 +0200
changeset 38045 808cbf1958ce
parent 37248 11a660dbbb8e
child 40334 5925b30a8f74
permissions -rw-r--r--
8153357: C2 creates incorrect cast after eliminating phi with unique input Summary: Refine C2's logic to determine the type of cast node added. Reviewed-by: kvn
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     1
/*
37248
11a660dbbb8e 8132524: Missing includes to resourceArea.hpp
jprovino
parents: 36611
diff changeset
     2
 * Copyright (c) 1997, 2016, 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: 5538
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 5538
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: 5538
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 "classfile/systemDictionary.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5901
diff changeset
    27
#include "memory/allocation.inline.hpp"
37248
11a660dbbb8e 8132524: Missing includes to resourceArea.hpp
jprovino
parents: 36611
diff changeset
    28
#include "memory/resourceArea.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5901
diff changeset
    29
#include "oops/objArrayKlass.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5901
diff changeset
    30
#include "opto/addnode.hpp"
35545
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
    31
#include "opto/castnode.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5901
diff changeset
    32
#include "opto/cfgnode.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5901
diff changeset
    33
#include "opto/connode.hpp"
23528
8f1a7f5e8066 8001532: C2 node files refactoring
morris
parents: 22799
diff changeset
    34
#include "opto/convertnode.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5901
diff changeset
    35
#include "opto/loopnode.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5901
diff changeset
    36
#include "opto/machnode.hpp"
23528
8f1a7f5e8066 8001532: C2 node files refactoring
morris
parents: 22799
diff changeset
    37
#include "opto/movenode.hpp"
8f1a7f5e8066 8001532: C2 node files refactoring
morris
parents: 22799
diff changeset
    38
#include "opto/narrowptrnode.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5901
diff changeset
    39
#include "opto/mulnode.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5901
diff changeset
    40
#include "opto/phaseX.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5901
diff changeset
    41
#include "opto/regmask.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5901
diff changeset
    42
#include "opto/runtime.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5901
diff changeset
    43
#include "opto/subnode.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5901
diff changeset
    44
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    45
// Portions of code courtesy of Clifford Click
489c9b5090e2 Initial load
duke
parents:
diff changeset
    46
489c9b5090e2 Initial load
duke
parents:
diff changeset
    47
// Optimization - Graph Style
489c9b5090e2 Initial load
duke
parents:
diff changeset
    48
489c9b5090e2 Initial load
duke
parents:
diff changeset
    49
//=============================================================================
489c9b5090e2 Initial load
duke
parents:
diff changeset
    50
//------------------------------Value------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
    51
// Compute the type of the RegionNode.
35551
36ef3841fb34 8146629: Make phase->is_IterGVN() accessible from Node::Identity and Node::Value
thartmann
parents: 35549
diff changeset
    52
const Type* RegionNode::Value(PhaseGVN* phase) const {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    53
  for( uint i=1; i<req(); ++i ) {       // For all paths in
489c9b5090e2 Initial load
duke
parents:
diff changeset
    54
    Node *n = in(i);            // Get Control source
489c9b5090e2 Initial load
duke
parents:
diff changeset
    55
    if( !n ) continue;          // Missing inputs are TOP
489c9b5090e2 Initial load
duke
parents:
diff changeset
    56
    if( phase->type(n) == Type::CONTROL )
489c9b5090e2 Initial load
duke
parents:
diff changeset
    57
      return Type::CONTROL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    58
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    59
  return Type::TOP;             // All paths dead?  Then so are we
489c9b5090e2 Initial load
duke
parents:
diff changeset
    60
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    61
489c9b5090e2 Initial load
duke
parents:
diff changeset
    62
//------------------------------Identity---------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
    63
// Check for Region being Identity.
35551
36ef3841fb34 8146629: Make phase->is_IterGVN() accessible from Node::Identity and Node::Value
thartmann
parents: 35549
diff changeset
    64
Node* RegionNode::Identity(PhaseGVN* phase) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    65
  // Cannot have Region be an identity, even if it has only 1 input.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    66
  // Phi users cannot have their Region input folded away for them,
489c9b5090e2 Initial load
duke
parents:
diff changeset
    67
  // since they need to select the proper data input
489c9b5090e2 Initial load
duke
parents:
diff changeset
    68
  return this;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    69
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    70
489c9b5090e2 Initial load
duke
parents:
diff changeset
    71
//------------------------------merge_region-----------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
    72
// If a Region flows into a Region, merge into one big happy merge.  This is
489c9b5090e2 Initial load
duke
parents:
diff changeset
    73
// hard to do if there is stuff that has to happen
489c9b5090e2 Initial load
duke
parents:
diff changeset
    74
static Node *merge_region(RegionNode *region, PhaseGVN *phase) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    75
  if( region->Opcode() != Op_Region ) // Do not do to LoopNodes
489c9b5090e2 Initial load
duke
parents:
diff changeset
    76
    return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    77
  Node *progress = NULL;        // Progress flag
489c9b5090e2 Initial load
duke
parents:
diff changeset
    78
  PhaseIterGVN *igvn = phase->is_IterGVN();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    79
489c9b5090e2 Initial load
duke
parents:
diff changeset
    80
  uint rreq = region->req();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    81
  for( uint i = 1; i < rreq; i++ ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    82
    Node *r = region->in(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    83
    if( r && r->Opcode() == Op_Region && // Found a region?
489c9b5090e2 Initial load
duke
parents:
diff changeset
    84
        r->in(0) == r &&        // Not already collapsed?
489c9b5090e2 Initial load
duke
parents:
diff changeset
    85
        r != region &&          // Avoid stupid situations
489c9b5090e2 Initial load
duke
parents:
diff changeset
    86
        r->outcnt() == 2 ) {    // Self user and 'region' user only?
489c9b5090e2 Initial load
duke
parents:
diff changeset
    87
      assert(!r->as_Region()->has_phi(), "no phi users");
489c9b5090e2 Initial load
duke
parents:
diff changeset
    88
      if( !progress ) {         // No progress
489c9b5090e2 Initial load
duke
parents:
diff changeset
    89
        if (region->has_phi()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    90
          return NULL;        // Only flatten if no Phi users
489c9b5090e2 Initial load
duke
parents:
diff changeset
    91
          // igvn->hash_delete( phi );
489c9b5090e2 Initial load
duke
parents:
diff changeset
    92
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    93
        igvn->hash_delete( region );
489c9b5090e2 Initial load
duke
parents:
diff changeset
    94
        progress = region;      // Making progress
489c9b5090e2 Initial load
duke
parents:
diff changeset
    95
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    96
      igvn->hash_delete( r );
489c9b5090e2 Initial load
duke
parents:
diff changeset
    97
489c9b5090e2 Initial load
duke
parents:
diff changeset
    98
      // Append inputs to 'r' onto 'region'
489c9b5090e2 Initial load
duke
parents:
diff changeset
    99
      for( uint j = 1; j < r->req(); j++ ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   100
        // Move an input from 'r' to 'region'
489c9b5090e2 Initial load
duke
parents:
diff changeset
   101
        region->add_req(r->in(j));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   102
        r->set_req(j, phase->C->top());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   103
        // Update phis of 'region'
489c9b5090e2 Initial load
duke
parents:
diff changeset
   104
        //for( uint k = 0; k < max; k++ ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   105
        //  Node *phi = region->out(k);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   106
        //  if( phi->is_Phi() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   107
        //    phi->add_req(phi->in(i));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   108
        //  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   109
        //}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   110
489c9b5090e2 Initial load
duke
parents:
diff changeset
   111
        rreq++;                 // One more input to Region
489c9b5090e2 Initial load
duke
parents:
diff changeset
   112
      } // Found a region to merge into Region
25913
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 24923
diff changeset
   113
      igvn->_worklist.push(r);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   114
      // Clobber pointer to the now dead 'r'
489c9b5090e2 Initial load
duke
parents:
diff changeset
   115
      region->set_req(i, phase->C->top());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   116
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   117
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   118
489c9b5090e2 Initial load
duke
parents:
diff changeset
   119
  return progress;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   120
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   121
489c9b5090e2 Initial load
duke
parents:
diff changeset
   122
489c9b5090e2 Initial load
duke
parents:
diff changeset
   123
489c9b5090e2 Initial load
duke
parents:
diff changeset
   124
//--------------------------------has_phi--------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   125
// Helper function: Return any PhiNode that uses this region or NULL
489c9b5090e2 Initial load
duke
parents:
diff changeset
   126
PhiNode* RegionNode::has_phi() const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   127
  for (DUIterator_Fast imax, i = fast_outs(imax); i < imax; i++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   128
    Node* phi = fast_out(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   129
    if (phi->is_Phi()) {   // Check for Phi users
489c9b5090e2 Initial load
duke
parents:
diff changeset
   130
      assert(phi->in(0) == (Node*)this, "phi uses region only via in(0)");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   131
      return phi->as_Phi();  // this one is good enough
489c9b5090e2 Initial load
duke
parents:
diff changeset
   132
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   133
  }
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
489c9b5090e2 Initial load
duke
parents:
diff changeset
   138
489c9b5090e2 Initial load
duke
parents:
diff changeset
   139
//-----------------------------has_unique_phi----------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   140
// Helper function: Return the only PhiNode that uses this region or NULL
489c9b5090e2 Initial load
duke
parents:
diff changeset
   141
PhiNode* RegionNode::has_unique_phi() const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   142
  // Check that only one use is a Phi
489c9b5090e2 Initial load
duke
parents:
diff changeset
   143
  PhiNode* only_phi = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   144
  for (DUIterator_Fast imax, i = fast_outs(imax); i < imax; i++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   145
    Node* phi = fast_out(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   146
    if (phi->is_Phi()) {   // Check for Phi users
489c9b5090e2 Initial load
duke
parents:
diff changeset
   147
      assert(phi->in(0) == (Node*)this, "phi uses region only via in(0)");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   148
      if (only_phi == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   149
        only_phi = phi->as_Phi();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   150
      } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   151
        return NULL;  // multiple phis
489c9b5090e2 Initial load
duke
parents:
diff changeset
   152
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   153
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   154
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   155
489c9b5090e2 Initial load
duke
parents:
diff changeset
   156
  return only_phi;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   157
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   158
489c9b5090e2 Initial load
duke
parents:
diff changeset
   159
489c9b5090e2 Initial load
duke
parents:
diff changeset
   160
//------------------------------check_phi_clipping-----------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   161
// Helper function for RegionNode's identification of FP clipping
489c9b5090e2 Initial load
duke
parents:
diff changeset
   162
// Check inputs to the Phi
489c9b5090e2 Initial load
duke
parents:
diff changeset
   163
static bool check_phi_clipping( PhiNode *phi, ConNode * &min, uint &min_idx, ConNode * &max, uint &max_idx, Node * &val, uint &val_idx ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   164
  min     = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   165
  max     = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   166
  val     = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   167
  min_idx = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   168
  max_idx = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   169
  val_idx = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   170
  uint  phi_max = phi->req();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   171
  if( phi_max == 4 ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   172
    for( uint j = 1; j < phi_max; ++j ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   173
      Node *n = phi->in(j);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   174
      int opcode = n->Opcode();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   175
      switch( opcode ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   176
      case Op_ConI:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   177
        {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   178
          if( min == NULL ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   179
            min     = n->Opcode() == Op_ConI ? (ConNode*)n : NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   180
            min_idx = j;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   181
          } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   182
            max     = n->Opcode() == Op_ConI ? (ConNode*)n : NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   183
            max_idx = j;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   184
            if( min->get_int() > max->get_int() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   185
              // Swap min and max
489c9b5090e2 Initial load
duke
parents:
diff changeset
   186
              ConNode *temp;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   187
              uint     temp_idx;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   188
              temp     = min;     min     = max;     max     = temp;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   189
              temp_idx = min_idx; min_idx = max_idx; max_idx = temp_idx;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   190
            }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   191
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   192
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   193
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   194
      default:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   195
        {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   196
          val = n;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   197
          val_idx = j;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   198
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   199
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   200
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   201
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   202
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   203
  return ( min && max && val && (min->get_int() <= 0) && (max->get_int() >=0) );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   204
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   205
489c9b5090e2 Initial load
duke
parents:
diff changeset
   206
489c9b5090e2 Initial load
duke
parents:
diff changeset
   207
//------------------------------check_if_clipping------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   208
// Helper function for RegionNode's identification of FP clipping
489c9b5090e2 Initial load
duke
parents:
diff changeset
   209
// Check that inputs to Region come from two IfNodes,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   210
//
489c9b5090e2 Initial load
duke
parents:
diff changeset
   211
//            If
489c9b5090e2 Initial load
duke
parents:
diff changeset
   212
//      False    True
489c9b5090e2 Initial load
duke
parents:
diff changeset
   213
//       If        |
489c9b5090e2 Initial load
duke
parents:
diff changeset
   214
//  False  True    |
489c9b5090e2 Initial load
duke
parents:
diff changeset
   215
//    |      |     |
489c9b5090e2 Initial load
duke
parents:
diff changeset
   216
//  RegionNode_inputs
489c9b5090e2 Initial load
duke
parents:
diff changeset
   217
//
489c9b5090e2 Initial load
duke
parents:
diff changeset
   218
static bool check_if_clipping( const RegionNode *region, IfNode * &bot_if, IfNode * &top_if ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   219
  top_if = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   220
  bot_if = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   221
489c9b5090e2 Initial load
duke
parents:
diff changeset
   222
  // Check control structure above RegionNode for (if  ( if  ) )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   223
  Node *in1 = region->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   224
  Node *in2 = region->in(2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   225
  Node *in3 = region->in(3);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   226
  // Check that all inputs are projections
489c9b5090e2 Initial load
duke
parents:
diff changeset
   227
  if( in1->is_Proj() && in2->is_Proj() && in3->is_Proj() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   228
    Node *in10 = in1->in(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   229
    Node *in20 = in2->in(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   230
    Node *in30 = in3->in(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   231
    // Check that #1 and #2 are ifTrue and ifFalse from same If
489c9b5090e2 Initial load
duke
parents:
diff changeset
   232
    if( in10 != NULL && in10->is_If() &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
   233
        in20 != NULL && in20->is_If() &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
   234
        in30 != NULL && in30->is_If() && in10 == in20 &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
   235
        (in1->Opcode() != in2->Opcode()) ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   236
      Node  *in100 = in10->in(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   237
      Node *in1000 = (in100 != NULL && in100->is_Proj()) ? in100->in(0) : NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   238
      // Check that control for in10 comes from other branch of IF from in3
489c9b5090e2 Initial load
duke
parents:
diff changeset
   239
      if( in1000 != NULL && in1000->is_If() &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
   240
          in30 == in1000 && (in3->Opcode() != in100->Opcode()) ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   241
        // Control pattern checks
489c9b5090e2 Initial load
duke
parents:
diff changeset
   242
        top_if = (IfNode*)in1000;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   243
        bot_if = (IfNode*)in10;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   244
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   245
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   246
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   247
489c9b5090e2 Initial load
duke
parents:
diff changeset
   248
  return (top_if != NULL);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   249
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   250
489c9b5090e2 Initial load
duke
parents:
diff changeset
   251
489c9b5090e2 Initial load
duke
parents:
diff changeset
   252
//------------------------------check_convf2i_clipping-------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   253
// Helper function for RegionNode's identification of FP clipping
489c9b5090e2 Initial load
duke
parents:
diff changeset
   254
// Verify that the value input to the phi comes from "ConvF2I; LShift; RShift"
489c9b5090e2 Initial load
duke
parents:
diff changeset
   255
static bool check_convf2i_clipping( PhiNode *phi, uint idx, ConvF2INode * &convf2i, Node *min, Node *max) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   256
  convf2i = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   257
489c9b5090e2 Initial load
duke
parents:
diff changeset
   258
  // Check for the RShiftNode
489c9b5090e2 Initial load
duke
parents:
diff changeset
   259
  Node *rshift = phi->in(idx);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   260
  assert( rshift, "Previous checks ensure phi input is present");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   261
  if( rshift->Opcode() != Op_RShiftI )  { return false; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   262
489c9b5090e2 Initial load
duke
parents:
diff changeset
   263
  // Check for the LShiftNode
489c9b5090e2 Initial load
duke
parents:
diff changeset
   264
  Node *lshift = rshift->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   265
  assert( lshift, "Previous checks ensure phi input is present");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   266
  if( lshift->Opcode() != Op_LShiftI )  { return false; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   267
489c9b5090e2 Initial load
duke
parents:
diff changeset
   268
  // Check for the ConvF2INode
489c9b5090e2 Initial load
duke
parents:
diff changeset
   269
  Node *conv = lshift->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   270
  if( conv->Opcode() != Op_ConvF2I ) { return false; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   271
489c9b5090e2 Initial load
duke
parents:
diff changeset
   272
  // Check that shift amounts are only to get sign bits set after F2I
489c9b5090e2 Initial load
duke
parents:
diff changeset
   273
  jint max_cutoff     = max->get_int();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   274
  jint min_cutoff     = min->get_int();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   275
  jint left_shift     = lshift->in(2)->get_int();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   276
  jint right_shift    = rshift->in(2)->get_int();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   277
  jint max_post_shift = nth_bit(BitsPerJavaInteger - left_shift - 1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   278
  if( left_shift != right_shift ||
489c9b5090e2 Initial load
duke
parents:
diff changeset
   279
      0 > left_shift || left_shift >= BitsPerJavaInteger ||
489c9b5090e2 Initial load
duke
parents:
diff changeset
   280
      max_post_shift < max_cutoff ||
489c9b5090e2 Initial load
duke
parents:
diff changeset
   281
      max_post_shift < -min_cutoff ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   282
    // Shifts are necessary but current transformation eliminates them
489c9b5090e2 Initial load
duke
parents:
diff changeset
   283
    return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   284
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   285
489c9b5090e2 Initial load
duke
parents:
diff changeset
   286
  // OK to return the result of ConvF2I without shifting
489c9b5090e2 Initial load
duke
parents:
diff changeset
   287
  convf2i = (ConvF2INode*)conv;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   288
  return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   289
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   290
489c9b5090e2 Initial load
duke
parents:
diff changeset
   291
489c9b5090e2 Initial load
duke
parents:
diff changeset
   292
//------------------------------check_compare_clipping-------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   293
// Helper function for RegionNode's identification of FP clipping
489c9b5090e2 Initial load
duke
parents:
diff changeset
   294
static bool check_compare_clipping( bool less_than, IfNode *iff, ConNode *limit, Node * & input ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   295
  Node *i1 = iff->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   296
  if ( !i1->is_Bool() ) { return false; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   297
  BoolNode *bool1 = i1->as_Bool();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   298
  if(       less_than && bool1->_test._test != BoolTest::le ) { return false; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   299
  else if( !less_than && bool1->_test._test != BoolTest::lt ) { return false; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   300
  const Node *cmpF = bool1->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   301
  if( cmpF->Opcode() != Op_CmpF )      { return false; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   302
  // Test that the float value being compared against
489c9b5090e2 Initial load
duke
parents:
diff changeset
   303
  // is equivalent to the int value used as a limit
489c9b5090e2 Initial load
duke
parents:
diff changeset
   304
  Node *nodef = cmpF->in(2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   305
  if( nodef->Opcode() != Op_ConF ) { return false; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   306
  jfloat conf = nodef->getf();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   307
  jint   coni = limit->get_int();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   308
  if( ((int)conf) != coni )        { return false; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   309
  input = cmpF->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   310
  return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   311
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   312
489c9b5090e2 Initial load
duke
parents:
diff changeset
   313
//------------------------------is_unreachable_region--------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   314
// Find if the Region node is reachable from the root.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   315
bool RegionNode::is_unreachable_region(PhaseGVN *phase) const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   316
  assert(req() == 2, "");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   317
489c9b5090e2 Initial load
duke
parents:
diff changeset
   318
  // First, cut the simple case of fallthrough region when NONE of
489c9b5090e2 Initial load
duke
parents:
diff changeset
   319
  // region's phis references itself directly or through a data node.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   320
  uint max = outcnt();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   321
  uint i;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   322
  for (i = 0; i < max; i++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   323
    Node* phi = raw_out(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   324
    if (phi != NULL && phi->is_Phi()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   325
      assert(phase->eqv(phi->in(0), this) && phi->req() == 2, "");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   326
      if (phi->outcnt() == 0)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   327
        continue; // Safe case - no loops
489c9b5090e2 Initial load
duke
parents:
diff changeset
   328
      if (phi->outcnt() == 1) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   329
        Node* u = phi->raw_out(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   330
        // Skip if only one use is an other Phi or Call or Uncommon trap.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   331
        // It is safe to consider this case as fallthrough.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   332
        if (u != NULL && (u->is_Phi() || u->is_CFG()))
489c9b5090e2 Initial load
duke
parents:
diff changeset
   333
          continue;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   334
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   335
      // Check when phi references itself directly or through an other node.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   336
      if (phi->as_Phi()->simple_data_loop_check(phi->in(1)) >= PhiNode::Unsafe)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   337
        break; // Found possible unsafe data loop.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   338
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   339
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   340
  if (i >= max)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   341
    return false; // An unsafe case was NOT found - don't need graph walk.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   342
489c9b5090e2 Initial load
duke
parents:
diff changeset
   343
  // Unsafe case - check if the Region node is reachable from root.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   344
  ResourceMark rm;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   345
489c9b5090e2 Initial load
duke
parents:
diff changeset
   346
  Arena *a = Thread::current()->resource_area();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   347
  Node_List nstack(a);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   348
  VectorSet visited(a);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   349
489c9b5090e2 Initial load
duke
parents:
diff changeset
   350
  // Mark all control nodes reachable from root outputs
489c9b5090e2 Initial load
duke
parents:
diff changeset
   351
  Node *n = (Node*)phase->C->root();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   352
  nstack.push(n);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   353
  visited.set(n->_idx);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   354
  while (nstack.size() != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   355
    n = nstack.pop();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   356
    uint max = n->outcnt();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   357
    for (uint i = 0; i < max; i++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   358
      Node* m = n->raw_out(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   359
      if (m != NULL && m->is_CFG()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   360
        if (phase->eqv(m, this)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   361
          return false; // We reached the Region node - it is not dead.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   362
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   363
        if (!visited.test_set(m->_idx))
489c9b5090e2 Initial load
duke
parents:
diff changeset
   364
          nstack.push(m);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   365
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   366
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   367
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   368
489c9b5090e2 Initial load
duke
parents:
diff changeset
   369
  return true; // The Region node is unreachable - it is dead.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   370
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   371
15113
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   372
bool RegionNode::try_clean_mem_phi(PhaseGVN *phase) {
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   373
  // Incremental inlining + PhaseStringOpts sometimes produce:
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   374
  //
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   375
  // cmpP with 1 top input
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   376
  //           |
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   377
  //          If
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   378
  //         /  \
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   379
  //   IfFalse  IfTrue  /- Some Node
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   380
  //         \  /      /    /
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   381
  //        Region    / /-MergeMem
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   382
  //             \---Phi
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   383
  //
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   384
  //
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   385
  // It's expected by PhaseStringOpts that the Region goes away and is
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   386
  // replaced by If's control input but because there's still a Phi,
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   387
  // the Region stays in the graph. The top input from the cmpP is
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   388
  // propagated forward and a subgraph that is useful goes away. The
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   389
  // code below replaces the Phi with the MergeMem so that the Region
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   390
  // is simplified.
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   391
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   392
  PhiNode* phi = has_unique_phi();
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   393
  if (phi && phi->type() == Type::MEMORY && req() == 3 && phi->is_diamond_phi(true)) {
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   394
    MergeMemNode* m = NULL;
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   395
    assert(phi->req() == 3, "same as region");
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   396
    for (uint i = 1; i < 3; ++i) {
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   397
      Node *mem = phi->in(i);
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   398
      if (mem && mem->is_MergeMem() && in(i)->outcnt() == 1) {
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   399
        // Nothing is control-dependent on path #i except the region itself.
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   400
        m = mem->as_MergeMem();
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   401
        uint j = 3 - i;
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   402
        Node* other = phi->in(j);
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   403
        if (other && other == m->base_memory()) {
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   404
          // m is a successor memory to other, and is not pinned inside the diamond, so push it out.
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   405
          // This will allow the diamond to collapse completely.
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   406
          phase->is_IterGVN()->replace_node(phi, m);
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   407
          return true;
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   408
        }
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   409
      }
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   410
    }
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   411
  }
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   412
  return false;
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   413
}
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   414
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   415
//------------------------------Ideal------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   416
// Return a node which is more "ideal" than the current node.  Must preserve
489c9b5090e2 Initial load
duke
parents:
diff changeset
   417
// the CFG, but we can still strip out dead paths.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   418
Node *RegionNode::Ideal(PhaseGVN *phase, bool can_reshape) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   419
  if( !can_reshape && !in(0) ) return NULL;     // Already degraded to a Copy
489c9b5090e2 Initial load
duke
parents:
diff changeset
   420
  assert(!in(0) || !in(0)->is_Root(), "not a specially hidden merge");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   421
489c9b5090e2 Initial load
duke
parents:
diff changeset
   422
  // Check for RegionNode with no Phi users and both inputs come from either
489c9b5090e2 Initial load
duke
parents:
diff changeset
   423
  // arm of the same IF.  If found, then the control-flow split is useless.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   424
  bool has_phis = false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   425
  if (can_reshape) {            // Need DU info to check for Phi users
489c9b5090e2 Initial load
duke
parents:
diff changeset
   426
    has_phis = (has_phi() != NULL);       // Cache result
15113
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   427
    if (has_phis && try_clean_mem_phi(phase)) {
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   428
      has_phis = false;
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   429
    }
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
   430
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   431
    if (!has_phis) {            // No Phi users?  Nothing merging?
489c9b5090e2 Initial load
duke
parents:
diff changeset
   432
      for (uint i = 1; i < req()-1; i++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   433
        Node *if1 = in(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   434
        if( !if1 ) continue;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   435
        Node *iff = if1->in(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   436
        if( !iff || !iff->is_If() ) continue;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   437
        for( uint j=i+1; j<req(); j++ ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   438
          if( in(j) && in(j)->in(0) == iff &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
   439
              if1->Opcode() != in(j)->Opcode() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   440
            // Add the IF Projections to the worklist. They (and the IF itself)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   441
            // will be eliminated if dead.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   442
            phase->is_IterGVN()->add_users_to_worklist(iff);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   443
            set_req(i, iff->in(0));// Skip around the useless IF diamond
489c9b5090e2 Initial load
duke
parents:
diff changeset
   444
            set_req(j, NULL);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   445
            return this;      // Record progress
489c9b5090e2 Initial load
duke
parents:
diff changeset
   446
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   447
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   448
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   449
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   450
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   451
489c9b5090e2 Initial load
duke
parents:
diff changeset
   452
  // Remove TOP or NULL input paths. If only 1 input path remains, this Region
489c9b5090e2 Initial load
duke
parents:
diff changeset
   453
  // degrades to a copy.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   454
  bool add_to_worklist = false;
25913
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 24923
diff changeset
   455
  bool modified = false;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   456
  int cnt = 0;                  // Count of values merging
489c9b5090e2 Initial load
duke
parents:
diff changeset
   457
  DEBUG_ONLY( int cnt_orig = req(); ) // Save original inputs count
489c9b5090e2 Initial load
duke
parents:
diff changeset
   458
  int del_it = 0;               // The last input path we delete
489c9b5090e2 Initial load
duke
parents:
diff changeset
   459
  // For all inputs...
489c9b5090e2 Initial load
duke
parents:
diff changeset
   460
  for( uint i=1; i<req(); ++i ){// For all paths in
489c9b5090e2 Initial load
duke
parents:
diff changeset
   461
    Node *n = in(i);            // Get the input
489c9b5090e2 Initial load
duke
parents:
diff changeset
   462
    if( n != NULL ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   463
      // Remove useless control copy inputs
489c9b5090e2 Initial load
duke
parents:
diff changeset
   464
      if( n->is_Region() && n->as_Region()->is_copy() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   465
        set_req(i, n->nonnull_req());
25913
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 24923
diff changeset
   466
        modified = true;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   467
        i--;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   468
        continue;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   469
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   470
      if( n->is_Proj() ) {      // Remove useless rethrows
489c9b5090e2 Initial load
duke
parents:
diff changeset
   471
        Node *call = n->in(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   472
        if (call->is_Call() && call->as_Call()->entry_point() == OptoRuntime::rethrow_stub()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   473
          set_req(i, call->in(0));
25913
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 24923
diff changeset
   474
          modified = true;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   475
          i--;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   476
          continue;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   477
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   478
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   479
      if( phase->type(n) == Type::TOP ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   480
        set_req(i, NULL);       // Ignore TOP inputs
25913
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 24923
diff changeset
   481
        modified = true;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   482
        i--;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   483
        continue;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   484
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   485
      cnt++;                    // One more value merging
489c9b5090e2 Initial load
duke
parents:
diff changeset
   486
489c9b5090e2 Initial load
duke
parents:
diff changeset
   487
    } else if (can_reshape) {   // Else found dead path with DU info
489c9b5090e2 Initial load
duke
parents:
diff changeset
   488
      PhaseIterGVN *igvn = phase->is_IterGVN();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   489
      del_req(i);               // Yank path from self
489c9b5090e2 Initial load
duke
parents:
diff changeset
   490
      del_it = i;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   491
      uint max = outcnt();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   492
      DUIterator j;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   493
      bool progress = true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   494
      while(progress) {         // Need to establish property over all users
489c9b5090e2 Initial load
duke
parents:
diff changeset
   495
        progress = false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   496
        for (j = outs(); has_out(j); j++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   497
          Node *n = out(j);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   498
          if( n->req() != req() && n->is_Phi() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   499
            assert( n->in(0) == this, "" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   500
            igvn->hash_delete(n); // Yank from hash before hacking edges
489c9b5090e2 Initial load
duke
parents:
diff changeset
   501
            n->set_req_X(i,NULL,igvn);// Correct DU info
489c9b5090e2 Initial load
duke
parents:
diff changeset
   502
            n->del_req(i);        // Yank path from Phis
489c9b5090e2 Initial load
duke
parents:
diff changeset
   503
            if( max != outcnt() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   504
              progress = true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   505
              j = refresh_out_pos(j);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   506
              max = outcnt();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   507
            }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   508
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   509
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   510
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   511
      add_to_worklist = true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   512
      i--;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   513
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   514
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   515
489c9b5090e2 Initial load
duke
parents:
diff changeset
   516
  if (can_reshape && cnt == 1) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   517
    // Is it dead loop?
489c9b5090e2 Initial load
duke
parents:
diff changeset
   518
    // If it is LoopNopde it had 2 (+1 itself) inputs and
489c9b5090e2 Initial load
duke
parents:
diff changeset
   519
    // one of them was cut. The loop is dead if it was EntryContol.
11189
c1ad8528ae68 7105605: Use EA info to optimize pointers compare
kvn
parents: 10258
diff changeset
   520
    // Loop node may have only one input because entry path
c1ad8528ae68 7105605: Use EA info to optimize pointers compare
kvn
parents: 10258
diff changeset
   521
    // is removed in PhaseIdealLoop::Dominators().
c1ad8528ae68 7105605: Use EA info to optimize pointers compare
kvn
parents: 10258
diff changeset
   522
    assert(!this->is_Loop() || cnt_orig <= 3, "Loop node should have 3 or less inputs");
c1ad8528ae68 7105605: Use EA info to optimize pointers compare
kvn
parents: 10258
diff changeset
   523
    if (this->is_Loop() && (del_it == LoopNode::EntryControl ||
c1ad8528ae68 7105605: Use EA info to optimize pointers compare
kvn
parents: 10258
diff changeset
   524
                            del_it == 0 && is_unreachable_region(phase)) ||
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   525
       !this->is_Loop() && has_phis && is_unreachable_region(phase)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   526
      // Yes,  the region will be removed during the next step below.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   527
      // Cut the backedge input and remove phis since no data paths left.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   528
      // We don't cut outputs to other nodes here since we need to put them
489c9b5090e2 Initial load
duke
parents:
diff changeset
   529
      // on the worklist.
30206
855fbaf99089 8075922: assert(t == t_no_spec) fails in phaseX.cpp
roland
parents: 29076
diff changeset
   530
      PhaseIterGVN *igvn = phase->is_IterGVN();
855fbaf99089 8075922: assert(t == t_no_spec) fails in phaseX.cpp
roland
parents: 29076
diff changeset
   531
      if (in(1)->outcnt() == 1) {
855fbaf99089 8075922: assert(t == t_no_spec) fails in phaseX.cpp
roland
parents: 29076
diff changeset
   532
        igvn->_worklist.push(in(1));
855fbaf99089 8075922: assert(t == t_no_spec) fails in phaseX.cpp
roland
parents: 29076
diff changeset
   533
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   534
      del_req(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   535
      cnt = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   536
      assert( req() == 1, "no more inputs expected" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   537
      uint max = outcnt();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   538
      bool progress = true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   539
      Node *top = phase->C->top();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   540
      DUIterator j;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   541
      while(progress) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   542
        progress = false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   543
        for (j = outs(); has_out(j); j++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   544
          Node *n = out(j);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   545
          if( n->is_Phi() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   546
            assert( igvn->eqv(n->in(0), this), "" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   547
            assert( n->req() == 2 &&  n->in(1) != NULL, "Only one data input expected" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   548
            // Break dead loop data path.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   549
            // Eagerly replace phis with top to avoid phis copies generation.
5901
c046f8e9c52b 6677629: PhaseIterGVN::subsume_node() should call hash_delete() and add_users_to_worklist()
kvn
parents: 5547
diff changeset
   550
            igvn->replace_node(n, top);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   551
            if( max != outcnt() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   552
              progress = true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   553
              j = refresh_out_pos(j);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   554
              max = outcnt();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   555
            }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   556
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   557
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   558
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   559
      add_to_worklist = true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   560
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   561
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   562
  if (add_to_worklist) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   563
    phase->is_IterGVN()->add_users_to_worklist(this); // Revisit collapsed Phis
489c9b5090e2 Initial load
duke
parents:
diff changeset
   564
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   565
489c9b5090e2 Initial load
duke
parents:
diff changeset
   566
  if( cnt <= 1 ) {              // Only 1 path in?
489c9b5090e2 Initial load
duke
parents:
diff changeset
   567
    set_req(0, NULL);           // Null control input for region copy
489c9b5090e2 Initial load
duke
parents:
diff changeset
   568
    if( cnt == 0 && !can_reshape) { // Parse phase - leave the node as it is.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   569
      // No inputs or all inputs are NULL.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   570
      return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   571
    } else if (can_reshape) {   // Optimization phase - remove the node
489c9b5090e2 Initial load
duke
parents:
diff changeset
   572
      PhaseIterGVN *igvn = phase->is_IterGVN();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   573
      Node *parent_ctrl;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   574
      if( cnt == 0 ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   575
        assert( req() == 1, "no inputs expected" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   576
        // During IGVN phase such region will be subsumed by TOP node
489c9b5090e2 Initial load
duke
parents:
diff changeset
   577
        // so region's phis will have TOP as control node.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   578
        // Kill phis here to avoid it. PhiNode::is_copy() will be always false.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   579
        // Also set other user's input to top.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   580
        parent_ctrl = phase->C->top();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   581
      } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   582
        // The fallthrough case since we already checked dead loops above.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   583
        parent_ctrl = in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   584
        assert(parent_ctrl != NULL, "Region is a copy of some non-null control");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   585
        assert(!igvn->eqv(parent_ctrl, this), "Close dead loop");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   586
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   587
      if (!add_to_worklist)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   588
        igvn->add_users_to_worklist(this); // Check for further allowed opts
489c9b5090e2 Initial load
duke
parents:
diff changeset
   589
      for (DUIterator_Last imin, i = last_outs(imin); i >= imin; --i) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   590
        Node* n = last_out(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   591
        igvn->hash_delete(n); // Remove from worklist before modifying edges
489c9b5090e2 Initial load
duke
parents:
diff changeset
   592
        if( n->is_Phi() ) {   // Collapse all Phis
489c9b5090e2 Initial load
duke
parents:
diff changeset
   593
          // Eagerly replace phis to avoid copies generation.
5901
c046f8e9c52b 6677629: PhaseIterGVN::subsume_node() should call hash_delete() and add_users_to_worklist()
kvn
parents: 5547
diff changeset
   594
          Node* in;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   595
          if( cnt == 0 ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   596
            assert( n->req() == 1, "No data inputs expected" );
5901
c046f8e9c52b 6677629: PhaseIterGVN::subsume_node() should call hash_delete() and add_users_to_worklist()
kvn
parents: 5547
diff changeset
   597
            in = parent_ctrl; // replaced by top
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   598
          } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   599
            assert( n->req() == 2 &&  n->in(1) != NULL, "Only one data input expected" );
5901
c046f8e9c52b 6677629: PhaseIterGVN::subsume_node() should call hash_delete() and add_users_to_worklist()
kvn
parents: 5547
diff changeset
   600
            in = n->in(1);               // replaced by unique input
c046f8e9c52b 6677629: PhaseIterGVN::subsume_node() should call hash_delete() and add_users_to_worklist()
kvn
parents: 5547
diff changeset
   601
            if( n->as_Phi()->is_unsafe_data_reference(in) )
c046f8e9c52b 6677629: PhaseIterGVN::subsume_node() should call hash_delete() and add_users_to_worklist()
kvn
parents: 5547
diff changeset
   602
              in = phase->C->top();      // replaced by top
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   603
          }
5901
c046f8e9c52b 6677629: PhaseIterGVN::subsume_node() should call hash_delete() and add_users_to_worklist()
kvn
parents: 5547
diff changeset
   604
          igvn->replace_node(n, in);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   605
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   606
        else if( n->is_Region() ) { // Update all incoming edges
489c9b5090e2 Initial load
duke
parents:
diff changeset
   607
          assert( !igvn->eqv(n, this), "Must be removed from DefUse edges");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   608
          uint uses_found = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   609
          for( uint k=1; k < n->req(); k++ ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   610
            if( n->in(k) == this ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   611
              n->set_req(k, parent_ctrl);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   612
              uses_found++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   613
            }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   614
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   615
          if( uses_found > 1 ) { // (--i) done at the end of the loop.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   616
            i -= (uses_found - 1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   617
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   618
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   619
        else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   620
          assert( igvn->eqv(n->in(0), this), "Expect RegionNode to be control parent");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   621
          n->set_req(0, parent_ctrl);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   622
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   623
#ifdef ASSERT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   624
        for( uint k=0; k < n->req(); k++ ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   625
          assert( !igvn->eqv(n->in(k), this), "All uses of RegionNode should be gone");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   626
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   627
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   628
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   629
      // Remove the RegionNode itself from DefUse info
489c9b5090e2 Initial load
duke
parents:
diff changeset
   630
      igvn->remove_dead_node(this);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   631
      return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   632
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   633
    return this;                // Record progress
489c9b5090e2 Initial load
duke
parents:
diff changeset
   634
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   635
489c9b5090e2 Initial load
duke
parents:
diff changeset
   636
489c9b5090e2 Initial load
duke
parents:
diff changeset
   637
  // If a Region flows into a Region, merge into one big happy merge.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   638
  if (can_reshape) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   639
    Node *m = merge_region(this, phase);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   640
    if (m != NULL)  return m;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   641
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   642
489c9b5090e2 Initial load
duke
parents:
diff changeset
   643
  // Check if this region is the root of a clipping idiom on floats
489c9b5090e2 Initial load
duke
parents:
diff changeset
   644
  if( ConvertFloat2IntClipping && can_reshape && req() == 4 ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   645
    // Check that only one use is a Phi and that it simplifies to two constants +
489c9b5090e2 Initial load
duke
parents:
diff changeset
   646
    PhiNode* phi = has_unique_phi();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   647
    if (phi != NULL) {          // One Phi user
489c9b5090e2 Initial load
duke
parents:
diff changeset
   648
      // Check inputs to the Phi
489c9b5090e2 Initial load
duke
parents:
diff changeset
   649
      ConNode *min;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   650
      ConNode *max;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   651
      Node    *val;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   652
      uint     min_idx;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   653
      uint     max_idx;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   654
      uint     val_idx;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   655
      if( check_phi_clipping( phi, min, min_idx, max, max_idx, val, val_idx )  ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   656
        IfNode *top_if;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   657
        IfNode *bot_if;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   658
        if( check_if_clipping( this, bot_if, top_if ) ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   659
          // Control pattern checks, now verify compares
489c9b5090e2 Initial load
duke
parents:
diff changeset
   660
          Node   *top_in = NULL;   // value being compared against
489c9b5090e2 Initial load
duke
parents:
diff changeset
   661
          Node   *bot_in = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   662
          if( check_compare_clipping( true,  bot_if, min, bot_in ) &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
   663
              check_compare_clipping( false, top_if, max, top_in ) ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   664
            if( bot_in == top_in ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   665
              PhaseIterGVN *gvn = phase->is_IterGVN();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   666
              assert( gvn != NULL, "Only had DefUse info in IterGVN");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   667
              // Only remaining check is that bot_in == top_in == (Phi's val + mods)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   668
489c9b5090e2 Initial load
duke
parents:
diff changeset
   669
              // Check for the ConvF2INode
489c9b5090e2 Initial load
duke
parents:
diff changeset
   670
              ConvF2INode *convf2i;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   671
              if( check_convf2i_clipping( phi, val_idx, convf2i, min, max ) &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
   672
                convf2i->in(1) == bot_in ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   673
                // Matched pattern, including LShiftI; RShiftI, replace with integer compares
489c9b5090e2 Initial load
duke
parents:
diff changeset
   674
                // max test
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 23528
diff changeset
   675
                Node *cmp   = gvn->register_new_node_with_optimizer(new CmpINode( convf2i, min ));
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 23528
diff changeset
   676
                Node *boo   = gvn->register_new_node_with_optimizer(new BoolNode( cmp, BoolTest::lt ));
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 23528
diff changeset
   677
                IfNode *iff = (IfNode*)gvn->register_new_node_with_optimizer(new IfNode( top_if->in(0), boo, PROB_UNLIKELY_MAG(5), top_if->_fcnt ));
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 23528
diff changeset
   678
                Node *if_min= gvn->register_new_node_with_optimizer(new IfTrueNode (iff));
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 23528
diff changeset
   679
                Node *ifF   = gvn->register_new_node_with_optimizer(new IfFalseNode(iff));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   680
                // min test
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 23528
diff changeset
   681
                cmp         = gvn->register_new_node_with_optimizer(new CmpINode( convf2i, max ));
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 23528
diff changeset
   682
                boo         = gvn->register_new_node_with_optimizer(new BoolNode( cmp, BoolTest::gt ));
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 23528
diff changeset
   683
                iff         = (IfNode*)gvn->register_new_node_with_optimizer(new IfNode( ifF, boo, PROB_UNLIKELY_MAG(5), bot_if->_fcnt ));
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 23528
diff changeset
   684
                Node *if_max= gvn->register_new_node_with_optimizer(new IfTrueNode (iff));
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 23528
diff changeset
   685
                ifF         = gvn->register_new_node_with_optimizer(new IfFalseNode(iff));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   686
                // update input edges to region node
489c9b5090e2 Initial load
duke
parents:
diff changeset
   687
                set_req_X( min_idx, if_min, gvn );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   688
                set_req_X( max_idx, if_max, gvn );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   689
                set_req_X( val_idx, ifF,    gvn );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   690
                // remove unnecessary 'LShiftI; RShiftI' idiom
489c9b5090e2 Initial load
duke
parents:
diff changeset
   691
                gvn->hash_delete(phi);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   692
                phi->set_req_X( val_idx, convf2i, gvn );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   693
                gvn->hash_find_insert(phi);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   694
                // Return transformed region node
489c9b5090e2 Initial load
duke
parents:
diff changeset
   695
                return this;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   696
              }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   697
            }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   698
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   699
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   700
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   701
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   702
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   703
25913
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 24923
diff changeset
   704
  return modified ? this : NULL;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   705
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   706
489c9b5090e2 Initial load
duke
parents:
diff changeset
   707
489c9b5090e2 Initial load
duke
parents:
diff changeset
   708
489c9b5090e2 Initial load
duke
parents:
diff changeset
   709
const RegMask &RegionNode::out_RegMask() const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   710
  return RegMask::Empty;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   711
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   712
489c9b5090e2 Initial load
duke
parents:
diff changeset
   713
// Find the one non-null required input.  RegionNode only
489c9b5090e2 Initial load
duke
parents:
diff changeset
   714
Node *Node::nonnull_req() const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   715
  assert( is_Region(), "" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   716
  for( uint i = 1; i < _cnt; i++ )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   717
    if( in(i) )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   718
      return in(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   719
  ShouldNotReachHere();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   720
  return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   721
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   722
489c9b5090e2 Initial load
duke
parents:
diff changeset
   723
489c9b5090e2 Initial load
duke
parents:
diff changeset
   724
//=============================================================================
489c9b5090e2 Initial load
duke
parents:
diff changeset
   725
// note that these functions assume that the _adr_type field is flattened
489c9b5090e2 Initial load
duke
parents:
diff changeset
   726
uint PhiNode::hash() const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   727
  const Type* at = _adr_type;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   728
  return TypeNode::hash() + (at ? at->hash() : 0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   729
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   730
uint PhiNode::cmp( const Node &n ) const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   731
  return TypeNode::cmp(n) && _adr_type == ((PhiNode&)n)._adr_type;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   732
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   733
static inline
489c9b5090e2 Initial load
duke
parents:
diff changeset
   734
const TypePtr* flatten_phi_adr_type(const TypePtr* at) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   735
  if (at == NULL || at == TypePtr::BOTTOM)  return at;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   736
  return Compile::current()->alias_type(at)->adr_type();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   737
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   738
489c9b5090e2 Initial load
duke
parents:
diff changeset
   739
//----------------------------make---------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   740
// create a new phi with edges matching r and set (initially) to x
489c9b5090e2 Initial load
duke
parents:
diff changeset
   741
PhiNode* PhiNode::make(Node* r, Node* x, const Type *t, const TypePtr* at) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   742
  uint preds = r->req();   // Number of predecessor paths
489c9b5090e2 Initial load
duke
parents:
diff changeset
   743
  assert(t != Type::MEMORY || at == flatten_phi_adr_type(at), "flatten at");
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 23528
diff changeset
   744
  PhiNode* p = new PhiNode(r, t, at);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   745
  for (uint j = 1; j < preds; j++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   746
    // Fill in all inputs, except those which the region does not yet have
489c9b5090e2 Initial load
duke
parents:
diff changeset
   747
    if (r->in(j) != NULL)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   748
      p->init_req(j, x);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   749
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   750
  return p;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   751
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   752
PhiNode* PhiNode::make(Node* r, Node* x) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   753
  const Type*    t  = x->bottom_type();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   754
  const TypePtr* at = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   755
  if (t == Type::MEMORY)  at = flatten_phi_adr_type(x->adr_type());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   756
  return make(r, x, t, at);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   757
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   758
PhiNode* PhiNode::make_blank(Node* r, Node* x) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   759
  const Type*    t  = x->bottom_type();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   760
  const TypePtr* at = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   761
  if (t == Type::MEMORY)  at = flatten_phi_adr_type(x->adr_type());
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 23528
diff changeset
   762
  return new PhiNode(r, t, at);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   763
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   764
489c9b5090e2 Initial load
duke
parents:
diff changeset
   765
489c9b5090e2 Initial load
duke
parents:
diff changeset
   766
//------------------------slice_memory-----------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   767
// create a new phi with narrowed memory type
489c9b5090e2 Initial load
duke
parents:
diff changeset
   768
PhiNode* PhiNode::slice_memory(const TypePtr* adr_type) const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   769
  PhiNode* mem = (PhiNode*) clone();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   770
  *(const TypePtr**)&mem->_adr_type = adr_type;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   771
  // convert self-loops, or else we get a bad graph
489c9b5090e2 Initial load
duke
parents:
diff changeset
   772
  for (uint i = 1; i < req(); i++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   773
    if ((const Node*)in(i) == this)  mem->set_req(i, mem);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   774
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   775
  mem->verify_adr_type();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   776
  return mem;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   777
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   778
247
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   779
//------------------------split_out_instance-----------------------------------
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   780
// Split out an instance type from a bottom phi.
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   781
PhiNode* PhiNode::split_out_instance(const TypePtr* at, PhaseIterGVN *igvn) const {
589
a44a1e70a3e4 6695810: null oop passed to encode_heap_oop_not_null
kvn
parents: 374
diff changeset
   782
  const TypeOopPtr *t_oop = at->isa_oopptr();
769
78e5090c7a20 6716441: error in meet with +DoEscapeAnalysis
kvn
parents: 767
diff changeset
   783
  assert(t_oop != NULL && t_oop->is_known_instance(), "expecting instance oopptr");
589
a44a1e70a3e4 6695810: null oop passed to encode_heap_oop_not_null
kvn
parents: 374
diff changeset
   784
  const TypePtr *t = adr_type();
a44a1e70a3e4 6695810: null oop passed to encode_heap_oop_not_null
kvn
parents: 374
diff changeset
   785
  assert(type() == Type::MEMORY &&
a44a1e70a3e4 6695810: null oop passed to encode_heap_oop_not_null
kvn
parents: 374
diff changeset
   786
         (t == TypePtr::BOTTOM || t == TypeRawPtr::BOTTOM ||
769
78e5090c7a20 6716441: error in meet with +DoEscapeAnalysis
kvn
parents: 767
diff changeset
   787
          t->isa_oopptr() && !t->is_oopptr()->is_known_instance() &&
955
723d7b577fba 6723160: Nightly failure: Error: meet not symmetric
kvn
parents: 781
diff changeset
   788
          t->is_oopptr()->cast_to_exactness(true)
723d7b577fba 6723160: Nightly failure: Error: meet not symmetric
kvn
parents: 781
diff changeset
   789
           ->is_oopptr()->cast_to_ptr_type(t_oop->ptr())
723d7b577fba 6723160: Nightly failure: Error: meet not symmetric
kvn
parents: 781
diff changeset
   790
           ->is_oopptr()->cast_to_instance_id(t_oop->instance_id()) == t_oop),
589
a44a1e70a3e4 6695810: null oop passed to encode_heap_oop_not_null
kvn
parents: 374
diff changeset
   791
         "bottom or raw memory required");
247
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   792
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   793
  // Check if an appropriate node already exists.
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   794
  Node *region = in(0);
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   795
  for (DUIterator_Fast kmax, k = region->fast_outs(kmax); k < kmax; k++) {
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   796
    Node* use = region->fast_out(k);
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   797
    if( use->is_Phi()) {
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   798
      PhiNode *phi2 = use->as_Phi();
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   799
      if (phi2->type() == Type::MEMORY && phi2->adr_type() == at) {
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   800
        return phi2;
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   801
      }
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   802
    }
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   803
  }
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   804
  Compile *C = igvn->C;
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   805
  Arena *a = Thread::current()->resource_area();
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   806
  Node_Array node_map = new Node_Array(a);
32202
7e7ad8b06f5b 8011858: Use Compile::live_nodes() instead of Compile::unique() in appropriate places
kvn
parents: 32084
diff changeset
   807
  Node_Stack stack(a, C->live_nodes() >> 4);
247
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   808
  PhiNode *nphi = slice_memory(at);
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   809
  igvn->register_new_node_with_optimizer( nphi );
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   810
  node_map.map(_idx, nphi);
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   811
  stack.push((Node *)this, 1);
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   812
  while(!stack.is_empty()) {
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   813
    PhiNode *ophi = stack.node()->as_Phi();
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   814
    uint i = stack.index();
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   815
    assert(i >= 1, "not control edge");
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   816
    stack.pop();
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   817
    nphi = node_map[ophi->_idx]->as_Phi();
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   818
    for (; i < ophi->req(); i++) {
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   819
      Node *in = ophi->in(i);
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   820
      if (in == NULL || igvn->type(in) == Type::TOP)
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   821
        continue;
17383
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 17008
diff changeset
   822
      Node *opt = MemNode::optimize_simple_memory_chain(in, t_oop, NULL, igvn);
247
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   823
      PhiNode *optphi = opt->is_Phi() ? opt->as_Phi() : NULL;
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   824
      if (optphi != NULL && optphi->adr_type() == TypePtr::BOTTOM) {
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   825
        opt = node_map[optphi->_idx];
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   826
        if (opt == NULL) {
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   827
          stack.push(ophi, i);
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   828
          nphi = optphi->slice_memory(at);
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   829
          igvn->register_new_node_with_optimizer( nphi );
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   830
          node_map.map(optphi->_idx, nphi);
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   831
          ophi = optphi;
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   832
          i = 0; // will get incremented at top of loop
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   833
          continue;
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   834
        }
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   835
      }
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   836
      nphi->set_req(i, opt);
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   837
    }
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   838
  }
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   839
  return nphi;
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   840
}
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
   841
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   842
//------------------------verify_adr_type--------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   843
#ifdef ASSERT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   844
void PhiNode::verify_adr_type(VectorSet& visited, const TypePtr* at) const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   845
  if (visited.test_set(_idx))  return;  //already visited
489c9b5090e2 Initial load
duke
parents:
diff changeset
   846
489c9b5090e2 Initial load
duke
parents:
diff changeset
   847
  // recheck constructor invariants:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   848
  verify_adr_type(false);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   849
489c9b5090e2 Initial load
duke
parents:
diff changeset
   850
  // recheck local phi/phi consistency:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   851
  assert(_adr_type == at || _adr_type == TypePtr::BOTTOM,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   852
         "adr_type must be consistent across phi nest");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   853
489c9b5090e2 Initial load
duke
parents:
diff changeset
   854
  // walk around
489c9b5090e2 Initial load
duke
parents:
diff changeset
   855
  for (uint i = 1; i < req(); i++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   856
    Node* n = in(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   857
    if (n == NULL)  continue;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   858
    const Node* np = in(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   859
    if (np->is_Phi()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   860
      np->as_Phi()->verify_adr_type(visited, at);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   861
    } else if (n->bottom_type() == Type::TOP
489c9b5090e2 Initial load
duke
parents:
diff changeset
   862
               || (n->is_Mem() && n->in(MemNode::Address)->bottom_type() == Type::TOP)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   863
      // ignore top inputs
489c9b5090e2 Initial load
duke
parents:
diff changeset
   864
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   865
      const TypePtr* nat = flatten_phi_adr_type(n->adr_type());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   866
      // recheck phi/non-phi consistency at leaves:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   867
      assert((nat != NULL) == (at != NULL), "");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   868
      assert(nat == at || nat == TypePtr::BOTTOM,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   869
             "adr_type must be consistent at leaves of phi nest");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   870
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   871
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   872
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   873
489c9b5090e2 Initial load
duke
parents:
diff changeset
   874
// Verify a whole nest of phis rooted at this one.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   875
void PhiNode::verify_adr_type(bool recursive) const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   876
  if (is_error_reported())  return;  // muzzle asserts when debugging an error
489c9b5090e2 Initial load
duke
parents:
diff changeset
   877
  if (Node::in_dump())      return;  // muzzle asserts when printing
489c9b5090e2 Initial load
duke
parents:
diff changeset
   878
489c9b5090e2 Initial load
duke
parents:
diff changeset
   879
  assert((_type == Type::MEMORY) == (_adr_type != NULL), "adr_type for memory phis only");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   880
489c9b5090e2 Initial load
duke
parents:
diff changeset
   881
  if (!VerifyAliases)       return;  // verify thoroughly only if requested
489c9b5090e2 Initial load
duke
parents:
diff changeset
   882
489c9b5090e2 Initial load
duke
parents:
diff changeset
   883
  assert(_adr_type == flatten_phi_adr_type(_adr_type),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   884
         "Phi::adr_type must be pre-normalized");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   885
489c9b5090e2 Initial load
duke
parents:
diff changeset
   886
  if (recursive) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   887
    VectorSet visited(Thread::current()->resource_area());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   888
    verify_adr_type(visited, _adr_type);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   889
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   890
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   891
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   892
489c9b5090e2 Initial load
duke
parents:
diff changeset
   893
489c9b5090e2 Initial load
duke
parents:
diff changeset
   894
//------------------------------Value------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   895
// Compute the type of the PhiNode
35551
36ef3841fb34 8146629: Make phase->is_IterGVN() accessible from Node::Identity and Node::Value
thartmann
parents: 35549
diff changeset
   896
const Type* PhiNode::Value(PhaseGVN* phase) const {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   897
  Node *r = in(0);              // RegionNode
489c9b5090e2 Initial load
duke
parents:
diff changeset
   898
  if( !r )                      // Copy or dead
489c9b5090e2 Initial load
duke
parents:
diff changeset
   899
    return in(1) ? phase->type(in(1)) : Type::TOP;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   900
489c9b5090e2 Initial load
duke
parents:
diff changeset
   901
  // Note: During parsing, phis are often transformed before their regions.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   902
  // This means we have to use type_or_null to defend against untyped regions.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   903
  if( phase->type_or_null(r) == Type::TOP )  // Dead code?
489c9b5090e2 Initial load
duke
parents:
diff changeset
   904
    return Type::TOP;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   905
489c9b5090e2 Initial load
duke
parents:
diff changeset
   906
  // Check for trip-counted loop.  If so, be smarter.
35549
3415401a6b6e 8145322: Code generated from unsafe loops can be slightly improved
roland
parents: 35545
diff changeset
   907
  CountedLoopNode* l = r->is_CountedLoop() ? r->as_CountedLoop() : NULL;
35600
26de15de8faa 8147853: "assert(t->meet(t0) == t) failed: Not monotonic" with sun/util/calendar/zi/TestZoneInfo310.java
roland
parents: 35578
diff changeset
   908
  if (l && ((const Node*)l->phi() == this)) { // Trip counted loop!
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   909
    // protect against init_trip() or limit() returning NULL
35600
26de15de8faa 8147853: "assert(t->meet(t0) == t) failed: Not monotonic" with sun/util/calendar/zi/TestZoneInfo310.java
roland
parents: 35578
diff changeset
   910
    if (l->can_be_counted_loop(phase)) {
26de15de8faa 8147853: "assert(t->meet(t0) == t) failed: Not monotonic" with sun/util/calendar/zi/TestZoneInfo310.java
roland
parents: 35578
diff changeset
   911
      const Node *init   = l->init_trip();
26de15de8faa 8147853: "assert(t->meet(t0) == t) failed: Not monotonic" with sun/util/calendar/zi/TestZoneInfo310.java
roland
parents: 35578
diff changeset
   912
      const Node *limit  = l->limit();
26de15de8faa 8147853: "assert(t->meet(t0) == t) failed: Not monotonic" with sun/util/calendar/zi/TestZoneInfo310.java
roland
parents: 35578
diff changeset
   913
      const Node* stride = l->stride();
26de15de8faa 8147853: "assert(t->meet(t0) == t) failed: Not monotonic" with sun/util/calendar/zi/TestZoneInfo310.java
roland
parents: 35578
diff changeset
   914
      if (init != NULL && limit != NULL && stride != NULL) {
26de15de8faa 8147853: "assert(t->meet(t0) == t) failed: Not monotonic" with sun/util/calendar/zi/TestZoneInfo310.java
roland
parents: 35578
diff changeset
   915
        const TypeInt* lo = phase->type(init)->isa_int();
26de15de8faa 8147853: "assert(t->meet(t0) == t) failed: Not monotonic" with sun/util/calendar/zi/TestZoneInfo310.java
roland
parents: 35578
diff changeset
   916
        const TypeInt* hi = phase->type(limit)->isa_int();
26de15de8faa 8147853: "assert(t->meet(t0) == t) failed: Not monotonic" with sun/util/calendar/zi/TestZoneInfo310.java
roland
parents: 35578
diff changeset
   917
        const TypeInt* stride_t = phase->type(stride)->isa_int();
26de15de8faa 8147853: "assert(t->meet(t0) == t) failed: Not monotonic" with sun/util/calendar/zi/TestZoneInfo310.java
roland
parents: 35578
diff changeset
   918
        if (lo != NULL && hi != NULL && stride_t != NULL) { // Dying loops might have TOP here
26de15de8faa 8147853: "assert(t->meet(t0) == t) failed: Not monotonic" with sun/util/calendar/zi/TestZoneInfo310.java
roland
parents: 35578
diff changeset
   919
          assert(stride_t->_hi >= stride_t->_lo, "bad stride type");
26de15de8faa 8147853: "assert(t->meet(t0) == t) failed: Not monotonic" with sun/util/calendar/zi/TestZoneInfo310.java
roland
parents: 35578
diff changeset
   920
          if (stride_t->_hi < 0) {          // Down-counter loop
26de15de8faa 8147853: "assert(t->meet(t0) == t) failed: Not monotonic" with sun/util/calendar/zi/TestZoneInfo310.java
roland
parents: 35578
diff changeset
   921
            swap(lo, hi);
26de15de8faa 8147853: "assert(t->meet(t0) == t) failed: Not monotonic" with sun/util/calendar/zi/TestZoneInfo310.java
roland
parents: 35578
diff changeset
   922
            return TypeInt::make(MIN2(lo->_lo, hi->_lo) , hi->_hi, 3);
26de15de8faa 8147853: "assert(t->meet(t0) == t) failed: Not monotonic" with sun/util/calendar/zi/TestZoneInfo310.java
roland
parents: 35578
diff changeset
   923
          } else if (stride_t->_lo >= 0) {
26de15de8faa 8147853: "assert(t->meet(t0) == t) failed: Not monotonic" with sun/util/calendar/zi/TestZoneInfo310.java
roland
parents: 35578
diff changeset
   924
            return TypeInt::make(lo->_lo, MAX2(lo->_hi, hi->_hi), 3);
26de15de8faa 8147853: "assert(t->meet(t0) == t) failed: Not monotonic" with sun/util/calendar/zi/TestZoneInfo310.java
roland
parents: 35578
diff changeset
   925
          }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   926
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   927
      }
35600
26de15de8faa 8147853: "assert(t->meet(t0) == t) failed: Not monotonic" with sun/util/calendar/zi/TestZoneInfo310.java
roland
parents: 35578
diff changeset
   928
    } else if (l->in(LoopNode::LoopBackControl) != NULL &&
26de15de8faa 8147853: "assert(t->meet(t0) == t) failed: Not monotonic" with sun/util/calendar/zi/TestZoneInfo310.java
roland
parents: 35578
diff changeset
   929
               in(LoopNode::EntryControl) != NULL &&
26de15de8faa 8147853: "assert(t->meet(t0) == t) failed: Not monotonic" with sun/util/calendar/zi/TestZoneInfo310.java
roland
parents: 35578
diff changeset
   930
               phase->type(l->in(LoopNode::LoopBackControl)) == Type::TOP) {
26de15de8faa 8147853: "assert(t->meet(t0) == t) failed: Not monotonic" with sun/util/calendar/zi/TestZoneInfo310.java
roland
parents: 35578
diff changeset
   931
      // During CCP, if we saturate the type of a counted loop's Phi
26de15de8faa 8147853: "assert(t->meet(t0) == t) failed: Not monotonic" with sun/util/calendar/zi/TestZoneInfo310.java
roland
parents: 35578
diff changeset
   932
      // before the special code for counted loop above has a chance
26de15de8faa 8147853: "assert(t->meet(t0) == t) failed: Not monotonic" with sun/util/calendar/zi/TestZoneInfo310.java
roland
parents: 35578
diff changeset
   933
      // to run (that is as long as the type of the backedge's control
26de15de8faa 8147853: "assert(t->meet(t0) == t) failed: Not monotonic" with sun/util/calendar/zi/TestZoneInfo310.java
roland
parents: 35578
diff changeset
   934
      // is top), we might end up with non monotonic types
26de15de8faa 8147853: "assert(t->meet(t0) == t) failed: Not monotonic" with sun/util/calendar/zi/TestZoneInfo310.java
roland
parents: 35578
diff changeset
   935
      return phase->type(in(LoopNode::EntryControl));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   936
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   937
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   938
489c9b5090e2 Initial load
duke
parents:
diff changeset
   939
  // Until we have harmony between classes and interfaces in the type
489c9b5090e2 Initial load
duke
parents:
diff changeset
   940
  // lattice, we must tread carefully around phis which implicitly
489c9b5090e2 Initial load
duke
parents:
diff changeset
   941
  // convert the one to the other.
767
64fb1fd7186d 6710487: More than half of JDI Regression tests hang with COOPs in -Xcomp mode
kvn
parents: 610
diff changeset
   942
  const TypePtr* ttp = _type->make_ptr();
64fb1fd7186d 6710487: More than half of JDI Regression tests hang with COOPs in -Xcomp mode
kvn
parents: 610
diff changeset
   943
  const TypeInstPtr* ttip = (ttp != NULL) ? ttp->isa_instptr() : NULL;
2019
086661823c2b 6788347: C2Compiler crash 6u7
never
parents: 1432
diff changeset
   944
  const TypeKlassPtr* ttkp = (ttp != NULL) ? ttp->isa_klassptr() : NULL;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   945
  bool is_intf = false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   946
  if (ttip != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   947
    ciKlass* k = ttip->klass();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   948
    if (k->is_loaded() && k->is_interface())
489c9b5090e2 Initial load
duke
parents:
diff changeset
   949
      is_intf = true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   950
  }
2019
086661823c2b 6788347: C2Compiler crash 6u7
never
parents: 1432
diff changeset
   951
  if (ttkp != NULL) {
086661823c2b 6788347: C2Compiler crash 6u7
never
parents: 1432
diff changeset
   952
    ciKlass* k = ttkp->klass();
086661823c2b 6788347: C2Compiler crash 6u7
never
parents: 1432
diff changeset
   953
    if (k->is_loaded() && k->is_interface())
086661823c2b 6788347: C2Compiler crash 6u7
never
parents: 1432
diff changeset
   954
      is_intf = true;
086661823c2b 6788347: C2Compiler crash 6u7
never
parents: 1432
diff changeset
   955
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   956
489c9b5090e2 Initial load
duke
parents:
diff changeset
   957
  // Default case: merge all inputs
489c9b5090e2 Initial load
duke
parents:
diff changeset
   958
  const Type *t = Type::TOP;        // Merged type starting value
489c9b5090e2 Initial load
duke
parents:
diff changeset
   959
  for (uint i = 1; i < req(); ++i) {// For all paths in
489c9b5090e2 Initial load
duke
parents:
diff changeset
   960
    // Reachable control path?
489c9b5090e2 Initial load
duke
parents:
diff changeset
   961
    if (r->in(i) && phase->type(r->in(i)) == Type::CONTROL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   962
      const Type* ti = phase->type(in(i));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   963
      // We assume that each input of an interface-valued Phi is a true
489c9b5090e2 Initial load
duke
parents:
diff changeset
   964
      // subtype of that interface.  This might not be true of the meet
489c9b5090e2 Initial load
duke
parents:
diff changeset
   965
      // of all the input types.  The lattice is not distributive in
489c9b5090e2 Initial load
duke
parents:
diff changeset
   966
      // such cases.  Ward off asserts in type.cpp by refusing to do
489c9b5090e2 Initial load
duke
parents:
diff changeset
   967
      // meets between interfaces and proper classes.
767
64fb1fd7186d 6710487: More than half of JDI Regression tests hang with COOPs in -Xcomp mode
kvn
parents: 610
diff changeset
   968
      const TypePtr* tip = ti->make_ptr();
64fb1fd7186d 6710487: More than half of JDI Regression tests hang with COOPs in -Xcomp mode
kvn
parents: 610
diff changeset
   969
      const TypeInstPtr* tiip = (tip != NULL) ? tip->isa_instptr() : NULL;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   970
      if (tiip) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   971
        bool ti_is_intf = false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   972
        ciKlass* k = tiip->klass();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   973
        if (k->is_loaded() && k->is_interface())
489c9b5090e2 Initial load
duke
parents:
diff changeset
   974
          ti_is_intf = true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   975
        if (is_intf != ti_is_intf)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   976
          { t = _type; break; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   977
      }
22799
83e58bac7980 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 22501
diff changeset
   978
      t = t->meet_speculative(ti);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   979
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   980
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   981
489c9b5090e2 Initial load
duke
parents:
diff changeset
   982
  // The worst-case type (from ciTypeFlow) should be consistent with "t".
489c9b5090e2 Initial load
duke
parents:
diff changeset
   983
  // That is, we expect that "t->higher_equal(_type)" holds true.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   984
  // There are various exceptions:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   985
  // - Inputs which are phis might in fact be widened unnecessarily.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   986
  //   For example, an input might be a widened int while the phi is a short.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   987
  // - Inputs might be BotPtrs but this phi is dependent on a null check,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   988
  //   and postCCP has removed the cast which encodes the result of the check.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   989
  // - The type of this phi is an interface, and the inputs are classes.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   990
  // - Value calls on inputs might produce fuzzy results.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   991
  //   (Occurrences of this case suggest improvements to Value methods.)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   992
  //
489c9b5090e2 Initial load
duke
parents:
diff changeset
   993
  // It is not possible to see Type::BOTTOM values as phi inputs,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   994
  // because the ciTypeFlow pre-pass produces verifier-quality types.
22799
83e58bac7980 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 22501
diff changeset
   995
  const Type* ft = t->filter_speculative(_type);  // Worst case type
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   996
489c9b5090e2 Initial load
duke
parents:
diff changeset
   997
#ifdef ASSERT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   998
  // The following logic has been moved into TypeOopPtr::filter.
22799
83e58bac7980 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 22501
diff changeset
   999
  const Type* jt = t->join_speculative(_type);
34202
5d19ca9c25a8 8141551: C2 can not handle returns with inccompatible interface arrays
simonis
parents: 32202
diff changeset
  1000
  if (jt->empty()) {           // Emptied out???
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1001
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1002
    // Check for evil case of 't' being a class and '_type' expecting an
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1003
    // interface.  This can happen because the bytecodes do not contain
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1004
    // enough type info to distinguish a Java-level interface variable
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1005
    // from a Java-level object variable.  If we meet 2 classes which
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1006
    // both implement interface I, but their meet is at 'j/l/O' which
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1007
    // doesn't implement I, we have no way to tell if the result should
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1008
    // be 'I' or 'j/l/O'.  Thus we'll pick 'j/l/O'.  If this then flows
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1009
    // into a Phi which "knows" it's an Interface type we'll have to
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1010
    // uplift the type.
34202
5d19ca9c25a8 8141551: C2 can not handle returns with inccompatible interface arrays
simonis
parents: 32202
diff changeset
  1011
    if (!t->empty() && ttip && ttip->is_loaded() && ttip->klass()->is_interface()) {
5d19ca9c25a8 8141551: C2 can not handle returns with inccompatible interface arrays
simonis
parents: 32202
diff changeset
  1012
      assert(ft == _type, ""); // Uplift to interface
5d19ca9c25a8 8141551: C2 can not handle returns with inccompatible interface arrays
simonis
parents: 32202
diff changeset
  1013
    } else if (!t->empty() && ttkp && ttkp->is_loaded() && ttkp->klass()->is_interface()) {
5d19ca9c25a8 8141551: C2 can not handle returns with inccompatible interface arrays
simonis
parents: 32202
diff changeset
  1014
      assert(ft == _type, ""); // Uplift to interface
5d19ca9c25a8 8141551: C2 can not handle returns with inccompatible interface arrays
simonis
parents: 32202
diff changeset
  1015
    } else {
5d19ca9c25a8 8141551: C2 can not handle returns with inccompatible interface arrays
simonis
parents: 32202
diff changeset
  1016
      // We also have to handle 'evil cases' of interface- vs. class-arrays
5d19ca9c25a8 8141551: C2 can not handle returns with inccompatible interface arrays
simonis
parents: 32202
diff changeset
  1017
      Type::get_arrays_base_elements(jt, _type, NULL, &ttip);
5d19ca9c25a8 8141551: C2 can not handle returns with inccompatible interface arrays
simonis
parents: 32202
diff changeset
  1018
      if (!t->empty() && ttip != NULL && ttip->is_loaded() && ttip->klass()->is_interface()) {
5d19ca9c25a8 8141551: C2 can not handle returns with inccompatible interface arrays
simonis
parents: 32202
diff changeset
  1019
          assert(ft == _type, "");   // Uplift to array of interface
5d19ca9c25a8 8141551: C2 can not handle returns with inccompatible interface arrays
simonis
parents: 32202
diff changeset
  1020
      } else {
5d19ca9c25a8 8141551: C2 can not handle returns with inccompatible interface arrays
simonis
parents: 32202
diff changeset
  1021
        // Otherwise it's something stupid like non-overlapping int ranges
5d19ca9c25a8 8141551: C2 can not handle returns with inccompatible interface arrays
simonis
parents: 32202
diff changeset
  1022
        // found on dying counted loops.
5d19ca9c25a8 8141551: C2 can not handle returns with inccompatible interface arrays
simonis
parents: 32202
diff changeset
  1023
        assert(ft == Type::TOP, ""); // Canonical empty value
5d19ca9c25a8 8141551: C2 can not handle returns with inccompatible interface arrays
simonis
parents: 32202
diff changeset
  1024
      }
5d19ca9c25a8 8141551: C2 can not handle returns with inccompatible interface arrays
simonis
parents: 32202
diff changeset
  1025
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1026
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1027
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1028
  else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1029
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1030
    // If we have an interface-typed Phi and we narrow to a class type, the join
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1031
    // should report back the class.  However, if we have a J/L/Object
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1032
    // class-typed Phi and an interface flows in, it's possible that the meet &
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1033
    // join report an interface back out.  This isn't possible but happens
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1034
    // because the type system doesn't interact well with interfaces.
767
64fb1fd7186d 6710487: More than half of JDI Regression tests hang with COOPs in -Xcomp mode
kvn
parents: 610
diff changeset
  1035
    const TypePtr *jtp = jt->make_ptr();
64fb1fd7186d 6710487: More than half of JDI Regression tests hang with COOPs in -Xcomp mode
kvn
parents: 610
diff changeset
  1036
    const TypeInstPtr *jtip = (jtp != NULL) ? jtp->isa_instptr() : NULL;
2019
086661823c2b 6788347: C2Compiler crash 6u7
never
parents: 1432
diff changeset
  1037
    const TypeKlassPtr *jtkp = (jtp != NULL) ? jtp->isa_klassptr() : NULL;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1038
    if( jtip && ttip ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1039
      if( jtip->is_loaded() &&  jtip->klass()->is_interface() &&
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 247
diff changeset
  1040
          ttip->is_loaded() && !ttip->klass()->is_interface() ) {
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 247
diff changeset
  1041
        assert(ft == ttip->cast_to_ptr_type(jtip->ptr()) ||
767
64fb1fd7186d 6710487: More than half of JDI Regression tests hang with COOPs in -Xcomp mode
kvn
parents: 610
diff changeset
  1042
               ft->isa_narrowoop() && ft->make_ptr() == ttip->cast_to_ptr_type(jtip->ptr()), "");
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 247
diff changeset
  1043
        jt = ft;
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 247
diff changeset
  1044
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1045
    }
2019
086661823c2b 6788347: C2Compiler crash 6u7
never
parents: 1432
diff changeset
  1046
    if( jtkp && ttkp ) {
086661823c2b 6788347: C2Compiler crash 6u7
never
parents: 1432
diff changeset
  1047
      if( jtkp->is_loaded() &&  jtkp->klass()->is_interface() &&
5120
c733ad68de42 6894807: No ClassCastException for HashAttributeSet constructors if run with -Xcomp
kvn
parents: 4016
diff changeset
  1048
          !jtkp->klass_is_exact() && // Keep exact interface klass (6894807)
2019
086661823c2b 6788347: C2Compiler crash 6u7
never
parents: 1432
diff changeset
  1049
          ttkp->is_loaded() && !ttkp->klass()->is_interface() ) {
086661823c2b 6788347: C2Compiler crash 6u7
never
parents: 1432
diff changeset
  1050
        assert(ft == ttkp->cast_to_ptr_type(jtkp->ptr()) ||
22501
ccc61eeabf49 8029464: assert(ft == ttkp->cast_to_ptr_type(jtkp->ptr()) || ft->isa_narrowoop()
roland
parents: 19979
diff changeset
  1051
               ft->isa_narrowklass() && ft->make_ptr() == ttkp->cast_to_ptr_type(jtkp->ptr()), "");
2019
086661823c2b 6788347: C2Compiler crash 6u7
never
parents: 1432
diff changeset
  1052
        jt = ft;
086661823c2b 6788347: C2Compiler crash 6u7
never
parents: 1432
diff changeset
  1053
      }
086661823c2b 6788347: C2Compiler crash 6u7
never
parents: 1432
diff changeset
  1054
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1055
    if (jt != ft && jt->base() == ft->base()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1056
      if (jt->isa_int() &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1057
          jt->is_int()->_lo == ft->is_int()->_lo &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1058
          jt->is_int()->_hi == ft->is_int()->_hi)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1059
        jt = ft;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1060
      if (jt->isa_long() &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1061
          jt->is_long()->_lo == ft->is_long()->_lo &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1062
          jt->is_long()->_hi == ft->is_long()->_hi)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1063
        jt = ft;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1064
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1065
    if (jt != ft) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1066
      tty->print("merge type:  "); t->dump(); tty->cr();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1067
      tty->print("kill type:   "); _type->dump(); tty->cr();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1068
      tty->print("join type:   "); jt->dump(); tty->cr();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1069
      tty->print("filter type: "); ft->dump(); tty->cr();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1070
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1071
    assert(jt == ft, "");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1072
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1073
#endif //ASSERT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1074
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1075
  // Deal with conversion problems found in data loops.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1076
  ft = phase->saturate(ft, phase->type_or_null(this), _type);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1077
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1078
  return ft;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1079
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1080
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1081
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1082
//------------------------------is_diamond_phi---------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1083
// Does this Phi represent a simple well-shaped diamond merge?  Return the
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1084
// index of the true path or 0 otherwise.
15113
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
  1085
// If check_control_only is true, do not inspect the If node at the
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
  1086
// top, and return -1 (not an edge number) on success.
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
  1087
int PhiNode::is_diamond_phi(bool check_control_only) const {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1088
  // Check for a 2-path merge
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1089
  Node *region = in(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1090
  if( !region ) return 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1091
  if( region->req() != 3 ) return 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1092
  if(         req() != 3 ) return 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1093
  // Check that both paths come from the same If
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1094
  Node *ifp1 = region->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1095
  Node *ifp2 = region->in(2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1096
  if( !ifp1 || !ifp2 ) return 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1097
  Node *iff = ifp1->in(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1098
  if( !iff || !iff->is_If() ) return 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1099
  if( iff != ifp2->in(0) ) return 0;
15113
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14828
diff changeset
  1100
  if (check_control_only)  return -1;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1101
  // Check for a proper bool/cmp
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1102
  const Node *b = iff->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1103
  if( !b->is_Bool() ) return 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1104
  const Node *cmp = b->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1105
  if( !cmp->is_Cmp() ) return 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1106
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1107
  // Check for branching opposite expected
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1108
  if( ifp2->Opcode() == Op_IfTrue ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1109
    assert( ifp1->Opcode() == Op_IfFalse, "" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1110
    return 2;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1111
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1112
    assert( ifp1->Opcode() == Op_IfTrue, "" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1113
    return 1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1114
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1115
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1116
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1117
//----------------------------check_cmove_id-----------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1118
// Check for CMove'ing a constant after comparing against the constant.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1119
// Happens all the time now, since if we compare equality vs a constant in
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1120
// the parser, we "know" the variable is constant on one path and we force
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1121
// it.  Thus code like "if( x==0 ) {/*EMPTY*/}" ends up inserting a
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1122
// conditional move: "x = (x==0)?0:x;".  Yucko.  This fix is slightly more
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1123
// general in that we don't need constants.  Since CMove's are only inserted
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1124
// in very special circumstances, we do it here on generic Phi's.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1125
Node* PhiNode::is_cmove_id(PhaseTransform* phase, int true_path) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1126
  assert(true_path !=0, "only diamond shape graph expected");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1127
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1128
  // is_diamond_phi() has guaranteed the correctness of the nodes sequence:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1129
  // phi->region->if_proj->ifnode->bool->cmp
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1130
  Node*     region = in(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1131
  Node*     iff    = region->in(1)->in(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1132
  BoolNode* b      = iff->in(1)->as_Bool();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1133
  Node*     cmp    = b->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1134
  Node*     tval   = in(true_path);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1135
  Node*     fval   = in(3-true_path);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1136
  Node*     id     = CMoveNode::is_cmove_id(phase, cmp, tval, fval, b);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1137
  if (id == NULL)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1138
    return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1139
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1140
  // Either value might be a cast that depends on a branch of 'iff'.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1141
  // Since the 'id' value will float free of the diamond, either
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1142
  // decast or return failure.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1143
  Node* ctl = id->in(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1144
  if (ctl != NULL && ctl->in(0) == iff) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1145
    if (id->is_ConstraintCast()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1146
      return id->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1147
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1148
      // Don't know how to disentangle this value.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1149
      return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1150
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1151
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1152
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1153
  return id;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1154
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1155
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1156
//------------------------------Identity---------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1157
// Check for Region being Identity.
35551
36ef3841fb34 8146629: Make phase->is_IterGVN() accessible from Node::Identity and Node::Value
thartmann
parents: 35549
diff changeset
  1158
Node* PhiNode::Identity(PhaseGVN* phase) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1159
  // Check for no merging going on
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1160
  // (There used to be special-case code here when this->region->is_Loop.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1161
  // It would check for a tributary phi on the backedge that the main phi
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1162
  // trivially, perhaps with a single cast.  The unique_input method
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1163
  // does all this and more, by reducing such tributaries to 'this'.)
35545
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1164
  Node* uin = unique_input(phase, false);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1165
  if (uin != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1166
    return uin;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1167
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1168
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1169
  int true_path = is_diamond_phi();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1170
  if (true_path != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1171
    Node* id = is_cmove_id(phase, true_path);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1172
    if (id != NULL)  return id;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1173
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1174
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1175
  return this;                     // No identity
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1176
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1177
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1178
//-----------------------------unique_input------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1179
// Find the unique value, discounting top, self-loops, and casts.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1180
// Return top if there are no inputs, and self if there are multiple.
35545
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1181
Node* PhiNode::unique_input(PhaseTransform* phase, bool uncast) {
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1182
  //  1) One unique direct input,
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1183
  // or if uncast is true:
35578
33d25acfb1fd 8146999: hotspot/test/compiler/c2/8007294/Test8007294.java test nightly failure
roland
parents: 35551
diff changeset
  1184
  //  2) some of the inputs have an intervening ConstraintCast
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1185
  //  3) an input is a self loop
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1186
  //
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1187
  //  1) input   or   2) input     or   3) input __
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1188
  //     /   \           /   \               \  /  \
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1189
  //     \   /          |    cast             phi  cast
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1190
  //      phi            \   /               /  \  /
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1191
  //                      phi               /    --
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1192
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1193
  Node* r = in(0);                      // RegionNode
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1194
  if (r == NULL)  return in(1);         // Already degraded to a Copy
35545
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1195
  Node* input = NULL; // The unique direct input (maybe uncasted = ConstraintCasts removed)
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1196
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1197
  for (uint i = 1, cnt = req(); i < cnt; ++i) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1198
    Node* rc = r->in(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1199
    if (rc == NULL || phase->type(rc) == Type::TOP)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1200
      continue;                 // ignore unreachable control path
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1201
    Node* n = in(i);
955
723d7b577fba 6723160: Nightly failure: Error: meet not symmetric
kvn
parents: 781
diff changeset
  1202
    if (n == NULL)
723d7b577fba 6723160: Nightly failure: Error: meet not symmetric
kvn
parents: 781
diff changeset
  1203
      continue;
35578
33d25acfb1fd 8146999: hotspot/test/compiler/c2/8007294/Test8007294.java test nightly failure
roland
parents: 35551
diff changeset
  1204
    Node* un = n;
33d25acfb1fd 8146999: hotspot/test/compiler/c2/8007294/Test8007294.java test nightly failure
roland
parents: 35551
diff changeset
  1205
    if (uncast) {
33d25acfb1fd 8146999: hotspot/test/compiler/c2/8007294/Test8007294.java test nightly failure
roland
parents: 35551
diff changeset
  1206
#ifdef ASSERT
33d25acfb1fd 8146999: hotspot/test/compiler/c2/8007294/Test8007294.java test nightly failure
roland
parents: 35551
diff changeset
  1207
      Node* m = un->uncast();
33d25acfb1fd 8146999: hotspot/test/compiler/c2/8007294/Test8007294.java test nightly failure
roland
parents: 35551
diff changeset
  1208
#endif
33d25acfb1fd 8146999: hotspot/test/compiler/c2/8007294/Test8007294.java test nightly failure
roland
parents: 35551
diff changeset
  1209
      while (un != NULL && un->req() == 2 && un->is_ConstraintCast()) {
33d25acfb1fd 8146999: hotspot/test/compiler/c2/8007294/Test8007294.java test nightly failure
roland
parents: 35551
diff changeset
  1210
        Node* next = un->in(1);
33d25acfb1fd 8146999: hotspot/test/compiler/c2/8007294/Test8007294.java test nightly failure
roland
parents: 35551
diff changeset
  1211
        if (phase->type(next)->isa_rawptr() && phase->type(un)->isa_oopptr()) {
33d25acfb1fd 8146999: hotspot/test/compiler/c2/8007294/Test8007294.java test nightly failure
roland
parents: 35551
diff changeset
  1212
          // risk exposing raw ptr at safepoint
33d25acfb1fd 8146999: hotspot/test/compiler/c2/8007294/Test8007294.java test nightly failure
roland
parents: 35551
diff changeset
  1213
          break;
33d25acfb1fd 8146999: hotspot/test/compiler/c2/8007294/Test8007294.java test nightly failure
roland
parents: 35551
diff changeset
  1214
        }
33d25acfb1fd 8146999: hotspot/test/compiler/c2/8007294/Test8007294.java test nightly failure
roland
parents: 35551
diff changeset
  1215
        un = next;
33d25acfb1fd 8146999: hotspot/test/compiler/c2/8007294/Test8007294.java test nightly failure
roland
parents: 35551
diff changeset
  1216
      }
33d25acfb1fd 8146999: hotspot/test/compiler/c2/8007294/Test8007294.java test nightly failure
roland
parents: 35551
diff changeset
  1217
      assert(m == un || un->in(1) == m, "Only expected at CheckCastPP from allocation");
33d25acfb1fd 8146999: hotspot/test/compiler/c2/8007294/Test8007294.java test nightly failure
roland
parents: 35551
diff changeset
  1218
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1219
    if (un == NULL || un == this || phase->type(un) == Type::TOP) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1220
      continue; // ignore if top, or in(i) and "this" are in a data cycle
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1221
    }
35545
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1222
    // Check for a unique input (maybe uncasted)
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1223
    if (input == NULL) {
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1224
      input = un;
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1225
    } else if (input != un) {
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1226
      input = NodeSentinel; // no unique input
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1227
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1228
  }
35545
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1229
  if (input == NULL) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1230
    return phase->C->top();        // no inputs
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1231
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1232
35545
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1233
  if (input != NodeSentinel) {
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1234
    return input;           // one unique direct input
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1235
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1236
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1237
  // Nothing.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1238
  return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1239
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1240
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1241
//------------------------------is_x2logic-------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1242
// Check for simple convert-to-boolean pattern
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1243
// If:(C Bool) Region:(IfF IfT) Phi:(Region 0 1)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1244
// Convert Phi to an ConvIB.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1245
static Node *is_x2logic( PhaseGVN *phase, PhiNode *phi, int true_path ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1246
  assert(true_path !=0, "only diamond shape graph expected");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1247
  // Convert the true/false index into an expected 0/1 return.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1248
  // Map 2->0 and 1->1.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1249
  int flipped = 2-true_path;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1250
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1251
  // is_diamond_phi() has guaranteed the correctness of the nodes sequence:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1252
  // phi->region->if_proj->ifnode->bool->cmp
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1253
  Node *region = phi->in(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1254
  Node *iff = region->in(1)->in(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1255
  BoolNode *b = (BoolNode*)iff->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1256
  const CmpNode *cmp = (CmpNode*)b->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1257
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1258
  Node *zero = phi->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1259
  Node *one  = phi->in(2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1260
  const Type *tzero = phase->type( zero );
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1261
  const Type *tone  = phase->type( one  );
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1262
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1263
  // Check for compare vs 0
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1264
  const Type *tcmp = phase->type(cmp->in(2));
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1265
  if( tcmp != TypeInt::ZERO && tcmp != TypePtr::NULL_PTR ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1266
    // Allow cmp-vs-1 if the other input is bounded by 0-1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1267
    if( !(tcmp == TypeInt::ONE && phase->type(cmp->in(1)) == TypeInt::BOOL) )
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1268
      return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1269
    flipped = 1-flipped;        // Test is vs 1 instead of 0!
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1270
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1271
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1272
  // Check for setting zero/one opposite expected
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1273
  if( tzero == TypeInt::ZERO ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1274
    if( tone == TypeInt::ONE ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1275
    } else return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1276
  } else if( tzero == TypeInt::ONE ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1277
    if( tone == TypeInt::ZERO ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1278
      flipped = 1-flipped;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1279
    } else return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1280
  } else return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1281
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1282
  // Check for boolean test backwards
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1283
  if( b->_test._test == BoolTest::ne ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1284
  } else if( b->_test._test == BoolTest::eq ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1285
    flipped = 1-flipped;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1286
  } else return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1287
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1288
  // Build int->bool conversion
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 23528
diff changeset
  1289
  Node *n = new Conv2BNode( cmp->in(1) );
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1290
  if( flipped )
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 23528
diff changeset
  1291
    n = new XorINode( phase->transform(n), phase->intcon(1) );
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1292
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1293
  return n;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1294
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1295
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1296
//------------------------------is_cond_add------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1297
// Check for simple conditional add pattern:  "(P < Q) ? X+Y : X;"
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1298
// To be profitable the control flow has to disappear; there can be no other
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1299
// values merging here.  We replace the test-and-branch with:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1300
// "(sgn(P-Q))&Y) + X".  Basically, convert "(P < Q)" into 0 or -1 by
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1301
// moving the carry bit from (P-Q) into a register with 'sbb EAX,EAX'.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1302
// Then convert Y to 0-or-Y and finally add.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1303
// This is a key transform for SpecJava _201_compress.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1304
static Node* is_cond_add(PhaseGVN *phase, PhiNode *phi, int true_path) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1305
  assert(true_path !=0, "only diamond shape graph expected");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1306
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1307
  // is_diamond_phi() has guaranteed the correctness of the nodes sequence:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1308
  // phi->region->if_proj->ifnode->bool->cmp
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1309
  RegionNode *region = (RegionNode*)phi->in(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1310
  Node *iff = region->in(1)->in(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1311
  BoolNode* b = iff->in(1)->as_Bool();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1312
  const CmpNode *cmp = (CmpNode*)b->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1313
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1314
  // Make sure only merging this one phi here
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1315
  if (region->has_unique_phi() != phi)  return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1316
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1317
  // Make sure each arm of the diamond has exactly one output, which we assume
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1318
  // is the region.  Otherwise, the control flow won't disappear.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1319
  if (region->in(1)->outcnt() != 1) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1320
  if (region->in(2)->outcnt() != 1) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1321
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1322
  // Check for "(P < Q)" of type signed int
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1323
  if (b->_test._test != BoolTest::lt)  return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1324
  if (cmp->Opcode() != Op_CmpI)        return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1325
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1326
  Node *p = cmp->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1327
  Node *q = cmp->in(2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1328
  Node *n1 = phi->in(  true_path);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1329
  Node *n2 = phi->in(3-true_path);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1330
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1331
  int op = n1->Opcode();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1332
  if( op != Op_AddI           // Need zero as additive identity
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1333
      /*&&op != Op_SubI &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1334
      op != Op_AddP &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1335
      op != Op_XorI &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1336
      op != Op_OrI*/ )
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1337
    return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1338
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1339
  Node *x = n2;
17008
fe66415573bf 6443505: Ideal() function for CmpLTMask
drchase
parents: 15113
diff changeset
  1340
  Node *y = NULL;
fe66415573bf 6443505: Ideal() function for CmpLTMask
drchase
parents: 15113
diff changeset
  1341
  if( x == n1->in(1) ) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1342
    y = n1->in(2);
17008
fe66415573bf 6443505: Ideal() function for CmpLTMask
drchase
parents: 15113
diff changeset
  1343
  } else if( x == n1->in(2) ) {
fe66415573bf 6443505: Ideal() function for CmpLTMask
drchase
parents: 15113
diff changeset
  1344
    y = n1->in(1);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1345
  } else return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1346
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1347
  // Not so profitable if compare and add are constants
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1348
  if( q->is_Con() && phase->type(q) != TypeInt::ZERO && y->is_Con() )
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1349
    return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1350
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 23528
diff changeset
  1351
  Node *cmplt = phase->transform( new CmpLTMaskNode(p,q) );
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 23528
diff changeset
  1352
  Node *j_and   = phase->transform( new AndINode(cmplt,y) );
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 23528
diff changeset
  1353
  return new AddINode(j_and,x);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1354
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1355
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1356
//------------------------------is_absolute------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1357
// Check for absolute value.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1358
static Node* is_absolute( PhaseGVN *phase, PhiNode *phi_root, int true_path) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1359
  assert(true_path !=0, "only diamond shape graph expected");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1360
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1361
  int  cmp_zero_idx = 0;        // Index of compare input where to look for zero
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1362
  int  phi_x_idx = 0;           // Index of phi input where to find naked x
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1363
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1364
  // ABS ends with the merge of 2 control flow paths.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1365
  // Find the false path from the true path. With only 2 inputs, 3 - x works nicely.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1366
  int false_path = 3 - true_path;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1367
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1368
  // is_diamond_phi() has guaranteed the correctness of the nodes sequence:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1369
  // phi->region->if_proj->ifnode->bool->cmp
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1370
  BoolNode *bol = phi_root->in(0)->in(1)->in(0)->in(1)->as_Bool();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1371
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1372
  // Check bool sense
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1373
  switch( bol->_test._test ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1374
  case BoolTest::lt: cmp_zero_idx = 1; phi_x_idx = true_path;  break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1375
  case BoolTest::le: cmp_zero_idx = 2; phi_x_idx = false_path; break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1376
  case BoolTest::gt: cmp_zero_idx = 2; phi_x_idx = true_path;  break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1377
  case BoolTest::ge: cmp_zero_idx = 1; phi_x_idx = false_path; break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1378
  default:           return NULL;                              break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1379
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1380
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1381
  // Test is next
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1382
  Node *cmp = bol->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1383
  const Type *tzero = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1384
  switch( cmp->Opcode() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1385
  case Op_CmpF:    tzero = TypeF::ZERO; break; // Float ABS
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1386
  case Op_CmpD:    tzero = TypeD::ZERO; break; // Double ABS
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1387
  default: return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1388
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1389
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1390
  // Find zero input of compare; the other input is being abs'd
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1391
  Node *x = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1392
  bool flip = false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1393
  if( phase->type(cmp->in(cmp_zero_idx)) == tzero ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1394
    x = cmp->in(3 - cmp_zero_idx);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1395
  } else if( phase->type(cmp->in(3 - cmp_zero_idx)) == tzero ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1396
    // The test is inverted, we should invert the result...
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1397
    x = cmp->in(cmp_zero_idx);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1398
    flip = true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1399
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1400
    return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1401
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1402
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1403
  // Next get the 2 pieces being selected, one is the original value
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1404
  // and the other is the negated value.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1405
  if( phi_root->in(phi_x_idx) != x ) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1406
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1407
  // Check other phi input for subtract node
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1408
  Node *sub = phi_root->in(3 - phi_x_idx);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1409
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1410
  // Allow only Sub(0,X) and fail out for all others; Neg is not OK
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1411
  if( tzero == TypeF::ZERO ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1412
    if( sub->Opcode() != Op_SubF ||
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1413
        sub->in(2) != x ||
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1414
        phase->type(sub->in(1)) != tzero ) return NULL;
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 23528
diff changeset
  1415
    x = new AbsFNode(x);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1416
    if (flip) {
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 23528
diff changeset
  1417
      x = new SubFNode(sub->in(1), phase->transform(x));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1418
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1419
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1420
    if( sub->Opcode() != Op_SubD ||
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1421
        sub->in(2) != x ||
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1422
        phase->type(sub->in(1)) != tzero ) return NULL;
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 23528
diff changeset
  1423
    x = new AbsDNode(x);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1424
    if (flip) {
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 23528
diff changeset
  1425
      x = new SubDNode(sub->in(1), phase->transform(x));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1426
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1427
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1428
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1429
  return x;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1430
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1431
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1432
//------------------------------split_once-------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1433
// Helper for split_flow_path
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1434
static void split_once(PhaseIterGVN *igvn, Node *phi, Node *val, Node *n, Node *newn) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1435
  igvn->hash_delete(n);         // Remove from hash before hacking edges
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1436
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1437
  uint j = 1;
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
  1438
  for (uint i = phi->req()-1; i > 0; i--) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
  1439
    if (phi->in(i) == val) {   // Found a path with val?
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1440
      // Add to NEW Region/Phi, no DU info
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1441
      newn->set_req( j++, n->in(i) );
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1442
      // Remove from OLD Region/Phi
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1443
      n->del_req(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1444
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1445
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1446
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1447
  // Register the new node but do not transform it.  Cannot transform until the
2131
98f9cef66a34 6810672: Comment typos
twisti
parents: 2019
diff changeset
  1448
  // entire Region/Phi conglomerate has been hacked as a single huge transform.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1449
  igvn->register_new_node_with_optimizer( newn );
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents: 7397
diff changeset
  1450
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1451
  // Now I can point to the new node.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1452
  n->add_req(newn);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1453
  igvn->_worklist.push(n);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1454
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1455
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1456
//------------------------------split_flow_path--------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1457
// Check for merging identical values and split flow paths
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1458
static Node* split_flow_path(PhaseGVN *phase, PhiNode *phi) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1459
  BasicType bt = phi->type()->basic_type();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1460
  if( bt == T_ILLEGAL || type2size[bt] <= 0 )
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1461
    return NULL;                // Bail out on funny non-value stuff
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1462
  if( phi->req() <= 3 )         // Need at least 2 matched inputs and a
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1463
    return NULL;                // third unequal input to be worth doing
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1464
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1465
  // Scan for a constant
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1466
  uint i;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1467
  for( i = 1; i < phi->req()-1; i++ ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1468
    Node *n = phi->in(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1469
    if( !n ) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1470
    if( phase->type(n) == Type::TOP ) return NULL;
13969
d2a189b83b87 7054512: Compress class pointers after perm gen removal
roland
parents: 13895
diff changeset
  1471
    if( n->Opcode() == Op_ConP || n->Opcode() == Op_ConN || n->Opcode() == Op_ConNKlass )
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1472
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1473
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1474
  if( i >= phi->req() )         // Only split for constants
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1475
    return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1476
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1477
  Node *val = phi->in(i);       // Constant to split for
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1478
  uint hit = 0;                 // Number of times it occurs
10258
10c77b8c8d3e 7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents: 9956
diff changeset
  1479
  Node *r = phi->region();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1480
2131
98f9cef66a34 6810672: Comment typos
twisti
parents: 2019
diff changeset
  1481
  for( ; i < phi->req(); i++ ){ // Count occurrences of constant
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1482
    Node *n = phi->in(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1483
    if( !n ) return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1484
    if( phase->type(n) == Type::TOP ) return NULL;
10258
10c77b8c8d3e 7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents: 9956
diff changeset
  1485
    if( phi->in(i) == val ) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1486
      hit++;
10258
10c77b8c8d3e 7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents: 9956
diff changeset
  1487
      if (PhaseIdealLoop::find_predicate(r->in(i)) != NULL) {
10c77b8c8d3e 7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents: 9956
diff changeset
  1488
        return NULL;            // don't split loop entry path
10c77b8c8d3e 7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents: 9956
diff changeset
  1489
      }
10c77b8c8d3e 7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents: 9956
diff changeset
  1490
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1491
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1492
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1493
  if( hit <= 1 ||               // Make sure we find 2 or more
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1494
      hit == phi->req()-1 )     // and not ALL the same value
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1495
    return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1496
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1497
  // Now start splitting out the flow paths that merge the same value.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1498
  // Split first the RegionNode.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1499
  PhaseIterGVN *igvn = phase->is_IterGVN();
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 23528
diff changeset
  1500
  RegionNode *newr = new RegionNode(hit+1);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1501
  split_once(igvn, phi, val, r, newr);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1502
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1503
  // Now split all other Phis than this one
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1504
  for (DUIterator_Fast kmax, k = r->fast_outs(kmax); k < kmax; k++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1505
    Node* phi2 = r->fast_out(k);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1506
    if( phi2->is_Phi() && phi2->as_Phi() != phi ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1507
      PhiNode *newphi = PhiNode::make_blank(newr, phi2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1508
      split_once(igvn, phi, val, phi2, newphi);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1509
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1510
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1511
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1512
  // Clean up this guy
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1513
  igvn->hash_delete(phi);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1514
  for( i = phi->req()-1; i > 0; i-- ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1515
    if( phi->in(i) == val ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1516
      phi->del_req(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1517
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1518
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1519
  phi->add_req(val);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1520
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1521
  return phi;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1522
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1523
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1524
//=============================================================================
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1525
//------------------------------simple_data_loop_check-------------------------
2131
98f9cef66a34 6810672: Comment typos
twisti
parents: 2019
diff changeset
  1526
//  Try to determining if the phi node in a simple safe/unsafe data loop.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1527
//  Returns:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1528
// enum LoopSafety { Safe = 0, Unsafe, UnsafeLoop };
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1529
// Safe       - safe case when the phi and it's inputs reference only safe data
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1530
//              nodes;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1531
// Unsafe     - the phi and it's inputs reference unsafe data nodes but there
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1532
//              is no reference back to the phi - need a graph walk
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1533
//              to determine if it is in a loop;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1534
// UnsafeLoop - unsafe case when the phi references itself directly or through
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1535
//              unsafe data node.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1536
//  Note: a safe data node is a node which could/never reference itself during
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1537
//  GVN transformations. For now it is Con, Proj, Phi, CastPP, CheckCastPP.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1538
//  I mark Phi nodes as safe node not only because they can reference itself
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1539
//  but also to prevent mistaking the fallthrough case inside an outer loop
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1540
//  as dead loop when the phi references itselfs through an other phi.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1541
PhiNode::LoopSafety PhiNode::simple_data_loop_check(Node *in) const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1542
  // It is unsafe loop if the phi node references itself directly.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1543
  if (in == (Node*)this)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1544
    return UnsafeLoop; // Unsafe loop
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1545
  // Unsafe loop if the phi node references itself through an unsafe data node.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1546
  // Exclude cases with null inputs or data nodes which could reference
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1547
  // itself (safe for dead loops).
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1548
  if (in != NULL && !in->is_dead_loop_safe()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1549
    // Check inputs of phi's inputs also.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1550
    // It is much less expensive then full graph walk.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1551
    uint cnt = in->req();
373
427fb4ca6a1e 6625997: CastPP, CheckCastPP and Proj nodes are not dead loop safe
kvn
parents: 360
diff changeset
  1552
    uint i = (in->is_Proj() && !in->is_CFG())  ? 0 : 1;
427fb4ca6a1e 6625997: CastPP, CheckCastPP and Proj nodes are not dead loop safe
kvn
parents: 360
diff changeset
  1553
    for (; i < cnt; ++i) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1554
      Node* m = in->in(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1555
      if (m == (Node*)this)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1556
        return UnsafeLoop; // Unsafe loop
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1557
      if (m != NULL && !m->is_dead_loop_safe()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1558
        // Check the most common case (about 30% of all cases):
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1559
        // phi->Load/Store->AddP->(ConP ConP Con)/(Parm Parm Con).
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1560
        Node *m1 = (m->is_AddP() && m->req() > 3) ? m->in(1) : NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1561
        if (m1 == (Node*)this)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1562
          return UnsafeLoop; // Unsafe loop
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1563
        if (m1 != NULL && m1 == m->in(2) &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1564
            m1->is_dead_loop_safe() && m->in(3)->is_Con()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1565
          continue; // Safe case
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1566
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1567
        // The phi references an unsafe node - need full analysis.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1568
        return Unsafe;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1569
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1570
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1571
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1572
  return Safe; // Safe case - we can optimize the phi node.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1573
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1574
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1575
//------------------------------is_unsafe_data_reference-----------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1576
// If phi can be reached through the data input - it is data loop.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1577
bool PhiNode::is_unsafe_data_reference(Node *in) const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1578
  assert(req() > 1, "");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1579
  // First, check simple cases when phi references itself directly or
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1580
  // through an other node.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1581
  LoopSafety safety = simple_data_loop_check(in);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1582
  if (safety == UnsafeLoop)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1583
    return true;  // phi references itself - unsafe loop
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1584
  else if (safety == Safe)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1585
    return false; // Safe case - phi could be replaced with the unique input.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1586
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1587
  // Unsafe case when we should go through data graph to determine
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1588
  // if the phi references itself.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1589
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1590
  ResourceMark rm;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1591
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1592
  Arena *a = Thread::current()->resource_area();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1593
  Node_List nstack(a);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1594
  VectorSet visited(a);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1595
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1596
  nstack.push(in); // Start with unique input.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1597
  visited.set(in->_idx);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1598
  while (nstack.size() != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1599
    Node* n = nstack.pop();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1600
    uint cnt = n->req();
373
427fb4ca6a1e 6625997: CastPP, CheckCastPP and Proj nodes are not dead loop safe
kvn
parents: 360
diff changeset
  1601
    uint i = (n->is_Proj() && !n->is_CFG()) ? 0 : 1;
427fb4ca6a1e 6625997: CastPP, CheckCastPP and Proj nodes are not dead loop safe
kvn
parents: 360
diff changeset
  1602
    for (; i < cnt; i++) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1603
      Node* m = n->in(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1604
      if (m == (Node*)this) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1605
        return true;    // Data loop
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1606
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1607
      if (m != NULL && !m->is_dead_loop_safe()) { // Only look for unsafe cases.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1608
        if (!visited.test_set(m->_idx))
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1609
          nstack.push(m);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1610
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1611
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1612
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1613
  return false; // The phi is not reachable from its inputs
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1614
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1615
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1616
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1617
//------------------------------Ideal------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1618
// Return a node which is more "ideal" than the current node.  Must preserve
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1619
// the CFG, but we can still strip out dead paths.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1620
Node *PhiNode::Ideal(PhaseGVN *phase, bool can_reshape) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1621
  // The next should never happen after 6297035 fix.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1622
  if( is_copy() )               // Already degraded to a Copy ?
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1623
    return NULL;                // No change
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1624
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1625
  Node *r = in(0);              // RegionNode
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1626
  assert(r->in(0) == NULL || !r->in(0)->is_Root(), "not a specially hidden merge");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1627
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1628
  // Note: During parsing, phis are often transformed before their regions.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1629
  // This means we have to use type_or_null to defend against untyped regions.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1630
  if( phase->type_or_null(r) == Type::TOP ) // Dead code?
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1631
    return NULL;                // No change
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1632
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1633
  Node *top = phase->C->top();
4016
9c11d6d519f7 6889300: assert(i != k || is_new || i->outcnt() > 0, "don't return dead nodes")
kvn
parents: 3603
diff changeset
  1634
  bool new_phi = (outcnt() == 0); // transforming new Phi
9956
a3f4ad8fee9e 7045506: assert(!can_reshape || !new_phi) failed: for igvn new phi should be hooked
kvn
parents: 9446
diff changeset
  1635
  // No change for igvn if new phi is not hooked
a3f4ad8fee9e 7045506: assert(!can_reshape || !new_phi) failed: for igvn new phi should be hooked
kvn
parents: 9446
diff changeset
  1636
  if (new_phi && can_reshape)
a3f4ad8fee9e 7045506: assert(!can_reshape || !new_phi) failed: for igvn new phi should be hooked
kvn
parents: 9446
diff changeset
  1637
    return NULL;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1638
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1639
  // The are 2 situations when only one valid phi's input is left
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1640
  // (in addition to Region input).
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1641
  // One: region is not loop - replace phi with this input.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1642
  // Two: region is loop - replace phi with top since this data path is dead
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1643
  //                       and we need to break the dead data loop.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1644
  Node* progress = NULL;        // Record if any progress made
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1645
  for( uint j = 1; j < req(); ++j ){ // For all paths in
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1646
    // Check unreachable control paths
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1647
    Node* rc = r->in(j);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1648
    Node* n = in(j);            // Get the input
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1649
    if (rc == NULL || phase->type(rc) == Type::TOP) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1650
      if (n != top) {           // Not already top?
14828
bb9dffedf46c 8005031: Some cleanup in c2 to prepare for incremental inlining support
roland
parents: 13969
diff changeset
  1651
        PhaseIterGVN *igvn = phase->is_IterGVN();
bb9dffedf46c 8005031: Some cleanup in c2 to prepare for incremental inlining support
roland
parents: 13969
diff changeset
  1652
        if (can_reshape && igvn != NULL) {
bb9dffedf46c 8005031: Some cleanup in c2 to prepare for incremental inlining support
roland
parents: 13969
diff changeset
  1653
          igvn->_worklist.push(r);
bb9dffedf46c 8005031: Some cleanup in c2 to prepare for incremental inlining support
roland
parents: 13969
diff changeset
  1654
        }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1655
        set_req(j, top);        // Nuke it down
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1656
        progress = this;        // Record progress
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1657
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1658
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1659
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1660
4016
9c11d6d519f7 6889300: assert(i != k || is_new || i->outcnt() > 0, "don't return dead nodes")
kvn
parents: 3603
diff changeset
  1661
  if (can_reshape && outcnt() == 0) {
9c11d6d519f7 6889300: assert(i != k || is_new || i->outcnt() > 0, "don't return dead nodes")
kvn
parents: 3603
diff changeset
  1662
    // set_req() above may kill outputs if Phi is referenced
9c11d6d519f7 6889300: assert(i != k || is_new || i->outcnt() > 0, "don't return dead nodes")
kvn
parents: 3603
diff changeset
  1663
    // only by itself on the dead (top) control path.
9c11d6d519f7 6889300: assert(i != k || is_new || i->outcnt() > 0, "don't return dead nodes")
kvn
parents: 3603
diff changeset
  1664
    return top;
9c11d6d519f7 6889300: assert(i != k || is_new || i->outcnt() > 0, "don't return dead nodes")
kvn
parents: 3603
diff changeset
  1665
  }
9c11d6d519f7 6889300: assert(i != k || is_new || i->outcnt() > 0, "don't return dead nodes")
kvn
parents: 3603
diff changeset
  1666
35545
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1667
  bool uncasted = false;
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1668
  Node* uin = unique_input(phase, false);
36611
eb4fe2f11579 8150804: C2 Compilation fails with assert(_base >= OopPtr && _base <= AryPtr) failed: Not a Java pointer
thartmann
parents: 35600
diff changeset
  1669
  if (uin == NULL && can_reshape) {
35545
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1670
    uncasted = true;
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1671
    uin = unique_input(phase, true);
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1672
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1673
  if (uin == top) {             // Simplest case: no alive inputs.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1674
    if (can_reshape)            // IGVN transformation
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1675
      return top;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1676
    else
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1677
      return NULL;              // Identity will return TOP
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1678
  } else if (uin != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1679
    // Only one not-NULL unique input path is left.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1680
    // Determine if this input is backedge of a loop.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1681
    // (Skip new phis which have no uses and dead regions).
11189
c1ad8528ae68 7105605: Use EA info to optimize pointers compare
kvn
parents: 10258
diff changeset
  1682
    if (outcnt() > 0 && r->in(0) != NULL) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1683
      // First, take the short cut when we know it is a loop and
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1684
      // the EntryControl data path is dead.
11189
c1ad8528ae68 7105605: Use EA info to optimize pointers compare
kvn
parents: 10258
diff changeset
  1685
      // Loop node may have only one input because entry path
c1ad8528ae68 7105605: Use EA info to optimize pointers compare
kvn
parents: 10258
diff changeset
  1686
      // is removed in PhaseIdealLoop::Dominators().
c1ad8528ae68 7105605: Use EA info to optimize pointers compare
kvn
parents: 10258
diff changeset
  1687
      assert(!r->is_Loop() || r->req() <= 3, "Loop node should have 3 or less inputs");
c1ad8528ae68 7105605: Use EA info to optimize pointers compare
kvn
parents: 10258
diff changeset
  1688
      bool is_loop = (r->is_Loop() && r->req() == 3);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1689
      // Then, check if there is a data loop when phi references itself directly
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1690
      // or through other data nodes.
11446
fd87432a895b 7128352: assert(obj_node == obj) failed
kvn
parents: 11189
diff changeset
  1691
      if (is_loop && !uin->eqv_uncast(in(LoopNode::EntryControl)) ||
11189
c1ad8528ae68 7105605: Use EA info to optimize pointers compare
kvn
parents: 10258
diff changeset
  1692
         !is_loop && is_unsafe_data_reference(uin)) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1693
        // Break this data loop to avoid creation of a dead loop.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1694
        if (can_reshape) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1695
          return top;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1696
        } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1697
          // We can't return top if we are in Parse phase - cut inputs only
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1698
          // let Identity to handle the case.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1699
          replace_edge(uin, top);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1700
          return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1701
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1702
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1703
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1704
35545
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1705
    if (uncasted) {
38045
808cbf1958ce 8153357: C2 creates incorrect cast after eliminating phi with unique input
zmajo
parents: 37248
diff changeset
  1706
      // Add a cast node between the phi to be removed and its unique input.
808cbf1958ce 8153357: C2 creates incorrect cast after eliminating phi with unique input
zmajo
parents: 37248
diff changeset
  1707
      // Wait until after parsing for the type information to propagate from the casts.
36611
eb4fe2f11579 8150804: C2 Compilation fails with assert(_base >= OopPtr && _base <= AryPtr) failed: Not a Java pointer
thartmann
parents: 35600
diff changeset
  1708
      assert(can_reshape, "Invalid during parsing");
35545
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1709
      const Type* phi_type = bottom_type();
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1710
      assert(phi_type->isa_int() || phi_type->isa_ptr(), "bad phi type");
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1711
      int opcode;
38045
808cbf1958ce 8153357: C2 creates incorrect cast after eliminating phi with unique input
zmajo
parents: 37248
diff changeset
  1712
      // Determine the type of cast to be added.
35545
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1713
      if (phi_type->isa_int()) {
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1714
        opcode = Op_CastII;
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1715
      } else {
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1716
        const Type* uin_type = phase->type(uin);
38045
808cbf1958ce 8153357: C2 creates incorrect cast after eliminating phi with unique input
zmajo
parents: 37248
diff changeset
  1717
        if ((phi_type->join(TypePtr::NOTNULL) == uin_type->join(TypePtr::NOTNULL)) ||
808cbf1958ce 8153357: C2 creates incorrect cast after eliminating phi with unique input
zmajo
parents: 37248
diff changeset
  1718
            (!phi_type->isa_oopptr() && !uin_type->isa_oopptr())) {
35545
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1719
          opcode = Op_CastPP;
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1720
        } else {
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1721
          opcode = Op_CheckCastPP;
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1722
        }
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1723
      }
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1724
      // Add a cast to carry the control dependency of the Phi that is
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1725
      // going away
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1726
      Node* cast = ConstraintCastNode::make_cast(opcode, r, uin, phi_type, true);
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1727
      cast = phase->transform(cast);
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1728
      // set all inputs to the new cast so the Phi is removed by Identity
36611
eb4fe2f11579 8150804: C2 Compilation fails with assert(_base >= OopPtr && _base <= AryPtr) failed: Not a Java pointer
thartmann
parents: 35600
diff changeset
  1729
      PhaseIterGVN* igvn = phase->is_IterGVN();
35545
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1730
      for (uint i = 1; i < req(); i++) {
36611
eb4fe2f11579 8150804: C2 Compilation fails with assert(_base >= OopPtr && _base <= AryPtr) failed: Not a Java pointer
thartmann
parents: 35600
diff changeset
  1731
        set_req_X(i, cast, igvn);
35545
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1732
      }
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1733
      uin = cast;
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1734
    }
a8f29dfd62b2 8139771: Eliminating CastPP nodes at Phis when they all come from a unique input may cause crash
roland
parents: 34202
diff changeset
  1735
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1736
    // One unique input.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1737
    debug_only(Node* ident = Identity(phase));
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1738
    // The unique input must eventually be detected by the Identity call.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1739
#ifdef ASSERT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1740
    if (ident != uin && !ident->is_top()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1741
      // print this output before failing assert
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1742
      r->dump(3);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1743
      this->dump(3);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1744
      ident->dump();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1745
      uin->dump();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1746
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1747
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1748
    assert(ident == uin || ident->is_top(), "Identity must clean this up");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1749
    return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1750
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1751
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1752
  Node* opt = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1753
  int true_path = is_diamond_phi();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1754
  if( true_path != 0 ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1755
    // Check for CMove'ing identity. If it would be unsafe,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1756
    // handle it here. In the safe case, let Identity handle it.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1757
    Node* unsafe_id = is_cmove_id(phase, true_path);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1758
    if( unsafe_id != NULL && is_unsafe_data_reference(unsafe_id) )
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1759
      opt = unsafe_id;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1760
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1761
    // Check for simple convert-to-boolean pattern
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1762
    if( opt == NULL )
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1763
      opt = is_x2logic(phase, this, true_path);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1764
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1765
    // Check for absolute value
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1766
    if( opt == NULL )
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1767
      opt = is_absolute(phase, this, true_path);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1768
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1769
    // Check for conditional add
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1770
    if( opt == NULL && can_reshape )
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1771
      opt = is_cond_add(phase, this, true_path);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1772
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1773
    // These 4 optimizations could subsume the phi:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1774
    // have to check for a dead data loop creation.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1775
    if( opt != NULL ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1776
      if( opt == unsafe_id || is_unsafe_data_reference(opt) ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1777
        // Found dead loop.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1778
        if( can_reshape )
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1779
          return top;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1780
        // We can't return top if we are in Parse phase - cut inputs only
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1781
        // to stop further optimizations for this phi. Identity will return TOP.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1782
        assert(req() == 3, "only diamond merge phi here");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1783
        set_req(1, top);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1784
        set_req(2, top);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1785
        return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1786
      } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1787
        return opt;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1788
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1789
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1790
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1791
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1792
  // Check for merging identical values and split flow paths
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1793
  if (can_reshape) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1794
    opt = split_flow_path(phase, this);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1795
    // This optimization only modifies phi - don't need to check for dead loop.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1796
    assert(opt == NULL || phase->eqv(opt, this), "do not elide phi");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1797
    if (opt != NULL)  return opt;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1798
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1799
5537
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1800
  if (in(1) != NULL && in(1)->Opcode() == Op_AddP && can_reshape) {
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1801
    // Try to undo Phi of AddP:
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1802
    // (Phi (AddP base base y) (AddP base2 base2 y))
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1803
    // becomes:
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1804
    // newbase := (Phi base base2)
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1805
    // (AddP newbase newbase y)
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1806
    //
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1807
    // This occurs as a result of unsuccessful split_thru_phi and
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1808
    // interferes with taking advantage of addressing modes. See the
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1809
    // clone_shift_expressions code in matcher.cpp
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1810
    Node* addp = in(1);
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1811
    const Type* type = addp->in(AddPNode::Base)->bottom_type();
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1812
    Node* y = addp->in(AddPNode::Offset);
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1813
    if (y != NULL && addp->in(AddPNode::Base) == addp->in(AddPNode::Address)) {
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1814
      // make sure that all the inputs are similar to the first one,
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1815
      // i.e. AddP with base == address and same offset as first AddP
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1816
      bool doit = true;
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1817
      for (uint i = 2; i < req(); i++) {
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1818
        if (in(i) == NULL ||
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1819
            in(i)->Opcode() != Op_AddP ||
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1820
            in(i)->in(AddPNode::Base) != in(i)->in(AddPNode::Address) ||
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1821
            in(i)->in(AddPNode::Offset) != y) {
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1822
          doit = false;
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1823
          break;
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1824
        }
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1825
        // Accumulate type for resulting Phi
22799
83e58bac7980 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 22501
diff changeset
  1826
        type = type->meet_speculative(in(i)->in(AddPNode::Base)->bottom_type());
5537
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1827
      }
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1828
      Node* base = NULL;
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1829
      if (doit) {
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1830
        // Check for neighboring AddP nodes in a tree.
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1831
        // If they have a base, use that it.
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1832
        for (DUIterator_Fast kmax, k = this->fast_outs(kmax); k < kmax; k++) {
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1833
          Node* u = this->fast_out(k);
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1834
          if (u->is_AddP()) {
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1835
            Node* base2 = u->in(AddPNode::Base);
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1836
            if (base2 != NULL && !base2->is_top()) {
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1837
              if (base == NULL)
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1838
                base = base2;
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1839
              else if (base != base2)
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1840
                { doit = false; break; }
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1841
            }
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1842
          }
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1843
        }
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1844
      }
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1845
      if (doit) {
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1846
        if (base == NULL) {
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 23528
diff changeset
  1847
          base = new PhiNode(in(0), type, NULL);
5537
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1848
          for (uint i = 1; i < req(); i++) {
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1849
            base->init_req(i, in(i)->in(AddPNode::Base));
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1850
          }
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1851
          phase->is_IterGVN()->register_new_node_with_optimizer(base);
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1852
        }
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 23528
diff changeset
  1853
        return new AddPNode(base, base, y);
5537
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1854
      }
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1855
    }
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1856
  }
c075180330cf 6937111: Restore optimization for Phi of AddP (6552204)
kvn
parents: 4016
diff changeset
  1857
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1858
  // Split phis through memory merges, so that the memory merges will go away.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1859
  // Piggy-back this transformation on the search for a unique input....
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1860
  // It will be as if the merged memory is the unique value of the phi.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1861
  // (Do not attempt this optimization unless parsing is complete.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1862
  // It would make the parser's memory-merge logic sick.)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1863
  // (MergeMemNode is not dead_loop_safe - need to check for dead loop.)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1864
  if (progress == NULL && can_reshape && type() == Type::MEMORY) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1865
    // see if this phi should be sliced
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1866
    uint merge_width = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1867
    bool saw_self = false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1868
    for( uint i=1; i<req(); ++i ) {// For all paths in
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1869
      Node *ii = in(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1870
      if (ii->is_MergeMem()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1871
        MergeMemNode* n = ii->as_MergeMem();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1872
        merge_width = MAX2(merge_width, n->req());
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1873
        saw_self = saw_self || phase->eqv(n->base_memory(), this);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1874
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1875
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1876
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1877
    // This restriction is temporarily necessary to ensure termination:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1878
    if (!saw_self && adr_type() == TypePtr::BOTTOM)  merge_width = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1879
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1880
    if (merge_width > Compile::AliasIdxRaw) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1881
      // found at least one non-empty MergeMem
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1882
      const TypePtr* at = adr_type();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1883
      if (at != TypePtr::BOTTOM) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1884
        // Patch the existing phi to select an input from the merge:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1885
        // Phi:AT1(...MergeMem(m0, m1, m2)...) into
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1886
        //     Phi:AT1(...m1...)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1887
        int alias_idx = phase->C->get_alias_index(at);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1888
        for (uint i=1; i<req(); ++i) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1889
          Node *ii = in(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1890
          if (ii->is_MergeMem()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1891
            MergeMemNode* n = ii->as_MergeMem();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1892
            // compress paths and change unreachable cycles to TOP
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1893
            // If not, we can update the input infinitely along a MergeMem cycle
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1894
            // Equivalent code is in MemNode::Ideal_common
1399
9648dfd4ce09 6384206: Phis which are later unneeded are impairing our ability to inline based on static types
never
parents: 955
diff changeset
  1895
            Node *m  = phase->transform(n);
9648dfd4ce09 6384206: Phis which are later unneeded are impairing our ability to inline based on static types
never
parents: 955
diff changeset
  1896
            if (outcnt() == 0) {  // Above transform() may kill us!
4016
9c11d6d519f7 6889300: assert(i != k || is_new || i->outcnt() > 0, "don't return dead nodes")
kvn
parents: 3603
diff changeset
  1897
              return top;
1399
9648dfd4ce09 6384206: Phis which are later unneeded are impairing our ability to inline based on static types
never
parents: 955
diff changeset
  1898
            }
2131
98f9cef66a34 6810672: Comment typos
twisti
parents: 2019
diff changeset
  1899
            // If transformed to a MergeMem, get the desired slice
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1900
            // Otherwise the returned node represents memory for every slice
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1901
            Node *new_mem = (m->is_MergeMem()) ?
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1902
                             m->as_MergeMem()->memory_at(alias_idx) : m;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1903
            // Update input if it is progress over what we have now
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1904
            if (new_mem != ii) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1905
              set_req(i, new_mem);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1906
              progress = this;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1907
            }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1908
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1909
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1910
      } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1911
        // We know that at least one MergeMem->base_memory() == this
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1912
        // (saw_self == true). If all other inputs also references this phi
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1913
        // (directly or through data nodes) - it is dead loop.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1914
        bool saw_safe_input = false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1915
        for (uint j = 1; j < req(); ++j) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1916
          Node *n = in(j);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1917
          if (n->is_MergeMem() && n->as_MergeMem()->base_memory() == this)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1918
            continue;              // skip known cases
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1919
          if (!is_unsafe_data_reference(n)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1920
            saw_safe_input = true; // found safe input
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1921
            break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1922
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1923
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1924
        if (!saw_safe_input)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1925
          return top; // all inputs reference back to this phi - dead loop
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1926
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1927
        // Phi(...MergeMem(m0, m1:AT1, m2:AT2)...) into
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1928
        //     MergeMem(Phi(...m0...), Phi:AT1(...m1...), Phi:AT2(...m2...))
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1929
        PhaseIterGVN *igvn = phase->is_IterGVN();
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 23528
diff changeset
  1930
        Node* hook = new Node(1);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1931
        PhiNode* new_base = (PhiNode*) clone();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1932
        // Must eagerly register phis, since they participate in loops.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1933
        if (igvn) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1934
          igvn->register_new_node_with_optimizer(new_base);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1935
          hook->add_req(new_base);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1936
        }
25930
eae8b7490d2c 8054033: Remove unused references to Compile*
thartmann
parents: 25913
diff changeset
  1937
        MergeMemNode* result = MergeMemNode::make(new_base);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1938
        for (uint i = 1; i < req(); ++i) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1939
          Node *ii = in(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1940
          if (ii->is_MergeMem()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1941
            MergeMemNode* n = ii->as_MergeMem();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1942
            for (MergeMemStream mms(result, n); mms.next_non_empty2(); ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1943
              // If we have not seen this slice yet, make a phi for it.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1944
              bool made_new_phi = false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1945
              if (mms.is_empty()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1946
                Node* new_phi = new_base->slice_memory(mms.adr_type(phase->C));
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1947
                made_new_phi = true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1948
                if (igvn) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1949
                  igvn->register_new_node_with_optimizer(new_phi);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1950
                  hook->add_req(new_phi);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1951
                }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1952
                mms.set_memory(new_phi);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1953
              }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1954
              Node* phi = mms.memory();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1955
              assert(made_new_phi || phi->in(i) == n, "replace the i-th merge by a slice");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1956
              phi->set_req(i, mms.memory2());
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1957
            }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1958
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1959
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1960
        // Distribute all self-loops.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1961
        { // (Extra braces to hide mms.)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1962
          for (MergeMemStream mms(result); mms.next_non_empty(); ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1963
            Node* phi = mms.memory();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1964
            for (uint i = 1; i < req(); ++i) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1965
              if (phi->in(i) == this)  phi->set_req(i, phi);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1966
            }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1967
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1968
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1969
        // now transform the new nodes, and return the mergemem
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1970
        for (MergeMemStream mms(result); mms.next_non_empty(); ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1971
          Node* phi = mms.memory();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1972
          mms.set_memory(phase->transform(phi));
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1973
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1974
        if (igvn) { // Unhook.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1975
          igvn->hash_delete(hook);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1976
          for (uint i = 1; i < hook->req(); i++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1977
            hook->set_req(i, NULL);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1978
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1979
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1980
        // Replace self with the result.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1981
        return result;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1982
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1983
    }
247
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
  1984
    //
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
  1985
    // Other optimizations on the memory chain
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
  1986
    //
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
  1987
    const TypePtr* at = adr_type();
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
  1988
    for( uint i=1; i<req(); ++i ) {// For all paths in
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
  1989
      Node *ii = in(i);
17383
3665c0901a0d 6934604: enable parts of EliminateAutoBox by default
kvn
parents: 17008
diff changeset
  1990
      Node *new_in = MemNode::optimize_memory_chain(ii, at, NULL, phase);
247
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
  1991
      if (ii != new_in ) {
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
  1992
        set_req(i, new_in);
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
  1993
        progress = this;
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
  1994
      }
2aeab9ac7fea 6674600: (Escape Analysis) Optimize memory graph for instance's fields
kvn
parents: 1
diff changeset
  1995
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1996
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1997
1400
afd034bb8c2e 6747051: Improve code and implicit null check generation for compressed oops
kvn
parents: 1399
diff changeset
  1998
#ifdef _LP64
13969
d2a189b83b87 7054512: Compress class pointers after perm gen removal
roland
parents: 13895
diff changeset
  1999
  // Push DecodeN/DecodeNKlass down through phi.
1400
afd034bb8c2e 6747051: Improve code and implicit null check generation for compressed oops
kvn
parents: 1399
diff changeset
  2000
  // The rest of phi graph will transform by split EncodeP node though phis up.
19979
ebe1dbb6e1aa 8015107: NPG: Use consistent naming for metaspace concepts
ehelin
parents: 17383
diff changeset
  2001
  if ((UseCompressedOops || UseCompressedClassPointers) && can_reshape && progress == NULL) {
1400
afd034bb8c2e 6747051: Improve code and implicit null check generation for compressed oops
kvn
parents: 1399
diff changeset
  2002
    bool may_push = true;
afd034bb8c2e 6747051: Improve code and implicit null check generation for compressed oops
kvn
parents: 1399
diff changeset
  2003
    bool has_decodeN = false;
13969
d2a189b83b87 7054512: Compress class pointers after perm gen removal
roland
parents: 13895
diff changeset
  2004
    bool is_decodeN = false;
1400
afd034bb8c2e 6747051: Improve code and implicit null check generation for compressed oops
kvn
parents: 1399
diff changeset
  2005
    for (uint i=1; i<req(); ++i) {// For all paths in
afd034bb8c2e 6747051: Improve code and implicit null check generation for compressed oops
kvn
parents: 1399
diff changeset
  2006
      Node *ii = in(i);
13969
d2a189b83b87 7054512: Compress class pointers after perm gen removal
roland
parents: 13895
diff changeset
  2007
      if (ii->is_DecodeNarrowPtr() && ii->bottom_type() == bottom_type()) {
3599
35bb709f2c62 6865031: Application gives bad result (throws bad exception) with compressed oops
kvn
parents: 3277
diff changeset
  2008
        // Do optimization if a non dead path exist.
3270
0a2825307027 6851282: JIT miscompilation results in null entry in array when using CompressedOops
kvn
parents: 2154
diff changeset
  2009
        if (ii->in(1)->bottom_type() != Type::TOP) {
0a2825307027 6851282: JIT miscompilation results in null entry in array when using CompressedOops
kvn
parents: 2154
diff changeset
  2010
          has_decodeN = true;
13969
d2a189b83b87 7054512: Compress class pointers after perm gen removal
roland
parents: 13895
diff changeset
  2011
          is_decodeN = ii->is_DecodeN();
3270
0a2825307027 6851282: JIT miscompilation results in null entry in array when using CompressedOops
kvn
parents: 2154
diff changeset
  2012
        }
1400
afd034bb8c2e 6747051: Improve code and implicit null check generation for compressed oops
kvn
parents: 1399
diff changeset
  2013
      } else if (!ii->is_Phi()) {
afd034bb8c2e 6747051: Improve code and implicit null check generation for compressed oops
kvn
parents: 1399
diff changeset
  2014
        may_push = false;
afd034bb8c2e 6747051: Improve code and implicit null check generation for compressed oops
kvn
parents: 1399
diff changeset
  2015
      }
afd034bb8c2e 6747051: Improve code and implicit null check generation for compressed oops
kvn
parents: 1399
diff changeset
  2016
    }
afd034bb8c2e 6747051: Improve code and implicit null check generation for compressed oops
kvn
parents: 1399
diff changeset
  2017
afd034bb8c2e 6747051: Improve code and implicit null check generation for compressed oops
kvn
parents: 1399
diff changeset
  2018
    if (has_decodeN && may_push) {
afd034bb8c2e 6747051: Improve code and implicit null check generation for compressed oops
kvn
parents: 1399
diff changeset
  2019
      PhaseIterGVN *igvn = phase->is_IterGVN();
3599
35bb709f2c62 6865031: Application gives bad result (throws bad exception) with compressed oops
kvn
parents: 3277
diff changeset
  2020
      // Make narrow type for new phi.
13969
d2a189b83b87 7054512: Compress class pointers after perm gen removal
roland
parents: 13895
diff changeset
  2021
      const Type* narrow_t;
d2a189b83b87 7054512: Compress class pointers after perm gen removal
roland
parents: 13895
diff changeset
  2022
      if (is_decodeN) {
d2a189b83b87 7054512: Compress class pointers after perm gen removal
roland
parents: 13895
diff changeset
  2023
        narrow_t = TypeNarrowOop::make(this->bottom_type()->is_ptr());
d2a189b83b87 7054512: Compress class pointers after perm gen removal
roland
parents: 13895
diff changeset
  2024
      } else {
d2a189b83b87 7054512: Compress class pointers after perm gen removal
roland
parents: 13895
diff changeset
  2025
        narrow_t = TypeNarrowKlass::make(this->bottom_type()->is_ptr());
d2a189b83b87 7054512: Compress class pointers after perm gen removal
roland
parents: 13895
diff changeset
  2026
      }
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 23528
diff changeset
  2027
      PhiNode* new_phi = new PhiNode(r, narrow_t);
1400
afd034bb8c2e 6747051: Improve code and implicit null check generation for compressed oops
kvn
parents: 1399
diff changeset
  2028
      uint orig_cnt = req();
afd034bb8c2e 6747051: Improve code and implicit null check generation for compressed oops
kvn
parents: 1399
diff changeset
  2029
      for (uint i=1; i<req(); ++i) {// For all paths in
afd034bb8c2e 6747051: Improve code and implicit null check generation for compressed oops
kvn
parents: 1399
diff changeset
  2030
        Node *ii = in(i);
afd034bb8c2e 6747051: Improve code and implicit null check generation for compressed oops
kvn
parents: 1399
diff changeset
  2031
        Node* new_ii = NULL;
13969
d2a189b83b87 7054512: Compress class pointers after perm gen removal
roland
parents: 13895
diff changeset
  2032
        if (ii->is_DecodeNarrowPtr()) {
1400
afd034bb8c2e 6747051: Improve code and implicit null check generation for compressed oops
kvn
parents: 1399
diff changeset
  2033
          assert(ii->bottom_type() == bottom_type(), "sanity");
afd034bb8c2e 6747051: Improve code and implicit null check generation for compressed oops
kvn
parents: 1399
diff changeset
  2034
          new_ii = ii->in(1);
afd034bb8c2e 6747051: Improve code and implicit null check generation for compressed oops
kvn
parents: 1399
diff changeset
  2035
        } else {
afd034bb8c2e 6747051: Improve code and implicit null check generation for compressed oops
kvn
parents: 1399
diff changeset
  2036
          assert(ii->is_Phi(), "sanity");
afd034bb8c2e 6747051: Improve code and implicit null check generation for compressed oops
kvn
parents: 1399
diff changeset
  2037
          if (ii->as_Phi() == this) {
afd034bb8c2e 6747051: Improve code and implicit null check generation for compressed oops
kvn
parents: 1399
diff changeset
  2038
            new_ii = new_phi;
afd034bb8c2e 6747051: Improve code and implicit null check generation for compressed oops
kvn
parents: 1399
diff changeset
  2039
          } else {
13969
d2a189b83b87 7054512: Compress class pointers after perm gen removal
roland
parents: 13895
diff changeset
  2040
            if (is_decodeN) {
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 23528
diff changeset
  2041
              new_ii = new EncodePNode(ii, narrow_t);
13969
d2a189b83b87 7054512: Compress class pointers after perm gen removal
roland
parents: 13895
diff changeset
  2042
            } else {
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 23528
diff changeset
  2043
              new_ii = new EncodePKlassNode(ii, narrow_t);
13969
d2a189b83b87 7054512: Compress class pointers after perm gen removal
roland
parents: 13895
diff changeset
  2044
            }
1400
afd034bb8c2e 6747051: Improve code and implicit null check generation for compressed oops
kvn
parents: 1399
diff changeset
  2045
            igvn->register_new_node_with_optimizer(new_ii);
afd034bb8c2e 6747051: Improve code and implicit null check generation for compressed oops
kvn
parents: 1399
diff changeset
  2046
          }
afd034bb8c2e 6747051: Improve code and implicit null check generation for compressed oops
kvn
parents: 1399
diff changeset
  2047
        }
afd034bb8c2e 6747051: Improve code and implicit null check generation for compressed oops
kvn
parents: 1399
diff changeset
  2048
        new_phi->set_req(i, new_ii);
afd034bb8c2e 6747051: Improve code and implicit null check generation for compressed oops
kvn
parents: 1399
diff changeset
  2049
      }
afd034bb8c2e 6747051: Improve code and implicit null check generation for compressed oops
kvn
parents: 1399
diff changeset
  2050
      igvn->register_new_node_with_optimizer(new_phi, this);
13969
d2a189b83b87 7054512: Compress class pointers after perm gen removal
roland
parents: 13895
diff changeset
  2051
      if (is_decodeN) {
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 23528
diff changeset
  2052
        progress = new DecodeNNode(new_phi, bottom_type());
13969
d2a189b83b87 7054512: Compress class pointers after perm gen removal
roland
parents: 13895
diff changeset
  2053
      } else {
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 23528
diff changeset
  2054
        progress = new DecodeNKlassNode(new_phi, bottom_type());
13969
d2a189b83b87 7054512: Compress class pointers after perm gen removal
roland
parents: 13895
diff changeset
  2055
      }
1400
afd034bb8c2e 6747051: Improve code and implicit null check generation for compressed oops
kvn
parents: 1399
diff changeset
  2056
    }
afd034bb8c2e 6747051: Improve code and implicit null check generation for compressed oops
kvn
parents: 1399
diff changeset
  2057
  }
afd034bb8c2e 6747051: Improve code and implicit null check generation for compressed oops
kvn
parents: 1399
diff changeset
  2058
#endif
afd034bb8c2e 6747051: Improve code and implicit null check generation for compressed oops
kvn
parents: 1399
diff changeset
  2059
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2060
  return progress;              // Return any progress
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2061
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2062
1432
44f076e3d2a4 6667595: Set probability FAIR for pre-, post- loops and ALWAYS for main loop
kvn
parents: 1400
diff changeset
  2063
//------------------------------is_tripcount-----------------------------------
44f076e3d2a4 6667595: Set probability FAIR for pre-, post- loops and ALWAYS for main loop
kvn
parents: 1400
diff changeset
  2064
bool PhiNode::is_tripcount() const {
44f076e3d2a4 6667595: Set probability FAIR for pre-, post- loops and ALWAYS for main loop
kvn
parents: 1400
diff changeset
  2065
  return (in(0) != NULL && in(0)->is_CountedLoop() &&
44f076e3d2a4 6667595: Set probability FAIR for pre-, post- loops and ALWAYS for main loop
kvn
parents: 1400
diff changeset
  2066
          in(0)->as_CountedLoop()->phi() == this);
44f076e3d2a4 6667595: Set probability FAIR for pre-, post- loops and ALWAYS for main loop
kvn
parents: 1400
diff changeset
  2067
}
44f076e3d2a4 6667595: Set probability FAIR for pre-, post- loops and ALWAYS for main loop
kvn
parents: 1400
diff changeset
  2068
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2069
//------------------------------out_RegMask------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2070
const RegMask &PhiNode::in_RegMask(uint i) const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2071
  return i ? out_RegMask() : RegMask::Empty;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2072
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2073
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2074
const RegMask &PhiNode::out_RegMask() const {
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 11446
diff changeset
  2075
  uint ideal_reg = _type->ideal_reg();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2076
  assert( ideal_reg != Node::NotAMachineReg, "invalid type at Phi" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2077
  if( ideal_reg == 0 ) return RegMask::Empty;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2078
  return *(Compile::current()->matcher()->idealreg2spillmask[ideal_reg]);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2079
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2080
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2081
#ifndef PRODUCT
32084
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2082
void PhiNode::related(GrowableArray<Node*> *in_rel, GrowableArray<Node*> *out_rel, bool compact) const {
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2083
  // For a PhiNode, the set of related nodes includes all inputs till level 2,
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2084
  // and all outputs till level 1. In compact mode, inputs till level 1 are
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2085
  // collected.
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2086
  this->collect_nodes(in_rel, compact ? 1 : 2, false, false);
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2087
  this->collect_nodes(out_rel, -1, false, false);
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2088
}
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2089
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2090
void PhiNode::dump_spec(outputStream *st) const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2091
  TypeNode::dump_spec(st);
1432
44f076e3d2a4 6667595: Set probability FAIR for pre-, post- loops and ALWAYS for main loop
kvn
parents: 1400
diff changeset
  2092
  if (is_tripcount()) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2093
    st->print(" #tripcount");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2094
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2095
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2096
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2097
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2098
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2099
//=============================================================================
35551
36ef3841fb34 8146629: Make phase->is_IterGVN() accessible from Node::Identity and Node::Value
thartmann
parents: 35549
diff changeset
  2100
const Type* GotoNode::Value(PhaseGVN* phase) const {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2101
  // If the input is reachable, then we are executed.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2102
  // If the input is not reachable, then we are not executed.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2103
  return phase->type(in(0));
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2104
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2105
35551
36ef3841fb34 8146629: Make phase->is_IterGVN() accessible from Node::Identity and Node::Value
thartmann
parents: 35549
diff changeset
  2106
Node* GotoNode::Identity(PhaseGVN* phase) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2107
  return in(0);                // Simple copy of incoming control
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2108
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2109
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2110
const RegMask &GotoNode::out_RegMask() const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2111
  return RegMask::Empty;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2112
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2113
32084
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2114
#ifndef PRODUCT
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2115
//-----------------------------related-----------------------------------------
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2116
// The related nodes of a GotoNode are all inputs at level 1, as well as the
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2117
// outputs at level 1. This is regardless of compact mode.
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2118
void GotoNode::related(GrowableArray<Node*> *in_rel, GrowableArray<Node*> *out_rel, bool compact) const {
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2119
  this->collect_nodes(in_rel, 1, false, false);
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2120
  this->collect_nodes(out_rel, -1, false, false);
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2121
}
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2122
#endif
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2123
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2124
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2125
//=============================================================================
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2126
const RegMask &JumpNode::out_RegMask() const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2127
  return RegMask::Empty;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2128
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2129
32084
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2130
#ifndef PRODUCT
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2131
//-----------------------------related-----------------------------------------
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2132
// The related nodes of a JumpNode are all inputs at level 1, as well as the
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2133
// outputs at level 2 (to include actual jump targets beyond projection nodes).
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2134
// This is regardless of compact mode.
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2135
void JumpNode::related(GrowableArray<Node*> *in_rel, GrowableArray<Node*> *out_rel, bool compact) const {
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2136
  this->collect_nodes(in_rel, 1, false, false);
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2137
  this->collect_nodes(out_rel, -2, false, false);
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2138
}
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2139
#endif
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2140
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2141
//=============================================================================
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2142
const RegMask &JProjNode::out_RegMask() const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2143
  return RegMask::Empty;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2144
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2145
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2146
//=============================================================================
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2147
const RegMask &CProjNode::out_RegMask() const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2148
  return RegMask::Empty;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2149
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2150
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2151
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2152
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2153
//=============================================================================
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2154
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2155
uint PCTableNode::hash() const { return Node::hash() + _size; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2156
uint PCTableNode::cmp( const Node &n ) const
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2157
{ return _size == ((PCTableNode&)n)._size; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2158
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2159
const Type *PCTableNode::bottom_type() const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2160
  const Type** f = TypeTuple::fields(_size);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2161
  for( uint i = 0; i < _size; i++ ) f[i] = Type::CONTROL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2162
  return TypeTuple::make(_size, f);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2163
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2164
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2165
//------------------------------Value------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2166
// Compute the type of the PCTableNode.  If reachable it is a tuple of
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2167
// Control, otherwise the table targets are not reachable
35551
36ef3841fb34 8146629: Make phase->is_IterGVN() accessible from Node::Identity and Node::Value
thartmann
parents: 35549
diff changeset
  2168
const Type* PCTableNode::Value(PhaseGVN* phase) const {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2169
  if( phase->type(in(0)) == Type::CONTROL )
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2170
    return bottom_type();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2171
  return Type::TOP;             // All paths dead?  Then so are we
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2172
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2173
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2174
//------------------------------Ideal------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2175
// Return a node which is more "ideal" than the current node.  Strip out
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2176
// control copies
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2177
Node *PCTableNode::Ideal(PhaseGVN *phase, bool can_reshape) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2178
  return remove_dead_region(phase, can_reshape) ? this : NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2179
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2180
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2181
//=============================================================================
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2182
uint JumpProjNode::hash() const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2183
  return Node::hash() + _dest_bci;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2184
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2185
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2186
uint JumpProjNode::cmp( const Node &n ) const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2187
  return ProjNode::cmp(n) &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2188
    _dest_bci == ((JumpProjNode&)n)._dest_bci;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2189
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2190
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2191
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2192
void JumpProjNode::dump_spec(outputStream *st) const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2193
  ProjNode::dump_spec(st);
32084
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2194
  st->print("@bci %d ",_dest_bci);
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2195
}
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2196
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2197
void JumpProjNode::dump_compact_spec(outputStream *st) const {
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2198
  ProjNode::dump_compact_spec(st);
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2199
  st->print("(%d)%d@%d", _switch_val, _proj_no, _dest_bci);
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2200
}
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2201
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2202
void JumpProjNode::related(GrowableArray<Node*> *in_rel, GrowableArray<Node*> *out_rel, bool compact) const {
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2203
  // The related nodes of a JumpProjNode are its inputs and outputs at level 1.
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2204
  this->collect_nodes(in_rel, 1, false, false);
7743e6943cdf 8004073: Implement C2 Ideal node specific dump() method
mhaupt
parents: 30206
diff changeset
  2205
  this->collect_nodes(out_rel, -1, false, false);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2206
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2207
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2208
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2209
//=============================================================================
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2210
//------------------------------Value------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2211
// Check for being unreachable, or for coming from a Rethrow.  Rethrow's cannot
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2212
// have the default "fall_through_index" path.
35551
36ef3841fb34 8146629: Make phase->is_IterGVN() accessible from Node::Identity and Node::Value
thartmann
parents: 35549
diff changeset
  2213
const Type* CatchNode::Value(PhaseGVN* phase) const {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2214
  // Unreachable?  Then so are all paths from here.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2215
  if( phase->type(in(0)) == Type::TOP ) return Type::TOP;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2216
  // First assume all paths are reachable
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2217
  const Type** f = TypeTuple::fields(_size);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2218
  for( uint i = 0; i < _size; i++ ) f[i] = Type::CONTROL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2219
  // Identify cases that will always throw an exception
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2220
  // () rethrow call
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2221
  // () virtual or interface call with NULL receiver
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2222
  // () call is a check cast with incompatible arguments
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2223
  if( in(1)->is_Proj() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2224
    Node *i10 = in(1)->in(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2225
    if( i10->is_Call() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2226
      CallNode *call = i10->as_Call();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2227
      // Rethrows always throw exceptions, never return
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2228
      if (call->entry_point() == OptoRuntime::rethrow_stub()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2229
        f[CatchProjNode::fall_through_index] = Type::TOP;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2230
      } else if( call->req() > TypeFunc::Parms ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2231
        const Type *arg0 = phase->type( call->in(TypeFunc::Parms) );
2131
98f9cef66a34 6810672: Comment typos
twisti
parents: 2019
diff changeset
  2232
        // Check for null receiver to virtual or interface calls
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2233
        if( call->is_CallDynamicJava() &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2234
            arg0->higher_equal(TypePtr::NULL_PTR) ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2235
          f[CatchProjNode::fall_through_index] = Type::TOP;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2236
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2237
      } // End of if not a runtime stub
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2238
    } // End of if have call above me
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2239
  } // End of slot 1 is not a projection
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2240
  return TypeTuple::make(_size, f);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2241
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2242
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2243
//=============================================================================
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2244
uint CatchProjNode::hash() const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2245
  return Node::hash() + _handler_bci;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2246
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2247
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2248
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2249
uint CatchProjNode::cmp( const Node &n ) const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2250
  return ProjNode::cmp(n) &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2251
    _handler_bci == ((CatchProjNode&)n)._handler_bci;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2252
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2253
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2254
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2255
//------------------------------Identity---------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2256
// If only 1 target is possible, choose it if it is the main control
35551
36ef3841fb34 8146629: Make phase->is_IterGVN() accessible from Node::Identity and Node::Value
thartmann
parents: 35549
diff changeset
  2257
Node* CatchProjNode::Identity(PhaseGVN* phase) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2258
  // If my value is control and no other value is, then treat as ID
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2259
  const TypeTuple *t = phase->type(in(0))->is_tuple();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2260
  if (t->field_at(_con) != Type::CONTROL)  return this;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2261
  // If we remove the last CatchProj and elide the Catch/CatchProj, then we
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2262
  // also remove any exception table entry.  Thus we must know the call
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2263
  // feeding the Catch will not really throw an exception.  This is ok for
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2264
  // the main fall-thru control (happens when we know a call can never throw
2131
98f9cef66a34 6810672: Comment typos
twisti
parents: 2019
diff changeset
  2265
  // an exception) or for "rethrow", because a further optimization will
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2266
  // yank the rethrow (happens when we inline a function that can throw an
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2267
  // exception and the caller has no handler).  Not legal, e.g., for passing
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2268
  // a NULL receiver to a v-call, or passing bad types to a slow-check-cast.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2269
  // These cases MUST throw an exception via the runtime system, so the VM
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2270
  // will be looking for a table entry.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2271
  Node *proj = in(0)->in(1);    // Expect a proj feeding CatchNode
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2272
  CallNode *call;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2273
  if (_con != TypeFunc::Control && // Bail out if not the main control.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2274
      !(proj->is_Proj() &&      // AND NOT a rethrow
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2275
        proj->in(0)->is_Call() &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2276
        (call = proj->in(0)->as_Call()) &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2277
        call->entry_point() == OptoRuntime::rethrow_stub()))
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2278
    return this;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2279
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2280
  // Search for any other path being control
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2281
  for (uint i = 0; i < t->cnt(); i++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2282
    if (i != _con && t->field_at(i) == Type::CONTROL)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2283
      return this;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2284
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2285
  // Only my path is possible; I am identity on control to the jump
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2286
  return in(0)->in(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2287
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2288
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2289
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2290
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2291
void CatchProjNode::dump_spec(outputStream *st) const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2292
  ProjNode::dump_spec(st);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2293
  st->print("@bci %d ",_handler_bci);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2294
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2295
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2296
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2297
//=============================================================================
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2298
//------------------------------Identity---------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2299
// Check for CreateEx being Identity.
35551
36ef3841fb34 8146629: Make phase->is_IterGVN() accessible from Node::Identity and Node::Value
thartmann
parents: 35549
diff changeset
  2300
Node* CreateExNode::Identity(PhaseGVN* phase) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2301
  if( phase->type(in(1)) == Type::TOP ) return in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2302
  if( phase->type(in(0)) == Type::TOP ) return in(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2303
  // We only come from CatchProj, unless the CatchProj goes away.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2304
  // If the CatchProj is optimized away, then we just carry the
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2305
  // exception oop through.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2306
  CallNode *call = in(1)->in(0)->as_Call();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2307
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2308
  return ( in(0)->is_CatchProj() && in(0)->in(0)->in(1) == in(1) )
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2309
    ? this
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2310
    : call->in(TypeFunc::Parms);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2311
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2312
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2313
//=============================================================================
374
585c671c9c0e 6663848: assert(i < Max(),"oob") in C2 with -Xcomp
never
parents: 373
diff changeset
  2314
//------------------------------Value------------------------------------------
585c671c9c0e 6663848: assert(i < Max(),"oob") in C2 with -Xcomp
never
parents: 373
diff changeset
  2315
// Check for being unreachable.
35551
36ef3841fb34 8146629: Make phase->is_IterGVN() accessible from Node::Identity and Node::Value
thartmann
parents: 35549
diff changeset
  2316
const Type* NeverBranchNode::Value(PhaseGVN* phase) const {
374
585c671c9c0e 6663848: assert(i < Max(),"oob") in C2 with -Xcomp
never
parents: 373
diff changeset
  2317
  if (!in(0) || in(0)->is_top()) return Type::TOP;
585c671c9c0e 6663848: assert(i < Max(),"oob") in C2 with -Xcomp
never
parents: 373
diff changeset
  2318
  return bottom_type();
585c671c9c0e 6663848: assert(i < Max(),"oob") in C2 with -Xcomp
never
parents: 373
diff changeset
  2319
}
585c671c9c0e 6663848: assert(i < Max(),"oob") in C2 with -Xcomp
never
parents: 373
diff changeset
  2320
585c671c9c0e 6663848: assert(i < Max(),"oob") in C2 with -Xcomp
never
parents: 373
diff changeset
  2321
//------------------------------Ideal------------------------------------------
585c671c9c0e 6663848: assert(i < Max(),"oob") in C2 with -Xcomp
never
parents: 373
diff changeset
  2322
// Check for no longer being part of a loop
585c671c9c0e 6663848: assert(i < Max(),"oob") in C2 with -Xcomp
never
parents: 373
diff changeset
  2323
Node *NeverBranchNode::Ideal(PhaseGVN *phase, bool can_reshape) {
585c671c9c0e 6663848: assert(i < Max(),"oob") in C2 with -Xcomp
never
parents: 373
diff changeset
  2324
  if (can_reshape && !in(0)->is_Loop()) {
585c671c9c0e 6663848: assert(i < Max(),"oob") in C2 with -Xcomp
never
parents: 373
diff changeset
  2325
    // Dead code elimination can sometimes delete this projection so
585c671c9c0e 6663848: assert(i < Max(),"oob") in C2 with -Xcomp
never
parents: 373
diff changeset
  2326
    // if it's not there, there's nothing to do.
585c671c9c0e 6663848: assert(i < Max(),"oob") in C2 with -Xcomp
never
parents: 373
diff changeset
  2327
    Node* fallthru = proj_out(0);
585c671c9c0e 6663848: assert(i < Max(),"oob") in C2 with -Xcomp
never
parents: 373
diff changeset
  2328
    if (fallthru != NULL) {
5901
c046f8e9c52b 6677629: PhaseIterGVN::subsume_node() should call hash_delete() and add_users_to_worklist()
kvn
parents: 5547
diff changeset
  2329
      phase->is_IterGVN()->replace_node(fallthru, in(0));
374
585c671c9c0e 6663848: assert(i < Max(),"oob") in C2 with -Xcomp
never
parents: 373
diff changeset
  2330
    }
585c671c9c0e 6663848: assert(i < Max(),"oob") in C2 with -Xcomp
never
parents: 373
diff changeset
  2331
    return phase->C->top();
585c671c9c0e 6663848: assert(i < Max(),"oob") in C2 with -Xcomp
never
parents: 373
diff changeset
  2332
  }
585c671c9c0e 6663848: assert(i < Max(),"oob") in C2 with -Xcomp
never
parents: 373
diff changeset
  2333
  return NULL;
585c671c9c0e 6663848: assert(i < Max(),"oob") in C2 with -Xcomp
never
parents: 373
diff changeset
  2334
}
585c671c9c0e 6663848: assert(i < Max(),"oob") in C2 with -Xcomp
never
parents: 373
diff changeset
  2335
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2336
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2337
void NeverBranchNode::format( PhaseRegAlloc *ra_, outputStream *st) const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2338
  st->print("%s", Name());
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2339
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  2340
#endif