hotspot/src/share/vm/opto/ifg.cpp
author iveresov
Thu, 22 Jan 2015 11:25:23 -0800
changeset 28723 0a36120cb225
parent 28643 a665e19ca007
child 33065 55892792936f
permissions -rw-r--r--
8071302: assert(!_reg_node[reg_lo] || edge_from_to(_reg_node[reg_lo], def)) failed: after block local Summary: Add merge nodes to node to block mapping Reviewed-by: kvn, vlivanov
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     1
/*
24424
2658d7834c6e 8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents: 23528
diff changeset
     2
 * Copyright (c) 1998, 2014, Oracle and/or its affiliates. All rights reserved.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
489c9b5090e2 Initial load
duke
parents:
diff changeset
     4
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
489c9b5090e2 Initial load
duke
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
489c9b5090e2 Initial load
duke
parents:
diff changeset
     7
 * published by the Free Software Foundation.
489c9b5090e2 Initial load
duke
parents:
diff changeset
     8
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
489c9b5090e2 Initial load
duke
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
489c9b5090e2 Initial load
duke
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
489c9b5090e2 Initial load
duke
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
489c9b5090e2 Initial load
duke
parents:
diff changeset
    13
 * accompanied this code).
489c9b5090e2 Initial load
duke
parents:
diff changeset
    14
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
489c9b5090e2 Initial load
duke
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
489c9b5090e2 Initial load
duke
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    18
 *
5547
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 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/indexSet.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    35
#include "opto/machnode.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    36
#include "opto/memnode.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    37
#include "opto/opcodes.hpp"
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    38
489c9b5090e2 Initial load
duke
parents:
diff changeset
    39
PhaseIFG::PhaseIFG( Arena *arena ) : Phase(Interference_Graph), _arena(arena) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    40
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    41
489c9b5090e2 Initial load
duke
parents:
diff changeset
    42
void PhaseIFG::init( uint maxlrg ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    43
  _maxlrg = maxlrg;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    44
  _yanked = new (_arena) VectorSet(_arena);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    45
  _is_square = false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    46
  // Make uninitialized adjacency lists
489c9b5090e2 Initial load
duke
parents:
diff changeset
    47
  _adjs = (IndexSet*)_arena->Amalloc(sizeof(IndexSet)*maxlrg);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    48
  // Also make empty live range structures
489c9b5090e2 Initial load
duke
parents:
diff changeset
    49
  _lrgs = (LRG *)_arena->Amalloc( maxlrg * sizeof(LRG) );
489c9b5090e2 Initial load
duke
parents:
diff changeset
    50
  memset(_lrgs,0,sizeof(LRG)*maxlrg);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    51
  // Init all to empty
489c9b5090e2 Initial load
duke
parents:
diff changeset
    52
  for( uint i = 0; i < maxlrg; i++ ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    53
    _adjs[i].initialize(maxlrg);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    54
    _lrgs[i].Set_All();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    55
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    56
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    57
489c9b5090e2 Initial load
duke
parents:
diff changeset
    58
// Add edge between vertices a & b.  These are sorted (triangular matrix),
489c9b5090e2 Initial load
duke
parents:
diff changeset
    59
// then the smaller number is inserted in the larger numbered array.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    60
int PhaseIFG::add_edge( uint a, uint b ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    61
  lrgs(a).invalid_degree();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    62
  lrgs(b).invalid_degree();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    63
  // Sort a and b, so that a is bigger
489c9b5090e2 Initial load
duke
parents:
diff changeset
    64
  assert( !_is_square, "only on triangular" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
    65
  if( a < b ) { uint tmp = a; a = b; b = tmp; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    66
  return _adjs[a].insert( b );
489c9b5090e2 Initial load
duke
parents:
diff changeset
    67
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    68
489c9b5090e2 Initial load
duke
parents:
diff changeset
    69
// Add an edge between 'a' and everything in the vector.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    70
void PhaseIFG::add_vector( uint a, IndexSet *vec ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    71
  // IFG is triangular, so do the inserts where 'a' < 'b'.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    72
  assert( !_is_square, "only on triangular" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
    73
  IndexSet *adjs_a = &_adjs[a];
489c9b5090e2 Initial load
duke
parents:
diff changeset
    74
  if( !vec->count() ) return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    75
489c9b5090e2 Initial load
duke
parents:
diff changeset
    76
  IndexSetIterator elements(vec);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    77
  uint neighbor;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    78
  while ((neighbor = elements.next()) != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    79
    add_edge( a, neighbor );
489c9b5090e2 Initial load
duke
parents:
diff changeset
    80
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    81
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    82
489c9b5090e2 Initial load
duke
parents:
diff changeset
    83
// Is there an edge between a and b?
489c9b5090e2 Initial load
duke
parents:
diff changeset
    84
int PhaseIFG::test_edge( uint a, uint b ) const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    85
  // Sort a and b, so that a is larger
489c9b5090e2 Initial load
duke
parents:
diff changeset
    86
  assert( !_is_square, "only on triangular" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
    87
  if( a < b ) { uint tmp = a; a = b; b = tmp; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    88
  return _adjs[a].member(b);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    89
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    90
489c9b5090e2 Initial load
duke
parents:
diff changeset
    91
// Convert triangular matrix to square matrix
489c9b5090e2 Initial load
duke
parents:
diff changeset
    92
void PhaseIFG::SquareUp() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    93
  assert( !_is_square, "only on triangular" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
    94
489c9b5090e2 Initial load
duke
parents:
diff changeset
    95
  // Simple transpose
489c9b5090e2 Initial load
duke
parents:
diff changeset
    96
  for( uint i = 0; i < _maxlrg; i++ ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    97
    IndexSetIterator elements(&_adjs[i]);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    98
    uint datum;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    99
    while ((datum = elements.next()) != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   100
      _adjs[datum].insert( i );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   101
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   102
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   103
  _is_square = true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   104
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   105
489c9b5090e2 Initial load
duke
parents:
diff changeset
   106
// Compute effective degree in bulk
489c9b5090e2 Initial load
duke
parents:
diff changeset
   107
void PhaseIFG::Compute_Effective_Degree() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   108
  assert( _is_square, "only on square" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   109
489c9b5090e2 Initial load
duke
parents:
diff changeset
   110
  for( uint i = 0; i < _maxlrg; i++ )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   111
    lrgs(i).set_degree(effective_degree(i));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   112
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   113
489c9b5090e2 Initial load
duke
parents:
diff changeset
   114
int PhaseIFG::test_edge_sq( uint a, uint b ) const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   115
  assert( _is_square, "only on square" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   116
  // Swap, so that 'a' has the lesser count.  Then binary search is on
489c9b5090e2 Initial load
duke
parents:
diff changeset
   117
  // the smaller of a's list and b's list.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   118
  if( neighbor_cnt(a) > neighbor_cnt(b) ) { uint tmp = a; a = b; b = tmp; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   119
  //return _adjs[a].unordered_member(b);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   120
  return _adjs[a].member(b);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   121
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   122
489c9b5090e2 Initial load
duke
parents:
diff changeset
   123
// Union edges of B into A
489c9b5090e2 Initial load
duke
parents:
diff changeset
   124
void PhaseIFG::Union( uint a, uint b ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   125
  assert( _is_square, "only on square" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   126
  IndexSet *A = &_adjs[a];
489c9b5090e2 Initial load
duke
parents:
diff changeset
   127
  IndexSetIterator b_elements(&_adjs[b]);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   128
  uint datum;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   129
  while ((datum = b_elements.next()) != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   130
    if(A->insert(datum)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   131
      _adjs[datum].insert(a);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   132
      lrgs(a).invalid_degree();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   133
      lrgs(datum).invalid_degree();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   134
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   135
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   136
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   137
489c9b5090e2 Initial load
duke
parents:
diff changeset
   138
// Yank a Node and all connected edges from the IFG.  Return a
489c9b5090e2 Initial load
duke
parents:
diff changeset
   139
// list of neighbors (edges) yanked.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   140
IndexSet *PhaseIFG::remove_node( uint a ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   141
  assert( _is_square, "only on square" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   142
  assert( !_yanked->test(a), "" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   143
  _yanked->set(a);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   144
489c9b5090e2 Initial load
duke
parents:
diff changeset
   145
  // I remove the LRG from all neighbors.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   146
  IndexSetIterator elements(&_adjs[a]);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   147
  LRG &lrg_a = lrgs(a);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   148
  uint datum;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   149
  while ((datum = elements.next()) != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   150
    _adjs[datum].remove(a);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   151
    lrgs(datum).inc_degree( -lrg_a.compute_degree(lrgs(datum)) );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   152
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   153
  return neighbors(a);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   154
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   155
489c9b5090e2 Initial load
duke
parents:
diff changeset
   156
// Re-insert a yanked Node.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   157
void PhaseIFG::re_insert( uint a ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   158
  assert( _is_square, "only on square" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   159
  assert( _yanked->test(a), "" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   160
  (*_yanked) >>= a;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   161
489c9b5090e2 Initial load
duke
parents:
diff changeset
   162
  IndexSetIterator elements(&_adjs[a]);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   163
  uint datum;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   164
  while ((datum = elements.next()) != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   165
    _adjs[datum].insert(a);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   166
    lrgs(datum).invalid_degree();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   167
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   168
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   169
489c9b5090e2 Initial load
duke
parents:
diff changeset
   170
// Compute the degree between 2 live ranges.  If both live ranges are
489c9b5090e2 Initial load
duke
parents:
diff changeset
   171
// aligned-adjacent powers-of-2 then we use the MAX size.  If either is
489c9b5090e2 Initial load
duke
parents:
diff changeset
   172
// mis-aligned (or for Fat-Projections, not-adjacent) then we have to
489c9b5090e2 Initial load
duke
parents:
diff changeset
   173
// MULTIPLY the sizes.  Inspect Brigg's thesis on register pairs to see why
489c9b5090e2 Initial load
duke
parents:
diff changeset
   174
// this is so.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   175
int LRG::compute_degree( LRG &l ) const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   176
  int tmp;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   177
  int num_regs = _num_regs;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   178
  int nregs = l.num_regs();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   179
  tmp =  (_fat_proj || l._fat_proj)     // either is a fat-proj?
489c9b5090e2 Initial load
duke
parents:
diff changeset
   180
    ? (num_regs * nregs)                // then use product
489c9b5090e2 Initial load
duke
parents:
diff changeset
   181
    : MAX2(num_regs,nregs);             // else use max
489c9b5090e2 Initial load
duke
parents:
diff changeset
   182
  return tmp;
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 effective degree for this live range.  If both live ranges are
489c9b5090e2 Initial load
duke
parents:
diff changeset
   186
// aligned-adjacent powers-of-2 then we use the MAX size.  If either is
489c9b5090e2 Initial load
duke
parents:
diff changeset
   187
// mis-aligned (or for Fat-Projections, not-adjacent) then we have to
489c9b5090e2 Initial load
duke
parents:
diff changeset
   188
// MULTIPLY the sizes.  Inspect Brigg's thesis on register pairs to see why
489c9b5090e2 Initial load
duke
parents:
diff changeset
   189
// this is so.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   190
int PhaseIFG::effective_degree( uint lidx ) const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   191
  int eff = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   192
  int num_regs = lrgs(lidx).num_regs();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   193
  int fat_proj = lrgs(lidx)._fat_proj;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   194
  IndexSet *s = neighbors(lidx);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   195
  IndexSetIterator elements(s);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   196
  uint nidx;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   197
  while((nidx = elements.next()) != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   198
    LRG &lrgn = lrgs(nidx);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   199
    int nregs = lrgn.num_regs();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   200
    eff += (fat_proj || lrgn._fat_proj) // either is a fat-proj?
489c9b5090e2 Initial load
duke
parents:
diff changeset
   201
      ? (num_regs * nregs)              // then use product
489c9b5090e2 Initial load
duke
parents:
diff changeset
   202
      : MAX2(num_regs,nregs);           // else use max
489c9b5090e2 Initial load
duke
parents:
diff changeset
   203
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   204
  return eff;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   205
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   206
489c9b5090e2 Initial load
duke
parents:
diff changeset
   207
489c9b5090e2 Initial load
duke
parents:
diff changeset
   208
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   209
void PhaseIFG::dump() const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   210
  tty->print_cr("-- Interference Graph --%s--",
489c9b5090e2 Initial load
duke
parents:
diff changeset
   211
                _is_square ? "square" : "triangular" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   212
  if( _is_square ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   213
    for( uint i = 0; i < _maxlrg; i++ ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   214
      tty->print( (*_yanked)[i] ? "XX " : "  ");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   215
      tty->print("L%d: { ",i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   216
      IndexSetIterator elements(&_adjs[i]);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   217
      uint datum;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   218
      while ((datum = elements.next()) != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   219
        tty->print("L%d ", datum);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   220
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   221
      tty->print_cr("}");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   222
489c9b5090e2 Initial load
duke
parents:
diff changeset
   223
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   224
    return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   225
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   226
489c9b5090e2 Initial load
duke
parents:
diff changeset
   227
  // Triangular
489c9b5090e2 Initial load
duke
parents:
diff changeset
   228
  for( uint i = 0; i < _maxlrg; i++ ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   229
    uint j;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   230
    tty->print( (*_yanked)[i] ? "XX " : "  ");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   231
    tty->print("L%d: { ",i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   232
    for( j = _maxlrg; j > i; j-- )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   233
      if( test_edge(j - 1,i) ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   234
        tty->print("L%d ",j - 1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   235
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   236
    tty->print("| ");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   237
    IndexSetIterator elements(&_adjs[i]);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   238
    uint datum;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   239
    while ((datum = elements.next()) != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   240
      tty->print("L%d ", datum);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   241
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   242
    tty->print("}\n");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   243
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   244
  tty->print("\n");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   245
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   246
489c9b5090e2 Initial load
duke
parents:
diff changeset
   247
void PhaseIFG::stats() const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   248
  ResourceMark rm;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   249
  int *h_cnt = NEW_RESOURCE_ARRAY(int,_maxlrg*2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   250
  memset( h_cnt, 0, sizeof(int)*_maxlrg*2 );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   251
  uint i;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   252
  for( i = 0; i < _maxlrg; i++ ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   253
    h_cnt[neighbor_cnt(i)]++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   254
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   255
  tty->print_cr("--Histogram of counts--");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   256
  for( i = 0; i < _maxlrg*2; i++ )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   257
    if( h_cnt[i] )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   258
      tty->print("%d/%d ",i,h_cnt[i]);
24424
2658d7834c6e 8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents: 23528
diff changeset
   259
  tty->cr();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   260
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   261
489c9b5090e2 Initial load
duke
parents:
diff changeset
   262
void PhaseIFG::verify( const PhaseChaitin *pc ) const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   263
  // IFG is square, sorted and no need for Find
489c9b5090e2 Initial load
duke
parents:
diff changeset
   264
  for( uint i = 0; i < _maxlrg; i++ ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   265
    assert(!((*_yanked)[i]) || !neighbor_cnt(i), "Is removed completely" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   266
    IndexSet *set = &_adjs[i];
489c9b5090e2 Initial load
duke
parents:
diff changeset
   267
    IndexSetIterator elements(set);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   268
    uint idx;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   269
    uint last = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   270
    while ((idx = elements.next()) != 0) {
17013
22a05c7f3314 8011621: live_ranges_in_separate_class.patch
neliasso
parents: 16618
diff changeset
   271
      assert(idx != i, "Must have empty diagonal");
22a05c7f3314 8011621: live_ranges_in_separate_class.patch
neliasso
parents: 16618
diff changeset
   272
      assert(pc->_lrg_map.find_const(idx) == idx, "Must not need Find");
22a05c7f3314 8011621: live_ranges_in_separate_class.patch
neliasso
parents: 16618
diff changeset
   273
      assert(_adjs[idx].member(i), "IFG not square");
22a05c7f3314 8011621: live_ranges_in_separate_class.patch
neliasso
parents: 16618
diff changeset
   274
      assert(!(*_yanked)[idx], "No yanked neighbors");
22a05c7f3314 8011621: live_ranges_in_separate_class.patch
neliasso
parents: 16618
diff changeset
   275
      assert(last < idx, "not sorted increasing");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   276
      last = idx;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   277
    }
17013
22a05c7f3314 8011621: live_ranges_in_separate_class.patch
neliasso
parents: 16618
diff changeset
   278
    assert(!lrgs(i)._degree_valid || effective_degree(i) == lrgs(i).degree(), "degree is valid but wrong");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   279
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   280
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   281
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   282
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   283
/*
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   284
 * Interfere this register with everything currently live.
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   285
 * Check for interference by checking overlap of regmasks.
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   286
 * Only interfere if acceptable register masks overlap.
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   287
 */
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   288
void PhaseChaitin::interfere_with_live(uint lid, IndexSet* liveout) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   289
  LRG& lrg = lrgs(lid);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   290
  const RegMask& rm = lrg.mask();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   291
  IndexSetIterator elements(liveout);
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   292
  uint interfering_lid = elements.next();
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   293
  while (interfering_lid != 0) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   294
    LRG& interfering_lrg = lrgs(interfering_lid);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   295
    if (rm.overlap(interfering_lrg.mask())) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   296
      _ifg->add_edge(lid, interfering_lid);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   297
    }
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   298
    interfering_lid = elements.next();
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   299
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   300
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   301
489c9b5090e2 Initial load
duke
parents:
diff changeset
   302
