hotspot/src/share/vm/opto/phaseX.cpp
author roland
Thu, 27 Nov 2014 16:54:49 +0100
changeset 27910 8653c71aea40
parent 27708 8a8710cb8fc4
child 30300 4b12a5b40064
permissions -rw-r--r--
8066045: opto/node.hpp:355, assert(i < _max) failed: oob: i=1, _max=1 Summary: code in PhaseIterGVN::add_users_to_worklist() from 8054478 makes incorrect assumption about graph shape Reviewed-by: iveresov
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     1
/*
24424
2658d7834c6e 8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents: 24016
diff changeset
     2
 * Copyright (c) 1997, 2014, 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: 4020
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 4020
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: 4020
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: 6752
diff changeset
    25
#include "precompiled.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6752
diff changeset
    26
#include "memory/allocation.inline.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6752
diff changeset
    27
#include "opto/block.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6752
diff changeset
    28
#include "opto/callnode.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6752
diff changeset
    29
#include "opto/cfgnode.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6752
diff changeset
    30
#include "opto/idealGraphPrinter.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6752
diff changeset
    31
#include "opto/loopnode.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6752
diff changeset
    32
#include "opto/machnode.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6752
diff changeset
    33
#include "opto/opcodes.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6752
diff changeset
    34
#include "opto/phaseX.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6752
diff changeset
    35
#include "opto/regalloc.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6752
diff changeset
    36
#include "opto/rootnode.hpp"
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    37
489c9b5090e2 Initial load
duke
parents:
diff changeset
    38
//=============================================================================
489c9b5090e2 Initial load
duke
parents:
diff changeset
    39
#define NODE_HASH_MINIMUM_SIZE    255
489c9b5090e2 Initial load
duke
parents:
diff changeset
    40
//------------------------------NodeHash---------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
    41
NodeHash::NodeHash(uint est_max_size) :
489c9b5090e2 Initial load
duke
parents:
diff changeset
    42
  _max( round_up(est_max_size < NODE_HASH_MINIMUM_SIZE ? NODE_HASH_MINIMUM_SIZE : est_max_size) ),
489c9b5090e2 Initial load
duke
parents:
diff changeset
    43
  _a(Thread::current()->resource_area()),
489c9b5090e2 Initial load
duke
parents:
diff changeset
    44
  _table( NEW_ARENA_ARRAY( _a , Node* , _max ) ), // (Node**)_a->Amalloc(_max * sizeof(Node*)) ),
26429
c5ca44ee3ce2 8056067: NodeHash debug variables are available in product build
thartmann
parents: 25930
diff changeset
    45
  _inserts(0), _insert_limit( insert_limit() )
c5ca44ee3ce2 8056067: NodeHash debug variables are available in product build
thartmann
parents: 25930
diff changeset
    46
#ifndef PRODUCT
c5ca44ee3ce2 8056067: NodeHash debug variables are available in product build
thartmann
parents: 25930
diff changeset
    47
  ,_look_probes(0), _lookup_hits(0), _lookup_misses(0),
c5ca44ee3ce2 8056067: NodeHash debug variables are available in product build
thartmann
parents: 25930
diff changeset
    48
  _delete_probes(0), _delete_hits(0), _delete_misses(0),
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    49
  _total_insert_probes(0), _total_inserts(0),
26429
c5ca44ee3ce2 8056067: NodeHash debug variables are available in product build
thartmann
parents: 25930
diff changeset
    50
  _insert_probes(0), _grows(0)
c5ca44ee3ce2 8056067: NodeHash debug variables are available in product build
thartmann
parents: 25930
diff changeset
    51
#endif
c5ca44ee3ce2 8056067: NodeHash debug variables are available in product build
thartmann
parents: 25930
diff changeset
    52
{
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    53
  // _sentinel must be in the current node space
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
    54
  _sentinel = new ProjNode(NULL, TypeFunc::Control);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    55
  memset(_table,0,sizeof(Node*)*_max);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    56
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    57
489c9b5090e2 Initial load
duke
parents:
diff changeset
    58
//------------------------------NodeHash---------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
    59
NodeHash::NodeHash(Arena *arena, uint est_max_size) :
489c9b5090e2 Initial load
duke
parents:
diff changeset
    60
  _max( round_up(est_max_size < NODE_HASH_MINIMUM_SIZE ? NODE_HASH_MINIMUM_SIZE : est_max_size) ),
489c9b5090e2 Initial load
duke
parents:
diff changeset
    61
  _a(arena),
489c9b5090e2 Initial load
duke
parents:
diff changeset
    62
  _table( NEW_ARENA_ARRAY( _a , Node* , _max ) ),
26429
c5ca44ee3ce2 8056067: NodeHash debug variables are available in product build
thartmann
parents: 25930
diff changeset
    63
  _inserts(0), _insert_limit( insert_limit() )
c5ca44ee3ce2 8056067: NodeHash debug variables are available in product build
thartmann
parents: 25930
diff changeset
    64
#ifndef PRODUCT
c5ca44ee3ce2 8056067: NodeHash debug variables are available in product build
thartmann
parents: 25930
diff changeset
    65
  ,_look_probes(0), _lookup_hits(0), _lookup_misses(0),
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    66
  _delete_probes(0), _delete_hits(0), _delete_misses(0),
489c9b5090e2 Initial load
duke
parents:
diff changeset
    67
  _total_insert_probes(0), _total_inserts(0),
26429
c5ca44ee3ce2 8056067: NodeHash debug variables are available in product build
thartmann
parents: 25930
diff changeset
    68
  _insert_probes(0), _grows(0)
c5ca44ee3ce2 8056067: NodeHash debug variables are available in product build
thartmann
parents: 25930
diff changeset
    69
#endif
c5ca44ee3ce2 8056067: NodeHash debug variables are available in product build
thartmann
parents: 25930
diff changeset
    70
{
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    71
  // _sentinel must be in the current node space
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
    72
  _sentinel = new ProjNode(NULL, TypeFunc::Control);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    73
  memset(_table,0,sizeof(Node*)*_max);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    74
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    75
489c9b5090e2 Initial load
duke
parents:
diff changeset
    76
//------------------------------NodeHash---------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
    77
NodeHash::NodeHash(NodeHash *nh) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    78
  debug_only(_table = (Node**)badAddress);   // interact correctly w/ operator=
489c9b5090e2 Initial load
duke
parents:
diff changeset
    79
  // just copy in all the fields
489c9b5090e2 Initial load
duke
parents:
diff changeset
    80
  *this = *nh;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    81
  // nh->_sentinel must be in the current node space
489c9b5090e2 Initial load
duke
parents:
diff changeset
    82
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    83
15113
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14623
diff changeset
    84
void NodeHash::replace_with(NodeHash *nh) {
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14623
diff changeset
    85
  debug_only(_table = (Node**)badAddress);   // interact correctly w/ operator=
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14623
diff changeset
    86
  // just copy in all the fields
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14623
diff changeset
    87
  *this = *nh;
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14623
diff changeset
    88
  // nh->_sentinel must be in the current node space
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14623
diff changeset
    89
}
823590505eb4 8005071: Incremental inlining for JSR 292
roland
parents: 14623
diff changeset
    90
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    91
//------------------------------hash_find--------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
    92
// Find in hash table
489c9b5090e2 Initial load
duke
parents:
diff changeset
    93
Node *NodeHash::hash_find( const Node *n ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    94
  // ((Node*)n)->set_hash( n->hash() );
489c9b5090e2 Initial load
duke
parents:
diff changeset
    95
  uint hash = n->hash();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    96
  if (hash == Node::NO_HASH) {
26429
c5ca44ee3ce2 8056067: NodeHash debug variables are available in product build
thartmann
parents: 25930
diff changeset
    97
    NOT_PRODUCT( _lookup_misses++ );
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    98
    return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    99
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   100
  uint key = hash & (_max-1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   101
  uint stride = key | 0x01;
26429
c5ca44ee3ce2 8056067: NodeHash debug variables are available in product build
thartmann
parents: 25930
diff changeset
   102
  NOT_PRODUCT( _look_probes++ );
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   103
  Node *k = _table[key];        // Get hashed value
489c9b5090e2 Initial load
duke
parents:
diff changeset
   104
  if( !k ) {                    // ?Miss?
26429
c5ca44ee3ce2 8056067: NodeHash debug variables are available in product build
thartmann
parents: 25930
diff changeset
   105
    NOT_PRODUCT( _lookup_misses++ );
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   106
    return NULL;                // Miss!
489c9b5090e2 Initial load
duke
parents:
diff changeset
   107
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   108
489c9b5090e2 Initial load
duke
parents:
diff changeset
   109
  int op = n->Opcode();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   110
  uint req = n->req();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   111
  while( 1 ) {                  // While probing hash table
489c9b5090e2 Initial load
duke
parents:
diff changeset
   112
    if( k->req() == req &&      // Same count of inputs
489c9b5090e2 Initial load
duke
parents:
diff changeset
   113
        k->Opcode() == op ) {   // Same Opcode
489c9b5090e2 Initial load
duke
parents:
diff changeset
   114
      for( uint i=0; i<req; i++ )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   115
        if( n->in(i)!=k->in(i)) // Different inputs?
489c9b5090e2 Initial load
duke
parents:
diff changeset
   116
          goto collision;       // "goto" is a speed hack...
489c9b5090e2 Initial load
duke
parents:
diff changeset
   117
      if( n->cmp(*k) ) {        // Check for any special bits
26429
c5ca44ee3ce2 8056067: NodeHash debug variables are available in product build
thartmann
parents: 25930
diff changeset
   118
        NOT_PRODUCT( _lookup_hits++ );
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   119
        return k;               // Hit!
489c9b5090e2 Initial load
duke
parents:
diff changeset
   120
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   121
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   122
  collision:
26429
c5ca44ee3ce2 8056067: NodeHash debug variables are available in product build
thartmann
parents: 25930
diff changeset
   123
    NOT_PRODUCT( _look_probes++ );
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   124
    key = (key + stride/*7*/) & (_max-1); // Stride through table with relative prime
489c9b5090e2 Initial load
duke
parents:
diff changeset
   125
    k = _table[key];            // Get hashed value
