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