// Actually build the interference graph.  Uses virtual registers only, no
489c9b5090e2 Initial load
duke
parents:
diff changeset
   303
// physical register masks.  This allows me to be very aggressive when
489c9b5090e2 Initial load
duke
parents:
diff changeset
   304
// coalescing copies.  Some of this aggressiveness will have to be undone
489c9b5090e2 Initial load
duke
parents:
diff changeset
   305
// later, but I'd rather get all the copies I can now (since unremoved copies
489c9b5090e2 Initial load
duke
parents:
diff changeset
   306
// at this point can end up in bad places).  Copies I re-insert later I have
489c9b5090e2 Initial load
duke
parents:
diff changeset
   307
// more opportunity to insert them in low-frequency locations.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   308
void PhaseChaitin::build_ifg_virtual( ) {
26913
9ad70cd32368 8058968: Compiler time traces should be improved
shade
parents: 24424
diff changeset
   309
  Compile::TracePhase tp("buildIFG_virt", &timers[_t_buildIFGvirtual]);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   310
489c9b5090e2 Initial load
duke
parents:
diff changeset
   311
  // For all blocks (in any order) do...
19330
49d6711171e6 8023003: Cleanup the public interface to PhaseCFG
adlertz
parents: 19279
diff changeset
   312
  for (uint i = 0; i < _cfg.number_of_blocks(); i++) {
49d6711171e6 8023003: Cleanup the public interface to PhaseCFG
adlertz
parents: 19279
diff changeset
   313
    Block* block = _cfg.get_block(i);
49d6711171e6 8023003: Cleanup the public interface to PhaseCFG
adlertz
parents: 19279
diff changeset
   314
    IndexSet* liveout = _live->live(block);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   315
489c9b5090e2 Initial load
duke
parents:
diff changeset
   316
    // The IFG is built by a single reverse pass over each basic block.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   317
    // Starting with the known live-out set, we remove things that get
489c9b5090e2 Initial load
duke
parents:
diff changeset
   318
    // defined and add things that become live (essentially executing one
489c9b5090e2 Initial load
duke
parents:
diff changeset
   319
    // pass of a standard LIVE analysis). Just before a Node defines a value
489c9b5090e2 Initial load
duke
parents:
diff changeset
   320
    // (and removes it from the live-ness set) that value is certainly live.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   321
    // The defined value interferes with everything currently live.  The
489c9b5090e2 Initial load
duke
parents:
diff changeset
   322
    // value is then removed from the live-ness set and it's inputs are
489c9b5090e2 Initial load
duke
parents:
diff changeset
   323
    // added to the live-ness set.
19330
49d6711171e6 8023003: Cleanup the public interface to PhaseCFG
adlertz
parents: 19279
diff changeset
   324
    for (uint j = block->end_idx() + 1; j > 1; j--) {
19717
7819ffdaf0ff 8023691: Create interface for nodes in class Block
adlertz
parents: 19330
diff changeset
   325
      Node* n = block->get_node(j - 1);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   326
489c9b5090e2 Initial load
duke
parents:
diff changeset
   327
      // Get value being defined
17013
22a05c7f3314 8011621: live_ranges_in_separate_class.patch
neliasso
parents: 16618
diff changeset
   328
      uint r = _lrg_map.live_range_id(n);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   329
489c9b5090e2 Initial load
duke
parents:
diff changeset
   330
      // Some special values do not allocate
17013
22a05c7f3314 8011621: live_ranges_in_separate_class.patch
neliasso
parents: 16618
diff changeset
   331
      if (r) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   332
489c9b5090e2 Initial load
duke
parents:
diff changeset
   333
        // Remove from live-out set
489c9b5090e2 Initial load
duke
parents:
diff changeset
   334
        liveout->remove(r);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   335
489c9b5090e2 Initial load
duke
parents:
diff changeset
   336
        // Copies do not define a new value and so do not interfere.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   337
        // Remove the copies source from the liveout set before interfering.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   338
        uint idx = n->is_Copy();
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   339
        if (idx != 0) {
17013
22a05c7f3314 8011621: live_ranges_in_separate_class.patch
neliasso
parents: 16618
diff changeset
   340
          liveout->remove(_lrg_map.live_range_id(n->in(idx)));
22a05c7f3314 8011621: live_ranges_in_separate_class.patch
neliasso
parents: 16618
diff changeset
   341
        }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   342
489c9b5090e2 Initial load
duke
parents:
diff changeset
   343
        // Interfere with everything live
17013
22a05c7f3314 8011621: live_ranges_in_separate_class.patch
neliasso
parents: 16618
diff changeset
   344
        interfere_with_live(r, liveout);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   345
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   346
489c9b5090e2 Initial load
duke
parents:
diff changeset
   347
      // Make all inputs live
17013
22a05c7f3314 8011621: live_ranges_in_separate_class.patch
neliasso
parents: 16618
diff changeset
   348
      if (!n->is_Phi()) {      // Phi function uses come from prior block
22a05c7f3314 8011621: live_ranges_in_separate_class.patch
neliasso
parents: 16618
diff changeset
   349
        for(uint k = 1; k < n->req(); k++) {
22a05c7f3314 8011621: live_ranges_in_separate_class.patch
neliasso
parents: 16618
diff changeset
   350
          liveout->insert(_lrg_map.live_range_id(n->in(k)));
22a05c7f3314 8011621: live_ranges_in_separate_class.patch
neliasso
parents: 16618
diff changeset
   351
        }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   352
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   353
489c9b5090e2 Initial load
duke
parents:
diff changeset
   354
      // 2-address instructions always have the defined value live
489c9b5090e2 Initial load
duke
parents:
diff changeset
   355
      // on entry to the instruction, even though it is being defined
489c9b5090e2 Initial load
duke
parents:
diff changeset
   356
      // by the instruction.  We pretend a virtual copy sits just prior
489c9b5090e2 Initial load
duke
parents:
diff changeset
   357
      // to the instruction and kills the src-def'd register.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   358
      // In other words, for 2-address instructions the defined value
489c9b5090e2 Initial load
duke
parents:
diff changeset
   359
      // interferes with all inputs.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   360
      uint idx;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   361
      if( n->is_Mach() && (idx = n->as_Mach()->two_adr()) ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   362
        const MachNode *mach = n->as_Mach();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   363
        // Sometimes my 2-address ADDs are commuted in a bad way.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   364
        // We generally want the USE-DEF register to refer to the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   365
        // loop-varying quantity, to avoid a copy.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   366
        uint op = mach->ideal_Opcode();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   367
        // Check that mach->num_opnds() == 3 to ensure instruction is
489c9b5090e2 Initial load
duke
parents:
diff changeset
   368
        // not subsuming constants, effectively excludes addI_cin_imm
489c9b5090e2 Initial load
duke
parents:
diff changeset
   369
        // Can NOT swap for instructions like addI_cin_imm since it
489c9b5090e2 Initial load
duke
parents:
diff changeset
   370
        // is adding zero to yhi + carry and the second ideal-input
489c9b5090e2 Initial load
duke
parents:
diff changeset
   371
        // points to the result of adding low-halves.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   372
        // Checking req() and num_opnds() does NOT distinguish addI_cout from addI_cout_imm
489c9b5090e2 Initial load
duke
parents:
diff changeset
   373
        if( (op == Op_AddI && mach->req() == 3 && mach->num_opnds() == 3) &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
   374
            n->in(1)->bottom_type()->base() == Type::Int &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
   375
            // See if the ADD is involved in a tight data loop the wrong way
489c9b5090e2 Initial load
duke
parents:
diff changeset
   376
            n->in(2)->is_Phi() &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
   377
            n->in(2)->in(2) == n ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   378
          Node *tmp = n->in(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   379
          n->set_req( 1, n->in(2) );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   380
          n->set_req( 2, tmp );
489c9b5090e2 Initial load
duke
parents:
diff changeset
   381
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   382
        // Defined value interferes with all inputs
17013
22a05c7f3314 8011621: live_ranges_in_separate_class.patch
neliasso
parents: 16618
diff changeset
   383
        uint lidx = _lrg_map.live_range_id(n->in(idx));
22a05c7f3314 8011621: live_ranges_in_separate_class.patch
neliasso
parents: 16618
diff changeset
   384
        for (uint k = 1; k < n->req(); k++) {
22a05c7f3314 8011621: live_ranges_in_separate_class.patch
neliasso
parents: 16618
diff changeset
   385
          uint kidx = _lrg_map.live_range_id(n->in(k));
22a05c7f3314 8011621: live_ranges_in_separate_class.patch
neliasso
parents: 16618
diff changeset
   386
          if (kidx != lidx) {
22a05c7f3314 8011621: live_ranges_in_separate_class.patch
neliasso
parents: 16618
diff changeset
   387
            _ifg->add_edge(r, kidx);
22a05c7f3314 8011621: live_ranges_in_separate_class.patch
neliasso
parents: 16618
diff changeset
   388
          }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   389
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   390
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   391
    } // End of forall instructions in block
489c9b5090e2 Initial load
duke
parents:
diff changeset
   392
  } // End of forall blocks
489c9b5090e2 Initial load
duke
parents:
diff changeset
   393
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   394
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   395
#ifdef ASSERT
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   396
uint PhaseChaitin::count_int_pressure(IndexSet* liveout) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   397
  IndexSetIterator elements(liveout);
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   398
  uint lidx = elements.next();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   399
  uint cnt = 0;
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   400
  while (lidx != 0) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   401
    LRG& lrg = lrgs(lidx);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   402
    if (lrg.mask_is_nonempty_and_up() &&
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   403
        !lrg.is_float_or_vector() &&
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   404
        lrg.mask().overlap(*Matcher::idealreg2regmask[Op_RegI])) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   405
      cnt += lrg.reg_pressure();
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   406
    }
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   407
    lidx = elements.next();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   408
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   409
  return cnt;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   410
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   411
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   412
uint PhaseChaitin::count_float_pressure(IndexSet* liveout) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   413
  IndexSetIterator elements(liveout);
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   414
  uint lidx = elements.next();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   415
  uint cnt = 0;
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   416
  while (lidx != 0) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   417
    LRG& lrg = lrgs(lidx);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   418
    if (lrg.mask_is_nonempty_and_up() && lrg.is_float_or_vector()) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   419
      cnt += lrg.reg_pressure();
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   420
    }
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   421
    lidx = elements.next();
1
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
}
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   425
#endif
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   426
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   427
/*
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   428
 * Adjust register pressure down by 1.  Capture last hi-to-low transition,
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   429
 */
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   430
void PhaseChaitin::lower_pressure(Block* b, uint location, LRG& lrg, IndexSet* liveout, Pressure& int_pressure, Pressure& float_pressure) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   431
  if (lrg.mask_is_nonempty_and_up()) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   432
    if (lrg.is_float_or_vector()) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   433
      float_pressure.lower(lrg, location);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   434
    } else {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   435
      // Do not count the SP and flag registers
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   436
      const RegMask& r = lrg.mask();
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   437
      if (r.overlap(*Matcher::idealreg2regmask[Op_RegI])) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   438
        int_pressure.lower(lrg, location);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   439
      }
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   440
    }
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   441
  }
