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