hotspot/src/share/vm/opto/ifg.cpp
author bharadwaj
Tue, 27 Nov 2012 17:24:15 -0800
changeset 14623 70c4c1be0a14
parent 13104 657b387034fb
child 16618 0d73b54bf055
permissions -rw-r--r--
7092905: C2: Keep track of the number of dead nodes Summary: keep an (almost) accurate running count of the reachable (live) flow graph nodes. Reviewed-by: kvn, twisti, jrose, vlivanov
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     1
/*
13104
657b387034fb 7119644: Increase superword's vector size up to 256 bits
kvn
parents: 7397
diff changeset
     2
 * Copyright (c) 1998, 2012, 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: 4760
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 4760
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: 4760
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: 5547
diff changeset
    25
#include "precompiled.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    26
#include "compiler/oopMap.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    27
#include "memory/allocation.inline.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    28
#include "opto/addnode.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    29
#include "opto/block.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    30
#include "opto/callnode.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    31
#include "opto/cfgnode.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    32
#include "opto/chaitin.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    33
#include "opto/coalesce.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    34
#include "opto/connode.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    35
#include "opto/indexSet.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    36
#include "opto/machnode.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    37
#include "opto/memnode.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    38
#include "opto/opcodes.hpp"
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    39
489c9b5090e2 Initial load
duke
parents:
diff changeset
    40
#define EXACT_PRESSURE 1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    41
489c9b5090e2 Initial load
duke
parents:
diff changeset
    42
//=============================================================================
489c9b5090e2 Initial load
duke
parents:
diff changeset
    43
//------------------------------IFG--------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
    44
PhaseIFG::PhaseIFG( Arena *arena ) : Phase(Interference_Graph), _arena(arena) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    45
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    46
489c9b5090e2 Initial load
duke
parents:
diff changeset
    47
//------------------------------init-------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
    48
void PhaseIFG::init( uint maxlrg ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    49
  _maxlrg = maxlrg;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    50
  _yanked = new (_arena) VectorSet(_arena);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    51
  _is_square = false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    52
  // Make uninitialized adjacency lists
489c9b5090e2 Initial load
duke
parents:
diff changeset
    53
  _adjs = (IndexSet*)_arena->Amalloc(sizeof(IndexSet)*maxlrg);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    54
  // Also make empty live range structures
489c9b5090e2 Initial load
duke
parents:
diff changeset
    55
  _lrgs = (LRG *)_arena->Amalloc( maxlrg * sizeof(LRG) );
489c9b5090e2 Initial load
duke
parents:
diff changeset
    56
  memset(_lrgs,0,sizeof(LRG)*maxlrg);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    57
  // Init all to empty
489c9b5090e2 Initial load
duke
parents:
diff changeset
    58
  for( uint i = 0; i < maxlrg; i++ ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    59
    _adjs[i].initialize(maxlrg);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    60
    _lrgs[i].Set_All();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    61
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    62
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    63
489c9b5090e2 Initial load
duke
parents:
diff changeset
    64
//------------------------------add--------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
    65
// Add edge between vertices a & b.  These are sorted (triangular matrix),
489c9b5090e2 Initial load
duke
parents:
diff changeset
    66
// then the smaller number is inserted in the larger numbered array.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    67
int PhaseIFG::add_edge( uint a, uint b ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    68
  lrgs(a).invalid_degree();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    69
  lrgs(b).invalid_degree();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    70
  // Sort a and b, so that a is bigger
489c9b5090e2 Initial load
duke
parents:
diff changeset
    71
  assert( !_is_square, "only on triangular" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
    72
  if( a < b ) { uint tmp = a; a = b; b = tmp; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    73
  return _adjs[a].insert( b );
489c9b5090e2 Initial load
duke
parents:
diff changeset
    74
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    75
489c9b5090e2 Initial load
duke
parents:
diff changeset
    76
//------------------------------add_vector-------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
    77
// Add an edge between 'a' and everything in the vector.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    78
void PhaseIFG::add_vector( uint a, IndexSet *vec ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    79
  // IFG is triangular, so do the inserts where 'a' < 'b'.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    80
  assert( !_is_square, "only on triangular" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
    81
  IndexSet *adjs_a = &_adjs[a];
489c9b5090e2 Initial load
duke
parents:
diff changeset
    82
  if( !vec->count() ) return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    83
489c9b5090e2 Initial load
duke
parents:
diff changeset
    84
  IndexSetIterator elements(vec);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    85
  uint neighbor;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    86
  while ((neighbor = elements.next()) != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    87
    add_edge( a, neighbor );
489c9b5090e2 Initial load
duke
parents:
diff changeset
    88
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    89
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    90
489c9b5090e2 Initial load
duke
parents:
diff changeset
    91
//------------------------------test-------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
    92
// Is there an edge between a and b?
489c9b5090e2 Initial load
duke
parents:
diff changeset
    93
int PhaseIFG::test_edge( uint a, uint b ) const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    94
  // Sort a and b, so that a is larger
489c9b5090e2 Initial load
duke
parents:
diff changeset
    95
  assert( !_is_square, "only on triangular" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
    96
  if( a < b ) { uint tmp = a; a = b; b = tmp; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    97
  return _adjs[a].member(b);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    98
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    99
489c9b5090e2 Initial load
duke
parents:
diff changeset
   100
//------------------------------SquareUp---------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   101
// Convert triangular matrix to square matrix
489c9b5090e2 Initial load
duke
parents:
diff changeset
   102
void PhaseIFG::SquareUp() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   103
  assert( !_is_square, "only on triangular" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   104
489c9b5090e2 Initial load
duke
parents:
diff changeset
   105
  // Simple transpose
489c9b5090e2 Initial load
duke
parents:
diff changeset
   106
  for( uint i = 0; i < _maxlrg; i++ ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   107
    IndexSetIterator elements(&_adjs[i]);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   108
    uint datum;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   109
    while ((datum = elements.next()) != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   110
      _adjs[datum].insert( i );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   111
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   112
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   113
  _is_square = true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   114
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   115
489c9b5090e2 Initial load
duke
parents:
diff changeset
   116
//------------------------------Compute_Effective_Degree-----------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   117
// Compute effective degree in bulk
489c9b5090e2 Initial load
duke
parents:
diff changeset
   118
void PhaseIFG::Compute_Effective_Degree() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   119
  assert( _is_square, "only on square" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   120
489c9b5090e2 Initial load
duke
parents:
diff changeset
   121
  for( uint i = 0; i < _maxlrg; i++ )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   122
    lrgs(i).set_degree(effective_degree(i));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   123
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   124
489c9b5090e2 Initial load
duke
parents:
diff changeset
   125
//------------------------------test_edge_sq-----------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   126
int PhaseIFG::test_edge_sq( uint a, uint b ) const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   127
  assert( _is_square, "only on square" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   128
  // Swap, so that 'a' has the lesser count.  Then binary search is on
489c9b5090e2 Initial load
duke
parents:
diff changeset
   129
  // the smaller of a's list and b's list.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   130
  if( neighbor_cnt(a) > neighbor_cnt(b) ) { uint tmp = a; a = b; b = tmp; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   131
  //return _adjs[a].unordered_member(b);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   132
  return _adjs[a].member(b);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   133
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   134
489c9b5090e2 Initial load
duke
parents:
diff changeset
   135
//------------------------------Union------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   136
// Union edges of B into A
489c9b5090e2 Initial load
duke
parents:
diff changeset
   137
void PhaseIFG::Union( uint a, uint b ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   138
  assert( _is_square, "only on square" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   139
  IndexSet *A = &_adjs[a];
489c9b5090e2 Initial load
duke
parents:
diff changeset
   140
  IndexSetIterator b_elements(&_adjs[b]);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   141
  uint datum;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   142
  while ((datum = b_elements.next()) != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   143
    if(A->insert(datum)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   144
      _adjs[datum].insert(a);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   145
      lrgs(a).invalid_degree();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   146
      lrgs(datum).invalid_degree();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   147
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   148
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   149
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   150
489c9b5090e2 Initial load
duke
parents:
diff changeset
   151
//------------------------------remove_node------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   152
// Yank a Node and all connected edges from the IFG.  Return a
489c9b5090e2 Initial load
duke
parents:
diff changeset
   153
// list of neighbors (edges) yanked.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   154
IndexSet *PhaseIFG::remove_node( uint a ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   155
  assert( _is_square, "only on square" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   156
  assert( !_yanked->test(a), "" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   157
  _yanked->set(a);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   158
489c9b5090e2 Initial load
duke
parents:
diff changeset
   159
  // I remove the LRG from all neighbors.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   160
  IndexSetIterator elements(&_adjs[a]);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   161
  LRG &lrg_a = lrgs(a);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   162
  uint datum;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   163
  while ((datum = elements.next()) != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   164
    _adjs[datum].remove(a);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   165
    lrgs(datum).inc_degree( -lrg_a.compute_degree(lrgs(datum)) );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   166
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   167
  return neighbors(a);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   168
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   169
489c9b5090e2 Initial load
duke
parents:
diff changeset
   170
//------------------------------re_insert--------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   171
// Re-insert a yanked Node.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   172
void PhaseIFG::re_insert( uint a ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   173
  assert( _is_square, "only on square" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   174
  assert( _yanked->test(a), "" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   175
  (*_yanked) >>= a;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   176
489c9b5090e2 Initial load
duke
parents:
diff changeset
   177
  IndexSetIterator elements(&_adjs[a]);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   178
  uint datum;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   179
  while ((datum = elements.next()) != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   180
    _adjs[datum].insert(a);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   181
    lrgs(datum).invalid_degree();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   182
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   183
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   184
489c9b5090e2 Initial load
duke
parents:
diff changeset
   185
//------------------------------compute_degree---------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   186
// Compute the degree between 2 live ranges.  If both live ranges are
489c9b5090e2 Initial load
duke
parents:
diff changeset
   187
// aligned-adjacent powers-of-2 then we use the MAX size.  If either is
489c9b5090e2 Initial load
duke
parents:
diff changeset
   188
// mis-aligned (or for Fat-Projections, not-adjacent) then we have to
489c9b5090e2 Initial load
duke
parents:
diff changeset
   189
// MULTIPLY the sizes.  Inspect Brigg's thesis on register pairs to see why
489c9b5090e2 Initial load
duke
parents:
diff changeset
   190
// this is so.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   191
int LRG::compute_degree( LRG &l ) const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   192
  int tmp;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   193
  int num_regs = _num_regs;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   194
  int nregs = l.num_regs();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   195
  tmp =  (_fat_proj || l._fat_proj)     // either is a fat-proj?
489c9b5090e2 Initial load
duke
parents:
diff changeset
   196
    ? (num_regs * nregs)                // then use product
489c9b5090e2 Initial load
duke
parents:
diff changeset
   197
    : MAX2(num_regs,nregs);             // else use max
489c9b5090e2 Initial load
duke
parents:
diff changeset
   198
  return tmp;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   199
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   200
489c9b5090e2 Initial load
duke
parents:
diff changeset
   201
//------------------------------effective_degree-------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   202
// Compute effective degree for this live range.  If both live ranges are
489c9b5090e2 Initial load
duke
parents:
diff changeset
   203
// aligned-adjacent powers-of-2 then we use the MAX size.  If either is
489c9b5090e2 Initial load
duke
parents:
diff changeset
   204
// mis-aligned (or for Fat-Projections, not-adjacent) then we have to
489c9b5090e2 Initial load
duke
parents:
diff changeset
   205
// MULTIPLY the sizes.  Inspect Brigg's thesis on register pairs to see why
489c9b5090e2 Initial load
duke
parents:
diff changeset
   206
// this is so.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   207
int PhaseIFG::effective_degree( uint lidx ) const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   208
  int eff = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   209
  int num_regs = lrgs(lidx).num_regs();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   210
  int fat_proj = lrgs(lidx)._fat_proj;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   211
  IndexSet *s = neighbors(lidx);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   212
  IndexSetIterator elements(s);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   213
  uint nidx;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   214
  while((nidx = elements.next()) != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   215
    LRG &lrgn = lrgs(nidx);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   216
    int nregs = lrgn.num_regs();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   217
    eff += (fat_proj || lrgn._fat_proj) // either is a fat-proj?
489c9b5090e2 Initial load
duke
parents:
diff changeset
   218
      ? (num_regs * nregs)              // then use product
489c9b5090e2 Initial load
duke
parents:
diff changeset
   219
      : MAX2(num_regs,nregs);           // else use max
489c9b5090e2 Initial load
duke
parents:
diff changeset
   220
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   221
  return eff;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   222
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   223
489c9b5090e2 Initial load
duke
parents:
diff changeset
   224
489c9b5090e2 Initial load
duke
parents:
diff changeset
   225
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   226
//------------------------------dump-------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   227
void PhaseIFG::dump() const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   228
  tty->print_cr("-- Interference Graph --%s--",
489c9b5090e2 Initial load
duke
parents:
diff changeset
   229
                _is_square ? "square" : "triangular" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   230
  if( _is_square ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   231
    for( uint i = 0; i < _maxlrg; i++ ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   232
      tty->print( (*_yanked)[i] ? "XX " : "  ");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   233
      tty->print("L%d: { ",i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   234
      IndexSetIterator elements(&_adjs[i]);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   235
      uint datum;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   236
      while ((datum = elements.next()) != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   237
        tty->print("L%d ", datum);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   238
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   239
      tty->print_cr("}");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   240
489c9b5090e2 Initial load
duke
parents:
diff changeset
   241
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   242
    return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   243
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   244
489c9b5090e2 Initial load
duke
parents:
diff changeset
   245
  // Triangular
489c9b5090e2 Initial load
duke
parents:
diff changeset
   246
  for( uint i = 0; i < _maxlrg; i++ ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   247
    uint j;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   248
    tty->print( (*_yanked)[i] ? "XX " : "  ");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   249
    tty->print("L%d: { ",i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   250
    for( j = _maxlrg; j > i; j-- )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   251
      if( test_edge(j - 1,i) ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   252
        tty->print("L%d ",j - 1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   253
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   254
    tty->print("| ");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   255
    IndexSetIterator elements(&_adjs[i]);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   256
    uint datum;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   257
    while ((datum = elements.next()) != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   258
      tty->print("L%d ", datum);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   259
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   260
    tty->print("}\n");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   261
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   262
  tty->print("\n");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   263
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   264
489c9b5090e2 Initial load
duke
parents:
diff changeset
   265
//------------------------------stats------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   266
void PhaseIFG::stats() const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   267
  ResourceMark rm;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   268
  int *h_cnt = NEW_RESOURCE_ARRAY(int,_maxlrg*2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   269
  memset( h_cnt, 0, sizeof(int)*_maxlrg*2 );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   270
  uint i;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   271
  for( i = 0; i < _maxlrg; i++ ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   272
    h_cnt[neighbor_cnt(i)]++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   273
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   274
  tty->print_cr("--Histogram of counts--");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   275
  for( i = 0; i < _maxlrg*2; i++ )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   276
    if( h_cnt[i] )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   277
      tty->print("%d/%d ",i,h_cnt[i]);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   278
  tty->print_cr("");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   279
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   280
489c9b5090e2 Initial load
duke
parents:
diff changeset
   281
//------------------------------verify-----------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   282
void PhaseIFG::verify( const PhaseChaitin *pc ) const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   283
  // IFG is square, sorted and no need for Find
489c9b5090e2 Initial load
duke
parents:
diff changeset
   284
  for( uint i = 0; i < _maxlrg; i++ ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   285
    assert(!((*_yanked)[i]) || !neighbor_cnt(i), "Is removed completely" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   286
    IndexSet *set = &_adjs[i];
489c9b5090e2 Initial load
duke
parents:
diff changeset
   287
    IndexSetIterator elements(set);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   288
    uint idx;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   289
    uint last = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   290
    while ((idx = elements.next()) != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   291
      assert( idx != i, "Must have empty diagonal");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   292
      assert( pc->Find_const(idx) == idx, "Must not need Find" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   293
      assert( _adjs[idx].member(i), "IFG not square" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   294
      assert( !(*_yanked)[idx], "No yanked neighbors" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   295
      assert( last < idx, "not sorted increasing");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   296
      last = idx;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   297
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   298
    assert( !lrgs(i)._degree_valid ||
489c9b5090e2 Initial load
duke
parents:
diff changeset
   299
            effective_degree(i) == lrgs(i).degree(), "degree is valid but wrong" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   300
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   301
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   302
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   303
489c9b5090e2 Initial load
duke
parents:
diff changeset
   304
//------------------------------interfere_with_live----------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   305
// Interfere this register with everything currently live.  Use the RegMasks
489c9b5090e2 Initial load
duke
parents:
diff changeset
   306
// to trim the set of possible interferences. Return a count of register-only
2131
98f9cef66a34 6810672: Comment typos
twisti
parents: 2030
diff changeset
   307
// interferences as an estimate of register pressure.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   308
void PhaseChaitin::interfere_with_live( uint r, IndexSet *liveout ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   309
  uint retval = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   310
  // Interfere with everything live.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   311
  const RegMask &rm = lrgs(r).mask();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   312
  // Check for interference by checking overlap of regmasks.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   313
  // Only interfere if acceptable register masks overlap.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   314
  IndexSetIterator elements(liveout);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   315
  uint l;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   316
  while( (l = elements.next()) != 0 )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   317
    if( rm.overlap( lrgs(l).mask() ) )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   318
      _ifg->add_edge( r, l );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   319
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   320
489c9b5090e2 Initial load
duke
parents:
diff changeset
   321
//------------------------------build_ifg_virtual------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   322
// Actually build the interference graph.  Uses virtual registers only, no
489c9b5090e2 Initial load
duke
parents:
diff changeset
   323
// physical register masks.  This allows me to be very aggressive when
489c9b5090e2 Initial load
duke
parents:
diff changeset
   324
// coalescing copies.  Some of this aggressiveness will have to be undone
489c9b5090e2 Initial load
duke
parents:
diff changeset
   325
// later, but I'd rather get all the copies I can now (since unremoved copies
489c9b5090e2 Initial load
duke
parents:
diff changeset
   326
// at this point can end up in bad places).  Copies I re-insert later I have
489c9b5090e2 Initial load
duke
parents:
diff changeset
   327
// more opportunity to insert them in low-frequency locations.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   328
void PhaseChaitin::build_ifg_virtual( ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   329
489c9b5090e2 Initial load
duke
parents:
diff changeset
   330
  // For all blocks (in any order) do...
489c9b5090e2 Initial load
duke
parents:
diff changeset
   331
  for( uint i=0; i<_cfg._num_blocks; i++ ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   332
    Block *b = _cfg._blocks[i];
489c9b5090e2 Initial load
duke
parents:
diff changeset
   333
    IndexSet *liveout = _live->live(b);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   334
489c9b5090e2 Initial load
duke
parents:
diff changeset
   335
    // The IFG is built by a single reverse pass over each basic block.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   336
    // Starting with the known live-out set, we remove things that get
489c9b5090e2 Initial load
duke
parents:
diff changeset
   337
    // defined and add things that become live (essentially executing one
489c9b5090e2 Initial load
duke
parents:
diff changeset
   338
    // pass of a standard LIVE analysis). Just before a Node defines a value
489c9b5090e2 Initial load
duke
parents:
diff changeset
   339
    // (and removes it from the live-ness set) that value is certainly live.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   340
    // The defined value interferes with everything currently live.  The
489c9b5090e2 Initial load
duke
parents:
diff changeset
   341
    // value is then removed from the live-ness set and it's inputs are
489c9b5090e2 Initial load
duke
parents:
diff changeset
   342
    // added to the live-ness set.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   343
    for( uint j = b->end_idx() + 1; j > 1; j-- ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   344
      Node *n = b->_nodes[j-1];
489c9b5090e2 Initial load
duke
parents:
diff changeset
   345
489c9b5090e2 Initial load
duke
parents:
diff changeset
   346
      // Get value being defined
489c9b5090e2 Initial load
duke
parents:
diff changeset
   347
      uint r = n2lidx(n);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   348
489c9b5090e2 Initial load
duke
parents:
diff changeset
   349
      // Some special values do not allocate
489c9b5090e2 Initial load
duke
parents:
diff changeset
   350
      if( r ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   351
489c9b5090e2 Initial load
duke
parents:
diff changeset
   352
        // Remove from live-out set
489c9b5090e2 Initial load
duke
parents:
diff changeset
   353
        liveout->remove(r);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   354
489c9b5090e2 Initial load
duke
parents:
diff changeset
   355
        // Copies do not define a new value and so do not interfere.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   356
        // Remove the copies source from the liveout set before interfering.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   357
        uint idx = n->is_Copy();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   358
        if( idx ) liveout->remove( n2lidx(n->in(idx)) );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   359
489c9b5090e2 Initial load
duke
parents:
diff changeset
   360
        // Interfere with everything live
489c9b5090e2 Initial load
duke
parents:
diff changeset
   361
        interfere_with_live( r, liveout );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   362
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   363
489c9b5090e2 Initial load
duke
parents:
diff changeset
   364
      // Make all inputs live
489c9b5090e2 Initial load
duke
parents:
diff changeset
   365
      if( !n->is_Phi() ) {      // Phi function uses come from prior block
489c9b5090e2 Initial load
duke
parents:
diff changeset
   366
        for( uint k = 1; k < n->req(); k++ )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   367
          liveout->insert( n2lidx(n->in(k)) );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   368
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   369
489c9b5090e2 Initial load
duke
parents:
diff changeset
   370
      // 2-address instructions always have the defined value live
489c9b5090e2 Initial load
duke
parents:
diff changeset
   371
      // on entry to the instruction, even though it is being defined
489c9b5090e2 Initial load
duke
parents:
diff changeset
   372
      // by the instruction.  We pretend a virtual copy sits just prior
489c9b5090e2 Initial load
duke
parents:
diff changeset
   373
      // to the instruction and kills the src-def'd register.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   374
      // In other words, for 2-address instructions the defined value
489c9b5090e2 Initial load
duke
parents:
diff changeset
   375
      // interferes with all inputs.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   376
      uint idx;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   377
      if( n->is_Mach() && (idx = n->as_Mach()->two_adr()) ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   378
        const MachNode *mach = n->as_Mach();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   379
        // Sometimes my 2-address ADDs are commuted in a bad way.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   380
        // We generally want the USE-DEF register to refer to the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   381
        // loop-varying quantity, to avoid a copy.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   382
        uint op = mach->ideal_Opcode();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   383
        // Check that mach->num_opnds() == 3 to ensure instruction is
489c9b5090e2 Initial load
duke
parents:
diff changeset
   384
        // not subsuming constants, effectively excludes addI_cin_imm
489c9b5090e2 Initial load
duke
parents:
diff changeset
   385
        // Can NOT swap for instructions like addI_cin_imm since it
489c9b5090e2 Initial load
duke
parents:
diff changeset
   386
        // is adding zero to yhi + carry and the second ideal-input
489c9b5090e2 Initial load
duke
parents:
diff changeset
   387
        // points to the result of adding low-halves.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   388
        // Checking req() and num_opnds() does NOT distinguish addI_cout from addI_cout_imm
489c9b5090e2 Initial load
duke
parents:
diff changeset
   389
        if( (op == Op_AddI && mach->req() == 3 && mach->num_opnds() == 3) &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
   390
            n->in(1)->bottom_type()->base() == Type::Int &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
   391
            // See if the ADD is involved in a tight data loop the wrong way
489c9b5090e2 Initial load
duke
parents:
diff changeset
   392
            n->in(2)->is_Phi() &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
   393
            n->in(2)->in(2) == n ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   394
          Node *tmp = n->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   395
          n->set_req( 1, n->in(2) );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   396
          n->set_req( 2, tmp );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   397
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   398
        // Defined value interferes with all inputs
489c9b5090e2 Initial load
duke
parents:
diff changeset
   399
        uint lidx = n2lidx(n->in(idx));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   400
        for( uint k = 1; k < n->req(); k++ ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   401
          uint kidx = n2lidx(n->in(k));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   402
          if( kidx != lidx )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   403
            _ifg->add_edge( r, kidx );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   404
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   405
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   406
    } // End of forall instructions in block
489c9b5090e2 Initial load
duke
parents:
diff changeset
   407
  } // End of forall blocks
489c9b5090e2 Initial load
duke
parents:
diff changeset
   408
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   409
489c9b5090e2 Initial load
duke
parents:
diff changeset
   410
//------------------------------count_int_pressure-----------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   411
uint PhaseChaitin::count_int_pressure( IndexSet *liveout ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   412
  IndexSetIterator elements(liveout);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   413
  uint lidx;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   414
  uint cnt = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   415
  while ((lidx = elements.next()) != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   416
    if( lrgs(lidx).mask().is_UP() &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
   417
        lrgs(lidx).mask_size() &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
   418
        !lrgs(lidx)._is_float &&
13104
657b387034fb 7119644: Increase superword's vector size up to 256 bits
kvn
parents: 7397
diff changeset
   419
        !lrgs(lidx)._is_vector &&
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   420
        lrgs(lidx).mask().overlap(*Matcher::idealreg2regmask[Op_RegI]) )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   421
      cnt += lrgs(lidx).reg_pressure();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   422
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   423
  return cnt;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   424
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   425
489c9b5090e2 Initial load
duke
parents:
diff changeset
   426
//------------------------------count_float_pressure---------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   427
uint PhaseChaitin::count_float_pressure( IndexSet *liveout ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   428
  IndexSetIterator elements(liveout);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   429
  uint lidx;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   430
  uint cnt = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   431
  while ((lidx = elements.next()) != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   432
    if( lrgs(lidx).mask().is_UP() &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
   433
        lrgs(lidx).mask_size() &&
13104
657b387034fb 7119644: Increase superword's vector size up to 256 bits
kvn
parents: 7397
diff changeset
   434
        (lrgs(lidx)._is_float || lrgs(lidx)._is_vector))
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   435
      cnt += lrgs(lidx).reg_pressure();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   436
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   437
  return cnt;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   438
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   439
489c9b5090e2 Initial load
duke
parents:
diff changeset
   440
//------------------------------lower_pressure---------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   441
// Adjust register pressure down by 1.  Capture last hi-to-low transition,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   442
static void lower_pressure( LRG *lrg, uint where, Block *b, uint *pressure, uint *hrp_index ) {
13104
657b387034fb 7119644: Increase superword's vector size up to 256 bits
kvn
parents: 7397
diff changeset
   443
  if (lrg->mask().is_UP() && lrg->mask_size()) {
657b387034fb 7119644: Increase superword's vector size up to 256 bits
kvn
parents: 7397
diff changeset
   444
    if (lrg->_is_float || lrg->_is_vector) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   445
      pressure[1] -= lrg->reg_pressure();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   446
      if( pressure[1] == (uint)FLOATPRESSURE ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   447
        hrp_index[1] = where;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   448
#ifdef EXACT_PRESSURE
489c9b5090e2 Initial load
duke
parents:
diff changeset
   449
      if( pressure[1] > b->_freg_pressure )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   450
        b->_freg_pressure = pressure[1]+1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   451
#else
489c9b5090e2 Initial load
duke
parents:
diff changeset
   452
        b->_freg_pressure = (uint)FLOATPRESSURE+1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   453
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   454
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   455
    } else if( lrg->mask().overlap(*Matcher::idealreg2regmask[Op_RegI]) ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   456
      pressure[0] -= lrg->reg_pressure();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   457
      if( pressure[0] == (uint)INTPRESSURE   ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   458
        hrp_index[0] = where;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   459
#ifdef EXACT_PRESSURE
489c9b5090e2 Initial load
duke
parents:
diff changeset
   460
      if( pressure[0] > b->_reg_pressure )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   461
        b->_reg_pressure = pressure[0]+1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   462
#else
489c9b5090e2 Initial load
duke
parents:
diff changeset
   463
        b->_reg_pressure = (uint)INTPRESSURE+1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   464
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   465
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   466
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   467
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   468
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   469
489c9b5090e2 Initial load
duke
parents:
diff changeset
   470
//------------------------------build_ifg_physical-----------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   471
// Build the interference graph using physical registers when available.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   472
// That is, if 2 live ranges are simultaneously alive but in their acceptable
489c9b5090e2 Initial load
duke
parents:
diff changeset
   473
// register sets do not overlap, then they do not interfere.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   474
uint PhaseChaitin::build_ifg_physical( ResourceArea *a ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   475
  NOT_PRODUCT( Compile::TracePhase t3("buildIFG", &_t_buildIFGphysical, TimeCompiler); )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   476
489c9b5090e2 Initial load
duke
parents:
diff changeset
   477
  uint spill_reg = LRG::SPILL_REG;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   478
  uint must_spill = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   479
489c9b5090e2 Initial load
duke
parents:
diff changeset
   480
  // For all blocks (in any order) do...
489c9b5090e2 Initial load
duke
parents:
diff changeset
   481
  for( uint i = 0; i < _cfg._num_blocks; i++ ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   482
    Block *b = _cfg._blocks[i];
489c9b5090e2 Initial load
duke
parents:
diff changeset
   483
    // Clone (rather than smash in place) the liveout info, so it is alive
489c9b5090e2 Initial load
duke
parents:
diff changeset
   484
    // for the "collect_gc_info" phase later.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   485
    IndexSet liveout(_live->live(b));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   486
    uint last_inst = b->end_idx();
2030
39d55e4534b4 6791852: assert(b->_nodes[insidx] == n,"got insidx set incorrectly")
kvn
parents: 1412
diff changeset
   487
    // Compute first nonphi node index
39d55e4534b4 6791852: assert(b->_nodes[insidx] == n,"got insidx set incorrectly")
kvn
parents: 1412
diff changeset
   488
    uint first_inst;
39d55e4534b4 6791852: assert(b->_nodes[insidx] == n,"got insidx set incorrectly")
kvn
parents: 1412
diff changeset
   489
    for( first_inst = 1; first_inst < last_inst; first_inst++ )
39d55e4534b4 6791852: assert(b->_nodes[insidx] == n,"got insidx set incorrectly")
kvn
parents: 1412
diff changeset
   490
      if( !b->_nodes[first_inst]->is_Phi() )
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   491
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   492
2030
39d55e4534b4 6791852: assert(b->_nodes[insidx] == n,"got insidx set incorrectly")
kvn
parents: 1412
diff changeset
   493
    // Spills could be inserted before CreateEx node which should be
39d55e4534b4 6791852: assert(b->_nodes[insidx] == n,"got insidx set incorrectly")
kvn
parents: 1412
diff changeset
   494
    // first instruction in block after Phis. Move CreateEx up.
39d55e4534b4 6791852: assert(b->_nodes[insidx] == n,"got insidx set incorrectly")
kvn
parents: 1412
diff changeset
   495
    for( uint insidx = first_inst; insidx < last_inst; insidx++ ) {
39d55e4534b4 6791852: assert(b->_nodes[insidx] == n,"got insidx set incorrectly")
kvn
parents: 1412
diff changeset
   496
      Node *ex = b->_nodes[insidx];
39d55e4534b4 6791852: assert(b->_nodes[insidx] == n,"got insidx set incorrectly")
kvn
parents: 1412
diff changeset
   497
      if( ex->is_SpillCopy() ) continue;
39d55e4534b4 6791852: assert(b->_nodes[insidx] == n,"got insidx set incorrectly")
kvn
parents: 1412
diff changeset
   498
      if( insidx > first_inst && ex->is_Mach() &&
39d55e4534b4 6791852: assert(b->_nodes[insidx] == n,"got insidx set incorrectly")
kvn
parents: 1412
diff changeset
   499
          ex->as_Mach()->ideal_Opcode() == Op_CreateEx ) {
39d55e4534b4 6791852: assert(b->_nodes[insidx] == n,"got insidx set incorrectly")
kvn
parents: 1412
diff changeset
   500
        // If the CreateEx isn't above all the MachSpillCopies
39d55e4534b4 6791852: assert(b->_nodes[insidx] == n,"got insidx set incorrectly")
kvn
parents: 1412
diff changeset
   501
        // then move it to the top.
39d55e4534b4 6791852: assert(b->_nodes[insidx] == n,"got insidx set incorrectly")
kvn
parents: 1412
diff changeset
   502
        b->_nodes.remove(insidx);
39d55e4534b4 6791852: assert(b->_nodes[insidx] == n,"got insidx set incorrectly")
kvn
parents: 1412
diff changeset
   503
        b->_nodes.insert(first_inst, ex);
39d55e4534b4 6791852: assert(b->_nodes[insidx] == n,"got insidx set incorrectly")
kvn
parents: 1412
diff changeset
   504
      }
39d55e4534b4 6791852: assert(b->_nodes[insidx] == n,"got insidx set incorrectly")
kvn
parents: 1412
diff changeset
   505
      // Stop once a CreateEx or any other node is found
39d55e4534b4 6791852: assert(b->_nodes[insidx] == n,"got insidx set incorrectly")
kvn
parents: 1412
diff changeset
   506
      break;
39d55e4534b4 6791852: assert(b->_nodes[insidx] == n,"got insidx set incorrectly")
kvn
parents: 1412
diff changeset
   507
    }
39d55e4534b4 6791852: assert(b->_nodes[insidx] == n,"got insidx set incorrectly")
kvn
parents: 1412
diff changeset
   508
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   509
    // Reset block's register pressure values for each ifg construction
489c9b5090e2 Initial load
duke
parents:
diff changeset
   510
    uint pressure[2], hrp_index[2];
489c9b5090e2 Initial load
duke
parents:
diff changeset
   511
    pressure[0] = pressure[1] = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   512
    hrp_index[0] = hrp_index[1] = last_inst+1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   513
    b->_reg_pressure = b->_freg_pressure = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   514
    // Liveout things are presumed live for the whole block.  We accumulate
489c9b5090e2 Initial load
duke
parents:
diff changeset
   515
    // 'area' accordingly.  If they get killed in the block, we'll subtract
489c9b5090e2 Initial load
duke
parents:
diff changeset
   516
    // the unused part of the block from the area.
2030
39d55e4534b4 6791852: assert(b->_nodes[insidx] == n,"got insidx set incorrectly")
kvn
parents: 1412
diff changeset
   517
    int inst_count = last_inst - first_inst;
1401
e5fdc8521d1f 6750588: assert(lrg._area >= 0,"negative spill area") running NSK stmp0101 test
rasbold
parents: 1057
diff changeset
   518
    double cost = (inst_count <= 0) ? 0.0 : b->_freq * double(inst_count);
e5fdc8521d1f 6750588: assert(lrg._area >= 0,"negative spill area") running NSK stmp0101 test
rasbold
parents: 1057
diff changeset
   519
    assert(!(cost < 0.0), "negative spill cost" );
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   520
    IndexSetIterator elements(&liveout);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   521
    uint lidx;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   522
    while ((lidx = elements.next()) != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   523
      LRG &lrg = lrgs(lidx);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   524
      lrg._area += cost;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   525
      // Compute initial register pressure
13104
657b387034fb 7119644: Increase superword's vector size up to 256 bits
kvn
parents: 7397
diff changeset
   526
      if (lrg.mask().is_UP() && lrg.mask_size()) {
657b387034fb 7119644: Increase superword's vector size up to 256 bits
kvn
parents: 7397
diff changeset
   527
        if (lrg._is_float || lrg._is_vector) {   // Count float pressure
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   528
          pressure[1] += lrg.reg_pressure();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   529
#ifdef EXACT_PRESSURE
489c9b5090e2 Initial load
duke
parents:
diff changeset
   530
          if( pressure[1] > b->_freg_pressure )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   531
            b->_freg_pressure = pressure[1];
489c9b5090e2 Initial load
duke
parents:
diff changeset
   532
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   533
          // Count int pressure, but do not count the SP, flags
489c9b5090e2 Initial load
duke
parents:
diff changeset
   534
        } else if( lrgs(lidx).mask().overlap(*Matcher::idealreg2regmask[Op_RegI]) ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   535
          pressure[0] += lrg.reg_pressure();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   536
#ifdef EXACT_PRESSURE
489c9b5090e2 Initial load
duke
parents:
diff changeset
   537
          if( pressure[0] > b->_reg_pressure )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   538
            b->_reg_pressure = pressure[0];
489c9b5090e2 Initial load
duke
parents:
diff changeset
   539
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   540
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   541
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   542
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   543
    assert( pressure[0] == count_int_pressure  (&liveout), "" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   544
    assert( pressure[1] == count_float_pressure(&liveout), "" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   545
489c9b5090e2 Initial load
duke
parents:
diff changeset
   546
    // The IFG is built by a single reverse pass over each basic block.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   547
    // Starting with the known live-out set, we remove things that get
489c9b5090e2 Initial load
duke
parents:
diff changeset
   548
    // defined and add things that become live (essentially executing one
489c9b5090e2 Initial load
duke
parents:
diff changeset
   549
    // pass of a standard LIVE analysis).  Just before a Node defines a value
489c9b5090e2 Initial load
duke
parents:
diff changeset
   550
    // (and removes it from the live-ness set) that value is certainly live.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   551
    // The defined value interferes with everything currently live.  The
489c9b5090e2 Initial load
duke
parents:
diff changeset
   552
    // value is then removed from the live-ness set and it's inputs are added
489c9b5090e2 Initial load
duke
parents:
diff changeset
   553
    // to the live-ness set.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   554
    uint j;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   555
    for( j = last_inst + 1; j > 1; j-- ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   556
      Node *n = b->_nodes[j - 1];
489c9b5090e2 Initial load
duke
parents:
diff changeset
   557
489c9b5090e2 Initial load
duke
parents:
diff changeset
   558
      // Get value being defined
489c9b5090e2 Initial load
duke
parents:
diff changeset
   559
      uint r = n2lidx(n);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   560
489c9b5090e2 Initial load
duke
parents:
diff changeset
   561
      // Some special values do not allocate
489c9b5090e2 Initial load
duke
parents:
diff changeset
   562
      if( r ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   563
        // A DEF normally costs block frequency; rematerialized values are
489c9b5090e2 Initial load
duke
parents:
diff changeset
   564
        // removed from the DEF sight, so LOWER costs here.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   565
        lrgs(r)._cost += n->rematerialize() ? 0 : b->_freq;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   566
489c9b5090e2 Initial load
duke
parents:
diff changeset
   567
        // If it is not live, then this instruction is dead.  Probably caused
489c9b5090e2 Initial load
duke
parents:
diff changeset
   568
        // by spilling and rematerialization.  Who cares why, yank this baby.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   569
        if( !liveout.member(r) && n->Opcode() != Op_SafePoint ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   570
          Node *def = n->in(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   571
          if( !n->is_Proj() ||
489c9b5090e2 Initial load
duke
parents:
diff changeset
   572
              // Could also be a flags-projection of a dead ADD or such.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   573
              (n2lidx(def) && !liveout.member(n2lidx(def)) ) ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   574
            b->_nodes.remove(j - 1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   575
            if( lrgs(r)._def == n ) lrgs(r)._def = 0;
14623
70c4c1be0a14 7092905: C2: Keep track of the number of dead nodes
bharadwaj
parents: 13104
diff changeset
   576
            n->disconnect_inputs(NULL, C);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   577
            _cfg._bbs.map(n->_idx,NULL);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   578
            n->replace_by(C->top());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   579
            // Since yanking a Node from block, high pressure moves up one
489c9b5090e2 Initial load
duke
parents:
diff changeset
   580
            hrp_index[0]--;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   581
            hrp_index[1]--;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   582
            continue;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   583
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   584
489c9b5090e2 Initial load
duke
parents:
diff changeset
   585
          // Fat-projections kill many registers which cannot be used to
489c9b5090e2 Initial load
duke
parents:
diff changeset
   586
          // hold live ranges.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   587
          if( lrgs(r)._fat_proj ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   588
            // Count the int-only registers
489c9b5090e2 Initial load
duke
parents:
diff changeset
   589
            RegMask itmp = lrgs(r).mask();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   590
            itmp.AND(*Matcher::idealreg2regmask[Op_RegI]);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   591
            int iregs = itmp.Size();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   592
#ifdef EXACT_PRESSURE
489c9b5090e2 Initial load
duke
parents:
diff changeset
   593
            if( pressure[0]+iregs > b->_reg_pressure )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   594
              b->_reg_pressure = pressure[0]+iregs;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   595
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   596
            if( pressure[0]       <= (uint)INTPRESSURE &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
   597
                pressure[0]+iregs >  (uint)INTPRESSURE ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   598
#ifndef EXACT_PRESSURE
489c9b5090e2 Initial load
duke
parents:
diff changeset
   599
              b->_reg_pressure = (uint)INTPRESSURE+1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   600
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   601
              hrp_index[0] = j-1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   602
            }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   603
            // Count the float-only registers
489c9b5090e2 Initial load
duke
parents:
diff changeset
   604
            RegMask ftmp = lrgs(r).mask();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   605
            ftmp.AND(*Matcher::idealreg2regmask[Op_RegD]);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   606
            int fregs = ftmp.Size();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   607
#ifdef EXACT_PRESSURE
489c9b5090e2 Initial load
duke
parents:
diff changeset
   608
            if( pressure[1]+fregs > b->_freg_pressure )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   609
              b->_freg_pressure = pressure[1]+fregs;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   610
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   611
            if( pressure[1]       <= (uint)FLOATPRESSURE &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
   612
                pressure[1]+fregs >  (uint)FLOATPRESSURE ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   613
#ifndef EXACT_PRESSURE
489c9b5090e2 Initial load
duke
parents:
diff changeset
   614
              b->_freg_pressure = (uint)FLOATPRESSURE+1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   615
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   616
              hrp_index[1] = j-1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   617
            }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   618
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   619
489c9b5090e2 Initial load
duke
parents:
diff changeset
   620
        } else {                // Else it is live
489c9b5090e2 Initial load
duke
parents:
diff changeset
   621
          // A DEF also ends 'area' partway through the block.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   622
          lrgs(r)._area -= cost;
1401
e5fdc8521d1f 6750588: assert(lrg._area >= 0,"negative spill area") running NSK stmp0101 test
rasbold
parents: 1057
diff changeset
   623
          assert(!(lrgs(r)._area < 0.0), "negative spill area" );
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   624
489c9b5090e2 Initial load
duke
parents:
diff changeset
   625
          // Insure high score for immediate-use spill copies so they get a color
489c9b5090e2 Initial load
duke
parents:
diff changeset
   626
          if( n->is_SpillCopy()
1057
44220ef9a775 6732194: Data corruption dependent on -server/-client/-Xbatch
never
parents: 1
diff changeset
   627
              && lrgs(r).is_singledef()        // MultiDef live range can still split
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   628
              && n->outcnt() == 1              // and use must be in this block
489c9b5090e2 Initial load
duke
parents:
diff changeset
   629
              && _cfg._bbs[n->unique_out()->_idx] == b ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   630
            // All single-use MachSpillCopy(s) that immediately precede their
489c9b5090e2 Initial load
duke
parents:
diff changeset
   631
            // use must color early.  If a longer live range steals their
489c9b5090e2 Initial load
duke
parents:
diff changeset
   632
            // color, the spill copy will split and may push another spill copy
489c9b5090e2 Initial load
duke
parents:
diff changeset
   633
            // further away resulting in an infinite spill-split-retry cycle.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   634
            // Assigning a zero area results in a high score() and a good
489c9b5090e2 Initial load
duke
parents:
diff changeset
   635
            // location in the simplify list.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   636
            //
489c9b5090e2 Initial load
duke
parents:
diff changeset
   637
489c9b5090e2 Initial load
duke
parents:
diff changeset
   638
            Node *single_use = n->unique_out();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   639
            assert( b->find_node(single_use) >= j, "Use must be later in block");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   640
            // Use can be earlier in block if it is a Phi, but then I should be a MultiDef
489c9b5090e2 Initial load
duke
parents:
diff changeset
   641
489c9b5090e2 Initial load
duke
parents:
diff changeset
   642
            // Find first non SpillCopy 'm' that follows the current instruction
489c9b5090e2 Initial load
duke
parents:
diff changeset
   643
            // (j - 1) is index for current instruction 'n'
489c9b5090e2 Initial load
duke
parents:
diff changeset
   644
            Node *m = n;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   645
            for( uint i = j; i <= last_inst && m->is_SpillCopy(); ++i ) { m = b->_nodes[i]; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   646
            if( m == single_use ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   647
              lrgs(r)._area = 0.0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   648
            }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   649
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   650
489c9b5090e2 Initial load
duke
parents:
diff changeset
   651
          // Remove from live-out set
489c9b5090e2 Initial load
duke
parents:
diff changeset
   652
          if( liveout.remove(r) ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   653
            // Adjust register pressure.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   654
            // Capture last hi-to-lo pressure transition
489c9b5090e2 Initial load
duke
parents:
diff changeset
   655
            lower_pressure( &lrgs(r), j-1, b, pressure, hrp_index );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   656
            assert( pressure[0] == count_int_pressure  (&liveout), "" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   657
            assert( pressure[1] == count_float_pressure(&liveout), "" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   658
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   659
489c9b5090e2 Initial load
duke
parents:
diff changeset
   660
          // Copies do not define a new value and so do not interfere.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   661
          // Remove the copies source from the liveout set before interfering.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   662
          uint idx = n->is_Copy();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   663
          if( idx ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   664
            uint x = n2lidx(n->in(idx));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   665
            if( liveout.remove( x ) ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   666
              lrgs(x)._area -= cost;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   667
              // Adjust register pressure.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   668
              lower_pressure( &lrgs(x), j-1, b, pressure, hrp_index );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   669
              assert( pressure[0] == count_int_pressure  (&liveout), "" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   670
              assert( pressure[1] == count_float_pressure(&liveout), "" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   671
            }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   672
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   673
        } // End of if live or not
489c9b5090e2 Initial load
duke
parents:
diff changeset
   674
489c9b5090e2 Initial load
duke
parents:
diff changeset
   675
        // Interfere with everything live.  If the defined value must
489c9b5090e2 Initial load
duke
parents:
diff changeset
   676
        // go in a particular register, just remove that register from
489c9b5090e2 Initial load
duke
parents:
diff changeset
   677
        // all conflicting parties and avoid the interference.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   678
489c9b5090e2 Initial load
duke
parents:
diff changeset
   679
        // Make exclusions for rematerializable defs.  Since rematerializable
489c9b5090e2 Initial load
duke
parents:
diff changeset
   680
        // DEFs are not bound but the live range is, some uses must be bound.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   681
        // If we spill live range 'r', it can rematerialize at each use site
489c9b5090e2 Initial load
duke
parents:
diff changeset
   682
        // according to its bindings.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   683
        const RegMask &rmask = lrgs(r).mask();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   684
        if( lrgs(r).is_bound() && !(n->rematerialize()) && rmask.is_NotEmpty() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   685
          // Check for common case
489c9b5090e2 Initial load
duke
parents:
diff changeset
   686
          int r_size = lrgs(r).num_regs();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   687
          OptoReg::Name r_reg = (r_size == 1) ? rmask.find_first_elem() : OptoReg::Physical;
13104
657b387034fb 7119644: Increase superword's vector size up to 256 bits
kvn
parents: 7397
diff changeset
   688
          // Smear odd bits
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   689
          IndexSetIterator elements(&liveout);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   690
          uint l;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   691
          while ((l = elements.next()) != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   692
            LRG &lrg = lrgs(l);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   693
            // If 'l' must spill already, do not further hack his bits.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   694
            // He'll get some interferences and be forced to spill later.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   695
            if( lrg._must_spill ) continue;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   696
            // Remove bound register(s) from 'l's choices
489c9b5090e2 Initial load
duke
parents:
diff changeset
   697
            RegMask old = lrg.mask();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   698
            uint old_size = lrg.mask_size();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   699
            // Remove the bits from LRG 'r' from LRG 'l' so 'l' no
489c9b5090e2 Initial load
duke
parents:
diff changeset
   700
            // longer interferes with 'r'.  If 'l' requires aligned
489c9b5090e2 Initial load
duke
parents:
diff changeset
   701
            // adjacent pairs, subtract out bit pairs.
13104
657b387034fb 7119644: Increase superword's vector size up to 256 bits
kvn
parents: 7397
diff changeset
   702
            assert(!lrg._is_vector || !lrg._fat_proj, "sanity");
657b387034fb 7119644: Increase superword's vector size up to 256 bits
kvn
parents: 7397
diff changeset
   703
            if (lrg.num_regs() > 1 && !lrg._fat_proj) {
657b387034fb 7119644: Increase superword's vector size up to 256 bits
kvn
parents: 7397
diff changeset
   704
              RegMask r2mask = rmask;
657b387034fb 7119644: Increase superword's vector size up to 256 bits
kvn
parents: 7397
diff changeset
   705
              // Leave only aligned set of bits.
657b387034fb 7119644: Increase superword's vector size up to 256 bits
kvn
parents: 7397
diff changeset
   706
              r2mask.smear_to_sets(lrg.num_regs());
657b387034fb 7119644: Increase superword's vector size up to 256 bits
kvn
parents: 7397
diff changeset
   707
              // It includes vector case.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   708
              lrg.SUBTRACT( r2mask );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   709
              lrg.compute_set_mask_size();
13104
657b387034fb 7119644: Increase superword's vector size up to 256 bits
kvn
parents: 7397
diff changeset
   710
            } else if( r_size != 1 ) { // fat proj
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   711
              lrg.SUBTRACT( rmask );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   712
              lrg.compute_set_mask_size();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   713
            } else {            // Common case: size 1 bound removal
489c9b5090e2 Initial load
duke
parents:
diff changeset
   714
              if( lrg.mask().Member(r_reg) ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   715
                lrg.Remove(r_reg);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   716
                lrg.set_mask_size(lrg.mask().is_AllStack() ? 65535:old_size-1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   717
              }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   718
            }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   719
            // If 'l' goes completely dry, it must spill.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   720
            if( lrg.not_free() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   721
              // Give 'l' some kind of reasonable mask, so he picks up
489c9b5090e2 Initial load
duke
parents:
diff changeset
   722
              // interferences (and will spill later).
489c9b5090e2 Initial load
duke
parents:
diff changeset
   723
              lrg.set_mask( old );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   724
              lrg.set_mask_size(old_size);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   725
              must_spill++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   726
              lrg._must_spill = 1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   727
              lrg.set_reg(OptoReg::Name(LRG::SPILL_REG));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   728
            }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   729
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   730
        } // End of if bound
489c9b5090e2 Initial load
duke
parents:
diff changeset
   731
489c9b5090e2 Initial load
duke
parents:
diff changeset
   732
        // Now interference with everything that is live and has
489c9b5090e2 Initial load
duke
parents:
diff changeset
   733
        // compatible register sets.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   734
        interfere_with_live(r,&liveout);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   735
489c9b5090e2 Initial load
duke
parents:
diff changeset
   736
      } // End of if normal register-allocated value
489c9b5090e2 Initial load
duke
parents:
diff changeset
   737
1401
e5fdc8521d1f 6750588: assert(lrg._area >= 0,"negative spill area") running NSK stmp0101 test
rasbold
parents: 1057
diff changeset
   738
      // Area remaining in the block
e5fdc8521d1f 6750588: assert(lrg._area >= 0,"negative spill area") running NSK stmp0101 test
rasbold
parents: 1057
diff changeset
   739
      inst_count--;
e5fdc8521d1f 6750588: assert(lrg._area >= 0,"negative spill area") running NSK stmp0101 test
rasbold
parents: 1057
diff changeset
   740
      cost = (inst_count <= 0) ? 0.0 : b->_freq * double(inst_count);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   741
489c9b5090e2 Initial load
duke
parents:
diff changeset
   742
      // Make all inputs live
489c9b5090e2 Initial load
duke
parents:
diff changeset
   743
      if( !n->is_Phi() ) {      // Phi function uses come from prior block
489c9b5090e2 Initial load
duke
parents:
diff changeset
   744
        JVMState* jvms = n->jvms();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   745
        uint debug_start = jvms ? jvms->debug_start() : 999999;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   746
        // Start loop at 1 (skip control edge) for most Nodes.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   747
        // SCMemProj's might be the sole use of a StoreLConditional.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   748
        // While StoreLConditionals set memory (the SCMemProj use)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   749
        // they also def flags; if that flag def is unused the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   750
        // allocator sees a flag-setting instruction with no use of
489c9b5090e2 Initial load
duke
parents:
diff changeset
   751
        // the flags and assumes it's dead.  This keeps the (useless)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   752
        // flag-setting behavior alive while also keeping the (useful)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   753
        // memory update effect.
4760
70aa3bc938c2 6921992: failure in verify scheduling after 6792161
never
parents: 4747
diff changeset
   754
        for( uint k = ((n->Opcode() == Op_SCMemProj) ? 0:1); k < n->req(); k++ ) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   755
          Node *def = n->in(k);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   756
          uint x = n2lidx(def);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   757
          if( !x ) continue;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   758
          LRG &lrg = lrgs(x);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   759
          // No use-side cost for spilling debug info
489c9b5090e2 Initial load
duke
parents:
diff changeset
   760
          if( k < debug_start )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   761
            // A USE costs twice block frequency (once for the Load, once
489c9b5090e2 Initial load
duke
parents:
diff changeset
   762
            // for a Load-delay).  Rematerialized uses only cost once.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   763
            lrg._cost += (def->rematerialize() ? b->_freq : (b->_freq + b->_freq));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   764
          // It is live now
489c9b5090e2 Initial load
duke
parents:
diff changeset
   765
          if( liveout.insert( x ) ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   766
            // Newly live things assumed live from here to top of block
489c9b5090e2 Initial load
duke
parents:
diff changeset
   767
            lrg._area += cost;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   768
            // Adjust register pressure
13104
657b387034fb 7119644: Increase superword's vector size up to 256 bits
kvn
parents: 7397
diff changeset
   769
            if (lrg.mask().is_UP() && lrg.mask_size()) {
657b387034fb 7119644: Increase superword's vector size up to 256 bits
kvn
parents: 7397
diff changeset
   770
              if (lrg._is_float || lrg._is_vector) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   771
                pressure[1] += lrg.reg_pressure();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   772
#ifdef EXACT_PRESSURE
489c9b5090e2 Initial load
duke
parents:
diff changeset
   773
                if( pressure[1] > b->_freg_pressure )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   774
                  b->_freg_pressure = pressure[1];
489c9b5090e2 Initial load
duke
parents:
diff changeset
   775
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   776
              } else if( lrg.mask().overlap(*Matcher::idealreg2regmask[Op_RegI]) ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   777
                pressure[0] += lrg.reg_pressure();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   778
#ifdef EXACT_PRESSURE
489c9b5090e2 Initial load
duke
parents:
diff changeset
   779
                if( pressure[0] > b->_reg_pressure )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   780
                  b->_reg_pressure = pressure[0];
489c9b5090e2 Initial load
duke
parents:
diff changeset
   781
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   782
              }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   783
            }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   784
            assert( pressure[0] == count_int_pressure  (&liveout), "" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   785
            assert( pressure[1] == count_float_pressure(&liveout), "" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   786
          }
1401
e5fdc8521d1f 6750588: assert(lrg._area >= 0,"negative spill area") running NSK stmp0101 test
rasbold
parents: 1057
diff changeset
   787
          assert(!(lrg._area < 0.0), "negative spill area" );
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   788
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   789
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   790
    } // End of reverse pass over all instructions in block
489c9b5090e2 Initial load
duke
parents:
diff changeset
   791
489c9b5090e2 Initial load
duke
parents:
diff changeset
   792
    // If we run off the top of the block with high pressure and
489c9b5090e2 Initial load
duke
parents:
diff changeset
   793
    // never see a hi-to-low pressure transition, just record that
489c9b5090e2 Initial load
duke
parents:
diff changeset
   794
    // the whole block is high pressure.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   795
    if( pressure[0] > (uint)INTPRESSURE   ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   796
      hrp_index[0] = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   797
#ifdef EXACT_PRESSURE
489c9b5090e2 Initial load
duke
parents:
diff changeset
   798
      if( pressure[0] > b->_reg_pressure )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   799
        b->_reg_pressure = pressure[0];
489c9b5090e2 Initial load
duke
parents:
diff changeset
   800
#else
489c9b5090e2 Initial load
duke
parents:
diff changeset
   801
      b->_reg_pressure = (uint)INTPRESSURE+1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   802
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   803
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   804
    if( pressure[1] > (uint)FLOATPRESSURE ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   805
      hrp_index[1] = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   806
#ifdef EXACT_PRESSURE
489c9b5090e2 Initial load
duke
parents:
diff changeset
   807
      if( pressure[1] > b->_freg_pressure )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   808
        b->_freg_pressure = pressure[1];
489c9b5090e2 Initial load
duke
parents:
diff changeset
   809
#else
489c9b5090e2 Initial load
duke
parents:
diff changeset
   810
      b->_freg_pressure = (uint)FLOATPRESSURE+1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   811
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   812
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   813
489c9b5090e2 Initial load
duke
parents:
diff changeset
   814
    // Compute high pressure indice; avoid landing in the middle of projnodes
489c9b5090e2 Initial load
duke
parents:
diff changeset
   815
    j = hrp_index[0];
489c9b5090e2 Initial load
duke
parents:
diff changeset
   816
    if( j < b->_nodes.size() && j < b->end_idx()+1 ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   817
      Node *cur = b->_nodes[j];
489c9b5090e2 Initial load
duke
parents:
diff changeset
   818
      while( cur->is_Proj() || (cur->is_MachNullCheck()) || cur->is_Catch() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   819
        j--;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   820
        cur = b->_nodes[j];
489c9b5090e2 Initial load
duke
parents:
diff changeset
   821
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   822
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   823
    b->_ihrp_index = j;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   824
    j = hrp_index[1];
489c9b5090e2 Initial load
duke
parents:
diff changeset
   825
    if( j < b->_nodes.size() && j < b->end_idx()+1 ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   826
      Node *cur = b->_nodes[j];
489c9b5090e2 Initial load
duke
parents:
diff changeset
   827
      while( cur->is_Proj() || (cur->is_MachNullCheck()) || cur->is_Catch() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   828
        j--;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   829
        cur = b->_nodes[j];
489c9b5090e2 Initial load
duke
parents:
diff changeset
   830
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   831
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   832
    b->_fhrp_index = j;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   833
489c9b5090e2 Initial load
duke
parents:
diff changeset
   834
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   835
    // Gather Register Pressure Statistics
489c9b5090e2 Initial load
duke
parents:
diff changeset
   836
    if( PrintOptoStatistics ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   837
      if( b->_reg_pressure > (uint)INTPRESSURE || b->_freg_pressure > (uint)FLOATPRESSURE )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   838
        _high_pressure++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   839
      else
489c9b5090e2 Initial load
duke
parents:
diff changeset
   840
        _low_pressure++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   841
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   842
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   843
  } // End of for all blocks
489c9b5090e2 Initial load
duke
parents:
diff changeset
   844
489c9b5090e2 Initial load
duke
parents:
diff changeset
   845
  return must_spill;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   846
}