22912
a8c042dca573 8032894: Remove dead code in Pressure::lower
adlertz
parents: 22804
diff changeset
   442
  assert(int_pressure.current_pressure() == count_int_pressure(liveout), "the int pressure is incorrect");
a8c042dca573 8032894: Remove dead code in Pressure::lower
adlertz
parents: 22804
diff changeset
   443
  assert(float_pressure.current_pressure() == count_float_pressure(liveout), "the float pressure is incorrect");
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   444
}
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   445
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   446
/* Go to the first non-phi index in a block */
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   447
static uint first_nonphi_index(Block* b) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   448
  uint i;
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   449
  uint end_idx = b->end_idx();
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   450
  for (i = 1; i < end_idx; i++) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   451
    Node* n = b->get_node(i);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   452
    if (!n->is_Phi()) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   453
      break;
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   454
    }
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   455
  }
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   456
  return i;
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   457
}
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   458
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   459
/*
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   460
 * Spills could be inserted before a CreateEx node which should be the first
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   461
 * instruction in a block after Phi nodes. If so, move the CreateEx node up.
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   462
 */
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   463
static void move_exception_node_up(Block* b, uint first_inst, uint last_inst) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   464
  for (uint i = first_inst; i < last_inst; i++) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   465
    Node* ex = b->get_node(i);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   466
    if (ex->is_SpillCopy()) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   467
      continue;
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   468
    }
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   469
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   470
    if (i > first_inst &&
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   471
        ex->is_Mach() && ex->as_Mach()->ideal_Opcode() == Op_CreateEx) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   472
      b->remove_node(i);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   473
      b->insert_node(ex, first_inst);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   474
    }
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   475
    // Stop once a CreateEx or any other node is found
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   476
    break;
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   477
  }
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   478
}
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   479
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   480
/*
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   481
 * When new live ranges are live, we raise the register pressure
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   482
 */
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   483
void PhaseChaitin::raise_pressure(Block* b, LRG& lrg, Pressure& int_pressure, Pressure& float_pressure) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   484
  if (lrg.mask_is_nonempty_and_up()) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   485
    if (lrg.is_float_or_vector()) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   486
      float_pressure.raise(lrg);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   487
    } else {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   488
      // Do not count the SP and flag registers
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   489
      const RegMask& rm = lrg.mask();
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   490
      if (rm.overlap(*Matcher::idealreg2regmask[Op_RegI])) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   491
        int_pressure.raise(lrg);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   492
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   493
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   494
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   495
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   496
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   497
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   498
/*
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   499
 * Computes the initial register pressure of a block, looking at all live
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   500
 * ranges in the liveout. The register pressure is computed for both float
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   501
 * and int/pointer registers.
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   502
 * Live ranges in the liveout are presumed live for the whole block.
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   503
 * We add the cost for the whole block to the area of the live ranges initially.
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   504
 * If a live range gets killed in the block, we'll subtract the unused part of
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   505
 * the block from the area.
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   506
 */
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   507
void PhaseChaitin::compute_initial_block_pressure(Block* b, IndexSet* liveout, Pressure& int_pressure, Pressure& float_pressure, double cost) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   508
  IndexSetIterator elements(liveout);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   509
  uint lid = elements.next();
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   510
  while (lid != 0) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   511
    LRG& lrg = lrgs(lid);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   512
    lrg._area += cost;
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   513
    raise_pressure(b, lrg, int_pressure, float_pressure);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   514
    lid = elements.next();
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   515
  }