489c9b5090e2 Initial load
duke
parents:
diff changeset
   126
    if( !k ) {                  // ?Miss?
26429
c5ca44ee3ce2 8056067: NodeHash debug variables are available in product build
thartmann
parents: 25930
diff changeset
   127
      NOT_PRODUCT( _lookup_misses++ );
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   128
      return NULL;              // Miss!
489c9b5090e2 Initial load
duke
parents:
diff changeset
   129
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   130
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   131
  ShouldNotReachHere();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   132
  return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   133
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   134
489c9b5090e2 Initial load
duke
parents:
diff changeset
   135
//------------------------------hash_find_insert-------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   136
// Find in hash table, insert if not already present
489c9b5090e2 Initial load
duke
parents:
diff changeset
   137
// Used to preserve unique entries in hash table
489c9b5090e2 Initial load
duke
parents:
diff changeset
   138
Node *NodeHash::hash_find_insert( Node *n ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   139
  // n->set_hash( );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   140
  uint hash = n->hash();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   141
  if (hash == Node::NO_HASH) {
26429
c5ca44ee3ce2 8056067: NodeHash debug variables are available in product build
thartmann
parents: 25930
diff changeset
   142
    NOT_PRODUCT( _lookup_misses++ );
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   143
    return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   144
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   145
  uint key = hash & (_max-1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   146
  uint stride = key | 0x01;     // stride must be relatively prime to table siz
489c9b5090e2 Initial load
duke
parents:
diff changeset
   147
  uint first_sentinel = 0;      // replace a sentinel if seen.
26429
c5ca44ee3ce2 8056067: NodeHash debug variables are available in product build
thartmann
parents: 25930
diff changeset
   148
  NOT_PRODUCT( _look_probes++ );
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   149
  Node *k = _table[key];        // Get hashed value
489c9b5090e2 Initial load
duke
parents:
diff changeset
   150
  if( !k ) {                    // ?Miss?
26429
c5ca44ee3ce2 8056067: NodeHash debug variables are available in product build
thartmann
parents: 25930
diff changeset
   151
    NOT_PRODUCT( _lookup_misses++ );
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   152
    _table[key] = n;            // Insert into table!
489c9b5090e2 Initial load
duke
parents:
diff changeset
   153
    debug_only(n->enter_hash_lock()); // Lock down the node while in the table.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   154
    check_grow();               // Grow table if insert hit limit
489c9b5090e2 Initial load
duke
parents:
diff changeset
   155
    return NULL;                // Miss!
489c9b5090e2 Initial load
duke
parents:
diff changeset
   156
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   157
  else if( k == _sentinel ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   158
    first_sentinel = key;      // Can insert here
489c9b5090e2 Initial load
duke
parents:
diff changeset
   159
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   160
489c9b5090e2 Initial load
duke
parents:
diff changeset
   161
  int op = n->Opcode();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   162
  uint req = n->req();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   163
  while( 1 ) {                  // While probing hash table
489c9b5090e2 Initial load
duke
parents:
diff changeset
   164
    if( k->req() == req &&      // Same count of inputs
489c9b5090e2 Initial load
duke
parents:
diff changeset
   165
        k->Opcode() == op ) {   // Same Opcode
489c9b5090e2 Initial load
duke
parents:
diff changeset
   166
      for( uint i=0; i<req; i++ )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   167
        if( n->in(i)!=k->in(i)) // Different inputs?
489c9b5090e2 Initial load
duke
parents:
diff changeset
   168
          goto collision;       // "goto" is a speed hack...
489c9b5090e2 Initial load
duke
parents:
diff changeset
   169
      if( n->cmp(*k) ) {        // Check for any special bits
26429
c5ca44ee3ce2 8056067: NodeHash debug variables are available in product build
thartmann
parents: 25930
diff changeset
   170
        NOT_PRODUCT( _lookup_hits++ );
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   171
        return k;               // Hit!
489c9b5090e2 Initial load
duke
parents:
diff changeset
   172
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   173
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   174
  collision:
26429
c5ca44ee3ce2 8056067: NodeHash debug variables are available in product build
thartmann
parents: 25930
diff changeset
   175
    NOT_PRODUCT( _look_probes++ );
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   176
    key = (key + stride) & (_max-1); // Stride through table w/ relative prime
489c9b5090e2 Initial load
duke
parents:
diff changeset
   177
    k = _table[key];            // Get hashed value
489c9b5090e2 Initial load
duke
parents:
diff changeset
   178
    if( !k ) {                  // ?Miss?
26429
c5ca44ee3ce2 8056067: NodeHash debug variables are available in product build
thartmann
parents: 25930
diff changeset
   179
      NOT_PRODUCT( _lookup_misses++ );
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   180
      key = (first_sentinel == 0) ? key : first_sentinel; // ?saw sentinel?
489c9b5090e2 Initial load
duke
parents:
diff changeset
   181
      _table[key] = n;          // Insert into table!
489c9b5090e2 Initial load
duke
parents:
diff changeset
   182
      debug_only(n->enter_hash_lock()); // Lock down the node while in the table.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   183
      check_grow();             // Grow table if insert hit limit
489c9b5090e2 Initial load
duke
parents:
diff changeset
   184
      return NULL;              // Miss!
489c9b5090e2 Initial load
duke
parents:
diff changeset
   185
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   186
    else if( first_sentinel == 0 && k == _sentinel ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   187
      first_sentinel = key;    // Can insert here
489c9b5090e2 Initial load
duke
parents:
diff changeset
   188
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   189
489c9b5090e2 Initial load
duke
parents:
diff changeset
   190
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   191
  ShouldNotReachHere();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   192
  return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   193
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   194
489c9b5090e2 Initial load
duke
parents:
diff changeset
   195
//------------------------------hash_insert------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   196
// Insert into hash table
489c9b5090e2 Initial load
duke
parents:
diff changeset
   197
void NodeHash::hash_insert( Node *n ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   198
  // // "conflict" comments -- print nodes that conflict
489c9b5090e2 Initial load
duke
parents:
diff changeset
   199
  // bool conflict = false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   200
  // n->set_hash();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   201
  uint hash = n->hash();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   202
  if (hash == Node::NO_HASH) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   203
    return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   204
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   205
  check_grow();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   206
  uint key = hash & (_max-1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   207
  uint stride = key | 0x01;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   208
489c9b5090e2 Initial load
duke
parents:
diff changeset
   209
  while( 1 ) {                  // While probing hash table
26429
c5ca44ee3ce2 8056067: NodeHash debug variables are available in product build
thartmann
parents: 25930
diff changeset
   210
    NOT_PRODUCT( _insert_probes++ );
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   211
    Node *k = _table[key];      // Get hashed value
489c9b5090e2 Initial load
duke
parents:
diff changeset
   212
    if( !k || (k == _sentinel) ) break;       // Found a slot
489c9b5090e2 Initial load
duke
parents:
diff changeset
   213
    assert( k != n, "already inserted" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   214
    // if( PrintCompilation && PrintOptoStatistics && Verbose ) { tty->print("  conflict: "); k->dump(); conflict = true; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   215
    key = (key + stride) & (_max-1); // Stride through table w/ relative prime
489c9b5090e2 Initial load
duke
parents:
diff changeset
   216
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   217
  _table[key] = n;              // Insert into table!
489c9b5090e2 Initial load
duke
parents:
diff changeset
   218
  debug_only(n->enter_hash_lock()); // Lock down the node while in the table.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   219
  // if( conflict ) { n->dump(); }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   220
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   221
489c9b5090e2 Initial load
duke
parents:
diff changeset
   222
//------------------------------hash_delete------------------------------------
2131
98f9cef66a34 6810672: Comment typos
twisti
parents: 1067
diff changeset
   223
// Replace in hash table with sentinel
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   224
bool NodeHash::hash_delete( const Node *n ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   225
  Node *k;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   226
  uint hash = n->hash();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   227
  if (hash == Node::NO_HASH) {
26429
c5ca44ee3ce2 8056067: NodeHash debug variables are available in product build
thartmann
parents: 25930
diff changeset
   228
    NOT_PRODUCT( _delete_misses++ );
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   229
    return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   230
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   231
  uint key = hash & (_max-1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   232
  uint stride = key | 0x01;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   233
  debug_only( uint counter = 0; );
2131
98f9cef66a34 6810672: Comment typos
twisti
parents: 1067
diff changeset
   234
  for( ; /* (k != NULL) && (k != _sentinel) */; ) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   235
    debug_only( counter++ );
26429
c5ca44ee3ce2 8056067: NodeHash debug variables are available in product build
thartmann
parents: 25930
diff changeset
   236
    NOT_PRODUCT( _delete_probes++ );
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   237
    k = _table[key];            // Get hashed value
489c9b5090e2 Initial load
duke
parents:
diff changeset
   238
    if( !k ) {                  // Miss?
26429
c5ca44ee3ce2 8056067: NodeHash debug variables are available in product build
thartmann
parents: 25930
diff changeset
   239
      NOT_PRODUCT( _delete_misses++ );
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   240
#ifdef ASSERT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   241
      if( VerifyOpto ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   242
        for( uint i=0; i < _max; i++ )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   243
          assert( _table[i] != n, "changed edges with rehashing" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   244
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   245
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   246
      return false;             // Miss! Not in chain
489c9b5090e2 Initial load
duke
parents:
diff changeset
   247
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   248
    else if( n == k ) {
26429
c5ca44ee3ce2 8056067: NodeHash debug variables are available in product build
thartmann
parents: 25930
diff changeset
   249
      NOT_PRODUCT( _delete_hits++ );
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   250
      _table[key] = _sentinel;  // Hit! Label as deleted entry
489c9b5090e2 Initial load
duke
parents:
diff changeset
   251
      debug_only(((Node*)n)->exit_hash_lock()); // Unlock the node upon removal from table.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   252
      return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   253
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   254
    else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   255
      // collision: move through table with prime offset
489c9b5090e2 Initial load
duke
parents:
diff changeset
   256
      key = (key + stride/*7*/) & (_max-1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   257
      assert( counter <= _insert_limit, "Cycle in hash-table");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   258
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   259
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   260
  ShouldNotReachHere();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   261
  return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   262
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   263
489c9b5090e2 Initial load
duke
parents:
diff changeset
   264
//------------------------------round_up---------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   265
// Round up to nearest power of 2
489c9b5090e2 Initial load
duke
parents:
diff changeset
   266
uint NodeHash::round_up( uint x ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   267
  x += (x>>2);                  // Add 25% slop
489c9b5090e2 Initial load
duke
parents:
diff changeset
   268
  if( x <16 ) return 16;        // Small stuff
489c9b5090e2 Initial load
duke
parents:
diff changeset
   269
  uint i=16;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   270
  while( i < x ) i <<= 1;       // Double to fit
489c9b5090e2 Initial load
duke
parents:
diff changeset
   271
  return i;                     // Return hash table size
489c9b5090e2 Initial load
duke
parents:
diff changeset
   272
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   273
489c9b5090e2 Initial load
duke
parents:
diff changeset
   274
//------------------------------grow-------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   275
// Grow _table to next power of 2 and insert old entries
489c9b5090e2 Initial load
duke
parents:
diff changeset
   276
void  NodeHash::grow() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   277
  // Record old state
489c9b5090e2 Initial load
duke
parents:
diff changeset
   278
  uint   old_max   = _max;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   279
  Node **old_table = _table;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   280
  // Construct new table with twice the space
26429
c5ca44ee3ce2 8056067: NodeHash debug variables are available in product build
thartmann
parents: 25930
diff changeset
   281
#ifndef PRODUCT
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   282
  _grows++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   283
  _total_inserts       += _inserts;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   284
  _total_insert_probes += _insert_probes;
26429
c5ca44ee3ce2 8056067: NodeHash debug variables are available in product build
thartmann
parents: 25930
diff changeset
   285
  _insert_probes   = 0;
c5ca44ee3ce2 8056067: NodeHash debug variables are available in product build
thartmann
parents: 25930
diff changeset
   286
#endif
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   287
  _inserts         = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   288
  _max     = _max << 1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   289
  _table   = NEW_ARENA_ARRAY( _a , Node* , _max ); // (Node**)_a->Amalloc( _max * sizeof(Node*) );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   290
  memset(_table,0,sizeof(Node*)*_max);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   291
  _insert_limit = insert_limit();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   292
  // Insert old entries into the new table
489c9b5090e2 Initial load
duke
parents:
diff changeset
   293
  for( uint i = 0; i < old_max; i++ ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   294
    Node *m = *old_table++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   295
    if( !m || m == _sentinel ) continue;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   296
    debug_only(m->exit_hash_lock()); // Unlock the node upon removal from old table.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   297
    hash_insert(m);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   298
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   299
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   300
489c9b5090e2 Initial load
duke
parents:
diff changeset
   301
//------------------------------clear------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   302
// Clear all entries in _table to NULL but keep storage
489c9b5090e2 Initial load
duke
parents:
diff changeset
   303
void  NodeHash::clear() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   304
#ifdef ASSERT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   305
  // Unlock all nodes upon removal from table.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   306
  for (uint i = 0; i < _max; i++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   307
    Node* n = _table[i];
489c9b5090e2 Initial load
duke
parents:
diff changeset
   308
    if (!n || n == _sentinel)  continue;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   309
    n->exit_hash_lock();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   310
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   311
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   312
489c9b5090e2 Initial load
duke
parents:
diff changeset
   313
  memset( _table, 0, _max * sizeof(Node*) );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   314
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   315
489c9b5090e2 Initial load
duke
parents:
diff changeset
   316
//-----------------------remove_useless_nodes----------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   317
// Remove useless nodes from value table,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   318
// implementation does not depend on hash function
489c9b5090e2 Initial load
duke
parents:
diff changeset
   319
void NodeHash::remove_useless_nodes(VectorSet &useful) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   320
489c9b5090e2 Initial load
duke
parents:
diff changeset
   321
  // Dead nodes in the hash table inherited from GVN should not replace
489c9b5090e2 Initial load
duke
parents:
diff changeset
   322
  // existing nodes, remove dead nodes.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   323
  uint max = size();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   324
  Node *sentinel_node = sentinel();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   325
  for( uint i = 0; i < max; ++i ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   326
    Node *n = at(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   327
    if(n != NULL && n != sentinel_node && !useful.test(n->_idx)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   328
      debug_only(n->exit_hash_lock()); // Unlock the node when removed
489c9b5090e2 Initial load
duke
parents:
diff changeset
   329
      _table[i] = sentinel_node;       // Replace with placeholder
489c9b5090e2 Initial load
duke
parents:
diff changeset
   330
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   331
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   332
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   333
22799
83e58bac7980 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 22234
diff changeset
   334
83e58bac7980 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 22234
diff changeset
   335
void NodeHash::check_no_speculative_types() {
83e58bac7980 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 22234
diff changeset
   336
#ifdef ASSERT
83e58bac7980 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 22234
diff changeset
   337
  uint max = size();
83e58bac7980 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 22234
diff changeset
   338
  Node *sentinel_node = sentinel();
83e58bac7980 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 22234
diff changeset
   339
  for (uint i = 0; i < max; ++i) {
83e58bac7980 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 22234
diff changeset
   340
    Node *n = at(i);
23525
e3eb08ead679 8031755: Type speculation should be used to optimize explicit null checks
roland
parents: 22799
diff changeset
   341
    if(n != NULL && n != sentinel_node && n->is_Type() && n->outcnt() > 0) {
22799
83e58bac7980 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 22234
diff changeset
   342
      TypeNode* tn = n->as_Type();
83e58bac7980 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 22234
diff changeset
   343
      const Type* t = tn->type();
83e58bac7980 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 22234
diff changeset
   344
      const Type* t_no_spec = t->remove_speculative();
83e58bac7980 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 22234
diff changeset
   345
      assert(t == t_no_spec, "dead node in hash table or missed node during speculative cleanup");
83e58bac7980 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 22234
diff changeset
   346
    }
83e58bac7980 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 22234
diff changeset
   347
  }
83e58bac7980 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 22234
diff changeset
   348
#endif
83e58bac7980 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 22234
diff changeset
   349
}
83e58bac7980 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 22234
diff changeset
   350
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   351
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   352
//------------------------------dump-------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   353
// Dump statistics for the hash table
489c9b5090e2 Initial load
duke
parents:
diff changeset
   354
void NodeHash::dump() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   355
  _total_inserts       += _inserts;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   356
  _total_insert_probes += _insert_probes;
10988
a3b2bd43ef4f 7107042: assert(no_dead_loop) failed: dead loop detected
kvn
parents: 10563
diff changeset
   357
  if (PrintCompilation && PrintOptoStatistics && Verbose && (_inserts > 0)) {
a3b2bd43ef4f 7107042: assert(no_dead_loop) failed: dead loop detected
kvn
parents: 10563
diff changeset
   358
    if (WizardMode) {
a3b2bd43ef4f 7107042: assert(no_dead_loop) failed: dead loop detected
kvn
parents: 10563
diff changeset
   359
      for (uint i=0; i<_max; i++) {
a3b2bd43ef4f 7107042: assert(no_dead_loop) failed: dead loop detected
kvn
parents: 10563
diff changeset
   360
        if (_table[i])
a3b2bd43ef4f 7107042: assert(no_dead_loop) failed: dead loop detected
kvn
parents: 10563
diff changeset
   361
          tty->print("%d/%d/%d ",i,_table[i]->hash()&(_max-1),_table[i]->_idx);
a3b2bd43ef4f 7107042: assert(no_dead_loop) failed: dead loop detected
kvn
parents: 10563
diff changeset
   362
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   363
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   364
    tty->print("\nGVN Hash stats:  %d grows to %d max_size\n", _grows, _max);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   365
    tty->print("  %d/%d (%8.1f%% full)\n", _inserts, _max, (double)_inserts/_max*100.0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   366
    tty->print("  %dp/(%dh+%dm) (%8.2f probes/lookup)\n", _look_probes, _lookup_hits, _lookup_misses, (double)_look_probes/(_lookup_hits+_lookup_misses));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   367
    tty->print("  %dp/%di (%8.2f probes/insert)\n", _total_insert_probes, _total_inserts, (double)_total_insert_probes/_total_inserts);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   368
    // sentinels increase lookup cost, but not insert cost
489c9b5090e2 Initial load
duke
parents:
diff changeset
   369
    assert((_lookup_misses+_lookup_hits)*4+100 >= _look_probes, "bad hash function");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   370
    assert( _inserts+(_inserts>>3) < _max, "table too full" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   371
    assert( _inserts*3+100 >= _insert_probes, "bad hash function" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   372
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   373
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   374
489c9b5090e2 Initial load
duke
parents:
diff changeset
   375
Node *NodeHash::find_index(uint idx) { // For debugging
489c9b5090e2 Initial load
duke
parents:
diff changeset
   376
  // Find an entry by its index value
489c9b5090e2 Initial load
duke
parents:
diff changeset
   377
  for( uint i = 0; i < _max; i++ ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   378
    Node *m = _table[i];
489c9b5090e2 Initial load
duke
parents:
diff changeset
   379
    if( !m || m == _sentinel ) continue;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   380
    if( m->_idx == (uint)idx ) return m;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   381
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   382
  return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   383
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   384
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   385
489c9b5090e2 Initial load
duke
parents:
diff changeset
   386
#ifdef ASSERT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   387
NodeHash::~NodeHash() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   388
  // Unlock all nodes upon destruction of table.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   389
  if (_table != (Node**)badAddress)  clear();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   390
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   391
489c9b5090e2 Initial load
duke
parents:
diff changeset
   392
void NodeHash::operator=(const NodeHash& nh) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   393
  // Unlock all nodes upon replacement of table.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   394
  if (&nh == this)  return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   395
  if (_table != (Node**)badAddress)  clear();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   396
  memcpy(this, &nh, sizeof(*this));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   397
  // Do not increment hash_lock counts again.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   398
  // Instead, be sure we never again use the source table.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   399
  ((NodeHash*)&nh)->_table = (Node**)badAddress;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   400
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   401
489c9b5090e2 Initial load
duke
parents:
diff changeset
   402
489c9b5090e2 Initial load
duke
parents:
diff changeset
   403
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   404
489c9b5090e2 Initial load
duke
parents:
diff changeset
   405
489c9b5090e2 Initial load
duke
parents:
diff changeset
   406
//=============================================================================
489c9b5090e2 Initial load
duke
parents:
diff changeset
   407
//------------------------------PhaseRemoveUseless-----------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   408
// 1) Use a breadthfirst walk to collect useful nodes reachable from root.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   409
PhaseRemoveUseless::PhaseRemoveUseless( PhaseGVN *gvn, Unique_Node_List *worklist ) : Phase(Remove_Useless),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   410
  _useful(Thread::current()->resource_area()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   411
489c9b5090e2 Initial load
duke
parents:
diff changeset
   412
  // Implementation requires 'UseLoopSafepoints == true' and an edge from root
489c9b5090e2 Initial load
duke
parents:
diff changeset
   413
  // to each SafePointNode at a backward branch.  Inserted in add_safepoint().
489c9b5090e2 Initial load
duke
parents:
diff changeset
   414
  if( !UseLoopSafepoints || !OptoRemoveUseless ) return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   415
489c9b5090e2 Initial load
duke
parents:
diff changeset
   416
  // Identify nodes that are reachable from below, useful.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   417
  C->identify_useful_nodes(_useful);
14623
70c4c1be0a14 7092905: C2: Keep track of the number of dead nodes
bharadwaj
parents: 13963
diff changeset
   418
  // Update dead node list
70c4c1be0a14 7092905: C2: Keep track of the number of dead nodes
bharadwaj
parents: 13963
diff changeset
   419
  C->update_dead_node_list(_useful);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   420
489c9b5090e2 Initial load
duke
parents:
diff changeset
   421
  // Remove all useless nodes from PhaseValues' recorded types
489c9b5090e2 Initial load
duke
parents:
diff changeset
   422
  // Must be done before disconnecting nodes to preserve hash-table-invariant
489c9b5090e2 Initial load
duke
parents:
diff changeset
   423
  gvn->remove_useless_nodes(_useful.member_set());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   424
489c9b5090e2 Initial load
duke
parents:
diff changeset
   425
  // Remove all useless nodes from future worklist
489c9b5090e2 Initial load
duke
parents:
diff changeset
   426
  worklist->remove_useless_nodes(_useful.member_set());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   427
489c9b5090e2 Initial load
duke
parents:
diff changeset
   428
  // Disconnect 'useless' nodes that are adjacent to useful nodes
489c9b5090e2 Initial load
duke
parents:
diff changeset
   429
  C->remove_useless_nodes(_useful);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   430
489c9b5090e2 Initial load
duke
parents:
diff changeset
   431
  // Remove edges from "root" to each SafePoint at a backward branch.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   432
  // They were inserted during parsing (see add_safepoint()) to make infinite
489c9b5090e2 Initial load
duke
parents:
diff changeset
   433
  // loops without calls or exceptions visible to root, i.e., useful.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   434
  Node *root = C->root();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   435
  if( root != NULL ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   436
    for( uint i = root->req(); i < root->len(); ++i ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   437
      Node *n = root->in(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   438
      if( n != NULL && n->is_SafePoint() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   439
        root->rm_prec(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   440
        --i;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   441
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   442
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   443
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   444
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   445
489c9b5090e2 Initial load
duke
parents:
diff changeset
   446
489c9b5090e2 Initial load
duke
parents:
diff changeset
   447
//=============================================================================
489c9b5090e2 Initial load
duke
parents:
diff changeset
   448
//------------------------------PhaseTransform---------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   449
PhaseTransform::PhaseTransform( PhaseNumber pnum ) : Phase(pnum),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   450
  _arena(Thread::current()->resource_area()),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   451
  _nodes(_arena),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   452
  _types(_arena)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   453
{
489c9b5090e2 Initial load
duke
parents:
diff changeset
   454
  init_con_caches();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   455
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   456
  clear_progress();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   457
  clear_transforms();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   458
  set_allow_progress(true);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   459
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   460
  // Force allocation for currently existing nodes
489c9b5090e2 Initial load
duke
parents:
diff changeset
   461
  _types.map(C->unique(), NULL);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   462
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   463
489c9b5090e2 Initial load
duke
parents:
diff changeset
   464
//------------------------------PhaseTransform---------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   465
PhaseTransform::PhaseTransform( Arena *arena, PhaseNumber pnum ) : Phase(pnum),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   466
  _arena(arena),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   467
  _nodes(arena),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   468
  _types(arena)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   469
{
489c9b5090e2 Initial load
duke
parents:
diff changeset
   470
  init_con_caches();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   471
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   472
  clear_progress();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   473
  clear_transforms();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   474
  set_allow_progress(true);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   475
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   476
  // Force allocation for currently existing nodes
489c9b5090e2 Initial load
duke
parents:
diff changeset
   477
  _types.map(C->unique(), NULL);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   478
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   479
489c9b5090e2 Initial load
duke
parents:
diff changeset
   480
//------------------------------PhaseTransform---------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   481
// Initialize with previously generated type information
489c9b5090e2 Initial load
duke
parents:
diff changeset
   482
PhaseTransform::PhaseTransform( PhaseTransform *pt, PhaseNumber pnum ) : Phase(pnum),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   483
  _arena(pt->_arena),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   484
  _nodes(pt->_nodes),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   485
  _types(pt->_types)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   486
{
489c9b5090e2 Initial load
duke
parents:
diff changeset
   487
  init_con_caches();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   488
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   489
  clear_progress();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   490
  clear_transforms();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   491
  set_allow_progress(true);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   492
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   493
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   494
489c9b5090e2 Initial load
duke
parents:
diff changeset
   495
void PhaseTransform::init_con_caches() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   496
  memset(_icons,0,sizeof(_icons));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   497
  memset(_lcons,0,sizeof(_lcons));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   498
  memset(_zcons,0,sizeof(_zcons));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   499
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   500
489c9b5090e2 Initial load
duke
parents:
diff changeset
   501
489c9b5090e2 Initial load
duke
parents:
diff changeset
   502
//--------------------------------find_int_type--------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   503
const TypeInt* PhaseTransform::find_int_type(Node* n) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   504
  if (n == NULL)  return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   505
  // Call type_or_null(n) to determine node's type since we might be in
489c9b5090e2 Initial load
duke
parents:
diff changeset
   506
  // parse phase and call n->Value() may return wrong type.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   507
  // (For example, a phi node at the beginning of loop parsing is not ready.)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   508
  const Type* t = type_or_null(n);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   509
  if (t == NULL)  return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   510
  return t->isa_int();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   511
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   512
489c9b5090e2 Initial load
duke
parents:
diff changeset
   513
489c9b5090e2 Initial load
duke
parents:
diff changeset
   514
//-------------------------------find_long_type--------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   515
const TypeLong* PhaseTransform::find_long_type(Node* n) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   516
  if (n == NULL)  return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   517
  // (See comment above on type_or_null.)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   518
  const Type* t = type_or_null(n);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   519
  if (t == NULL)  return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   520
  return t->isa_long();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   521
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   522
489c9b5090e2 Initial load
duke
parents:
diff changeset
   523
489c9b5090e2 Initial load
duke
parents:
diff changeset
   524
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   525
void PhaseTransform::dump_old2new_map() const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   526
  _nodes.dump();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   527
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   528
489c9b5090e2 Initial load
duke
parents:
diff changeset
   529
void PhaseTransform::dump_new( uint nidx ) const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   530
  for( uint i=0; i<_nodes.Size(); i++ )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   531
    if( _nodes[i] && _nodes[i]->_idx == nidx ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   532
      _nodes[i]->dump();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   533
      tty->cr();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   534
      tty->print_cr("Old index= %d",i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   535
      return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   536
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   537
  tty->print_cr("Node %d not found in the new indices", nidx);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   538
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   539
489c9b5090e2 Initial load
duke
parents:
diff changeset
   540
//------------------------------dump_types-------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   541
void PhaseTransform::dump_types( ) const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   542
  _types.dump();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   543
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   544
489c9b5090e2 Initial load
duke
parents:
diff changeset
   545
//------------------------------dump_nodes_and_types---------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   546
void PhaseTransform::dump_nodes_and_types(const Node *root, uint depth, bool only_ctrl) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   547
  VectorSet visited(Thread::current()->resource_area());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   548
  dump_nodes_and_types_recur( root, depth, only_ctrl, visited );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   549
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   550
489c9b5090e2 Initial load
duke
parents:
diff changeset
   551
//------------------------------dump_nodes_and_types_recur---------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   552
void PhaseTransform::dump_nodes_and_types_recur( const Node *n, uint depth, bool only_ctrl, VectorSet &visited) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   553
  if( !n ) return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   554
  if( depth == 0 ) return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   555
  if( visited.test_set(n->_idx) ) return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   556
  for( uint i=0; i<n->len(); i++ ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   557
    if( only_ctrl && !(n->is_Region()) && i != TypeFunc::Control ) continue;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   558
    dump_nodes_and_types_recur( n->in(i), depth-1, only_ctrl, visited );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   559
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   560
  n->dump();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   561
  if (type_or_null(n) != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   562
    tty->print("      "); type(n)->dump(); tty->cr();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   563
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   564
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   565
489c9b5090e2 Initial load
duke
parents:
diff changeset
   566
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   567
489c9b5090e2 Initial load
duke
parents:
diff changeset
   568
489c9b5090e2 Initial load
duke
parents:
diff changeset
   569
//=============================================================================
489c9b5090e2 Initial load
duke
parents:
diff changeset
   570
//------------------------------PhaseValues------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   571
// Set minimum table size to "255"
489c9b5090e2 Initial load
duke
parents:
diff changeset
   572
PhaseValues::PhaseValues( Arena *arena, uint est_max_size ) : PhaseTransform(arena, GVN), _table(arena, est_max_size) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   573
  NOT_PRODUCT( clear_new_values(); )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   574
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   575
489c9b5090e2 Initial load
duke
parents:
diff changeset
   576
//------------------------------PhaseValues------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   577
// Set minimum table size to "255"
489c9b5090e2 Initial load
duke
parents:
diff changeset
   578
PhaseValues::PhaseValues( PhaseValues *ptv ) : PhaseTransform( ptv, GVN ),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   579
  _table(&ptv->_table) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   580
  NOT_PRODUCT( clear_new_values(); )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   581
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   582
489c9b5090e2 Initial load
duke
parents:
diff changeset
   583
//------------------------------PhaseValues------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   584
// Used by +VerifyOpto.  Clear out hash table but copy _types array.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   585
PhaseValues::PhaseValues( PhaseValues *ptv, const char *dummy ) : PhaseTransform( ptv, GVN ),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   586
  _table(ptv->arena(),ptv->_table.size()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   587
  NOT_PRODUCT( clear_new_values(); )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   588
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   589
489c9b5090e2 Initial load
duke
parents:
diff changeset
   590
//------------------------------~PhaseValues-----------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   591
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   592
PhaseValues::~PhaseValues() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   593
  _table.dump();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   594
489c9b5090e2 Initial load
duke
parents:
diff changeset
   595
  // Statistics for value progress and efficiency
489c9b5090e2 Initial load
duke
parents:
diff changeset
   596
  if( PrintCompilation && Verbose && WizardMode ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   597
    tty->print("\n%sValues: %d nodes ---> %d/%d (%d)",
489c9b5090e2 Initial load
duke
parents:
diff changeset
   598
      is_IterGVN() ? "Iter" : "    ", C->unique(), made_progress(), made_transforms(), made_new_values());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   599
    if( made_transforms() != 0 ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   600
      tty->print_cr("  ratio %f", made_progress()/(float)made_transforms() );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   601
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   602
      tty->cr();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   603
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   604
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   605
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   606
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   607
489c9b5090e2 Initial load
duke
parents:
diff changeset
   608
//------------------------------makecon----------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   609
ConNode* PhaseTransform::makecon(const Type *t) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   610
  assert(t->singleton(), "must be a constant");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   611
  assert(!t->empty() || t == Type::TOP, "must not be vacuous range");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   612
  switch (t->base()) {  // fast paths
489c9b5090e2 Initial load
duke
parents:
diff changeset
   613
  case Type::Half:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   614
  case Type::Top:  return (ConNode*) C->top();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   615
  case Type::Int:  return intcon( t->is_int()->get_con() );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   616
  case Type::Long: return longcon( t->is_long()->get_con() );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   617
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   618
  if (t->is_zero_type())
489c9b5090e2 Initial load
duke
parents:
diff changeset
   619
    return zerocon(t->basic_type());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   620
  return uncached_makecon(t);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   621
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   622
489c9b5090e2 Initial load
duke
parents:
diff changeset
   623
//--------------------------uncached_makecon-----------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   624
// Make an idealized constant - one of ConINode, ConPNode, etc.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   625
ConNode* PhaseValues::uncached_makecon(const Type *t) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   626
  assert(t->singleton(), "must be a constant");
25930
eae8b7490d2c 8054033: Remove unused references to Compile*
thartmann
parents: 25913
diff changeset
   627
  ConNode* x = ConNode::make(t);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   628
  ConNode* k = (ConNode*)hash_find_insert(x); // Value numbering
489c9b5090e2 Initial load
duke
parents:
diff changeset
   629
  if (k == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   630
    set_type(x, t);             // Missed, provide type mapping
489c9b5090e2 Initial load
duke
parents:
diff changeset
   631
    GrowableArray<Node_Notes*>* nna = C->node_note_array();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   632
    if (nna != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   633
      Node_Notes* loc = C->locate_node_notes(nna, x->_idx, true);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   634
      loc->clear(); // do not put debug info on constants
489c9b5090e2 Initial load
duke
parents:
diff changeset
   635
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   636
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   637
    x->destruct();              // Hit, destroy duplicate constant
489c9b5090e2 Initial load
duke
parents:
diff changeset
   638
    x = k;                      // use existing constant
489c9b5090e2 Initial load
duke
parents:
diff changeset
   639
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   640
  return x;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   641
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   642
489c9b5090e2 Initial load
duke
parents:
diff changeset
   643
//------------------------------intcon-----------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   644
// Fast integer constant.  Same as "transform(new ConINode(TypeInt::make(i)))"
489c9b5090e2 Initial load
duke
parents:
diff changeset
   645
ConINode* PhaseTransform::intcon(int i) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   646
  // Small integer?  Check cache! Check that cached node is not dead
489c9b5090e2 Initial load
duke
parents:
diff changeset
   647
  if (i >= _icon_min && i <= _icon_max) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   648
    ConINode* icon = _icons[i-_icon_min];
489c9b5090e2 Initial load
duke
parents:
diff changeset
   649
    if (icon != NULL && icon->in(TypeFunc::Control) != NULL)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   650
      return icon;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   651
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   652
  ConINode* icon = (ConINode*) uncached_makecon(TypeInt::make(i));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   653
  assert(icon->is_Con(), "");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   654
  if (i >= _icon_min && i <= _icon_max)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   655
    _icons[i-_icon_min] = icon;   // Cache small integers
489c9b5090e2 Initial load
duke
parents:
diff changeset
   656
  return icon;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   657
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   658
489c9b5090e2 Initial load
duke
parents:
diff changeset
   659
//------------------------------longcon----------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   660
// Fast long constant.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   661
ConLNode* PhaseTransform::longcon(jlong l) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   662
  // Small integer?  Check cache! Check that cached node is not dead
489c9b5090e2 Initial load
duke
parents:
diff changeset
   663
  if (l >= _lcon_min && l <= _lcon_max) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   664
    ConLNode* lcon = _lcons[l-_lcon_min];
489c9b5090e2 Initial load
duke
parents:
diff changeset
   665
    if (lcon != NULL && lcon->in(TypeFunc::Control) != NULL)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   666
      return lcon;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   667
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   668
  ConLNode* lcon = (ConLNode*) uncached_makecon(TypeLong::make(l));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   669
  assert(lcon->is_Con(), "");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   670
  if (l >= _lcon_min && l <= _lcon_max)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   671
    _lcons[l-_lcon_min] = lcon;      // Cache small integers
489c9b5090e2 Initial load
duke
parents:
diff changeset
   672
  return lcon;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   673
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   674
489c9b5090e2 Initial load
duke
parents:
diff changeset
   675
//------------------------------zerocon-----------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   676
// Fast zero or null constant. Same as "transform(ConNode::make(Type::get_zero_type(bt)))"
489c9b5090e2 Initial load
duke
parents:
diff changeset
   677
ConNode* PhaseTransform::zerocon(BasicType bt) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   678
  assert((uint)bt <= _zcon_max, "domain check");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   679
  ConNode* zcon = _zcons[bt];
489c9b5090e2 Initial load
duke
parents:
diff changeset
   680
  if (zcon != NULL && zcon->in(TypeFunc::Control) != NULL)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   681
    return zcon;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   682
  zcon = (ConNode*) uncached_makecon(Type::get_zero_type(bt));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   683
  _zcons[bt] = zcon;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   684
  return zcon;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   685
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   686
489c9b5090e2 Initial load
duke
parents:
diff changeset
   687
489c9b5090e2 Initial load
duke
parents:
diff changeset
   688
489c9b5090e2 Initial load
duke
parents:
diff changeset
   689
//=============================================================================
489c9b5090e2 Initial load
duke
parents:
diff changeset
   690
//------------------------------transform--------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   691
// Return a node which computes the same function as this node, but in a
214
c5d9b4456687 6667605: (Escape Analysis) inline java constructors when EA is on
kvn
parents: 1
diff changeset
   692
// faster or cheaper fashion.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   693
Node *PhaseGVN::transform( Node *n ) {
214
c5d9b4456687 6667605: (Escape Analysis) inline java constructors when EA is on
kvn
parents: 1
diff changeset
   694
  return transform_no_reclaim(n);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   695
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   696
489c9b5090e2 Initial load
duke
parents:
diff changeset
   697
//------------------------------transform--------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   698
// Return a node which computes the same function as this node, but
489c9b5090e2 Initial load
duke
parents:
diff changeset
   699
// in a faster or cheaper fashion.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   700
Node *PhaseGVN::transform_no_reclaim( Node *n ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   701
  NOT_PRODUCT( set_transforms(); )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   702
489c9b5090e2 Initial load
duke
parents:
diff changeset
   703
  // Apply the Ideal call in a loop until it no longer applies
489c9b5090e2 Initial load
duke
parents:
diff changeset
   704
  Node *k = n;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   705
  NOT_PRODUCT( uint loop_count = 0; )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   706
  while( 1 ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   707
    Node *i = k->Ideal(this, /*can_reshape=*/false);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   708
    if( !i ) break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   709
    assert( i->_idx >= k->_idx, "Idealize should return new nodes, use Identity to return old nodes" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   710
    k = i;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   711
    assert(loop_count++ < K, "infinite loop in PhaseGVN::transform");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   712
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   713
  NOT_PRODUCT( if( loop_count != 0 ) { set_progress(); } )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   714
489c9b5090e2 Initial load
duke
parents:
diff changeset
   715
489c9b5090e2 Initial load
duke
parents:
diff changeset
   716
  // If brand new node, make space in type array.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   717
  ensure_type_or_null(k);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   718
489c9b5090e2 Initial load
duke
parents:
diff changeset
   719
  // Since I just called 'Value' to compute the set of run-time values
489c9b5090e2 Initial load
duke
parents:
diff changeset
   720
  // for this Node, and 'Value' is non-local (and therefore expensive) I'll
489c9b5090e2 Initial load
duke
parents:
diff changeset
   721
  // cache Value.  Later requests for the local phase->type of this Node can
489c9b5090e2 Initial load
duke
parents:
diff changeset
   722
  // use the cached Value instead of suffering with 'bottom_type'.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   723
  const Type *t = k->Value(this); // Get runtime Value set
489c9b5090e2 Initial load
duke
parents:
diff changeset
   724
  assert(t != NULL, "value sanity");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   725
  if (type_or_null(k) != t) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   726
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   727
    // Do not count initial visit to node as a transformation
489c9b5090e2 Initial load
duke
parents:
diff changeset
   728
    if (type_or_null(k) == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   729
      inc_new_values();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   730
      set_progress();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   731
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   732
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   733
    set_type(k, t);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   734
    // If k is a TypeNode, capture any more-precise type permanently into Node
489c9b5090e2 Initial load
duke
parents:
diff changeset
   735
    k->raise_bottom_type(t);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   736
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   737
489c9b5090e2 Initial load
duke
parents:
diff changeset
   738
  if( t->singleton() && !k->is_Con() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   739
    NOT_PRODUCT( set_progress(); )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   740
    return makecon(t);          // Turn into a constant
489c9b5090e2 Initial load
duke
parents:
diff changeset
   741
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   742
489c9b5090e2 Initial load
duke
parents:
diff changeset
   743
  // Now check for Identities
489c9b5090e2 Initial load
duke
parents:
diff changeset
   744
  Node *i = k->Identity(this);  // Look for a nearby replacement
489c9b5090e2 Initial load
duke
parents:
diff changeset
   745
  if( i != k ) {                // Found? Return replacement!
489c9b5090e2 Initial load
duke
parents:
diff changeset
   746
    NOT_PRODUCT( set_progress(); )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   747
    return i;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   748
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   749
489c9b5090e2 Initial load
duke
parents:
diff changeset
   750
  // Global Value Numbering
489c9b5090e2 Initial load
duke
parents:
diff changeset
   751
  i = hash_find_insert(k);      // Insert if new
489c9b5090e2 Initial load
duke
parents:
diff changeset
   752
  if( i && (i != k) ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   753
    // Return the pre-existing node
489c9b5090e2 Initial load
duke
parents:
diff changeset
   754
    NOT_PRODUCT( set_progress(); )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   755
    return i;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   756
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   757
489c9b5090e2 Initial load
duke
parents:
diff changeset
   758
  // Return Idealized original
489c9b5090e2 Initial load
duke
parents:
diff changeset
   759
  return k;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   760
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   761
489c9b5090e2 Initial load
duke
parents:
diff changeset
   762
#ifdef ASSERT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   763
//------------------------------dead_loop_check--------------------------------
2131
98f9cef66a34 6810672: Comment typos
twisti
parents: 1067
diff changeset
   764
// Check for a simple dead loop when a data node references itself directly
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   765
// or through an other data node excluding cons and phis.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   766
void PhaseGVN::dead_loop_check( Node *n ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   767
  // Phi may reference itself in a loop
489c9b5090e2 Initial load
duke
parents:
diff changeset
   768
  if (n != NULL && !n->is_dead_loop_safe() && !n->is_CFG()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   769
    // Do 2 levels check and only data inputs.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   770
    bool no_dead_loop = true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   771
    uint cnt = n->req();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   772
    for (uint i = 1; i < cnt && no_dead_loop; i++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   773
      Node *in = n->in(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   774
      if (in == n) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   775
        no_dead_loop = false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   776
      } else if (in != NULL && !in->is_dead_loop_safe()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   777
        uint icnt = in->req();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   778
        for (uint j = 1; j < icnt && no_dead_loop; j++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   779
          if (in->in(j) == n || in->in(j) == in)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   780
            no_dead_loop = false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   781
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   782
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   783
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   784
    if (!no_dead_loop) n->dump(3);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   785
    assert(no_dead_loop, "dead loop detected");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   786
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   787
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   788
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   789
489c9b5090e2 Initial load
duke
parents:
diff changeset
   790
//=============================================================================
489c9b5090e2 Initial load
duke
parents:
diff changeset
   791
//------------------------------PhaseIterGVN-----------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   792
// Initialize hash table to fresh and clean for +VerifyOpto
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 238
diff changeset
   793
PhaseIterGVN::PhaseIterGVN( PhaseIterGVN *igvn, const char *dummy ) : PhaseGVN(igvn,dummy), _worklist( ),
13312
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
   794
                                                                      _stack(C->unique() >> 1),
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 238
diff changeset
   795
                                                                      _delay_transform(false) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   796
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   797
489c9b5090e2 Initial load
duke
parents:
diff changeset
   798
//------------------------------PhaseIterGVN-----------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   799
// Initialize with previous PhaseIterGVN info; used by PhaseCCP
489c9b5090e2 Initial load
duke
parents:
diff changeset
   800
PhaseIterGVN::PhaseIterGVN( PhaseIterGVN *igvn ) : PhaseGVN(igvn),
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 238
diff changeset
   801
                                                   _worklist( igvn->_worklist ),
13312
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
   802
                                                   _stack( igvn->_stack ),
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 238
diff changeset
   803
                                                   _delay_transform(igvn->_delay_transform)
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   804
{
489c9b5090e2 Initial load
duke
parents:
diff changeset
   805
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   806
489c9b5090e2 Initial load
duke
parents:
diff changeset
   807
//------------------------------PhaseIterGVN-----------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   808
// Initialize with previous PhaseGVN info from Parser
489c9b5090e2 Initial load
duke
parents:
diff changeset
   809
PhaseIterGVN::PhaseIterGVN( PhaseGVN *gvn ) : PhaseGVN(gvn),
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 238
diff changeset
   810
                                              _worklist(*C->for_igvn()),
13312
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
   811
                                              _stack(C->unique() >> 1),
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 238
diff changeset
   812
                                              _delay_transform(false)
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   813
{
489c9b5090e2 Initial load
duke
parents:
diff changeset
   814
  uint max;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   815
489c9b5090e2 Initial load
duke
parents:
diff changeset
   816
  // Dead nodes in the hash table inherited from GVN were not treated as
489c9b5090e2 Initial load
duke
parents:
diff changeset
   817
  // roots during def-use info creation; hence they represent an invisible
489c9b5090e2 Initial load
duke
parents:
diff changeset
   818
  // use.  Clear them out.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   819
  max = _table.size();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   820
  for( uint i = 0; i < max; ++i ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   821
    Node *n = _table.at(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   822
    if(n != NULL && n != _table.sentinel() && n->outcnt() == 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   823
      if( n->is_top() ) continue;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   824
      assert( false, "Parse::remove_useless_nodes missed this node");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   825
      hash_delete(n);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   826
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   827
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   828
489c9b5090e2 Initial load
duke
parents:
diff changeset
   829
  // Any Phis or Regions on the worklist probably had uses that could not
489c9b5090e2 Initial load
duke
parents:
diff changeset
   830
  // make more progress because the uses were made while the Phis and Regions
489c9b5090e2 Initial load
duke
parents:
diff changeset
   831
  // were in half-built states.  Put all uses of Phis and Regions on worklist.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   832
  max = _worklist.size();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   833
  for( uint j = 0; j < max; j++ ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   834
    Node *n = _worklist.at(j);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   835
    uint uop = n->Opcode();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   836
    if( uop == Op_Phi || uop == Op_Region ||
489c9b5090e2 Initial load
duke
parents:
diff changeset
   837
        n->is_Type() ||
489c9b5090e2 Initial load
duke
parents:
diff changeset
   838
        n->is_Mem() )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   839
      add_users_to_worklist(n);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   840
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   841
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   842
24015
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   843
/**
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   844
 * Initialize worklist for each node.
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   845
 */
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   846
void PhaseIterGVN::init_worklist(Node* first) {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   847
  Unique_Node_List to_process;
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   848
  to_process.push(first);
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   849
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   850
  while (to_process.size() > 0) {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   851
    Node* n = to_process.pop();
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   852
    if (!_worklist.member(n)) {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   853
      _worklist.push(n);
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   854
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   855
      uint cnt = n->req();
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   856
      for(uint i = 0; i < cnt; i++) {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   857
        Node* m = n->in(i);
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   858
        if (m != NULL) {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   859
          to_process.push(m);
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   860
        }
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   861
      }
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   862
    }
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   863
  }
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   864
}
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   865
489c9b5090e2 Initial load
duke
parents:
diff changeset
   866
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   867
void PhaseIterGVN::verify_step(Node* n) {
24015
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   868
  if (VerifyIterativeGVN) {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   869
    _verify_window[_verify_counter % _verify_window_size] = n;
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   870
    ++_verify_counter;
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   871
    ResourceMark rm;
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   872
    ResourceArea* area = Thread::current()->resource_area();
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   873
    VectorSet old_space(area), new_space(area);
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   874
    if (C->unique() < 1000 ||
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   875
        0 == _verify_counter % (C->unique() < 10000 ? 10 : 100)) {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   876
      ++_verify_full_passes;
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   877
      Node::verify_recur(C->root(), -1, old_space, new_space);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   878
    }
24015
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   879
    const int verify_depth = 4;
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   880
    for ( int i = 0; i < _verify_window_size; i++ ) {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   881
      Node* n = _verify_window[i];
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   882
      if ( n == NULL )  continue;
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   883
      if( n->in(0) == NodeSentinel ) {  // xform_idom
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   884
        _verify_window[i] = n->in(1);
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   885
        --i; continue;
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   886
      }
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   887
      // Typical fanout is 1-2, so this call visits about 6 nodes.
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   888
      Node::verify_recur(n, verify_depth, old_space, new_space);
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   889
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   890
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   891
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   892
24015
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   893
void PhaseIterGVN::trace_PhaseIterGVN(Node* n, Node* nn, const Type* oldtype) {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   894
  if (TraceIterativeGVN) {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   895
    uint wlsize = _worklist.size();
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   896
    const Type* newtype = type_or_null(n);
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   897
    if (nn != n) {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   898
      // print old node
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   899
      tty->print("< ");
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   900
      if (oldtype != newtype && oldtype != NULL) {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   901
        oldtype->dump();
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   902
      }
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   903
      do { tty->print("\t"); } while (tty->position() < 16);
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   904
      tty->print("<");
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   905
      n->dump();
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   906
    }
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   907
    if (oldtype != newtype || nn != n) {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   908
      // print new node and/or new type
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   909
      if (oldtype == NULL) {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   910
        tty->print("* ");
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   911
      } else if (nn != n) {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   912
        tty->print("> ");
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   913
      } else {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   914
        tty->print("= ");
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   915
      }
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   916
      if (newtype == NULL) {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   917
        tty->print("null");
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   918
      } else {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   919
        newtype->dump();
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   920
      }
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   921
      do { tty->print("\t"); } while (tty->position() < 16);
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   922
      nn->dump();
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   923
    }
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   924
    if (Verbose && wlsize < _worklist.size()) {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   925
      tty->print("  Push {");
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   926
      while (wlsize != _worklist.size()) {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   927
        Node* pushed = _worklist.at(wlsize++);
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   928
        tty->print(" %d", pushed->_idx);
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   929
      }
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   930
      tty->print_cr(" }");
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   931
    }
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   932
    if (nn != n) {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   933
      // ignore n, it might be subsumed
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   934
      verify_step((Node*) NULL);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   935
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   936
  }
24015
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   937
}
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   938
24015
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   939
void PhaseIterGVN::init_verifyPhaseIterGVN() {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   940
  _verify_counter = 0;
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   941
  _verify_full_passes = 0;
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   942
  for (int i = 0; i < _verify_window_size; i++) {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   943
    _verify_window[i] = NULL;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   944
  }
25913
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 25739
diff changeset
   945
#ifdef ASSERT
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 25739
diff changeset
   946
  // Verify that all modified nodes are on _worklist
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 25739
diff changeset
   947
  Unique_Node_List* modified_list = C->modified_nodes();
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 25739
diff changeset
   948
  while (modified_list != NULL && modified_list->size()) {
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 25739
diff changeset
   949
    Node* n = modified_list->pop();
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 25739
diff changeset
   950
    if (n->outcnt() != 0 && !n->is_Con() && !_worklist.member(n)) {
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 25739
diff changeset
   951
      n->dump();
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 25739
diff changeset
   952
      assert(false, "modified node is not on IGVN._worklist");
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 25739
diff changeset
   953
    }
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 25739
diff changeset
   954
  }
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 25739
diff changeset
   955
#endif
24015
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   956
}
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   957
24015
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   958
void PhaseIterGVN::verify_PhaseIterGVN() {
25913
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 25739
diff changeset
   959
#ifdef ASSERT
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 25739
diff changeset
   960
  // Verify nodes with changed inputs.
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 25739
diff changeset
   961
  Unique_Node_List* modified_list = C->modified_nodes();
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 25739
diff changeset
   962
  while (modified_list != NULL && modified_list->size()) {
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 25739
diff changeset
   963
    Node* n = modified_list->pop();
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 25739
diff changeset
   964
    if (n->outcnt() != 0 && !n->is_Con()) { // skip dead and Con nodes
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 25739
diff changeset
   965
      n->dump();
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 25739
diff changeset
   966
      assert(false, "modified node was not processed by IGVN.transform_old()");
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 25739
diff changeset
   967
    }
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 25739
diff changeset
   968
  }
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 25739
diff changeset
   969
#endif
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 25739
diff changeset
   970
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   971
  C->verify_graph_edges();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   972
  if( VerifyOpto && allow_progress() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   973
    // Must turn off allow_progress to enable assert and break recursion
489c9b5090e2 Initial load
duke
parents:
diff changeset
   974
    C->root()->verify();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   975
    { // Check if any progress was missed using IterGVN
489c9b5090e2 Initial load
duke
parents:
diff changeset
   976
      // Def-Use info enables transformations not attempted in wash-pass
489c9b5090e2 Initial load
duke
parents:
diff changeset
   977
      // e.g. Region/Phi cleanup, ...
489c9b5090e2 Initial load
duke
parents:
diff changeset
   978
      // Null-check elision -- may not have reached fixpoint
489c9b5090e2 Initial load
duke
parents:
diff changeset
   979
      //                       do not propagate to dominated nodes
489c9b5090e2 Initial load
duke
parents:
diff changeset
   980
      ResourceMark rm;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   981
      PhaseIterGVN igvn2(this,"Verify"); // Fresh and clean!
489c9b5090e2 Initial load
duke
parents:
diff changeset
   982
      // Fill worklist completely
489c9b5090e2 Initial load
duke
parents:
diff changeset
   983
      igvn2.init_worklist(C->root());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   984
489c9b5090e2 Initial load
duke
parents:
diff changeset
   985
      igvn2.set_allow_progress(false);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   986
      igvn2.optimize();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   987
      igvn2.set_allow_progress(true);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   988
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   989
  }
24015
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   990
  if (VerifyIterativeGVN && PrintOpto) {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   991
    if (_verify_counter == _verify_full_passes) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   992
      tty->print_cr("VerifyIterativeGVN: %d transforms and verify passes",
24424
2658d7834c6e 8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents: 24016
diff changeset
   993
                    (int) _verify_full_passes);
24015
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   994
    } else {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   995
      tty->print_cr("VerifyIterativeGVN: %d transforms, %d full verify passes",
24424
2658d7834c6e 8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents: 24016
diff changeset
   996
                  (int) _verify_counter, (int) _verify_full_passes);
24015
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
   997
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   998
  }
25913
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 25739
diff changeset
   999
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 25739
diff changeset
  1000
#ifdef ASSERT
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 25739
diff changeset
  1001
  while (modified_list->size()) {
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 25739
diff changeset
  1002
    Node* n = modified_list->pop();
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 25739
diff changeset
  1003
    n->dump();
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 25739
diff changeset
  1004
    assert(false, "VerifyIterativeGVN: new modified node was added");
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 25739
diff changeset
  1005
  }
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 25739
diff changeset
  1006
#endif
24015
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1007
}
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1008
#endif /* PRODUCT */
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1009
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1010
#ifdef ASSERT
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1011
/**
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1012
 * Dumps information that can help to debug the problem. A debug
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1013
 * build fails with an assert.
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1014
 */
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1015
void PhaseIterGVN::dump_infinite_loop_info(Node* n) {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1016
  n->dump(4);
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1017
  _worklist.dump();
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1018
  assert(false, "infinite loop in PhaseIterGVN::optimize");
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1019
}
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1020
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1021
/**
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1022
 * Prints out information about IGVN if the 'verbose' option is used.
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1023
 */
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1024
void PhaseIterGVN::trace_PhaseIterGVN_verbose(Node* n, int num_processed) {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1025
  if (TraceIterativeGVN && Verbose) {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1026
    tty->print("  Pop ");
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1027
    n->dump();
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1028
    if ((num_processed % 100) == 0) {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1029
      _worklist.print_set();
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1030
    }
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1031
  }
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1032
}
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1033
#endif /* ASSERT */
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1034
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1035
void PhaseIterGVN::optimize() {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1036
  DEBUG_ONLY(uint num_processed  = 0;)
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1037
  NOT_PRODUCT(init_verifyPhaseIterGVN();)
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1038
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1039
  uint loop_count = 0;
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1040
  // Pull from worklist and transform the node. If the node has changed,
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1041
  // update edge info and put uses on worklist.
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1042
  while(_worklist.size()) {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1043
    if (C->check_node_count(NodeLimitFudgeFactor * 2, "Out of nodes")) {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1044
      return;
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1045
    }
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1046
    Node* n  = _worklist.pop();
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1047
    if (++loop_count >= K * C->live_nodes()) {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1048
      DEBUG_ONLY(dump_infinite_loop_info(n);)
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1049
      C->record_method_not_compilable("infinite loop in PhaseIterGVN::optimize");
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1050
      return;
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1051
    }
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1052
    DEBUG_ONLY(trace_PhaseIterGVN_verbose(n, num_processed++);)
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1053
    if (n->outcnt() != 0) {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1054
      NOT_PRODUCT(const Type* oldtype = type_or_null(n));
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1055
      // Do the transformation
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1056
      Node* nn = transform_old(n);
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1057
      NOT_PRODUCT(trace_PhaseIterGVN(n, nn, oldtype);)
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1058
    } else if (!n->is_top()) {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1059
      remove_dead_node(n);
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1060
    }
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1061
  }
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1062
  NOT_PRODUCT(verify_PhaseIterGVN();)
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1063
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1064
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1065
24015
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1066
/**
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1067
 * Register a new node with the optimizer.  Update the types array, the def-use
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1068
 * info.  Put on worklist.
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1069
 */
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1070
Node* PhaseIterGVN::register_new_node_with_optimizer(Node* n, Node* orig) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1071
  set_type_bottom(n);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1072
  _worklist.push(n);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1073
  if (orig != NULL)  C->copy_node_notes_to(n, orig);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1074
  return n;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1075
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1076
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1077
//------------------------------transform--------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1078
// Non-recursive: idealize Node 'n' with respect to its inputs and its value
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1079
Node *PhaseIterGVN::transform( Node *n ) {
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 238
diff changeset
  1080
  if (_delay_transform) {
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 238
diff changeset
  1081
    // Register the node but don't optimize for now
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 238
diff changeset
  1082
    register_new_node_with_optimizer(n);
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 238
diff changeset
  1083
    return n;
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 238
diff changeset
  1084
  }
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 238
diff changeset
  1085
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1086
  // If brand new node, make space in type array, and give it a type.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1087
  ensure_type_or_null(n);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1088
  if (type_or_null(n) == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1089
    set_type_bottom(n);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1090
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1091
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1092
  return transform_old(n);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1093
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1094
24015
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1095
Node *PhaseIterGVN::transform_old(Node* n) {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1096
  DEBUG_ONLY(uint loop_count = 0;);
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1097
  NOT_PRODUCT(set_transforms());
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1098
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1099
  // Remove 'n' from hash table in case it gets modified
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1100
  _table.hash_delete(n);
24015
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1101
  if (VerifyIterativeGVN) {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1102
   assert(!_table.find_index(n->_idx), "found duplicate entry in table");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1103
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1104
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1105
  // Apply the Ideal call in a loop until it no longer applies
24015
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1106
  Node* k = n;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1107
  DEBUG_ONLY(dead_loop_check(k);)
1067
f82e0a8cd438 6736417: Fastdebug C2 crashes in StoreBNode::Ideal
kvn
parents: 670
diff changeset
  1108
  DEBUG_ONLY(bool is_new = (k->outcnt() == 0);)
25913
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 25739
diff changeset
  1109
  C->remove_modified_node(k);
24015
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1110
  Node* i = k->Ideal(this, /*can_reshape=*/true);
1067
f82e0a8cd438 6736417: Fastdebug C2 crashes in StoreBNode::Ideal
kvn
parents: 670
diff changeset
  1111
  assert(i != k || is_new || i->outcnt() > 0, "don't return dead nodes");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1112
#ifndef PRODUCT
24015
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1113
  verify_step(k);
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1114
  if (i && VerifyOpto ) {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1115
    if (!allow_progress()) {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1116
      if (i->is_Add() && (i->outcnt() == 1)) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1117
        // Switched input to left side because this is the only use
24015
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1118
      } else if (i->is_If() && (i->in(0) == NULL)) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1119
        // This IF is dead because it is dominated by an equivalent IF When
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1120
        // dominating if changed, info is not propagated sparsely to 'this'
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1121
        // Propagating this info further will spuriously identify other
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1122
        // progress.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1123
        return i;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1124
      } else
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1125
        set_progress();
24015
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1126
    } else {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1127
      set_progress();
24015
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1128
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1129
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1130
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1131
24015
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1132
  while (i != NULL) {
25739
0d9a6751cb1d 8051303: 'optimized' build broken by JDK-8039425
anoll
parents: 24923
diff changeset
  1133
#ifdef ASSERT
24015
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1134
    if (loop_count >= K) {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1135
      dump_infinite_loop_info(i);
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1136
    }
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1137
    loop_count++;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1138
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1139
    assert((i->_idx >= k->_idx) || i->is_top(), "Idealize should return new nodes, use Identity to return old nodes");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1140
    // Made a change; put users of original Node on worklist
24015
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1141
    add_users_to_worklist(k);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1142
    // Replacing root of transform tree?
24015
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1143
    if (k != i) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1144
      // Make users of old Node now use new.
24015
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1145
      subsume_node(k, i);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1146
      k = i;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1147
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1148
    DEBUG_ONLY(dead_loop_check(k);)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1149
    // Try idealizing again
1067
f82e0a8cd438 6736417: Fastdebug C2 crashes in StoreBNode::Ideal
kvn
parents: 670
diff changeset
  1150
    DEBUG_ONLY(is_new = (k->outcnt() == 0);)
25913
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 25739
diff changeset
  1151
    C->remove_modified_node(k);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1152
    i = k->Ideal(this, /*can_reshape=*/true);
24015
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1153
    assert(i != k || is_new || (i->outcnt() > 0), "don't return dead nodes");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1154
#ifndef PRODUCT
24015
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1155
    verify_step(k);
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1156
    if (i && VerifyOpto) {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1157
      set_progress();
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1158
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1159
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1160
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1161
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1162
  // If brand new node, make space in type array.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1163
  ensure_type_or_null(k);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1164
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1165
  // See what kind of values 'k' takes on at runtime
24015
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1166
  const Type* t = k->Value(this);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1167
  assert(t != NULL, "value sanity");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1168
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1169
  // Since I just called 'Value' to compute the set of run-time values
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1170
  // for this Node, and 'Value' is non-local (and therefore expensive) I'll
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1171
  // cache Value.  Later requests for the local phase->type of this Node can
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1172
  // use the cached Value instead of suffering with 'bottom_type'.
24015
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1173
  if (type_or_null(k) != t) {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1174
#ifndef PRODUCT
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1175
    inc_new_values();
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1176
    set_progress();
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1177
#endif
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1178
    set_type(k, t);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1179
    // If k is a TypeNode, capture any more-precise type permanently into Node
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1180
    k->raise_bottom_type(t);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1181
    // Move users of node to worklist
24015
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1182
    add_users_to_worklist(k);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1183
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1184
  // If 'k' computes a constant, replace it with a constant
24015
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1185
  if (t->singleton() && !k->is_Con()) {
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1186
    NOT_PRODUCT(set_progress();)
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1187
    Node* con = makecon(t);     // Make a constant
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1188
    add_users_to_worklist(k);
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1189
    subsume_node(k, con);       // Everybody using k now uses con
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1190
    return con;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1191
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1192
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1193
  // Now check for Identities
24015
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1194
  i = k->Identity(this);      // Look for a nearby replacement
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1195
  if (i != k) {                // Found? Return replacement!
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1196
    NOT_PRODUCT(set_progress();)
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1197
    add_users_to_worklist(k);
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1198
    subsume_node(k, i);       // Everybody using k now uses i
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1199
    return i;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1200
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1201
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1202
  // Global Value Numbering
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1203
  i = hash_find_insert(k);      // Check for pre-existing node
24015
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1204
  if (i && (i != k)) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1205
    // Return the pre-existing node if it isn't dead
24015
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1206
    NOT_PRODUCT(set_progress();)
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1207
    add_users_to_worklist(k);
e54063b2eb53 8039425: Code cleanup: PhaseIterGVN::optimize()
anoll
parents: 23528
diff changeset
  1208
    subsume_node(k, i);       // Everybody using k now uses i
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1209
    return i;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1210
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1211
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1212
  // Return Idealized original
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1213
  return k;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1214
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1215
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1216
//---------------------------------saturate------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1217
const Type* PhaseIterGVN::saturate(const Type* new_type, const Type* old_type,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1218
                                   const Type* limit_type) const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1219
  return new_type->narrow(old_type);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1220
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1221
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1222
//------------------------------remove_globally_dead_node----------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1223
// Kill a globally dead Node.  All uses are also globally dead and are
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1224
// aggressively trimmed.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1225
void PhaseIterGVN::remove_globally_dead_node( Node *dead ) {
13312
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1226
  enum DeleteProgress {
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1227
    PROCESS_INPUTS,
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1228
    PROCESS_OUTPUTS
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1229
  };
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1230
  assert(_stack.is_empty(), "not empty");
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1231
  _stack.push(dead, PROCESS_INPUTS);
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1232
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1233
  while (_stack.is_nonempty()) {
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1234
    dead = _stack.node();
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1235
    uint progress_state = _stack.index();
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1236
    assert(dead != C->root(), "killing root, eh?");
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1237
    assert(!dead->is_top(), "add check for top when pushing");
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1238
    NOT_PRODUCT( set_progress(); )
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1239
    if (progress_state == PROCESS_INPUTS) {
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1240
      // After following inputs, continue to outputs
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1241
      _stack.set_index(PROCESS_OUTPUTS);
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1242
      if (!dead->is_Con()) { // Don't kill cons but uses
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1243
        bool recurse = false;
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1244
        // Remove from hash table
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1245
        _table.hash_delete( dead );
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1246
        // Smash all inputs to 'dead', isolating him completely
16619
a0e531dcc9e9 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 15813
diff changeset
  1247
        for (uint i = 0; i < dead->req(); i++) {
13312
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1248
          Node *in = dead->in(i);
16619
a0e531dcc9e9 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 15813
diff changeset
  1249
          if (in != NULL && in != C->top()) {  // Points to something?
a0e531dcc9e9 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 15813
diff changeset
  1250
            int nrep = dead->replace_edge(in, NULL);  // Kill edges
a0e531dcc9e9 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 15813
diff changeset
  1251
            assert((nrep > 0), "sanity");
a0e531dcc9e9 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 15813
diff changeset
  1252
            if (in->outcnt() == 0) { // Made input go dead?
13312
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1253
              _stack.push(in, PROCESS_INPUTS); // Recursively remove
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1254
              recurse = true;
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1255
            } else if (in->outcnt() == 1 &&
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1256
                       in->has_special_unique_user()) {
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1257
              _worklist.push(in->unique_out());
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1258
            } else if (in->outcnt() <= 2 && dead->is_Phi()) {
16619
a0e531dcc9e9 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 15813
diff changeset
  1259
              if (in->Opcode() == Op_Region) {
13312
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1260
                _worklist.push(in);
16619
a0e531dcc9e9 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 15813
diff changeset
  1261
              } else if (in->is_Store()) {
13312
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1262
                DUIterator_Fast imax, i = in->fast_outs(imax);
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1263
                _worklist.push(in->fast_out(i));
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1264
                i++;
16619
a0e531dcc9e9 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 15813
diff changeset
  1265
                if (in->outcnt() == 2) {
13312
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1266
                  _worklist.push(in->fast_out(i));
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1267
                  i++;
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1268
                }
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1269
                assert(!(i < imax), "sanity");
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1270
              }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1271
            }
15813
6efd4c793e47 8007294: ReduceFieldZeroing doesn't check for dependent load and can lead to incorrect execution
roland
parents: 15618
diff changeset
  1272
            if (ReduceFieldZeroing && dead->is_Load() && i == MemNode::Memory &&
6efd4c793e47 8007294: ReduceFieldZeroing doesn't check for dependent load and can lead to incorrect execution
roland
parents: 15618
diff changeset
  1273
                in->is_Proj() && in->in(0) != NULL && in->in(0)->is_Initialize()) {
6efd4c793e47 8007294: ReduceFieldZeroing doesn't check for dependent load and can lead to incorrect execution
roland
parents: 15618
diff changeset
  1274
              // A Load that directly follows an InitializeNode is
6efd4c793e47 8007294: ReduceFieldZeroing doesn't check for dependent load and can lead to incorrect execution
roland
parents: 15618
diff changeset
  1275
              // going away. The Stores that follow are candidates
6efd4c793e47 8007294: ReduceFieldZeroing doesn't check for dependent load and can lead to incorrect execution
roland
parents: 15618
diff changeset
  1276
              // again to be captured by the InitializeNode.
6efd4c793e47 8007294: ReduceFieldZeroing doesn't check for dependent load and can lead to incorrect execution
roland
parents: 15618
diff changeset
  1277
              for (DUIterator_Fast jmax, j = in->fast_outs(jmax); j < jmax; j++) {
6efd4c793e47 8007294: ReduceFieldZeroing doesn't check for dependent load and can lead to incorrect execution
roland
parents: 15618
diff changeset
  1278
                Node *n = in->fast_out(j);
6efd4c793e47 8007294: ReduceFieldZeroing doesn't check for dependent load and can lead to incorrect execution
roland
parents: 15618
diff changeset
  1279
                if (n->is_Store()) {
6efd4c793e47 8007294: ReduceFieldZeroing doesn't check for dependent load and can lead to incorrect execution
roland
parents: 15618
diff changeset
  1280
                  _worklist.push(n);
6efd4c793e47 8007294: ReduceFieldZeroing doesn't check for dependent load and can lead to incorrect execution
roland
parents: 15618
diff changeset
  1281
                }
6efd4c793e47 8007294: ReduceFieldZeroing doesn't check for dependent load and can lead to incorrect execution
roland
parents: 15618
diff changeset
  1282
              }
6efd4c793e47 8007294: ReduceFieldZeroing doesn't check for dependent load and can lead to incorrect execution
roland
parents: 15618
diff changeset
  1283
            }
16619
a0e531dcc9e9 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 15813
diff changeset
  1284
          } // if (in != NULL && in != C->top())
a0e531dcc9e9 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 15813
diff changeset
  1285
        } // for (uint i = 0; i < dead->req(); i++)
13312
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1286
        if (recurse) {
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1287
          continue;
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1288
        }
16619
a0e531dcc9e9 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 15813
diff changeset
  1289
      } // if (!dead->is_Con())
a0e531dcc9e9 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 15813
diff changeset
  1290
    } // if (progress_state == PROCESS_INPUTS)
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1291
13312
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1292
    // Aggressively kill globally dead uses
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1293
    // (Rather than pushing all the outs at once, we push one at a time,
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1294
    // plus the parent to resume later, because of the indefinite number
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1295
    // of edge deletions per loop trip.)
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1296
    if (dead->outcnt() > 0) {
16619
a0e531dcc9e9 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 15813
diff changeset
  1297
      // Recursively remove output edges
13312
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1298
      _stack.push(dead->raw_out(0), PROCESS_INPUTS);
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1299
    } else {
16619
a0e531dcc9e9 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 15813
diff changeset
  1300
      // Finished disconnecting all input and output edges.
13312
a8e45429e01e 7147464: Java crashed while executing method with over 8k of dneg operations
dlong
parents: 10988
diff changeset
  1301
      _stack.pop();
16619
a0e531dcc9e9 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 15813
diff changeset
  1302
      // Remove dead node from iterative worklist
a0e531dcc9e9 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 15813
diff changeset
  1303
      _worklist.remove(dead);
25913
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 25739
diff changeset
  1304
      C->remove_modified_node(dead);
16619
a0e531dcc9e9 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 15813
diff changeset
  1305
      // Constant node that has no out-edges and has only one in-edge from
a0e531dcc9e9 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 15813
diff changeset
  1306
      // root is usually dead. However, sometimes reshaping walk makes
a0e531dcc9e9 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 15813
diff changeset
  1307
      // it reachable by adding use edges. So, we will NOT count Con nodes
a0e531dcc9e9 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 15813
diff changeset
  1308
      // as dead to be conservative about the dead node count at any
a0e531dcc9e9 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 15813
diff changeset
  1309
      // given time.
a0e531dcc9e9 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 15813
diff changeset
  1310
      if (!dead->is_Con()) {
a0e531dcc9e9 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 15813
diff changeset
  1311
        C->record_dead_node(dead->_idx);
a0e531dcc9e9 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 15813
diff changeset
  1312
      }
a0e531dcc9e9 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 15813
diff changeset
  1313
      if (dead->is_macro()) {
a0e531dcc9e9 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 15813
diff changeset
  1314
        C->remove_macro_node(dead);
a0e531dcc9e9 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 15813
diff changeset
  1315
      }
a0e531dcc9e9 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 15813
diff changeset
  1316
      if (dead->is_expensive()) {
a0e531dcc9e9 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 15813
diff changeset
  1317
        C->remove_expensive_node(dead);
a0e531dcc9e9 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 15813
diff changeset
  1318
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1319
    }
16619
a0e531dcc9e9 8004640: C2 assert failure in memnode.cpp: NULL+offs not RAW address
kvn
parents: 15813
diff changeset
  1320
  } // while (_stack.is_nonempty())
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1321
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1322
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1323
//------------------------------subsume_node-----------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1324
// Remove users from node 'old' and add them to node 'nn'.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1325
void PhaseIterGVN::subsume_node( Node *old, Node *nn ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1326
  assert( old != hash_find(old), "should already been removed" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1327
  assert( old != C->top(), "cannot subsume top node");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1328
  // Copy debug or profile information to the new version:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1329
  C->copy_node_notes_to(nn, old);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1330
  // Move users of node 'old' to node 'nn'
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1331
  for (DUIterator_Last imin, i = old->last_outs(imin); i >= imin; ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1332
    Node* use = old->last_out(i);  // for each use...
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1333
    // use might need re-hashing (but it won't if it's a new node)
25913
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 25739
diff changeset
  1334
    rehash_node_delayed(use);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1335
    // Update use-def info as well
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1336
    // We remove all occurrences of old within use->in,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1337
    // so as to avoid rehashing any node more than once.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1338
    // The hash table probe swamps any outer loop overhead.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1339
    uint num_edges = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1340
    for (uint jmax = use->len(), j = 0; j < jmax; j++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1341
      if (use->in(j) == old) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1342
        use->set_req(j, nn);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1343
        ++num_edges;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1344
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1345
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1346
    i -= num_edges;    // we deleted 1 or more copies of this edge
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1347
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1348
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1349
  // Smash all inputs to 'old', isolating him completely
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24479
diff changeset
  1350
  Node *temp = new Node(1);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1351
  temp->init_req(0,nn);     // Add a use to nn to prevent him from dying
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1352
  remove_dead_node( old );
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1353
  temp->del_req(0);         // Yank bogus edge
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1354
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1355
  if( VerifyIterativeGVN ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1356
    for ( int i = 0; i < _verify_window_size; i++ ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1357
      if ( _verify_window[i] == old )
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1358
        _verify_window[i] = nn;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1359
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1360
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1361
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1362
  _worklist.remove(temp);   // this can be necessary
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1363
  temp->destruct();         // reuse the _idx of this little guy
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1364
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1365
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1366
//------------------------------add_users_to_worklist--------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1367
void PhaseIterGVN::add_users_to_worklist0( Node *n ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1368
  for (DUIterator_Fast imax, i = n->fast_outs(imax); i < imax; i++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1369
    _worklist.push(n->fast_out(i));  // Push on worklist
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1370
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1371
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1372
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1373
void PhaseIterGVN::add_users_to_worklist( Node *n ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1374
  add_users_to_worklist0(n);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1375
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1376
  // Move users of node to worklist
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1377
  for (DUIterator_Fast imax, i = n->fast_outs(imax); i < imax; i++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1378
    Node* use = n->fast_out(i); // Get use
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1379
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1380
    if( use->is_Multi() ||      // Multi-definer?  Push projs on worklist
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1381
        use->is_Store() )       // Enable store/load same address
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1382
      add_users_to_worklist0(use);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1383
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1384
    // If we changed the receiver type to a call, we need to revisit
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1385
    // the Catch following the call.  It's looking for a non-NULL
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1386
    // receiver to know when to enable the regular fall-through path
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1387
    // in addition to the NullPtrException path.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1388
    if (use->is_CallDynamicJava() && n == use->in(TypeFunc::Parms)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1389
      Node* p = use->as_CallDynamicJava()->proj_out(TypeFunc::Control);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1390
      if (p != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1391
        add_users_to_worklist0(p);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1392
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1393
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1394
27708
8a8710cb8fc4 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 26429
diff changeset
  1395
    uint use_op = use->Opcode();
8a8710cb8fc4 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 26429
diff changeset
  1396
    if(use->is_Cmp()) {       // Enable CMP/BOOL optimization
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1397
      add_users_to_worklist(use); // Put Bool on worklist
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1398
      if (use->outcnt() > 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1399
        Node* bol = use->raw_out(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1400
        if (bol->outcnt() > 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1401
          Node* iff = bol->raw_out(0);
27708
8a8710cb8fc4 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 26429
diff changeset
  1402
          if (use_op == Op_CmpI &&
8a8710cb8fc4 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 26429
diff changeset
  1403
              iff->is_CountedLoopEnd()) {
8a8710cb8fc4 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 26429
diff changeset
  1404
            CountedLoopEndNode* cle = iff->as_CountedLoopEnd();
8a8710cb8fc4 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 26429
diff changeset
  1405
            if (cle->limit() == n && cle->phi() != NULL) {
8a8710cb8fc4 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 26429
diff changeset
  1406
              // If an opaque node feeds into the limit condition of a
8a8710cb8fc4 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 26429
diff changeset
  1407
              // CountedLoop, we need to process the Phi node for the
8a8710cb8fc4 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 26429
diff changeset
  1408
              // induction variable when the opaque node is removed:
8a8710cb8fc4 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 26429
diff changeset
  1409
              // the range of values taken by the Phi is now known and
8a8710cb8fc4 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 26429
diff changeset
  1410
              // so its type is also known.
8a8710cb8fc4 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 26429
diff changeset
  1411
              _worklist.push(cle->phi());
8a8710cb8fc4 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 26429
diff changeset
  1412
            }
8a8710cb8fc4 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 26429
diff changeset
  1413
          } else if (iff->outcnt() == 2) {
8a8710cb8fc4 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 26429
diff changeset
  1414
            // Look for the 'is_x2logic' pattern: "x ? : 0 : 1" and put the
8a8710cb8fc4 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 26429
diff changeset
  1415
            // phi merging either 0 or 1 onto the worklist
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1416
            Node* ifproj0 = iff->raw_out(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1417
            Node* ifproj1 = iff->raw_out(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1418
            if (ifproj0->outcnt() > 0 && ifproj1->outcnt() > 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1419
              Node* region0 = ifproj0->raw_out(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1420
              Node* region1 = ifproj1->raw_out(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1421
              if( region0 == region1 )
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1422
                add_users_to_worklist0(region0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1423
            }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1424
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1425
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1426
      }
27708
8a8710cb8fc4 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 26429
diff changeset
  1427
      if (use_op == Op_CmpI) {
8a8710cb8fc4 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 26429
diff changeset
  1428
        Node* in1 = use->in(1);
8a8710cb8fc4 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 26429
diff changeset
  1429
        for (uint i = 0; i < in1->outcnt(); i++) {
8a8710cb8fc4 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 26429
diff changeset
  1430
          if (in1->raw_out(i)->Opcode() == Op_CastII) {
8a8710cb8fc4 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 26429
diff changeset
  1431
            Node* castii = in1->raw_out(i);
8a8710cb8fc4 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 26429
diff changeset
  1432
            if (castii->in(0) != NULL && castii->in(0)->in(0) != NULL && castii->in(0)->in(0)->is_If()) {
8a8710cb8fc4 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 26429
diff changeset
  1433
              Node* ifnode = castii->in(0)->in(0);
27910
8653c71aea40 8066045: opto/node.hpp:355, assert(i < _max) failed: oob: i=1, _max=1
roland
parents: 27708
diff changeset
  1434
              if (ifnode->in(1) != NULL && ifnode->in(1)->is_Bool() && ifnode->in(1)->in(1) == use) {
27708
8a8710cb8fc4 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 26429
diff changeset
  1435
                // Reprocess a CastII node that may depend on an
8a8710cb8fc4 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 26429
diff changeset
  1436
                // opaque node value when the opaque node is
8a8710cb8fc4 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 26429
diff changeset
  1437
                // removed. In case it carries a dependency we can do
8a8710cb8fc4 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 26429
diff changeset
  1438
                // a better job of computing its type.
8a8710cb8fc4 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 26429
diff changeset
  1439
                _worklist.push(castii);
8a8710cb8fc4 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 26429
diff changeset
  1440
              }
8a8710cb8fc4 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 26429
diff changeset
  1441
            }
8a8710cb8fc4 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 26429
diff changeset
  1442
          }
8a8710cb8fc4 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 26429
diff changeset
  1443
        }
8a8710cb8fc4 8054478: C2: Incorrectly compiled char[] array access crashes JVM
roland
parents: 26429
diff changeset
  1444
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1445
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1446
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1447
    // If changed Cast input, check Phi users for simple cycles
238
803c80713999 6674588: (Escape Analysis) Improve Escape Analysis code
kvn
parents: 214
diff changeset
  1448
    if( use->is_ConstraintCast() || use->is_CheckCastPP() ) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1449
      for (DUIterator_Fast i2max, i2 = use->fast_outs(i2max); i2 < i2max; i2++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1450
        Node* u = use->fast_out(i2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1451
        if (u->is_Phi())
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1452
          _worklist.push(u);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1453
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1454
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1455
    // If changed LShift inputs, check RShift users for useless sign-ext
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1456
    if( use_op == Op_LShiftI ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1457
      for (DUIterator_Fast i2max, i2 = use->fast_outs(i2max); i2 < i2max; i2++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1458
        Node* u = use->fast_out(i2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1459
        if (u->Opcode() == Op_RShiftI)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1460
          _worklist.push(u);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1461
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1462
    }
24479
20eb4c752fb5 8042786: Proper fix for 8032566
kvn
parents: 24424
diff changeset
  1463
    // If changed AddI/SubI inputs, check CmpU for range check optimization.
20eb4c752fb5 8042786: Proper fix for 8032566
kvn
parents: 24424
diff changeset
  1464
    if (use_op == Op_AddI || use_op == Op_SubI) {
20eb4c752fb5 8042786: Proper fix for 8032566
kvn
parents: 24424
diff changeset
  1465
      for (DUIterator_Fast i2max, i2 = use->fast_outs(i2max); i2 < i2max; i2++) {
20eb4c752fb5 8042786: Proper fix for 8032566
kvn
parents: 24424
diff changeset
  1466
        Node* u = use->fast_out(i2);
20eb4c752fb5 8042786: Proper fix for 8032566
kvn
parents: 24424
diff changeset
  1467
        if (u->is_Cmp() && (u->Opcode() == Op_CmpU)) {
20eb4c752fb5 8042786: Proper fix for 8032566
kvn
parents: 24424
diff changeset
  1468
          _worklist.push(u);
20eb4c752fb5 8042786: Proper fix for 8032566
kvn
parents: 24424
diff changeset
  1469
        }
20eb4c752fb5 8042786: Proper fix for 8032566
kvn
parents: 24424
diff changeset
  1470
      }
20eb4c752fb5 8042786: Proper fix for 8032566
kvn
parents: 24424
diff changeset
  1471
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1472
    // If changed AddP inputs, check Stores for loop invariant
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1473
    if( use_op == Op_AddP ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1474
      for (DUIterator_Fast i2max, i2 = use->fast_outs(i2max); i2 < i2max; i2++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1475
        Node* u = use->fast_out(i2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1476
        if (u->is_Mem())
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1477
          _worklist.push(u);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1478
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1479
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1480
    // If changed initialization activity, check dependent Stores
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1481
    if (use_op == Op_Allocate || use_op == Op_AllocateArray) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1482
      InitializeNode* init = use->as_Allocate()->initialization();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1483
      if (init != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1484
        Node* imem = init->proj_out(TypeFunc::Memory);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1485
        if (imem != NULL)  add_users_to_worklist0(imem);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1486
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1487
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1488
    if (use_op == Op_Initialize) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1489
      Node* imem = use->as_Initialize()->proj_out(TypeFunc::Memory);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1490
      if (imem != NULL)  add_users_to_worklist0(imem);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1491
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1492
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1493
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1494
21099
46e6bbecd9e5 8024070: C2 needs some form of type speculation
roland
parents: 19717
diff changeset
  1495
/**
46e6bbecd9e5 8024070: C2 needs some form of type speculation
roland
parents: 19717
diff changeset
  1496
 * Remove the speculative part of all types that we know of
46e6bbecd9e5 8024070: C2 needs some form of type speculation
roland
parents: 19717
diff changeset
  1497
 */
46e6bbecd9e5 8024070: C2 needs some form of type speculation
roland
parents: 19717
diff changeset
  1498
void PhaseIterGVN::remove_speculative_types()  {
46e6bbecd9e5 8024070: C2 needs some form of type speculation
roland
parents: 19717
diff changeset
  1499
  assert(UseTypeSpeculation, "speculation is off");
46e6bbecd9e5 8024070: C2 needs some form of type speculation
roland
parents: 19717
diff changeset
  1500
  for (uint i = 0; i < _types.Size(); i++)  {
46e6bbecd9e5 8024070: C2 needs some form of type speculation
roland
parents: 19717
diff changeset
  1501
    const Type* t = _types.fast_lookup(i);
22799
83e58bac7980 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 22234
diff changeset
  1502
    if (t != NULL) {
83e58bac7980 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 22234
diff changeset
  1503
      _types.map(i, t->remove_speculative());
21099
46e6bbecd9e5 8024070: C2 needs some form of type speculation
roland
parents: 19717
diff changeset
  1504
    }
46e6bbecd9e5 8024070: C2 needs some form of type speculation
roland
parents: 19717
diff changeset
  1505
  }
22799
83e58bac7980 8027422: assert(_gvn.type(obj)->higher_equal(tjp)) failed: cast_up is no longer needed
roland
parents: 22234
diff changeset
  1506
  _table.check_no_speculative_types();
21099
46e6bbecd9e5 8024070: C2 needs some form of type speculation
roland
parents: 19717
diff changeset
  1507
}
46e6bbecd9e5 8024070: C2 needs some form of type speculation
roland
parents: 19717
diff changeset
  1508
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1509
//=============================================================================
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1510
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1511
uint PhaseCCP::_total_invokes   = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1512
uint PhaseCCP::_total_constants = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1513
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1514
//------------------------------PhaseCCP---------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1515
// Conditional Constant Propagation, ala Wegman & Zadeck
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1516
PhaseCCP::PhaseCCP( PhaseIterGVN *igvn ) : PhaseIterGVN(igvn) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1517
  NOT_PRODUCT( clear_constants(); )
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1518
  assert( _worklist.size() == 0, "" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1519
  // Clear out _nodes from IterGVN.  Must be clear to transform call.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1520
  _nodes.clear();               // Clear out from IterGVN
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1521
  analyze();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1522
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1523
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1524
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1525
//------------------------------~PhaseCCP--------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1526
PhaseCCP::~PhaseCCP() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1527
  inc_invokes();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1528
  _total_constants += count_constants();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1529
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1530
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1531
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1532
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1533
#ifdef ASSERT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1534
static bool ccp_type_widens(const Type* t, const Type* t0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1535
  assert(t->meet(t0) == t, "Not monotonic");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1536
  switch (t->base() == t0->base() ? t->base() : Type::Top) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1537
  case Type::Int:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1538
    assert(t0->isa_int()->_widen <= t->isa_int()->_widen, "widen increases");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1539
    break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1540
  case Type::Long:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1541
    assert(t0->isa_long()->_widen <= t->isa_long()->_widen, "widen increases");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1542
    break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1543
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1544
  return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1545
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1546
#endif //ASSERT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1547
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1548
//------------------------------analyze----------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1549
void PhaseCCP::analyze() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1550
  // Initialize all types to TOP, optimistic analysis
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1551
  for (int i = C->unique() - 1; i >= 0; i--)  {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1552
    _types.map(i,Type::TOP);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1553
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1554
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1555
  // Push root onto worklist
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1556
  Unique_Node_List worklist;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1557
  worklist.push(C->root());
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1558
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1559
  // Pull from worklist; compute new value; push changes out.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1560
  // This loop is the meat of CCP.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1561
  while( worklist.size() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1562
    Node *n = worklist.pop();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1563
    const Type *t = n->Value(this);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1564
    if (t != type(n)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1565
      assert(ccp_type_widens(t, type(n)), "ccp type must widen");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1566
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1567
      if( TracePhaseCCP ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1568
        t->dump();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1569
        do { tty->print("\t"); } while (tty->position() < 16);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1570
        n->dump();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1571
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1572
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1573
      set_type(n, t);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1574
      for (DUIterator_Fast imax, i = n->fast_outs(imax); i < imax; i++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1575
        Node* m = n->fast_out(i);   // Get user
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1576
        if( m->is_Region() ) {  // New path to Region?  Must recheck Phis too
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1577
          for (DUIterator_Fast i2max, i2 = m->fast_outs(i2max); i2 < i2max; i2++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1578
            Node* p = m->fast_out(i2); // Propagate changes to uses
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1579
            if( p->bottom_type() != type(p) ) // If not already bottomed out
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1580
              worklist.push(p); // Propagate change to user
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1581
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1582
        }
2131
98f9cef66a34 6810672: Comment typos
twisti
parents: 1067
diff changeset
  1583
        // If we changed the receiver type to a call, we need to revisit
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1584
        // the Catch following the call.  It's looking for a non-NULL
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1585
        // receiver to know when to enable the regular fall-through path
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1586
        // in addition to the NullPtrException path
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1587
        if (m->is_Call()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1588
          for (DUIterator_Fast i2max, i2 = m->fast_outs(i2max); i2 < i2max; i2++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1589
            Node* p = m->fast_out(i2);  // Propagate changes to uses
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1590
            if (p->is_Proj() && p->as_Proj()->_con == TypeFunc::Control && p->outcnt() == 1)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1591
              worklist.push(p->unique_out());
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1592
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1593
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1594
        if( m->bottom_type() != type(m) ) // If not already bottomed out
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1595
          worklist.push(m);     // Propagate change to user
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1596
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1597
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1598
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1599
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1600
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1601
//------------------------------do_transform-----------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1602
// Top level driver for the recursive transformer
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1603
void PhaseCCP::do_transform() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1604
  // Correct leaves of new-space Nodes; they point to old-space.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1605
  C->set_root( transform(C->root())->as_Root() );
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1606
  assert( C->top(),  "missing TOP node" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1607
  assert( C->root(), "missing root" );
24009
ec494183af7e 8034216: assert(false) failed: infinite loop in PhaseIterGVN::optimize
anoll
parents: 23528
diff changeset
  1608
ec494183af7e 8034216: assert(false) failed: infinite loop in PhaseIterGVN::optimize
anoll
parents: 23528
diff changeset
  1609
  // Eagerly remove castPP nodes here. CastPP nodes might not be
ec494183af7e 8034216: assert(false) failed: infinite loop in PhaseIterGVN::optimize
anoll
parents: 23528
diff changeset
  1610
  // removed in the subsequent IGVN phase if a node that changes
ec494183af7e 8034216: assert(false) failed: infinite loop in PhaseIterGVN::optimize
anoll
parents: 23528
diff changeset
  1611
  // in(1) of a castPP is processed prior to the castPP node.
ec494183af7e 8034216: assert(false) failed: infinite loop in PhaseIterGVN::optimize
anoll
parents: 23528
diff changeset
  1612
  for (uint i = 0; i < _worklist.size(); i++) {
ec494183af7e 8034216: assert(false) failed: infinite loop in PhaseIterGVN::optimize
anoll
parents: 23528
diff changeset
  1613
    Node* n = _worklist.at(i);
ec494183af7e 8034216: assert(false) failed: infinite loop in PhaseIterGVN::optimize
anoll
parents: 23528
diff changeset
  1614
ec494183af7e 8034216: assert(false) failed: infinite loop in PhaseIterGVN::optimize
anoll
parents: 23528
diff changeset
  1615
    if (n->is_ConstraintCast()) {
ec494183af7e 8034216: assert(false) failed: infinite loop in PhaseIterGVN::optimize
anoll
parents: 23528
diff changeset
  1616
      Node* nn = n->Identity(this);
ec494183af7e 8034216: assert(false) failed: infinite loop in PhaseIterGVN::optimize
anoll
parents: 23528
diff changeset
  1617
      if (nn != n) {
ec494183af7e 8034216: assert(false) failed: infinite loop in PhaseIterGVN::optimize
anoll
parents: 23528
diff changeset
  1618
        replace_node(n, nn);
ec494183af7e 8034216: assert(false) failed: infinite loop in PhaseIterGVN::optimize
anoll
parents: 23528
diff changeset
  1619
        --i;
ec494183af7e 8034216: assert(false) failed: infinite loop in PhaseIterGVN::optimize
anoll
parents: 23528
diff changeset
  1620
      }
ec494183af7e 8034216: assert(false) failed: infinite loop in PhaseIterGVN::optimize
anoll
parents: 23528
diff changeset
  1621
    }
ec494183af7e 8034216: assert(false) failed: infinite loop in PhaseIterGVN::optimize
anoll
parents: 23528
diff changeset
  1622
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1623
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1624
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1625
//------------------------------transform--------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1626
// Given a Node in old-space, clone him into new-space.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1627
// Convert any of his old-space children into new-space children.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1628
Node *PhaseCCP::transform( Node *n ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1629
  Node *new_node = _nodes[n->_idx]; // Check for transformed node
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1630
  if( new_node != NULL )
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1631
    return new_node;                // Been there, done that, return old answer
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1632
  new_node = transform_once(n);     // Check for constant
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1633
  _nodes.map( n->_idx, new_node );  // Flag as having been cloned
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1634
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1635
  // Allocate stack of size _nodes.Size()/2 to avoid frequent realloc
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1636
  GrowableArray <Node *> trstack(C->unique() >> 1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1637
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1638
  trstack.push(new_node);           // Process children of cloned node
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1639
  while ( trstack.is_nonempty() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1640
    Node *clone = trstack.pop();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1641
    uint cnt = clone->req();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1642
    for( uint i = 0; i < cnt; i++ ) {          // For all inputs do
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1643
      Node *input = clone->in(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1644
      if( input != NULL ) {                    // Ignore NULLs
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1645
        Node *new_input = _nodes[input->_idx]; // Check for cloned input node
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1646
        if( new_input == NULL ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1647
          new_input = transform_once(input);   // Check for constant
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1648
          _nodes.map( input->_idx, new_input );// Flag as having been cloned
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1649
          trstack.push(new_input);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1650
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1651
        assert( new_input == clone->in(i), "insanity check");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1652
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1653
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1654
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1655
  return new_node;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1656
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1657
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1658
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1659
//------------------------------transform_once---------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1660
// For PhaseCCP, transformation is IDENTITY unless Node computed a constant.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1661
Node *PhaseCCP::transform_once( Node *n ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1662
  const Type *t = type(n);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1663
  // Constant?  Use constant Node instead
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1664
  if( t->singleton() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1665
    Node *nn = n;               // Default is to return the original constant
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1666
    if( t == Type::TOP ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1667
      // cache my top node on the Compile instance
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1668
      if( C->cached_top_node() == NULL || C->cached_top_node()->in(0) == NULL ) {
25930
eae8b7490d2c 8054033: Remove unused references to Compile*
thartmann
parents: 25913
diff changeset
  1669
        C->set_cached_top_node(ConNode::make(Type::TOP));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1670
        set_type(C->top(), Type::TOP);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1671
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1672
      nn = C->top();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1673
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1674
    if( !n->is_Con() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1675
      if( t != Type::TOP ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1676
        nn = makecon(t);        // ConNode::make(t);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1677
        NOT_PRODUCT( inc_constants(); )
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1678
      } else if( n->is_Region() ) { // Unreachable region
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1679
        // Note: nn == C->top()
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1680
        n->set_req(0, NULL);        // Cut selfreference
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1681
        // Eagerly remove dead phis to avoid phis copies creation.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1682
        for (DUIterator i = n->outs(); n->has_out(i); i++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1683
          Node* m = n->out(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1684
          if( m->is_Phi() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1685
            assert(type(m) == Type::TOP, "Unreachable region should not have live phis.");
5901
c046f8e9c52b 6677629: PhaseIterGVN::subsume_node() should call hash_delete() and add_users_to_worklist()
kvn
parents: 5547
diff changeset
  1686
            replace_node(m, nn);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1687
            --i; // deleted this phi; rescan starting with next position
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1688
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1689
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1690
      }
5901
c046f8e9c52b 6677629: PhaseIterGVN::subsume_node() should call hash_delete() and add_users_to_worklist()
kvn
parents: 5547
diff changeset
  1691
      replace_node(n,nn);       // Update DefUse edges for new constant
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1692
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1693
    return nn;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1694
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1695
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1696
  // If x is a TypeNode, capture any more-precise type permanently into Node
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1697
  if (t != n->bottom_type()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1698
    hash_delete(n);             // changing bottom type may force a rehash
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1699
    n->raise_bottom_type(t);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1700
    _worklist.push(n);          // n re-enters the hash table via the worklist
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1701
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1702
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1703
  // Idealize graph using DU info.  Must clone() into new-space.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1704
  // DU info is generally used to show profitability, progress or safety
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1705
  // (but generally not needed for correctness).
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1706
  Node *nn = n->Ideal_DU_postCCP(this);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1707
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1708
  // TEMPORARY fix to ensure that 2nd GVN pass eliminates NULL checks
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1709
  switch( n->Opcode() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1710
  case Op_FastLock:      // Revisit FastLocks for lock coarsening
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1711
  case Op_If:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1712
  case Op_CountedLoopEnd:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1713
  case Op_Region:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1714
  case Op_Loop:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1715
  case Op_CountedLoop:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1716
  case Op_Conv2B:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1717
  case Op_Opaque1:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1718
  case Op_Opaque2:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1719
    _worklist.push(n);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1720
    break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1721
  default:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1722
    break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1723
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1724
  if( nn ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1725
    _worklist.push(n);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1726
    // Put users of 'n' onto worklist for second igvn transform
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1727
    add_users_to_worklist(n);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1728
    return nn;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1729
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1730
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1731
  return  n;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1732
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1733
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1734
//---------------------------------saturate------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1735
const Type* PhaseCCP::saturate(const Type* new_type, const Type* old_type,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1736
                               const Type* limit_type) const {
4012
579b7bad9983 6885584: A particular class structure causes large allocation spike for jit
never
parents: 3682
diff changeset
  1737
  const Type* wide_type = new_type->widen(old_type, limit_type);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1738
  if (wide_type != new_type) {          // did we widen?
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1739
    // If so, we may have widened beyond the limit type.  Clip it back down.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1740
    new_type = wide_type->filter(limit_type);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1741
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1742
  return new_type;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1743
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1744
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1745
//------------------------------print_statistics-------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1746
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1747
void PhaseCCP::print_statistics() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1748
  tty->print_cr("CCP: %d  constants found: %d", _total_invokes, _total_constants);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1749
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1750
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1751
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1752
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1753
//=============================================================================
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1754
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1755
uint PhasePeephole::_total_peepholes = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1756
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1757
//------------------------------PhasePeephole----------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1758
// Conditional Constant Propagation, ala Wegman & Zadeck
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1759
PhasePeephole::PhasePeephole( PhaseRegAlloc *regalloc, PhaseCFG &cfg )
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1760
  : PhaseTransform(Peephole), _regalloc(regalloc), _cfg(cfg) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1761
  NOT_PRODUCT( clear_peepholes(); )
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1762
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1763
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1764
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1765
//------------------------------~PhasePeephole---------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1766
PhasePeephole::~PhasePeephole() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1767
  _total_peepholes += count_peepholes();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1768
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1769
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1770
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1771
//------------------------------transform--------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1772
Node *PhasePeephole::transform( Node *n ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1773
  ShouldNotCallThis();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1774
  return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1775
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1776
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1777
//------------------------------do_transform-----------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1778
void PhasePeephole::do_transform() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1779
  bool method_name_not_printed = true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1780
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1781
  // Examine each basic block
19330
49d6711171e6 8023003: Cleanup the public interface to PhaseCFG
adlertz
parents: 17383
diff changeset
  1782
  for (uint block_number = 1; block_number < _cfg.number_of_blocks(); ++block_number) {
49d6711171e6 8023003: Cleanup the public interface to PhaseCFG
adlertz
parents: 17383
diff changeset
  1783
    Block* block = _cfg.get_block(block_number);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1784
    bool block_not_printed = true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1785
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1786
    // and each instruction within a block
19717
7819ffdaf0ff 8023691: Create interface for nodes in class Block
adlertz
parents: 19330
diff changeset
  1787
    uint end_index = block->number_of_nodes();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1788
    // block->end_idx() not valid after PhaseRegAlloc
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1789
    for( uint instruction_index = 1; instruction_index < end_index; ++instruction_index ) {
19717
7819ffdaf0ff 8023691: Create interface for nodes in class Block
adlertz
parents: 19330
diff changeset
  1790
      Node     *n = block->get_node(instruction_index);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1791
      if( n->is_Mach() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1792
        MachNode *m = n->as_Mach();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1793
        int deleted_count = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1794
        // check for peephole opportunities
25930
eae8b7490d2c 8054033: Remove unused references to Compile*
thartmann
parents: 25913
diff changeset
  1795
        MachNode *m2 = m->peephole(block, instruction_index, _regalloc, deleted_count);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1796
        if( m2 != NULL ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1797
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1798
          if( PrintOptoPeephole ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1799
            // Print method, first time only
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1800
            if( C->method() && method_name_not_printed ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1801
              C->method()->print_short_name(); tty->cr();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1802
              method_name_not_printed = false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1803
            }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1804
            // Print this block
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1805
            if( Verbose && block_not_printed) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1806
              tty->print_cr("in block");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1807
              block->dump();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1808
              block_not_printed = false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1809
            }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1810
            // Print instructions being deleted
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1811
            for( int i = (deleted_count - 1); i >= 0; --i ) {
19717
7819ffdaf0ff 8023691: Create interface for nodes in class Block
adlertz
parents: 19330
diff changeset
  1812
              block->get_node(instruction_index-i)->as_Mach()->format(_regalloc); tty->cr();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1813
            }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1814
            tty->print_cr("replaced with");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1815
            // Print new instruction
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1816
            m2->format(_regalloc);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1817
            tty->print("\n\n");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1818
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1819
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1820
          // Remove old nodes from basic block and update instruction_index
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1821
          // (old nodes still exist and may have edges pointing to them
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1822
          //  as register allocation info is stored in the allocator using
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1823
          //  the node index to live range mappings.)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1824
          uint safe_instruction_index = (instruction_index - deleted_count);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1825
          for( ; (instruction_index > safe_instruction_index); --instruction_index ) {
19717
7819ffdaf0ff 8023691: Create interface for nodes in class Block
adlertz
parents: 19330
diff changeset
  1826
            block->remove_node( instruction_index );
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1827
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1828
          // install new node after safe_instruction_index
19717
7819ffdaf0ff 8023691: Create interface for nodes in class Block
adlertz
parents: 19330
diff changeset
  1829
          block->insert_node(m2, safe_instruction_index + 1);
7819ffdaf0ff 8023691: Create interface for nodes in class Block
adlertz
parents: 19330
diff changeset
  1830
          end_index = block->number_of_nodes() - 1; // Recompute new block size
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1831
          NOT_PRODUCT( inc_peepholes(); )
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1832
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1833
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1834
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1835
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1836
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1837
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1838
//------------------------------print_statistics-------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1839
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1840
void PhasePeephole::print_statistics() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1841
  tty->print_cr("Peephole: peephole rules applied: %d",  _total_peepholes);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1842
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1843
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1844
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1845
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1846
//=============================================================================
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1847
//------------------------------set_req_X--------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1848
void Node::set_req_X( uint i, Node *n, PhaseIterGVN *igvn ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1849
  assert( is_not_dead(n), "can not use dead node");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1850
  assert( igvn->hash_find(this) != this, "Need to remove from hash before changing edges" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1851
  Node *old = in(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1852
  set_req(i, n);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1853
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1854
  // old goes dead?
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1855
  if( old ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1856
    switch (old->outcnt()) {
3682
42de755d7d6e 6866651: Regression: simple int sum crashes jvm (build 1.6.0_14-b08 and 1.7.0-ea-b59)
cfang
parents: 2131
diff changeset
  1857
    case 0:
42de755d7d6e 6866651: Regression: simple int sum crashes jvm (build 1.6.0_14-b08 and 1.7.0-ea-b59)
cfang
parents: 2131
diff changeset
  1858
      // Put into the worklist to kill later. We do not kill it now because the
42de755d7d6e 6866651: Regression: simple int sum crashes jvm (build 1.6.0_14-b08 and 1.7.0-ea-b59)
cfang
parents: 2131
diff changeset
  1859
      // recursive kill will delete the current node (this) if dead-loop exists
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1860
      if (!old->is_top())
3682
42de755d7d6e 6866651: Regression: simple int sum crashes jvm (build 1.6.0_14-b08 and 1.7.0-ea-b59)
cfang
parents: 2131
diff changeset
  1861
        igvn->_worklist.push( old );
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1862
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1863
    case 1:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1864
      if( old->is_Store() || old->has_special_unique_user() )
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1865
        igvn->add_users_to_worklist( old );
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1866
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1867
    case 2:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1868
      if( old->is_Store() )
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1869
        igvn->add_users_to_worklist( old );
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1870
      if( old->Opcode() == Op_Region )
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1871
        igvn->_worklist.push(old);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1872
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1873
    case 3:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1874
      if( old->Opcode() == Op_Region ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1875
        igvn->_worklist.push(old);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1876
        igvn->add_users_to_worklist( old );
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1877
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1878
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1879
    default:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1880
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1881
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1882
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1883
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1884
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1885
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1886
//-------------------------------replace_by-----------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1887
// Using def-use info, replace one node for another.  Follow the def-use info
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1888
// to all users of the OLD node.  Then make all uses point to the NEW node.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1889
void Node::replace_by(Node *new_node) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1890
  assert(!is_top(), "top node has no DU info");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1891
  for (DUIterator_Last imin, i = last_outs(imin); i >= imin; ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1892
    Node* use = last_out(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1893
    uint uses_found = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1894
    for (uint j = 0; j < use->len(); j++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1895
      if (use->in(j) == this) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1896
        if (j < use->req())
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1897
              use->set_req(j, new_node);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1898
        else  use->set_prec(j, new_node);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1899
        uses_found++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1900
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1901
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1902
    i -= uses_found;    // we deleted 1 or more copies of this edge
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1903
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1904
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1905
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1906
//=============================================================================
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1907
//-----------------------------------------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1908
void Type_Array::grow( uint i ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1909
  if( !_max ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1910
    _max = 1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1911
    _types = (const Type**)_a->Amalloc( _max * sizeof(Type*) );
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1912
    _types[0] = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1913
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1914
  uint old = _max;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1915
  while( i >= _max ) _max <<= 1;        // Double to fit
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1916
  _types = (const Type**)_a->Arealloc( _types, old*sizeof(Type*),_max*sizeof(Type*));
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1917
  memset( &_types[old], 0, (_max-old)*sizeof(Type*) );
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1918
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1919
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1920
//------------------------------dump-------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1921
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1922
void Type_Array::dump() const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1923
  uint max = Size();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1924
  for( uint i = 0; i < max; i++ ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1925
    if( _types[i] != NULL ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1926
      tty->print("  %d\t== ", i); _types[i]->dump(); tty->cr();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1927
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1928
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1929
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1930
#endif