22912
a8c042dca573 8032894: Remove dead code in Pressure::lower
adlertz
parents: 22804
diff changeset
   516
  assert(int_pressure.current_pressure() == count_int_pressure(liveout), "the int pressure is incorrect");
a8c042dca573 8032894: Remove dead code in Pressure::lower
adlertz
parents: 22804
diff changeset
   517
  assert(float_pressure.current_pressure() == count_float_pressure(liveout), "the float pressure is incorrect");
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   518
}
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   519
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   520
/*
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   521
 * Remove dead node if it's not used.
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   522
 * We only remove projection nodes if the node "defining" the projection is
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   523
 * dead, for example on x86, if we have a dead Add node we remove its
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   524
 * RFLAGS node.
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   525
 */
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   526
bool PhaseChaitin::remove_node_if_not_used(Block* b, uint location, Node* n, uint lid, IndexSet* liveout) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   527
  Node* def = n->in(0);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   528
  if (!n->is_Proj() ||
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   529
      (_lrg_map.live_range_id(def) && !liveout->member(_lrg_map.live_range_id(def)))) {
27704
88c9f3b507ff 8065618: C2 RA incorrectly removes kill projections
kvn
parents: 26913
diff changeset
   530
    if (n->is_MachProj()) {
88c9f3b507ff 8065618: C2 RA incorrectly removes kill projections
kvn
parents: 26913
diff changeset
   531
      // Don't remove KILL projections if their "defining" nodes have
88c9f3b507ff 8065618: C2 RA incorrectly removes kill projections
kvn
parents: 26913
diff changeset
   532
      // memory effects (have SCMemProj projection node) -
88c9f3b507ff 8065618: C2 RA incorrectly removes kill projections
kvn
parents: 26913
diff changeset
   533
      // they are not dead even when their result is not used.
88c9f3b507ff 8065618: C2 RA incorrectly removes kill projections
kvn
parents: 26913
diff changeset
   534
      // For example, compareAndSwapL (and other CAS) and EncodeISOArray nodes.
88c9f3b507ff 8065618: C2 RA incorrectly removes kill projections
kvn
parents: 26913
diff changeset
   535
      // The method add_input_to_liveout() keeps such nodes alive (put them on liveout list)
88c9f3b507ff 8065618: C2 RA incorrectly removes kill projections
kvn
parents: 26913
diff changeset
   536
      // when it sees SCMemProj node in a block. Unfortunately SCMemProj node could be placed
88c9f3b507ff 8065618: C2 RA incorrectly removes kill projections
kvn
parents: 26913
diff changeset
   537
      // in block in such order that KILL MachProj nodes are processed first.
28643
a665e19ca007 8066312: Add new Node* Node::find_out(int opc) method.
zmajo
parents: 27704
diff changeset
   538
      if (def->has_out_with(Op_SCMemProj)) {
a665e19ca007 8066312: Add new Node* Node::find_out(int opc) method.
zmajo
parents: 27704
diff changeset
   539
        return false;
27704
88c9f3b507ff 8065618: C2 RA incorrectly removes kill projections
kvn
parents: 26913
diff changeset
   540
      }
88c9f3b507ff 8065618: C2 RA incorrectly removes kill projections
kvn
parents: 26913
diff changeset
   541
    }
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   542
    b->remove_node(location);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   543
    LRG& lrg = lrgs(lid);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   544
    if (lrg._def == n) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   545
      lrg._def = 0;
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   546
    }
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   547
    n->disconnect_inputs(NULL, C);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   548
    _cfg.unmap_node_from_block(n);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   549
    n->replace_by(C->top());
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   550
    return true;
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   551
  }
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   552
  return false;
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   553
}
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   554
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   555
/*
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   556
 * When encountering a fat projection, we might go from a low to high to low
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   557
 * (since the fat proj only lives at this instruction) going backwards in the
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   558
 * block. If we find a low to high transition, we record it.
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   559
 */
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   560
void PhaseChaitin::check_for_high_pressure_transition_at_fatproj(uint& block_reg_pressure, uint location, LRG& lrg, Pressure& pressure, const int op_regtype) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   561
  RegMask mask_tmp = lrg.mask();
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   562
  mask_tmp.AND(*Matcher::idealreg2regmask[op_regtype]);
22912
a8c042dca573 8032894: Remove dead code in Pressure::lower
adlertz
parents: 22804
diff changeset
   563
  pressure.check_pressure_at_fatproj(location, mask_tmp);
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   564
}
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   565
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   566
/*
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   567
 * Insure high score for immediate-use spill copies so they get a color.
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   568
 * All single-use MachSpillCopy(s) that immediately precede their
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   569
 * use must color early.  If a longer live range steals their
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   570
 * color, the spill copy will split and may push another spill copy
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   571
 * further away resulting in an infinite spill-split-retry cycle.
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   572
 * Assigning a zero area results in a high score() and a good
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   573
 * location in the simplify list.
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   574
 */
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   575
void PhaseChaitin::assign_high_score_to_immediate_copies(Block* b, Node* n, LRG& lrg, uint next_inst, uint last_inst) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   576
  if (n->is_SpillCopy() &&
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   577
      lrg.is_singledef() && // A multi defined live range can still split
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   578
      n->outcnt() == 1 &&   // and use must be in this block
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   579
      _cfg.get_block_for_node(n->unique_out()) == b) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   580
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   581
    Node* single_use = n->unique_out();
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   582
    assert(b->find_node(single_use) >= next_inst, "Use must be later in block");
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   583
    // Use can be earlier in block if it is a Phi, but then I should be a MultiDef
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   584
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   585
    // Find first non SpillCopy 'm' that follows the current instruction
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   586
    // (current_inst - 1) is index for current instruction 'n'
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   587
    Node* m = n;
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   588
    for (uint i = next_inst; i <= last_inst && m->is_SpillCopy(); ++i) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   589
      m = b->get_node(i);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   590
    }
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   591
    if (m == single_use) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   592
      lrg._area = 0.0;
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   593
    }
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   594
  }
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   595
}
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   596
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   597
/*
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   598
 * Copies do not define a new value and so do not interfere.
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   599
 * Remove the copies source from the liveout set before interfering.
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   600
 */
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   601
void PhaseChaitin::remove_interference_from_copy(Block* b, uint location, uint lid_copy, IndexSet* liveout, double cost, Pressure& int_pressure, Pressure& float_pressure) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   602
  if (liveout->remove(lid_copy)) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   603
    LRG& lrg_copy = lrgs(lid_copy);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   604
    lrg_copy._area -= cost;
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   605
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   606
    // Lower register pressure since copy and definition can share the same register
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   607
    lower_pressure(b, location, lrg_copy, liveout, int_pressure, float_pressure);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   608
  }
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   609
}
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   610
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   611
/*
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   612
 * The defined value must go in a particular register. Remove that register from
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   613
 * all conflicting parties and avoid the interference.
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   614
 */
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   615
void PhaseChaitin::remove_bound_register_from_interfering_live_ranges(LRG& lrg, IndexSet* liveout, uint& must_spill) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   616
  // Check for common case
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   617
  const RegMask& rm = lrg.mask();
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   618
  int r_size = lrg.num_regs();
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   619
  // Smear odd bits
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   620
  IndexSetIterator elements(liveout);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   621
  uint l = elements.next();
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   622
  while (l != 0) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   623
    LRG& interfering_lrg = lrgs(l);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   624
    // If 'l' must spill already, do not further hack his bits.
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   625
    // He'll get some interferences and be forced to spill later.
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   626
    if (interfering_lrg._must_spill) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   627
      l = elements.next();
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   628
      continue;
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   629
    }
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   630
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   631
    // Remove bound register(s) from 'l's choices
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   632
    RegMask old = interfering_lrg.mask();
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   633
    uint old_size = interfering_lrg.mask_size();
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   634
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   635
    // Remove the bits from LRG 'rm' from LRG 'l' so 'l' no
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   636
    // longer interferes with 'rm'.  If 'l' requires aligned
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   637
    // adjacent pairs, subtract out bit pairs.
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   638
    assert(!interfering_lrg._is_vector || !interfering_lrg._fat_proj, "sanity");
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   639
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   640
    if (interfering_lrg.num_regs() > 1 && !interfering_lrg._fat_proj) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   641
      RegMask r2mask = rm;
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   642
      // Leave only aligned set of bits.
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   643
      r2mask.smear_to_sets(interfering_lrg.num_regs());
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   644
      // It includes vector case.
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   645
      interfering_lrg.SUBTRACT(r2mask);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   646
      interfering_lrg.compute_set_mask_size();
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   647
    } else if (r_size != 1) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   648
      // fat proj
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   649
      interfering_lrg.SUBTRACT(rm);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   650
      interfering_lrg.compute_set_mask_size();
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   651
    } else {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   652
      // Common case: size 1 bound removal
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   653
      OptoReg::Name r_reg = rm.find_first_elem();
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   654
      if (interfering_lrg.mask().Member(r_reg)) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   655
        interfering_lrg.Remove(r_reg);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   656
        interfering_lrg.set_mask_size(interfering_lrg.mask().is_AllStack() ? LRG::AllStack_size : old_size - 1);
19330
49d6711171e6 8023003: Cleanup the public interface to PhaseCFG
adlertz
parents: 19279
diff changeset
   657
      }
49d6711171e6 8023003: Cleanup the public interface to PhaseCFG
adlertz
parents: 19279
diff changeset
   658
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   659
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   660
    // If 'l' goes completely dry, it must spill.
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   661
    if (interfering_lrg.not_free()) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   662
      // Give 'l' some kind of reasonable mask, so it picks up
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   663
      // interferences (and will spill later).
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   664
      interfering_lrg.set_mask(old);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   665
      interfering_lrg.set_mask_size(old_size);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   666
      must_spill++;
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   667
      interfering_lrg._must_spill = 1;
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   668
      interfering_lrg.set_reg(OptoReg::Name(LRG::SPILL_REG));
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   669
    }
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   670
    l = elements.next();
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   671
  }
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   672
}
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   673
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   674
/*
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   675
 * Start loop at 1 (skip control edge) for most Nodes. SCMemProj's might be the
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   676
 * sole use of a StoreLConditional. While StoreLConditionals set memory (the
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   677
 * SCMemProj use) they also def flags; if that flag def is unused the allocator
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   678
 * sees a flag-setting instruction with no use of the flags and assumes it's
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   679
 * dead.  This keeps the (useless) flag-setting behavior alive while also
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   680
 * keeping the (useful) memory update effect.
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   681
 */
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   682
void PhaseChaitin::add_input_to_liveout(Block* b, Node* n, IndexSet* liveout, double cost, Pressure& int_pressure, Pressure& float_pressure) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   683
  JVMState* jvms = n->jvms();
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   684
  uint debug_start = jvms ? jvms->debug_start() : 999999;
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   685
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   686
  for (uint k = ((n->Opcode() == Op_SCMemProj) ? 0:1); k < n->req(); k++) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   687
    Node* def = n->in(k);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   688
    uint lid = _lrg_map.live_range_id(def);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   689
    if (!lid) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   690
      continue;
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   691
    }
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   692
    LRG& lrg = lrgs(lid);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   693
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   694
    // No use-side cost for spilling debug info
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   695
    if (k < debug_start) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   696
      // A USE costs twice block frequency (once for the Load, once
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   697
      // for a Load-delay).  Rematerialized uses only cost once.
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   698
      lrg._cost += (def->rematerialize() ? b->_freq : (b->_freq * 2));
2030
39d55e4534b4 6791852: assert(b->_nodes[insidx] == n,"got insidx set incorrectly")
kvn
parents: 1412
diff changeset
   699
    }
39d55e4534b4 6791852: assert(b->_nodes[insidx] == n,"got insidx set incorrectly")
kvn
parents: 1412
diff changeset
   700
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   701
    if (liveout->insert(lid)) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   702
      // Newly live things assumed live from here to top of block
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   703
      lrg._area += cost;
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   704
      raise_pressure(b, lrg, int_pressure, float_pressure);
22912
a8c042dca573 8032894: Remove dead code in Pressure::lower
adlertz
parents: 22804
diff changeset
   705
      assert(int_pressure.current_pressure() == count_int_pressure(liveout), "the int pressure is incorrect");
a8c042dca573 8032894: Remove dead code in Pressure::lower
adlertz
parents: 22804
diff changeset
   706
      assert(float_pressure.current_pressure() == count_float_pressure(liveout), "the float pressure is incorrect");
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   707
    }
22915
231c85af5482 8033260: assert(lrg._area >= 0.0) failed: negative spill area
adlertz
parents: 22912
diff changeset
   708
    assert(lrg._area >= 0.0, "negative spill area" );
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   709
  }
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   710
}
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   711
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   712
/*
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   713
 * If we run off the top of the block with high pressure just record that the
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   714
 * whole block is high pressure. (Even though we might have a transition
22912
a8c042dca573 8032894: Remove dead code in Pressure::lower
adlertz
parents: 22804
diff changeset
   715
 * later down in the block)
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   716
 */
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   717
void PhaseChaitin::check_for_high_pressure_block(Pressure& pressure) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   718
  // current pressure now means the pressure before the first instruction in the block
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   719
  // (since we have stepped through all instructions backwards)
22912
a8c042dca573 8032894: Remove dead code in Pressure::lower
adlertz
parents: 22804
diff changeset
   720
  if (pressure.current_pressure() > pressure.high_pressure_limit()) {
a8c042dca573 8032894: Remove dead code in Pressure::lower
adlertz
parents: 22804
diff changeset
   721
    pressure.set_high_pressure_index_to_block_start();
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   722
  }
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   723
}
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   724
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   725
/*
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   726
 * Compute high pressure indice; avoid landing in the middle of projnodes
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   727
 * and set the high pressure index for the block
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   728
 */
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   729
void PhaseChaitin::adjust_high_pressure_index(Block* b, uint& block_hrp_index, Pressure& pressure) {
22912
a8c042dca573 8032894: Remove dead code in Pressure::lower
adlertz
parents: 22804
diff changeset
   730
  uint i = pressure.high_pressure_index();
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   731
  if (i < b->number_of_nodes() && i < b->end_idx() + 1) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   732
    Node* cur = b->get_node(i);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   733
    while (cur->is_Proj() || (cur->is_MachNullCheck()) || cur->is_Catch()) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   734
      cur = b->get_node(--i);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   735
    }
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   736
  }
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   737
  block_hrp_index = i;
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   738
}
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   739
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   740
/* Build an interference graph:
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   741
 *   That is, if 2 live ranges are simultaneously alive but in their acceptable
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   742
 *   register sets do not overlap, then they do not interfere. The IFG is built
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   743
 *   by a single reverse pass over each basic block. Starting with the known
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   744
 *   live-out set, we remove things that get defined and add things that become
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   745
 *   live (essentially executing one pass of a standard LIVE analysis). Just
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   746
 *   before a Node defines a value (and removes it from the live-ness set) that
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   747
 *   value is certainly live. The defined value interferes with everything
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   748
 *   currently live. The value is then removed from the live-ness set and it's
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   749
 *   inputs are added to the live-ness set.
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   750
 * Compute register pressure for each block:
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   751
 *   We store the biggest register pressure for each block and also the first
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   752
 *   low to high register pressure transition within the block (if any).
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   753
 */
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   754
uint PhaseChaitin::build_ifg_physical( ResourceArea *a ) {
26913
9ad70cd32368 8058968: Compiler time traces should be improved
shade
parents: 24424
diff changeset
   755
  Compile::TracePhase tp("buildIFG", &timers[_t_buildIFGphysical]);
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   756
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   757
  uint must_spill = 0;
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   758
  for (uint i = 0; i < _cfg.number_of_blocks(); i++) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   759
    Block* block = _cfg.get_block(i);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   760
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   761
    // Clone (rather than smash in place) the liveout info, so it is alive
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   762
    // for the "collect_gc_info" phase later.
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   763
    IndexSet liveout(_live->live(block));
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   764
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   765
    uint first_inst = first_nonphi_index(block);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   766
    uint last_inst = block->end_idx();
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   767
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   768
    move_exception_node_up(block, first_inst, last_inst);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   769
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   770
    Pressure int_pressure(last_inst + 1, INTPRESSURE);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   771
    Pressure float_pressure(last_inst + 1, FLOATPRESSURE);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   772
    block->_reg_pressure = 0;
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   773
    block->_freg_pressure = 0;
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   774
2030
39d55e4534b4 6791852: assert(b->_nodes[insidx] == n,"got insidx set incorrectly")
kvn
parents: 1412
diff changeset
   775
    int inst_count = last_inst - first_inst;
19330
49d6711171e6 8023003: Cleanup the public interface to PhaseCFG
adlertz
parents: 19279
diff changeset
   776
    double cost = (inst_count <= 0) ? 0.0 : block->_freq * double(inst_count);
22915
231c85af5482 8033260: assert(lrg._area >= 0.0) failed: negative spill area
adlertz
parents: 22912
diff changeset
   777
    assert(cost >= 0.0, "negative spill cost" );
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   778
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   779
    compute_initial_block_pressure(block, &liveout, int_pressure, float_pressure, cost);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   780
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   781
    for (uint location = last_inst; location > 0; location--) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   782
      Node* n = block->get_node(location);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   783
      uint lid = _lrg_map.live_range_id(n);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   784
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   785
      if(lid) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   786
        LRG& lrg = lrgs(lid);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   787
489c9b5090e2 Initial load
duke
parents:
diff changeset
   788
        // A DEF normally costs block frequency; rematerialized values are
489c9b5090e2 Initial load
duke
parents:
diff changeset
   789
        // removed from the DEF sight, so LOWER costs here.
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   790
        lrg._cost += n->rematerialize() ? 0 : block->_freq;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   791
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   792
        if (!liveout.member(lid) && n->Opcode() != Op_SafePoint) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   793
          if (remove_node_if_not_used(block, location, n, lid, &liveout)) {
22912
a8c042dca573 8032894: Remove dead code in Pressure::lower
adlertz
parents: 22804
diff changeset
   794
            float_pressure.lower_high_pressure_index();
a8c042dca573 8032894: Remove dead code in Pressure::lower
adlertz
parents: 22804
diff changeset
   795
            int_pressure.lower_high_pressure_index();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   796
            continue;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   797
          }
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   798
          if (lrg._fat_proj) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   799
            check_for_high_pressure_transition_at_fatproj(block->_reg_pressure, location, lrg, int_pressure, Op_RegI);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   800
            check_for_high_pressure_transition_at_fatproj(block->_freg_pressure, location, lrg, float_pressure, Op_RegD);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   801
          }
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   802
        } else {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   803
          // A live range ends at its definition, remove the remaining area.
22915
231c85af5482 8033260: assert(lrg._area >= 0.0) failed: negative spill area
adlertz
parents: 22912
diff changeset
   804
          // If the cost is +Inf (which might happen in extreme cases), the lrg area will also be +Inf,
231c85af5482 8033260: assert(lrg._area >= 0.0) failed: negative spill area
adlertz
parents: 22912
diff changeset
   805
          // and +Inf - +Inf = NaN. So let's not do that subtraction.
231c85af5482 8033260: assert(lrg._area >= 0.0) failed: negative spill area
adlertz
parents: 22912
diff changeset
   806
          if (g_isfinite(cost)) {
231c85af5482 8033260: assert(lrg._area >= 0.0) failed: negative spill area
adlertz
parents: 22912
diff changeset
   807
            lrg._area -= cost;
231c85af5482 8033260: assert(lrg._area >= 0.0) failed: negative spill area
adlertz
parents: 22912
diff changeset
   808
          }
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   809
          assert(lrg._area >= 0.0, "negative spill area" );
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   810
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   811
          assign_high_score_to_immediate_copies(block, n, lrg, location + 1, last_inst);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   812
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   813
          if (liveout.remove(lid)) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   814
            lower_pressure(block, location, lrg, &liveout, int_pressure, float_pressure);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   815
          }
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   816
          uint copy_idx = n->is_Copy();
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   817
          if (copy_idx) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   818
            uint lid_copy = _lrg_map.live_range_id(n->in(copy_idx));
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   819
            remove_interference_from_copy(block, location, lid_copy, &liveout, cost, int_pressure, float_pressure);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   820
          }
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   821
        }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   822
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   823
        // Since rematerializable DEFs are not bound but the live range is,
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   824
        // some uses must be bound. If we spill live range 'r', it can
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   825
        // rematerialize at each use site according to its bindings.
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   826
        if (lrg.is_bound() && !n->rematerialize() && lrg.mask().is_NotEmpty()) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   827
          remove_bound_register_from_interfering_live_ranges(lrg, &liveout, must_spill);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   828
        }
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   829
        interfere_with_live(lid, &liveout);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   830
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   831
1401
e5fdc8521d1f 6750588: assert(lrg._area >= 0,"negative spill area") running NSK stmp0101 test
rasbold
parents: 1057
diff changeset
   832
      // Area remaining in the block
e5fdc8521d1f 6750588: assert(lrg._area >= 0,"negative spill area") running NSK stmp0101 test
rasbold
parents: 1057
diff changeset
   833
      inst_count--;
19330
49d6711171e6 8023003: Cleanup the public interface to PhaseCFG
adlertz
parents: 19279
diff changeset
   834
      cost = (inst_count <= 0) ? 0.0 : block->_freq * double(inst_count);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   835
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   836
      if (!n->is_Phi()) {
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   837
        add_input_to_liveout(block, n, &liveout, cost, int_pressure, float_pressure);
19330
49d6711171e6 8023003: Cleanup the public interface to PhaseCFG
adlertz
parents: 19279
diff changeset
   838
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   839
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   840
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   841
    check_for_high_pressure_block(int_pressure);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   842
    check_for_high_pressure_block(float_pressure);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   843
    adjust_high_pressure_index(block, block->_ihrp_index, int_pressure);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   844
    adjust_high_pressure_index(block, block->_fhrp_index, float_pressure);
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   845
    // set the final_pressure as the register pressure for the block
22912
a8c042dca573 8032894: Remove dead code in Pressure::lower
adlertz
parents: 22804
diff changeset
   846
    block->_reg_pressure = int_pressure.final_pressure();
a8c042dca573 8032894: Remove dead code in Pressure::lower
adlertz
parents: 22804
diff changeset
   847
    block->_freg_pressure = float_pressure.final_pressure();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   848
489c9b5090e2 Initial load
duke
parents:
diff changeset
   849
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   850
    // Gather Register Pressure Statistics
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   851
    if (PrintOptoStatistics) {
22912
a8c042dca573 8032894: Remove dead code in Pressure::lower
adlertz
parents: 22804
diff changeset
   852
      if (block->_reg_pressure > int_pressure.high_pressure_limit() || block->_freg_pressure > float_pressure.high_pressure_limit()) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   853
        _high_pressure++;
19330
49d6711171e6 8023003: Cleanup the public interface to PhaseCFG
adlertz
parents: 19279
diff changeset
   854
      } else {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   855
        _low_pressure++;
19330
49d6711171e6 8023003: Cleanup the public interface to PhaseCFG
adlertz
parents: 19279
diff changeset
   856
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   857
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   858
#endif
22804
401135897b65 8031498: Cleanup and re-factorize PhaseChaitin::build_ifg_physical
adlertz
parents: 22234
diff changeset
   859
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   860
489c9b5090e2 Initial load
duke
parents:
diff changeset
   861
  return must_spill;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   862
}