src/hotspot/share/opto/lcm.cpp
author lkorinth
Wed, 13 Nov 2019 11:37:29 +0100
changeset 59053 ba6c248cae19
parent 55563 d56b192c73e9
permissions -rw-r--r--
8232365: Implementation for JEP 363: Remove the Concurrent Mark Sweep (CMS) Garbage Collector Reviewed-by: kbarrett, tschatzl, erikj, coleenp, dholmes
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     1
/*
59053
ba6c248cae19 8232365: Implementation for JEP 363: Remove the Concurrent Mark Sweep (CMS) Garbage Collector
lkorinth
parents: 55563
diff changeset
     2
 * Copyright (c) 1998, 2019, 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: 4580
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 4580
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: 4580
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: 6418
diff changeset
    25
#include "precompiled.hpp"
49480
d7df2dd501ce 8199809: Don't include frame.inline.hpp and other.inline.hpp from .hpp files
coleenp
parents: 47216
diff changeset
    26
#include "asm/macroAssembler.inline.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6418
diff changeset
    27
#include "memory/allocation.inline.hpp"
54780
f8d182aedc92 8223136: Move compressed oops functions to CompressedOops class
stefank
parents: 54423
diff changeset
    28
#include "oops/compressedOops.hpp"
25715
d5a8dbdc5150 8049325: Introduce and clean up umbrella headers for the files in the cpu subdirectories.
goetz
parents: 24923
diff changeset
    29
#include "opto/ad.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6418
diff changeset
    30
#include "opto/block.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6418
diff changeset
    31
#include "opto/c2compiler.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6418
diff changeset
    32
#include "opto/callnode.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6418
diff changeset
    33
#include "opto/cfgnode.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6418
diff changeset
    34
#include "opto/machnode.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6418
diff changeset
    35
#include "opto/runtime.hpp"
33065
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
    36
#include "opto/chaitin.hpp"
29081
c61eb4914428 8072911: Remove includes of oop.inline.hpp from .hpp files
stefank
parents: 28188
diff changeset
    37
#include "runtime/sharedRuntime.hpp"
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    38
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6418
diff changeset
    39
// Optimization - Graph Style
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    40
22852
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
    41
// Check whether val is not-null-decoded compressed oop,
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
    42
// i.e. will grab into the base of the heap if it represents NULL.
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
    43
static bool accesses_heap_base_zone(Node *val) {
54780
f8d182aedc92 8223136: Move compressed oops functions to CompressedOops class
stefank
parents: 54423
diff changeset
    44
  if (CompressedOops::base() != NULL) { // Implies UseCompressedOops.
22852
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
    45
    if (val && val->is_Mach()) {
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
    46
      if (val->as_Mach()->ideal_Opcode() == Op_DecodeN) {
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
    47
        // This assumes all Decodes with TypePtr::NotNull are matched to nodes that
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
    48
        // decode NULL to point to the heap base (Decode_NN).
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
    49
        if (val->bottom_type()->is_oopptr()->ptr() == TypePtr::NotNull) {
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
    50
          return true;
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
    51
        }
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
    52
      }
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
    53
      // Must recognize load operation with Decode matched in memory operand.
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
    54
      // We should not reach here exept for PPC/AIX, as os::zero_page_read_protected()
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
    55
      // returns true everywhere else. On PPC, no such memory operands
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
    56
      // exist, therefore we did not yet implement a check for such operands.
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
    57
      NOT_AIX(Unimplemented());
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
    58
    }
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
    59
  }
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
    60
  return false;
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
    61
}
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
    62
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
    63
static bool needs_explicit_null_check_for_read(Node *val) {
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
    64
  // On some OSes (AIX) the page at address 0 is only write protected.
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
    65
  // If so, only Store operations will trap.
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
    66
  if (os::zero_page_read_protected()) {
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
    67
    return false;  // Implicit null check will work.
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
    68
  }
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
    69
  // Also a read accessing the base of a heap-based compressed heap will trap.
54780
f8d182aedc92 8223136: Move compressed oops functions to CompressedOops class
stefank
parents: 54423
diff changeset
    70
  if (accesses_heap_base_zone(val) &&         // Hits the base zone page.
f8d182aedc92 8223136: Move compressed oops functions to CompressedOops class
stefank
parents: 54423
diff changeset
    71
      CompressedOops::use_implicit_null_checks()) { // Base zone page is protected.
22852
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
    72
    return false;
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
    73
  }
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
    74
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
    75
  return true;
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
    76
}
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
    77
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    78
//------------------------------implicit_null_check----------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
    79
// Detect implicit-null-check opportunities.  Basically, find NULL checks
489c9b5090e2 Initial load
duke
parents:
diff changeset
    80
// with suitable memory ops nearby.  Use the memory op to do the NULL check.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    81
// I can generate a memory op if there is not one nearby.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    82
// The proj is the control projection for the not-null case.
5698
091095915ee6 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 4580
diff changeset
    83
// The val is the pointer being checked for nullness or
091095915ee6 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 4580
diff changeset
    84
// decodeHeapOop_not_null node if it did not fold into address.
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
    85
void PhaseCFG::implicit_null_check(Block* block, Node *proj, Node *val, int allowed_reasons) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    86
  // Assume if null check need for 0 offset then always needed
489c9b5090e2 Initial load
duke
parents:
diff changeset
    87
  // Intel solaris doesn't support any null checks yet and no
489c9b5090e2 Initial load
duke
parents:
diff changeset
    88
  // mechanism exists (yet) to set the switches at an os_cpu level
489c9b5090e2 Initial load
duke
parents:
diff changeset
    89
  if( !ImplicitNullChecks || MacroAssembler::needs_explicit_null_check(0)) return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    90
489c9b5090e2 Initial load
duke
parents:
diff changeset
    91
  // Make sure the ptr-is-null path appears to be uncommon!
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
    92
  float f = block->end()->as_MachIf()->_prob;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    93
  if( proj->Opcode() == Op_IfTrue ) f = 1.0f - f;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    94
  if( f > PROB_UNLIKELY_MAG(4) ) return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    95
489c9b5090e2 Initial load
duke
parents:
diff changeset
    96
  uint bidx = 0;                // Capture index of value into memop
489c9b5090e2 Initial load
duke
parents:
diff changeset
    97
  bool was_store;               // Memory op is a store op
489c9b5090e2 Initial load
duke
parents:
diff changeset
    98
489c9b5090e2 Initial load
duke
parents:
diff changeset
    99
  // Get the successor block for if the test ptr is non-null
489c9b5090e2 Initial load
duke
parents:
diff changeset
   100
  Block* not_null_block;  // this one goes with the proj
489c9b5090e2 Initial load
duke
parents:
diff changeset
   101
  Block* null_block;
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   102
  if (block->get_node(block->number_of_nodes()-1) == proj) {
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   103
    null_block     = block->_succs[0];
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   104
    not_null_block = block->_succs[1];
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   105
  } else {
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   106
    assert(block->get_node(block->number_of_nodes()-2) == proj, "proj is one or the other");
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   107
    not_null_block = block->_succs[0];
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   108
    null_block     = block->_succs[1];
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   109
  }
1136
9f6351874414 6746907: Improve implicit null check generation
kvn
parents: 961
diff changeset
   110
  while (null_block->is_Empty() == Block::empty_with_goto) {
9f6351874414 6746907: Improve implicit null check generation
kvn
parents: 961
diff changeset
   111
    null_block     = null_block->_succs[0];
9f6351874414 6746907: Improve implicit null check generation
kvn
parents: 961
diff changeset
   112
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   113
489c9b5090e2 Initial load
duke
parents:
diff changeset
   114
  // Search the exception block for an uncommon trap.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   115
  // (See Parse::do_if and Parse::do_ifnull for the reason
489c9b5090e2 Initial load
duke
parents:
diff changeset
   116
  // we need an uncommon trap.  Briefly, we need a way to
489c9b5090e2 Initial load
duke
parents:
diff changeset
   117
  // detect failure of this optimization, as in 6366351.)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   118
  {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   119
    bool found_trap = false;
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   120
    for (uint i1 = 0; i1 < null_block->number_of_nodes(); i1++) {
19717
7819ffdaf0ff 8023691: Create interface for nodes in class Block
adlertz
parents: 19330
diff changeset
   121
      Node* nn = null_block->get_node(i1);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   122
      if (nn->is_MachCall() &&
6418
6671edbd230e 6978355: renaming for 6961697
twisti
parents: 6188
diff changeset
   123
          nn->as_MachCall()->entry_point() == SharedRuntime::uncommon_trap_blob()->entry_point()) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   124
        const Type* trtype = nn->in(TypeFunc::Parms)->bottom_type();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   125
        if (trtype->isa_int() && trtype->is_int()->is_con()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   126
          jint tr_con = trtype->is_int()->get_con();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   127
          Deoptimization::DeoptReason reason = Deoptimization::trap_request_reason(tr_con);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   128
          Deoptimization::DeoptAction action = Deoptimization::trap_request_action(tr_con);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   129
          assert((int)reason < (int)BitsPerInt, "recode bit map");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   130
          if (is_set_nth_bit(allowed_reasons, (int) reason)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   131
              && action != Deoptimization::Action_none) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   132
            // This uncommon trap is sure to recompile, eventually.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   133
            // When that happens, C->too_many_traps will prevent
489c9b5090e2 Initial load
duke
parents:
diff changeset
   134
            // this transformation from happening again.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   135
            found_trap = true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   136
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   137
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   138
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   139
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   140
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   141
    if (!found_trap) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   142
      // We did not find an uncommon trap.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   143
      return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   144
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   145
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   146
5698
091095915ee6 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 4580
diff changeset
   147
  // Check for decodeHeapOop_not_null node which did not fold into address
091095915ee6 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 4580
diff changeset
   148
  bool is_decoden = ((intptr_t)val) & 1;
091095915ee6 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 4580
diff changeset
   149
  val = (Node*)(((intptr_t)val) & ~1);
091095915ee6 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 4580
diff changeset
   150
091095915ee6 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 4580
diff changeset
   151
  assert(!is_decoden || (val->in(0) == NULL) && val->is_Mach() &&
091095915ee6 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 4580
diff changeset
   152
         (val->as_Mach()->ideal_Opcode() == Op_DecodeN), "sanity");
091095915ee6 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 4580
diff changeset
   153
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   154
  // Search the successor block for a load or store who's base value is also
489c9b5090e2 Initial load
duke
parents:
diff changeset
   155
  // the tested value.  There may be several.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   156
  Node_List *out = new Node_List(Thread::current()->resource_area());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   157
  MachNode *best = NULL;        // Best found so far
489c9b5090e2 Initial load
duke
parents:
diff changeset
   158
  for (DUIterator i = val->outs(); val->has_out(i); i++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   159
    Node *m = val->out(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   160
    if( !m->is_Mach() ) continue;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   161
    MachNode *mach = m->as_Mach();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   162
    was_store = false;
6188
95ea4d66089a 6973329: C2 with Zero based COOP produces code with broken anti-dependency on x86
kvn
parents: 6180
diff changeset
   163
    int iop = mach->ideal_Opcode();
95ea4d66089a 6973329: C2 with Zero based COOP produces code with broken anti-dependency on x86
kvn
parents: 6180
diff changeset
   164
    switch( iop ) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   165
    case Op_LoadB:
13104
657b387034fb 7119644: Increase superword's vector size up to 256 bits
kvn
parents: 11567
diff changeset
   166
    case Op_LoadUB:
2022
28ce8115a91d 6796746: rename LoadC (char) opcode class to LoadUS (unsigned short)
twisti
parents: 1397
diff changeset
   167
    case Op_LoadUS:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   168
    case Op_LoadD:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   169
    case Op_LoadF:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   170
    case Op_LoadI:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   171
    case Op_LoadL:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   172
    case Op_LoadP:
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 1
diff changeset
   173
    case Op_LoadN:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   174
    case Op_LoadS:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   175
    case Op_LoadKlass:
590
2954744d7bba 6703890: Compressed Oops: add LoadNKlass node to generate narrow oops (32-bits) compare instructions
kvn
parents: 372
diff changeset
   176
    case Op_LoadNKlass:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   177
    case Op_LoadRange:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   178
    case Op_LoadD_unaligned:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   179
    case Op_LoadL_unaligned:
4580
db05951a970c 6912517: JIT bug compiles out (and stops running) code that needs to be run. Causes NPE.
kvn
parents: 4566
diff changeset
   180
      assert(mach->in(2) == val, "should be address");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   181
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   182
    case Op_StoreB:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   183
    case Op_StoreC:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   184
    case Op_StoreCM:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   185
    case Op_StoreD:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   186
    case Op_StoreF:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   187
    case Op_StoreI:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   188
    case Op_StoreL:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   189
    case Op_StoreP:
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 1
diff changeset
   190
    case Op_StoreN:
13969
d2a189b83b87 7054512: Compress class pointers after perm gen removal
roland
parents: 13895
diff changeset
   191
    case Op_StoreNKlass:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   192
      was_store = true;         // Memory op is a store op
489c9b5090e2 Initial load
duke
parents:
diff changeset
   193
      // Stores will have their address in slot 2 (memory in slot 1).
489c9b5090e2 Initial load
duke
parents:
diff changeset
   194
      // If the value being nul-checked is in another slot, it means we
489c9b5090e2 Initial load
duke
parents:
diff changeset
   195
      // are storing the checked value, which does NOT check the value!
489c9b5090e2 Initial load
duke
parents:
diff changeset
   196
      if( mach->in(2) != val ) continue;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   197
      break;                    // Found a memory op?
489c9b5090e2 Initial load
duke
parents:
diff changeset
   198
    case Op_StrComp:
2348
4e71ed4c2709 6761600: Use sse 4.2 in intrinsics
cfang
parents: 2259
diff changeset
   199
    case Op_StrEquals:
4e71ed4c2709 6761600: Use sse 4.2 in intrinsics
cfang
parents: 2259
diff changeset
   200
    case Op_StrIndexOf:
33628
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 33065
diff changeset
   201
    case Op_StrIndexOfChar:
595
a2be4c89de81 6695049: (coll) Create an x86 intrinsic for Arrays.equals
rasbold
parents: 590
diff changeset
   202
    case Op_AryEq:
33628
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 33065
diff changeset
   203
    case Op_StrInflatedCopy:
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 33065
diff changeset
   204
    case Op_StrCompressedCopy:
15242
695bb216be99 6896617: Optimize sun.nio.cs.ISO_8859_1$Encode.encodeArrayLoop() on x86
kvn
parents: 14623
diff changeset
   205
    case Op_EncodeISOArray:
33628
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 33065
diff changeset
   206
    case Op_HasNegatives:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   207
      // Not a legit memory op for implicit null check regardless of
489c9b5090e2 Initial load
duke
parents:
diff changeset
   208
      // embedded loads
489c9b5090e2 Initial load
duke
parents:
diff changeset
   209
      continue;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   210
    default:                    // Also check for embedded loads
489c9b5090e2 Initial load
duke
parents:
diff changeset
   211
      if( !mach->needs_anti_dependence_check() )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   212
        continue;               // Not an memory op; skip it
6188
95ea4d66089a 6973329: C2 with Zero based COOP produces code with broken anti-dependency on x86
kvn
parents: 6180
diff changeset
   213
      if( must_clone[iop] ) {
95ea4d66089a 6973329: C2 with Zero based COOP produces code with broken anti-dependency on x86
kvn
parents: 6180
diff changeset
   214
        // Do not move nodes which produce flags because
95ea4d66089a 6973329: C2 with Zero based COOP produces code with broken anti-dependency on x86
kvn
parents: 6180
diff changeset
   215
        // RA will try to clone it to place near branch and
95ea4d66089a 6973329: C2 with Zero based COOP produces code with broken anti-dependency on x86
kvn
parents: 6180
diff changeset
   216
        // it will cause recompilation, see clone_node().
95ea4d66089a 6973329: C2 with Zero based COOP produces code with broken anti-dependency on x86
kvn
parents: 6180
diff changeset
   217
        continue;
95ea4d66089a 6973329: C2 with Zero based COOP produces code with broken anti-dependency on x86
kvn
parents: 6180
diff changeset
   218
      }
4580
db05951a970c 6912517: JIT bug compiles out (and stops running) code that needs to be run. Causes NPE.
kvn
parents: 4566
diff changeset
   219
      {
5698
091095915ee6 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 4580
diff changeset
   220
        // Check that value is used in memory address in
091095915ee6 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 4580
diff changeset
   221
        // instructions with embedded load (CmpP val1,(val2+off)).
4580
db05951a970c 6912517: JIT bug compiles out (and stops running) code that needs to be run. Causes NPE.
kvn
parents: 4566
diff changeset
   222
        Node* base;
db05951a970c 6912517: JIT bug compiles out (and stops running) code that needs to be run. Causes NPE.
kvn
parents: 4566
diff changeset
   223
        Node* index;
db05951a970c 6912517: JIT bug compiles out (and stops running) code that needs to be run. Causes NPE.
kvn
parents: 4566
diff changeset
   224
        const MachOper* oper = mach->memory_inputs(base, index);
db05951a970c 6912517: JIT bug compiles out (and stops running) code that needs to be run. Causes NPE.
kvn
parents: 4566
diff changeset
   225
        if (oper == NULL || oper == (MachOper*)-1) {
db05951a970c 6912517: JIT bug compiles out (and stops running) code that needs to be run. Causes NPE.
kvn
parents: 4566
diff changeset
   226
          continue;             // Not an memory op; skip it
db05951a970c 6912517: JIT bug compiles out (and stops running) code that needs to be run. Causes NPE.
kvn
parents: 4566
diff changeset
   227
        }
db05951a970c 6912517: JIT bug compiles out (and stops running) code that needs to be run. Causes NPE.
kvn
parents: 4566
diff changeset
   228
        if (val == base ||
46630
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 44403
diff changeset
   229
            (val == index && val->bottom_type()->isa_narrowoop())) {
4580
db05951a970c 6912517: JIT bug compiles out (and stops running) code that needs to be run. Causes NPE.
kvn
parents: 4566
diff changeset
   230
          break;                // Found it
db05951a970c 6912517: JIT bug compiles out (and stops running) code that needs to be run. Causes NPE.
kvn
parents: 4566
diff changeset
   231
        } else {
db05951a970c 6912517: JIT bug compiles out (and stops running) code that needs to be run. Causes NPE.
kvn
parents: 4566
diff changeset
   232
          continue;             // Skip it
db05951a970c 6912517: JIT bug compiles out (and stops running) code that needs to be run. Causes NPE.
kvn
parents: 4566
diff changeset
   233
        }
db05951a970c 6912517: JIT bug compiles out (and stops running) code that needs to be run. Causes NPE.
kvn
parents: 4566
diff changeset
   234
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   235
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   236
    }
22852
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
   237
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
   238
    // On some OSes (AIX) the page at address 0 is only write protected.
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
   239
    // If so, only Store operations will trap.
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
   240
    // But a read accessing the base of a heap-based compressed heap will trap.
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
   241
    if (!was_store && needs_explicit_null_check_for_read(val)) {
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
   242
      continue;
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
   243
    }
1063026e8cee 8028471: PPC64 (part 215): opto: Extend ImplicitNullCheck optimization.
goetz
parents: 22838
diff changeset
   244
40862
3f9cd7a4bfa7 8162496: missing precedence edge for anti_dependence
kvn
parents: 40086
diff changeset
   245
    // Check that node's control edge is not-null block's head or dominates it,
3f9cd7a4bfa7 8162496: missing precedence edge for anti_dependence
kvn
parents: 40086
diff changeset
   246
    // otherwise we can't hoist it because there are other control dependencies.
3f9cd7a4bfa7 8162496: missing precedence edge for anti_dependence
kvn
parents: 40086
diff changeset
   247
    Node* ctrl = mach->in(0);
3f9cd7a4bfa7 8162496: missing precedence edge for anti_dependence
kvn
parents: 40086
diff changeset
   248
    if (ctrl != NULL && !(ctrl == not_null_block->head() ||
3f9cd7a4bfa7 8162496: missing precedence edge for anti_dependence
kvn
parents: 40086
diff changeset
   249
        get_block_for_node(ctrl)->dominates(not_null_block))) {
3f9cd7a4bfa7 8162496: missing precedence edge for anti_dependence
kvn
parents: 40086
diff changeset
   250
      continue;
3f9cd7a4bfa7 8162496: missing precedence edge for anti_dependence
kvn
parents: 40086
diff changeset
   251
    }
3f9cd7a4bfa7 8162496: missing precedence edge for anti_dependence
kvn
parents: 40086
diff changeset
   252
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   253
    // check if the offset is not too high for implicit exception
489c9b5090e2 Initial load
duke
parents:
diff changeset
   254
    {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   255
      intptr_t offset = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   256
      const TypePtr *adr_type = NULL;  // Do not need this return value here
489c9b5090e2 Initial load
duke
parents:
diff changeset
   257
      const Node* base = mach->get_base_and_disp(offset, adr_type);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   258
      if (base == NULL || base == NodeSentinel) {
44403
1b4e73257070 8176518: C2: Invalid ImplicitNullChecks with non-protected heap base
mdoerr
parents: 42081
diff changeset
   259
        // Narrow oop address doesn't have base, only index.
1b4e73257070 8176518: C2: Invalid ImplicitNullChecks with non-protected heap base
mdoerr
parents: 42081
diff changeset
   260
        // Give up if offset is beyond page size or if heap base is not protected.
1b4e73257070 8176518: C2: Invalid ImplicitNullChecks with non-protected heap base
mdoerr
parents: 42081
diff changeset
   261
        if (val->bottom_type()->isa_narrowoop() &&
1b4e73257070 8176518: C2: Invalid ImplicitNullChecks with non-protected heap base
mdoerr
parents: 42081
diff changeset
   262
            (MacroAssembler::needs_explicit_null_check(offset) ||
54780
f8d182aedc92 8223136: Move compressed oops functions to CompressedOops class
stefank
parents: 54423
diff changeset
   263
             !CompressedOops::use_implicit_null_checks()))
44403
1b4e73257070 8176518: C2: Invalid ImplicitNullChecks with non-protected heap base
mdoerr
parents: 42081
diff changeset
   264
          continue;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   265
        // cannot reason about it; is probably not implicit null exception
489c9b5090e2 Initial load
duke
parents:
diff changeset
   266
      } else {
2254
f13dda645a4b 6791178: Specialize for zero as the compressed oop vm heap base
kvn
parents: 2022
diff changeset
   267
        const TypePtr* tptr;
54780
f8d182aedc92 8223136: Move compressed oops functions to CompressedOops class
stefank
parents: 54423
diff changeset
   268
        if (UseCompressedOops && (CompressedOops::shift() == 0 ||
f8d182aedc92 8223136: Move compressed oops functions to CompressedOops class
stefank
parents: 54423
diff changeset
   269
                                  CompressedKlassPointers::shift() == 0)) {
2254
f13dda645a4b 6791178: Specialize for zero as the compressed oop vm heap base
kvn
parents: 2022
diff changeset
   270
          // 32-bits narrow oop can be the base of address expressions
17384
4e6ea5fa04ad 8014189: JVM crash with SEGV in ConnectionGraph::record_for_escape_analysis()
kvn
parents: 15871
diff changeset
   271
          tptr = base->get_ptr_type();
2254
f13dda645a4b 6791178: Specialize for zero as the compressed oop vm heap base
kvn
parents: 2022
diff changeset
   272
        } else {
f13dda645a4b 6791178: Specialize for zero as the compressed oop vm heap base
kvn
parents: 2022
diff changeset
   273
          // only regular oops are expected here
f13dda645a4b 6791178: Specialize for zero as the compressed oop vm heap base
kvn
parents: 2022
diff changeset
   274
          tptr = base->bottom_type()->is_ptr();
f13dda645a4b 6791178: Specialize for zero as the compressed oop vm heap base
kvn
parents: 2022
diff changeset
   275
        }
44403
1b4e73257070 8176518: C2: Invalid ImplicitNullChecks with non-protected heap base
mdoerr
parents: 42081
diff changeset
   276
        // Give up if offset is not a compile-time constant.
1b4e73257070 8176518: C2: Invalid ImplicitNullChecks with non-protected heap base
mdoerr
parents: 42081
diff changeset
   277
        if (offset == Type::OffsetBot || tptr->_offset == Type::OffsetBot)
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   278
          continue;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   279
        offset += tptr->_offset; // correct if base is offseted
44403
1b4e73257070 8176518: C2: Invalid ImplicitNullChecks with non-protected heap base
mdoerr
parents: 42081
diff changeset
   280
        // Give up if reference is beyond page size.
1b4e73257070 8176518: C2: Invalid ImplicitNullChecks with non-protected heap base
mdoerr
parents: 42081
diff changeset
   281
        if (MacroAssembler::needs_explicit_null_check(offset))
1b4e73257070 8176518: C2: Invalid ImplicitNullChecks with non-protected heap base
mdoerr
parents: 42081
diff changeset
   282
          continue;
1b4e73257070 8176518: C2: Invalid ImplicitNullChecks with non-protected heap base
mdoerr
parents: 42081
diff changeset
   283
        // Give up if base is a decode node and the heap base is not protected.
1b4e73257070 8176518: C2: Invalid ImplicitNullChecks with non-protected heap base
mdoerr
parents: 42081
diff changeset
   284
        if (base->is_Mach() && base->as_Mach()->ideal_Opcode() == Op_DecodeN &&
54780
f8d182aedc92 8223136: Move compressed oops functions to CompressedOops class
stefank
parents: 54423
diff changeset
   285
            !CompressedOops::use_implicit_null_checks())
44403
1b4e73257070 8176518: C2: Invalid ImplicitNullChecks with non-protected heap base
mdoerr
parents: 42081
diff changeset
   286
          continue;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   287
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   288
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   289
489c9b5090e2 Initial load
duke
parents:
diff changeset
   290
    // Check ctrl input to see if the null-check dominates the memory op
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   291
    Block *cb = get_block_for_node(mach);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   292
    cb = cb->_idom;             // Always hoist at least 1 block
489c9b5090e2 Initial load
duke
parents:
diff changeset
   293
    if( !was_store ) {          // Stores can be hoisted only one block
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   294
      while( cb->_dom_depth > (block->_dom_depth + 1))
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   295
        cb = cb->_idom;         // Hoist loads as far as we want
489c9b5090e2 Initial load
duke
parents:
diff changeset
   296
      // The non-null-block should dominate the memory op, too. Live
489c9b5090e2 Initial load
duke
parents:
diff changeset
   297
      // range spilling will insert a spill in the non-null-block if it is
489c9b5090e2 Initial load
duke
parents:
diff changeset
   298
      // needs to spill the memory op for an implicit null check.
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   299
      if (cb->_dom_depth == (block->_dom_depth + 1)) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   300
        if (cb != not_null_block) continue;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   301
        cb = cb->_idom;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   302
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   303
    }
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   304
    if( cb != block ) continue;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   305
489c9b5090e2 Initial load
duke
parents:
diff changeset
   306
    // Found a memory user; see if it can be hoisted to check-block
489c9b5090e2 Initial load
duke
parents:
diff changeset
   307
    uint vidx = 0;              // Capture index of value into memop
489c9b5090e2 Initial load
duke
parents:
diff changeset
   308
    uint j;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   309
    for( j = mach->req()-1; j > 0; j-- ) {
5698
091095915ee6 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 4580
diff changeset
   310
      if( mach->in(j) == val ) {
091095915ee6 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 4580
diff changeset
   311
        vidx = j;
091095915ee6 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 4580
diff changeset
   312
        // Ignore DecodeN val which could be hoisted to where needed.
091095915ee6 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 4580
diff changeset
   313
        if( is_decoden ) continue;
091095915ee6 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 4580
diff changeset
   314
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   315
      // Block of memory-op input
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   316
      Block *inb = get_block_for_node(mach->in(j));
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   317
      Block *b = block;          // Start from nul check
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   318
      while( b != inb && b->_dom_depth > inb->_dom_depth )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   319
        b = b->_idom;           // search upwards for input
489c9b5090e2 Initial load
duke
parents:
diff changeset
   320
      // See if input dominates null check
489c9b5090e2 Initial load
duke
parents:
diff changeset
   321
      if( b != inb )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   322
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   323
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   324
    if( j > 0 )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   325
      continue;
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   326
    Block *mb = get_block_for_node(mach);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   327
    // Hoisting stores requires more checks for the anti-dependence case.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   328
    // Give up hoisting if we have to move the store past any load.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   329
    if( was_store ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   330
      Block *b = mb;            // Start searching here for a local load
489c9b5090e2 Initial load
duke
parents:
diff changeset
   331
      // mach use (faulting) trying to hoist
489c9b5090e2 Initial load
duke
parents:
diff changeset
   332
      // n might be blocker to hoisting
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   333
      while( b != block ) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   334
        uint k;
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   335
        for( k = 1; k < b->number_of_nodes(); k++ ) {
19717
7819ffdaf0ff 8023691: Create interface for nodes in class Block
adlertz
parents: 19330
diff changeset
   336
          Node *n = b->get_node(k);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   337
          if( n->needs_anti_dependence_check() &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
   338
              n->in(LoadNode::Memory) == mach->in(StoreNode::Memory) )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   339
            break;              // Found anti-dependent load
489c9b5090e2 Initial load
duke
parents:
diff changeset
   340
        }
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   341
        if( k < b->number_of_nodes() )
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   342
          break;                // Found anti-dependent load
489c9b5090e2 Initial load
duke
parents:
diff changeset
   343
        // Make sure control does not do a merge (would have to check allpaths)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   344
        if( b->num_preds() != 2 ) break;
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   345
        b = get_block_for_node(b->pred(1)); // Move up to predecessor block
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   346
      }
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   347
      if( b != block ) continue;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   348
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   349
489c9b5090e2 Initial load
duke
parents:
diff changeset
   350
    // Make sure this memory op is not already being used for a NullCheck
489c9b5090e2 Initial load
duke
parents:
diff changeset
   351
    Node *e = mb->end();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   352
    if( e->is_MachNullCheck() && e->in(1) == mach )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   353
      continue;                 // Already being used as a NULL check
489c9b5090e2 Initial load
duke
parents:
diff changeset
   354
489c9b5090e2 Initial load
duke
parents:
diff changeset
   355
    // Found a candidate!  Pick one with least dom depth - the highest
489c9b5090e2 Initial load
duke
parents:
diff changeset
   356
    // in the dom tree should be closest to the null check.
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   357
    if (best == NULL || get_block_for_node(mach)->_dom_depth < get_block_for_node(best)->_dom_depth) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   358
      best = mach;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   359
      bidx = vidx;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   360
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   361
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   362
  // No candidate!
19279
4be3c2e6663c 8022284: Hide internal data structure in PhaseCFG
adlertz
parents: 17384
diff changeset
   363
  if (best == NULL) {
4be3c2e6663c 8022284: Hide internal data structure in PhaseCFG
adlertz
parents: 17384
diff changeset
   364
    return;
4be3c2e6663c 8022284: Hide internal data structure in PhaseCFG
adlertz
parents: 17384
diff changeset
   365
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   366
489c9b5090e2 Initial load
duke
parents:
diff changeset
   367
  // ---- Found an implicit null check
36336
7006dd73b206 8150720: Cleanup code around PrintOptoStatistics
redestad
parents: 35539
diff changeset
   368
#ifndef PRODUCT
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   369
  extern int implicit_null_checks;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   370
  implicit_null_checks++;
36336
7006dd73b206 8150720: Cleanup code around PrintOptoStatistics
redestad
parents: 35539
diff changeset
   371
#endif
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   372
5698
091095915ee6 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 4580
diff changeset
   373
  if( is_decoden ) {
091095915ee6 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 4580
diff changeset
   374
    // Check if we need to hoist decodeHeapOop_not_null first.
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   375
    Block *valb = get_block_for_node(val);
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   376
    if( block != valb && block->_dom_depth < valb->_dom_depth ) {
5698
091095915ee6 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 4580
diff changeset
   377
      // Hoist it up to the end of the test block.
091095915ee6 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 4580
diff changeset
   378
      valb->find_remove(val);
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   379
      block->add_inst(val);
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   380
      map_node_to_block(val, block);
5698
091095915ee6 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 4580
diff changeset
   381
      // DecodeN on x86 may kill flags. Check for flag-killing projections
091095915ee6 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 4580
diff changeset
   382
      // that also need to be hoisted.
091095915ee6 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 4580
diff changeset
   383
      for (DUIterator_Fast jmax, j = val->fast_outs(jmax); j < jmax; j++) {
091095915ee6 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 4580
diff changeset
   384
        Node* n = val->fast_out(j);
10255
bab46e6f7661 7069452: Cleanup NodeFlags
kvn
parents: 10019
diff changeset
   385
        if( n->is_MachProj() ) {
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   386
          get_block_for_node(n)->find_remove(n);
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   387
          block->add_inst(n);
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   388
          map_node_to_block(n, block);
5698
091095915ee6 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 4580
diff changeset
   389
        }
091095915ee6 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 4580
diff changeset
   390
      }
091095915ee6 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 4580
diff changeset
   391
    }
091095915ee6 6954029: Improve implicit null check generation with compressed oops
kvn
parents: 4580
diff changeset
   392
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   393
  // Hoist the memory candidate up to the end of the test block.
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   394
  Block *old_block = get_block_for_node(best);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   395
  old_block->find_remove(best);
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   396
  block->add_inst(best);
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   397
  map_node_to_block(best, block);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   398
40862
3f9cd7a4bfa7 8162496: missing precedence edge for anti_dependence
kvn
parents: 40086
diff changeset
   399
  // Move the control dependence if it is pinned to not-null block.
3f9cd7a4bfa7 8162496: missing precedence edge for anti_dependence
kvn
parents: 40086
diff changeset
   400
  // Don't change it in other cases: NULL or dominating control.
3f9cd7a4bfa7 8162496: missing precedence edge for anti_dependence
kvn
parents: 40086
diff changeset
   401
  if (best->in(0) == not_null_block->head()) {
3f9cd7a4bfa7 8162496: missing precedence edge for anti_dependence
kvn
parents: 40086
diff changeset
   402
    // Set it to control edge of null check.
3f9cd7a4bfa7 8162496: missing precedence edge for anti_dependence
kvn
parents: 40086
diff changeset
   403
    best->set_req(0, proj->in(0)->in(0));
3f9cd7a4bfa7 8162496: missing precedence edge for anti_dependence
kvn
parents: 40086
diff changeset
   404
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   405
489c9b5090e2 Initial load
duke
parents:
diff changeset
   406
  // Check for flag-killing projections that also need to be hoisted
489c9b5090e2 Initial load
duke
parents:
diff changeset
   407
  // Should be DU safe because no edge updates.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   408
  for (DUIterator_Fast jmax, j = best->fast_outs(jmax); j < jmax; j++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   409
    Node* n = best->fast_out(j);
10255
bab46e6f7661 7069452: Cleanup NodeFlags
kvn
parents: 10019
diff changeset
   410
    if( n->is_MachProj() ) {
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   411
      get_block_for_node(n)->find_remove(n);
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   412
      block->add_inst(n);
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   413
      map_node_to_block(n, block);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   414
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   415
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   416
489c9b5090e2 Initial load
duke
parents:
diff changeset
   417
  // proj==Op_True --> ne test; proj==Op_False --> eq test.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   418
  // One of two graph shapes got matched:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   419
  //   (IfTrue  (If (Bool NE (CmpP ptr NULL))))
489c9b5090e2 Initial load
duke
parents:
diff changeset
   420
  //   (IfFalse (If (Bool EQ (CmpP ptr NULL))))
489c9b5090e2 Initial load
duke
parents:
diff changeset
   421
  // NULL checks are always branch-if-eq.  If we see a IfTrue projection
489c9b5090e2 Initial load
duke
parents:
diff changeset
   422
  // then we are replacing a 'ne' test with a 'eq' NULL check test.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   423
  // We need to flip the projections to keep the same semantics.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   424
  if( proj->Opcode() == Op_IfTrue ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   425
    // Swap order of projections in basic block to swap branch targets
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   426
    Node *tmp1 = block->get_node(block->end_idx()+1);
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   427
    Node *tmp2 = block->get_node(block->end_idx()+2);
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   428
    block->map_node(tmp2, block->end_idx()+1);
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   429
    block->map_node(tmp1, block->end_idx()+2);
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 22911
diff changeset
   430
    Node *tmp = new Node(C->top()); // Use not NULL input
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   431
    tmp1->replace_by(tmp);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   432
    tmp2->replace_by(tmp1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   433
    tmp->replace_by(tmp2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   434
    tmp->destruct();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   435
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   436
489c9b5090e2 Initial load
duke
parents:
diff changeset
   437
  // Remove the existing null check; use a new implicit null check instead.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   438
  // Since schedule-local needs precise def-use info, we need to correct
489c9b5090e2 Initial load
duke
parents:
diff changeset
   439
  // it as well.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   440
  Node *old_tst = proj->in(0);
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 22911
diff changeset
   441
  MachNode *nul_chk = new MachNullCheckNode(old_tst->in(0),best,bidx);
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   442
  block->map_node(nul_chk, block->end_idx());
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   443
  map_node_to_block(nul_chk, block);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   444
  // Redirect users of old_test to nul_chk
489c9b5090e2 Initial load
duke
parents:
diff changeset
   445
  for (DUIterator_Last i2min, i2 = old_tst->last_outs(i2min); i2 >= i2min; --i2)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   446
    old_tst->last_out(i2)->set_req(0, nul_chk);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   447
  // Clean-up any dead code
28188
08f2d6e29ceb 8066763: fatal error "assert(false) failed: unexpected yanked node" in postaloc.cpp:139
thartmann
parents: 27454
diff changeset
   448
  for (uint i3 = 0; i3 < old_tst->req(); i3++) {
08f2d6e29ceb 8066763: fatal error "assert(false) failed: unexpected yanked node" in postaloc.cpp:139
thartmann
parents: 27454
diff changeset
   449
    Node* in = old_tst->in(i3);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   450
    old_tst->set_req(i3, NULL);
28188
08f2d6e29ceb 8066763: fatal error "assert(false) failed: unexpected yanked node" in postaloc.cpp:139
thartmann
parents: 27454
diff changeset
   451
    if (in->outcnt() == 0) {
08f2d6e29ceb 8066763: fatal error "assert(false) failed: unexpected yanked node" in postaloc.cpp:139
thartmann
parents: 27454
diff changeset
   452
      // Remove dead input node
08f2d6e29ceb 8066763: fatal error "assert(false) failed: unexpected yanked node" in postaloc.cpp:139
thartmann
parents: 27454
diff changeset
   453
      in->disconnect_inputs(NULL, C);
08f2d6e29ceb 8066763: fatal error "assert(false) failed: unexpected yanked node" in postaloc.cpp:139
thartmann
parents: 27454
diff changeset
   454
      block->find_remove(in);
08f2d6e29ceb 8066763: fatal error "assert(false) failed: unexpected yanked node" in postaloc.cpp:139
thartmann
parents: 27454
diff changeset
   455
    }
08f2d6e29ceb 8066763: fatal error "assert(false) failed: unexpected yanked node" in postaloc.cpp:139
thartmann
parents: 27454
diff changeset
   456
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   457
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   458
  latency_from_uses(nul_chk);
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   459
  latency_from_uses(best);
40862
3f9cd7a4bfa7 8162496: missing precedence edge for anti_dependence
kvn
parents: 40086
diff changeset
   460
3f9cd7a4bfa7 8162496: missing precedence edge for anti_dependence
kvn
parents: 40086
diff changeset
   461
  // insert anti-dependences to defs in this block
3f9cd7a4bfa7 8162496: missing precedence edge for anti_dependence
kvn
parents: 40086
diff changeset
   462
  if (! best->needs_anti_dependence_check()) {
3f9cd7a4bfa7 8162496: missing precedence edge for anti_dependence
kvn
parents: 40086
diff changeset
   463
    for (uint k = 1; k < block->number_of_nodes(); k++) {
3f9cd7a4bfa7 8162496: missing precedence edge for anti_dependence
kvn
parents: 40086
diff changeset
   464
      Node *n = block->get_node(k);
3f9cd7a4bfa7 8162496: missing precedence edge for anti_dependence
kvn
parents: 40086
diff changeset
   465
      if (n->needs_anti_dependence_check() &&
3f9cd7a4bfa7 8162496: missing precedence edge for anti_dependence
kvn
parents: 40086
diff changeset
   466
          n->in(LoadNode::Memory) == best->in(StoreNode::Memory)) {
3f9cd7a4bfa7 8162496: missing precedence edge for anti_dependence
kvn
parents: 40086
diff changeset
   467
        // Found anti-dependent load
3f9cd7a4bfa7 8162496: missing precedence edge for anti_dependence
kvn
parents: 40086
diff changeset
   468
        insert_anti_dependences(block, n);
3f9cd7a4bfa7 8162496: missing precedence edge for anti_dependence
kvn
parents: 40086
diff changeset
   469
      }
3f9cd7a4bfa7 8162496: missing precedence edge for anti_dependence
kvn
parents: 40086
diff changeset
   470
    }
3f9cd7a4bfa7 8162496: missing precedence edge for anti_dependence
kvn
parents: 40086
diff changeset
   471
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   472
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   473
489c9b5090e2 Initial load
duke
parents:
diff changeset
   474
489c9b5090e2 Initial load
duke
parents:
diff changeset
   475
//------------------------------select-----------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   476
// Select a nice fellow from the worklist to schedule next. If there is only
489c9b5090e2 Initial load
duke
parents:
diff changeset
   477
// one choice, then use it. Projections take top priority for correctness
489c9b5090e2 Initial load
duke
parents:
diff changeset
   478
// reasons - if I see a projection, then it is next.  There are a number of
489c9b5090e2 Initial load
duke
parents:
diff changeset
   479
// other special cases, for instructions that consume condition codes, et al.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   480
// These are chosen immediately. Some instructions are required to immediately
489c9b5090e2 Initial load
duke
parents:
diff changeset
   481
// precede the last instruction in the block, and these are taken last. Of the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   482
// remaining cases (most), choose the instruction with the greatest latency
489c9b5090e2 Initial load
duke
parents:
diff changeset
   483
// (that is, the most number of pseudo-cycles required to the end of the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   484
// routine). If there is a tie, choose the instruction with the most inputs.
33065
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   485
Node* PhaseCFG::select(
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   486
  Block* block,
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   487
  Node_List &worklist,
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   488
  GrowableArray<int> &ready_cnt,
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   489
  VectorSet &next_call,
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   490
  uint sched_slot,
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   491
  intptr_t* recalc_pressure_nodes) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   492
489c9b5090e2 Initial load
duke
parents:
diff changeset
   493
  // If only a single entry on the stack, use it
489c9b5090e2 Initial load
duke
parents:
diff changeset
   494
  uint cnt = worklist.size();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   495
  if (cnt == 1) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   496
    Node *n = worklist[0];
489c9b5090e2 Initial load
duke
parents:
diff changeset
   497
    worklist.map(0,worklist.pop());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   498
    return n;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   499
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   500
489c9b5090e2 Initial load
duke
parents:
diff changeset
   501
  uint choice  = 0; // Bigger is most important
489c9b5090e2 Initial load
duke
parents:
diff changeset
   502
  uint latency = 0; // Bigger is scheduled first
489c9b5090e2 Initial load
duke
parents:
diff changeset
   503
  uint score   = 0; // Bigger is better
961
7fb3b13d4205 6726999: nsk/stress/jck12a/jck12a010 assert(n != null,"Bad immediate dominator info.")
kvn
parents: 670
diff changeset
   504
  int idx = -1;     // Index in worklist
15871
b04dd94da4e6 8009120: Fuzz instruction scheduling in HotSpot compilers
shade
parents: 15242
diff changeset
   505
  int cand_cnt = 0; // Candidate count
33065
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   506
  bool block_size_threshold_ok = (block->number_of_nodes() > 10) ? true : false;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   507
489c9b5090e2 Initial load
duke
parents:
diff changeset
   508
  for( uint i=0; i<cnt; i++ ) { // Inspect entire worklist
489c9b5090e2 Initial load
duke
parents:
diff changeset
   509
    // Order in worklist is used to break ties.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   510
    // See caller for how this is used to delay scheduling
489c9b5090e2 Initial load
duke
parents:
diff changeset
   511
    // of induction variable increments to after the other
489c9b5090e2 Initial load
duke
parents:
diff changeset
   512
    // uses of the phi are scheduled.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   513
    Node *n = worklist[i];      // Get Node on worklist
489c9b5090e2 Initial load
duke
parents:
diff changeset
   514
489c9b5090e2 Initial load
duke
parents:
diff changeset
   515
    int iop = n->is_Mach() ? n->as_Mach()->ideal_Opcode() : 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   516
    if( n->is_Proj() ||         // Projections always win
489c9b5090e2 Initial load
duke
parents:
diff changeset
   517
        n->Opcode()== Op_Con || // So does constant 'Top'
489c9b5090e2 Initial load
duke
parents:
diff changeset
   518
        iop == Op_CreateEx ||   // Create-exception must start block
489c9b5090e2 Initial load
duke
parents:
diff changeset
   519
        iop == Op_CheckCastPP
489c9b5090e2 Initial load
duke
parents:
diff changeset
   520
        ) {
27454
cbd4c87b9b55 8059780: SPECjvm2008-MPEG performance regressions on x64 platforms
kvn
parents: 25932
diff changeset
   521
      worklist.map(i,worklist.pop());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   522
      return n;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   523
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   524
489c9b5090e2 Initial load
duke
parents:
diff changeset
   525
    // Final call in a block must be adjacent to 'catch'
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   526
    Node *e = block->end();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   527
    if( e->is_Catch() && e->in(0)->in(0) == n )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   528
      continue;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   529
489c9b5090e2 Initial load
duke
parents:
diff changeset
   530
    // Memory op for an implicit null check has to be at the end of the block
489c9b5090e2 Initial load
duke
parents:
diff changeset
   531
    if( e->is_MachNullCheck() && e->in(1) == n )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   532
      continue;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   533
13104
657b387034fb 7119644: Increase superword's vector size up to 256 bits
kvn
parents: 11567
diff changeset
   534
    // Schedule IV increment last.
34210
2dafc56da253 8143307: Crash in C2 local code motion
goetz
parents: 33638
diff changeset
   535
    if (e->is_Mach() && e->as_Mach()->ideal_Opcode() == Op_CountedLoopEnd) {
2dafc56da253 8143307: Crash in C2 local code motion
goetz
parents: 33638
diff changeset
   536
      // Cmp might be matched into CountedLoopEnd node.
2dafc56da253 8143307: Crash in C2 local code motion
goetz
parents: 33638
diff changeset
   537
      Node *cmp = (e->in(1)->ideal_reg() == Op_RegFlags) ? e->in(1) : e;
2dafc56da253 8143307: Crash in C2 local code motion
goetz
parents: 33638
diff changeset
   538
      if (cmp->req() > 1 && cmp->in(1) == n && n->is_iteratively_computed()) {
2dafc56da253 8143307: Crash in C2 local code motion
goetz
parents: 33638
diff changeset
   539
        continue;
2dafc56da253 8143307: Crash in C2 local code motion
goetz
parents: 33638
diff changeset
   540
      }
2dafc56da253 8143307: Crash in C2 local code motion
goetz
parents: 33638
diff changeset
   541
    }
13104
657b387034fb 7119644: Increase superword's vector size up to 256 bits
kvn
parents: 11567
diff changeset
   542
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   543
    uint n_choice  = 2;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   544
489c9b5090e2 Initial load
duke
parents:
diff changeset
   545
    // See if this instruction is consumed by a branch. If so, then (as the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   546
    // branch is the last instruction in the basic block) force it to the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   547
    // end of the basic block
489c9b5090e2 Initial load
duke
parents:
diff changeset
   548
    if ( must_clone[iop] ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   549
      // See if any use is a branch
489c9b5090e2 Initial load
duke
parents:
diff changeset
   550
      bool found_machif = false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   551
489c9b5090e2 Initial load
duke
parents:
diff changeset
   552
      for (DUIterator_Fast jmax, j = n->fast_outs(jmax); j < jmax; j++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   553
        Node* use = n->fast_out(j);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   554
489c9b5090e2 Initial load
duke
parents:
diff changeset
   555
        // The use is a conditional branch, make them adjacent
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   556
        if (use->is_MachIf() && get_block_for_node(use) == block) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   557
          found_machif = true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   558
          break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   559
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   560
489c9b5090e2 Initial load
duke
parents:
diff changeset
   561
        // More than this instruction pending for successor to be ready,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   562
        // don't choose this if other opportunities are ready
11567
512b2c76e3b7 7116050: C2/ARM: memory stomping error with DivideMcTests
roland
parents: 11196
diff changeset
   563
        if (ready_cnt.at(use->_idx) > 1)
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   564
          n_choice = 1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   565
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   566
489c9b5090e2 Initial load
duke
parents:
diff changeset
   567
      // loop terminated, prefer not to use this instruction
489c9b5090e2 Initial load
duke
parents:
diff changeset
   568
      if (found_machif)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   569
        continue;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   570
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   571
489c9b5090e2 Initial load
duke
parents:
diff changeset
   572
    // See if this has a predecessor that is "must_clone", i.e. sets the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   573
    // condition code. If so, choose this first
489c9b5090e2 Initial load
duke
parents:
diff changeset
   574
    for (uint j = 0; j < n->req() ; j++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   575
      Node *inn = n->in(j);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   576
      if (inn) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   577
        if (inn->is_Mach() && must_clone[inn->as_Mach()->ideal_Opcode()] ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   578
          n_choice = 3;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   579
          break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   580
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   581
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   582
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   583
489c9b5090e2 Initial load
duke
parents:
diff changeset
   584
    // MachTemps should be scheduled last so they are near their uses
489c9b5090e2 Initial load
duke
parents:
diff changeset
   585
    if (n->is_MachTemp()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   586
      n_choice = 1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   587
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   588
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   589
    uint n_latency = get_latency_for_node(n);
33065
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   590
    uint n_score = n->req();   // Many inputs get high score to break ties
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   591
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   592
    if (OptoRegScheduling && block_size_threshold_ok) {
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   593
      if (recalc_pressure_nodes[n->_idx] == 0x7fff7fff) {
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   594
        _regalloc->_scratch_int_pressure.init(_regalloc->_sched_int_pressure.high_pressure_limit());
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   595
        _regalloc->_scratch_float_pressure.init(_regalloc->_sched_float_pressure.high_pressure_limit());
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   596
        // simulate the notion that we just picked this node to schedule
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   597
        n->add_flag(Node::Flag_is_scheduled);
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   598
        // now caculate its effect upon the graph if we did
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   599
        adjust_register_pressure(n, block, recalc_pressure_nodes, false);
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   600
        // return its state for finalize in case somebody else wins
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   601
        n->remove_flag(Node::Flag_is_scheduled);
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   602
        // now save the two final pressure components of register pressure, limiting pressure calcs to short size
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   603
        short int_pressure = (short)_regalloc->_scratch_int_pressure.current_pressure();
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   604
        short float_pressure = (short)_regalloc->_scratch_float_pressure.current_pressure();
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   605
        recalc_pressure_nodes[n->_idx] = int_pressure;
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   606
        recalc_pressure_nodes[n->_idx] |= (float_pressure << 16);
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   607
      }
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   608
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   609
      if (_scheduling_for_pressure) {
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   610
        latency = n_latency;
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   611
        if (n_choice != 3) {
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   612
          // Now evaluate each register pressure component based on threshold in the score.
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   613
          // In general the defining register type will dominate the score, ergo we will not see register pressure grow on both banks
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   614
          // on a single instruction, but we might see it shrink on both banks.
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   615
          // For each use of register that has a register class that is over the high pressure limit, we build n_score up for
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   616
          // live ranges that terminate on this instruction.
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   617
          if (_regalloc->_sched_int_pressure.current_pressure() > _regalloc->_sched_int_pressure.high_pressure_limit()) {
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   618
            short int_pressure = (short)recalc_pressure_nodes[n->_idx];
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   619
            n_score = (int_pressure < 0) ? ((score + n_score) - int_pressure) : (int_pressure > 0) ? 1 : n_score;
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   620
          }
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   621
          if (_regalloc->_sched_float_pressure.current_pressure() > _regalloc->_sched_float_pressure.high_pressure_limit()) {
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   622
            short float_pressure = (short)(recalc_pressure_nodes[n->_idx] >> 16);
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   623
            n_score = (float_pressure < 0) ? ((score + n_score) - float_pressure) : (float_pressure > 0) ? 1 : n_score;
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   624
          }
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   625
        } else {
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   626
          // make sure we choose these candidates
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   627
          score = 0;
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   628
        }
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   629
      }
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   630
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   631
489c9b5090e2 Initial load
duke
parents:
diff changeset
   632
    // Keep best latency found
15871
b04dd94da4e6 8009120: Fuzz instruction scheduling in HotSpot compilers
shade
parents: 15242
diff changeset
   633
    cand_cnt++;
b04dd94da4e6 8009120: Fuzz instruction scheduling in HotSpot compilers
shade
parents: 15242
diff changeset
   634
    if (choice < n_choice ||
b04dd94da4e6 8009120: Fuzz instruction scheduling in HotSpot compilers
shade
parents: 15242
diff changeset
   635
        (choice == n_choice &&
b04dd94da4e6 8009120: Fuzz instruction scheduling in HotSpot compilers
shade
parents: 15242
diff changeset
   636
         ((StressLCM && Compile::randomized_select(cand_cnt)) ||
b04dd94da4e6 8009120: Fuzz instruction scheduling in HotSpot compilers
shade
parents: 15242
diff changeset
   637
          (!StressLCM &&
b04dd94da4e6 8009120: Fuzz instruction scheduling in HotSpot compilers
shade
parents: 15242
diff changeset
   638
           (latency < n_latency ||
b04dd94da4e6 8009120: Fuzz instruction scheduling in HotSpot compilers
shade
parents: 15242
diff changeset
   639
            (latency == n_latency &&
b04dd94da4e6 8009120: Fuzz instruction scheduling in HotSpot compilers
shade
parents: 15242
diff changeset
   640
             (score < n_score))))))) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   641
      choice  = n_choice;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   642
      latency = n_latency;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   643
      score   = n_score;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   644
      idx     = i;               // Also keep index in worklist
489c9b5090e2 Initial load
duke
parents:
diff changeset
   645
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   646
  } // End of for all ready nodes in worklist
489c9b5090e2 Initial load
duke
parents:
diff changeset
   647
51078
fc6cfe40e32a 8207049: Minor improvements of compiler code.
goetz
parents: 50525
diff changeset
   648
  guarantee(idx >= 0, "index should be set");
961
7fb3b13d4205 6726999: nsk/stress/jck12a/jck12a010 assert(n != null,"Bad immediate dominator info.")
kvn
parents: 670
diff changeset
   649
  Node *n = worklist[(uint)idx];      // Get the winner
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   650
27454
cbd4c87b9b55 8059780: SPECjvm2008-MPEG performance regressions on x64 platforms
kvn
parents: 25932
diff changeset
   651
  worklist.map((uint)idx, worklist.pop());     // Compress worklist
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   652
  return n;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   653
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   654
33065
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   655
//-------------------------adjust_register_pressure----------------------------
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   656
void PhaseCFG::adjust_register_pressure(Node* n, Block* block, intptr_t* recalc_pressure_nodes, bool finalize_mode) {
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   657
  PhaseLive* liveinfo = _regalloc->get_live();
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   658
  IndexSet* liveout = liveinfo->live(block);
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   659
  // first adjust the register pressure for the sources
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   660
  for (uint i = 1; i < n->req(); i++) {
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   661
    bool lrg_ends = false;
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   662
    Node *src_n = n->in(i);
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   663
    if (src_n == NULL) continue;
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   664
    if (!src_n->is_Mach()) continue;
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   665
    uint src = _regalloc->_lrg_map.find(src_n);
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   666
    if (src == 0) continue;
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   667
    LRG& lrg_src = _regalloc->lrgs(src);
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   668
    // detect if the live range ends or not
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   669
    if (liveout->member(src) == false) {
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   670
      lrg_ends = true;
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   671
      for (DUIterator_Fast jmax, j = src_n->fast_outs(jmax); j < jmax; j++) {
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   672
        Node* m = src_n->fast_out(j); // Get user
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   673
        if (m == n) continue;
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   674
        if (!m->is_Mach()) continue;
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   675
        MachNode *mach = m->as_Mach();
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   676
        bool src_matches = false;
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   677
        int iop = mach->ideal_Opcode();
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   678
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   679
        switch (iop) {
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   680
        case Op_StoreB:
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   681
        case Op_StoreC:
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   682
        case Op_StoreCM:
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   683
        case Op_StoreD:
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   684
        case Op_StoreF:
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   685
        case Op_StoreI:
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   686
        case Op_StoreL:
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   687
        case Op_StoreP:
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   688
        case Op_StoreN:
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   689
        case Op_StoreVector:
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   690
        case Op_StoreNKlass:
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   691
          for (uint k = 1; k < m->req(); k++) {
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   692
            Node *in = m->in(k);
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   693
            if (in == src_n) {
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   694
              src_matches = true;
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   695
              break;
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   696
            }
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   697
          }
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   698
          break;
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   699
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   700
        default:
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   701
          src_matches = true;
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   702
          break;
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   703
        }
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   704
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   705
        // If we have a store as our use, ignore the non source operands
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   706
        if (src_matches == false) continue;
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   707
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   708
        // Mark every unscheduled use which is not n with a recalculation
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   709
        if ((get_block_for_node(m) == block) && (!m->is_scheduled())) {
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   710
          if (finalize_mode && !m->is_Phi()) {
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   711
            recalc_pressure_nodes[m->_idx] = 0x7fff7fff;
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   712
          }
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   713
          lrg_ends = false;
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   714
        }
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   715
      }
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   716
    }
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   717
    // if none, this live range ends and we can adjust register pressure
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   718
    if (lrg_ends) {
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   719
      if (finalize_mode) {
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   720
        _regalloc->lower_pressure(block, 0, lrg_src, NULL, _regalloc->_sched_int_pressure, _regalloc->_sched_float_pressure);
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   721
      } else {
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   722
        _regalloc->lower_pressure(block, 0, lrg_src, NULL, _regalloc->_scratch_int_pressure, _regalloc->_scratch_float_pressure);
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   723
      }
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   724
    }
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   725
  }
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   726
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   727
  // now add the register pressure from the dest and evaluate which heuristic we should use:
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   728
  // 1.) The default, latency scheduling
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   729
  // 2.) Register pressure scheduling based on the high pressure limit threshold for int or float register stacks
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   730
  uint dst = _regalloc->_lrg_map.find(n);
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   731
  if (dst != 0) {
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   732
    LRG& lrg_dst = _regalloc->lrgs(dst);
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   733
    if (finalize_mode) {
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   734
      _regalloc->raise_pressure(block, lrg_dst, _regalloc->_sched_int_pressure, _regalloc->_sched_float_pressure);
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   735
      // check to see if we fall over the register pressure cliff here
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   736
      if (_regalloc->_sched_int_pressure.current_pressure() > _regalloc->_sched_int_pressure.high_pressure_limit()) {
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   737
        _scheduling_for_pressure = true;
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   738
      } else if (_regalloc->_sched_float_pressure.current_pressure() > _regalloc->_sched_float_pressure.high_pressure_limit()) {
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   739
        _scheduling_for_pressure = true;
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   740
      } else {
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   741
        // restore latency scheduling mode
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   742
        _scheduling_for_pressure = false;
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   743
      }
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   744
    } else {
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   745
      _regalloc->raise_pressure(block, lrg_dst, _regalloc->_scratch_int_pressure, _regalloc->_scratch_float_pressure);
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   746
    }
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   747
  }
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   748
}
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   749
489c9b5090e2 Initial load
duke
parents:
diff changeset
   750
//------------------------------set_next_call----------------------------------
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   751
void PhaseCFG::set_next_call(Block* block, Node* n, VectorSet& next_call) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   752
  if( next_call.test_set(n->_idx) ) return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   753
  for( uint i=0; i<n->len(); i++ ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   754
    Node *m = n->in(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   755
    if( !m ) continue;  // must see all nodes in block that precede call
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   756
    if (get_block_for_node(m) == block) {
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   757
      set_next_call(block, m, next_call);
19279
4be3c2e6663c 8022284: Hide internal data structure in PhaseCFG
adlertz
parents: 17384
diff changeset
   758
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   759
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   760
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   761
489c9b5090e2 Initial load
duke
parents:
diff changeset
   762
//------------------------------needed_for_next_call---------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   763
// Set the flag 'next_call' for each Node that is needed for the next call to
489c9b5090e2 Initial load
duke
parents:
diff changeset
   764
// be scheduled.  This flag lets me bias scheduling so Nodes needed for the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   765
// next subroutine call get priority - basically it moves things NOT needed
489c9b5090e2 Initial load
duke
parents:
diff changeset
   766
// for the next call till after the call.  This prevents me from trying to
489c9b5090e2 Initial load
duke
parents:
diff changeset
   767
// carry lots of stuff live across a call.
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   768
void PhaseCFG::needed_for_next_call(Block* block, Node* this_call, VectorSet& next_call) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   769
  // Find the next control-defining Node in this block
489c9b5090e2 Initial load
duke
parents:
diff changeset
   770
  Node* call = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   771
  for (DUIterator_Fast imax, i = this_call->fast_outs(imax); i < imax; i++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   772
    Node* m = this_call->fast_out(i);
19724
1e308c6b0de3 8024095: Missing brackets in local scheduling code.
adlertz
parents: 19721
diff changeset
   773
    if (get_block_for_node(m) == block && // Local-block user
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   774
        m != this_call &&       // Not self-start node
19724
1e308c6b0de3 8024095: Missing brackets in local scheduling code.
adlertz
parents: 19721
diff changeset
   775
        m->is_MachCall()) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   776
      call = m;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   777
      break;
19724
1e308c6b0de3 8024095: Missing brackets in local scheduling code.
adlertz
parents: 19721
diff changeset
   778
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   779
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   780
  if (call == NULL)  return;    // No next call (e.g., block end is near)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   781
  // Set next-call for all inputs to this call
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   782
  set_next_call(block, call, next_call);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   783
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   784
11196
a310a659c580 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 10255
diff changeset
   785
//------------------------------add_call_kills-------------------------------------
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   786
// helper function that adds caller save registers to MachProjNode
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   787
static void add_call_kills(MachProjNode *proj, RegMask& regs, const char* save_policy, bool exclude_soe) {
11196
a310a659c580 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 10255
diff changeset
   788
  // Fill in the kill mask for the call
a310a659c580 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 10255
diff changeset
   789
  for( OptoReg::Name r = OptoReg::Name(0); r < _last_Mach_Reg; r=OptoReg::add(r,1) ) {
a310a659c580 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 10255
diff changeset
   790
    if( !regs.Member(r) ) {     // Not already defined by the call
a310a659c580 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 10255
diff changeset
   791
      // Save-on-call register?
a310a659c580 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 10255
diff changeset
   792
      if ((save_policy[r] == 'C') ||
a310a659c580 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 10255
diff changeset
   793
          (save_policy[r] == 'A') ||
a310a659c580 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 10255
diff changeset
   794
          ((save_policy[r] == 'E') && exclude_soe)) {
a310a659c580 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 10255
diff changeset
   795
        proj->_rout.Insert(r);
a310a659c580 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 10255
diff changeset
   796
      }
a310a659c580 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 10255
diff changeset
   797
    }
a310a659c580 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 10255
diff changeset
   798
  }
a310a659c580 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 10255
diff changeset
   799
}
a310a659c580 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 10255
diff changeset
   800
a310a659c580 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 10255
diff changeset
   801
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   802
//------------------------------sched_call-------------------------------------
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   803
uint PhaseCFG::sched_call(Block* block, uint node_cnt, Node_List& worklist, GrowableArray<int>& ready_cnt, MachCallNode* mcall, VectorSet& next_call) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   804
  RegMask regs;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   805
489c9b5090e2 Initial load
duke
parents:
diff changeset
   806
  // Schedule all the users of the call right now.  All the users are
489c9b5090e2 Initial load
duke
parents:
diff changeset
   807
  // projection Nodes, so they must be scheduled next to the call.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   808
  // Collect all the defined registers.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   809
  for (DUIterator_Fast imax, i = mcall->fast_outs(imax); i < imax; i++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   810
    Node* n = mcall->fast_out(i);
10255
bab46e6f7661 7069452: Cleanup NodeFlags
kvn
parents: 10019
diff changeset
   811
    assert( n->is_MachProj(), "" );
11567
512b2c76e3b7 7116050: C2/ARM: memory stomping error with DivideMcTests
roland
parents: 11196
diff changeset
   812
    int n_cnt = ready_cnt.at(n->_idx)-1;
512b2c76e3b7 7116050: C2/ARM: memory stomping error with DivideMcTests
roland
parents: 11196
diff changeset
   813
    ready_cnt.at_put(n->_idx, n_cnt);
512b2c76e3b7 7116050: C2/ARM: memory stomping error with DivideMcTests
roland
parents: 11196
diff changeset
   814
    assert( n_cnt == 0, "" );
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   815
    // Schedule next to call
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   816
    block->map_node(n, node_cnt++);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   817
    // Collect defined registers
489c9b5090e2 Initial load
duke
parents:
diff changeset
   818
    regs.OR(n->out_RegMask());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   819
    // Check for scheduling the next control-definer
489c9b5090e2 Initial load
duke
parents:
diff changeset
   820
    if( n->bottom_type() == Type::CONTROL )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   821
      // Warm up next pile of heuristic bits
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   822
      needed_for_next_call(block, n, next_call);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   823
489c9b5090e2 Initial load
duke
parents:
diff changeset
   824
    // Children of projections are now all ready
489c9b5090e2 Initial load
duke
parents:
diff changeset
   825
    for (DUIterator_Fast jmax, j = n->fast_outs(jmax); j < jmax; j++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   826
      Node* m = n->fast_out(j); // Get user
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   827
      if(get_block_for_node(m) != block) {
19279
4be3c2e6663c 8022284: Hide internal data structure in PhaseCFG
adlertz
parents: 17384
diff changeset
   828
        continue;
4be3c2e6663c 8022284: Hide internal data structure in PhaseCFG
adlertz
parents: 17384
diff changeset
   829
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   830
      if( m->is_Phi() ) continue;
33065
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   831
      int m_cnt = ready_cnt.at(m->_idx) - 1;
11567
512b2c76e3b7 7116050: C2/ARM: memory stomping error with DivideMcTests
roland
parents: 11196
diff changeset
   832
      ready_cnt.at_put(m->_idx, m_cnt);
512b2c76e3b7 7116050: C2/ARM: memory stomping error with DivideMcTests
roland
parents: 11196
diff changeset
   833
      if( m_cnt == 0 )
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   834
        worklist.push(m);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   835
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   836
489c9b5090e2 Initial load
duke
parents:
diff changeset
   837
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   838
489c9b5090e2 Initial load
duke
parents:
diff changeset
   839
  // Act as if the call defines the Frame Pointer.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   840
  // Certainly the FP is alive and well after the call.
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   841
  regs.Insert(_matcher.c_frame_pointer());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   842
489c9b5090e2 Initial load
duke
parents:
diff changeset
   843
  // Set all registers killed and not already defined by the call.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   844
  uint r_cnt = mcall->tf()->range()->cnt();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   845
  int op = mcall->ideal_Opcode();
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 22911
diff changeset
   846
  MachProjNode *proj = new MachProjNode( mcall, r_cnt+1, RegMask::Empty, MachProjNode::fat_proj );
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   847
  map_node_to_block(proj, block);
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   848
  block->insert_node(proj, node_cnt++);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   849
489c9b5090e2 Initial load
duke
parents:
diff changeset
   850
  // Select the right register save policy.
33589
7cbd1b2c139b 8139040: Fix initializations before ShouldNotReachHere() etc. and enable -Wuninitialized on linux.
goetz
parents: 33065
diff changeset
   851
  const char *save_policy = NULL;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   852
  switch (op) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   853
    case Op_CallRuntime:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   854
    case Op_CallLeaf:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   855
    case Op_CallLeafNoFP:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   856
      // Calling C code so use C calling convention
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   857
      save_policy = _matcher._c_reg_save_policy;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   858
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   859
489c9b5090e2 Initial load
duke
parents:
diff changeset
   860
    case Op_CallStaticJava:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   861
    case Op_CallDynamicJava:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   862
      // Calling Java code so use Java calling convention
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   863
      save_policy = _matcher._register_save_policy;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   864
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   865
489c9b5090e2 Initial load
duke
parents:
diff changeset
   866
    default:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   867
      ShouldNotReachHere();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   868
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   869
489c9b5090e2 Initial load
duke
parents:
diff changeset
   870
  // When using CallRuntime mark SOE registers as killed by the call
489c9b5090e2 Initial load
duke
parents:
diff changeset
   871
  // so values that could show up in the RegisterMap aren't live in a
489c9b5090e2 Initial load
duke
parents:
diff changeset
   872
  // callee saved register since the register wouldn't know where to
489c9b5090e2 Initial load
duke
parents:
diff changeset
   873
  // find them.  CallLeaf and CallLeafNoFP are ok because they can't
489c9b5090e2 Initial load
duke
parents:
diff changeset
   874
  // have debug info on them.  Strictly speaking this only needs to be
489c9b5090e2 Initial load
duke
parents:
diff changeset
   875
  // done for oops since idealreg2debugmask takes care of debug info
489c9b5090e2 Initial load
duke
parents:
diff changeset
   876
  // references but there no way to handle oops differently than other
489c9b5090e2 Initial load
duke
parents:
diff changeset
   877
  // pointers as far as the kill mask goes.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   878
  bool exclude_soe = op == Op_CallRuntime;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   879
4566
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4470
diff changeset
   880
  // If the call is a MethodHandle invoke, we need to exclude the
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4470
diff changeset
   881
  // register which is used to save the SP value over MH invokes from
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4470
diff changeset
   882
  // the mask.  Otherwise this register could be used for
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4470
diff changeset
   883
  // deoptimization information.
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4470
diff changeset
   884
  if (op == Op_CallStaticJava) {
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4470
diff changeset
   885
    MachCallStaticJavaNode* mcallstaticjava = (MachCallStaticJavaNode*) mcall;
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4470
diff changeset
   886
    if (mcallstaticjava->_method_handle_invoke)
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4470
diff changeset
   887
      proj->_rout.OR(Matcher::method_handle_invoke_SP_save_mask());
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4470
diff changeset
   888
  }
b363f6ef4068 6829187: compiler optimizations required for JSR 292
twisti
parents: 4470
diff changeset
   889
11196
a310a659c580 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 10255
diff changeset
   890
  add_call_kills(proj, regs, save_policy, exclude_soe);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   891
489c9b5090e2 Initial load
duke
parents:
diff changeset
   892
  return node_cnt;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   893
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   894
489c9b5090e2 Initial load
duke
parents:
diff changeset
   895
489c9b5090e2 Initial load
duke
parents:
diff changeset
   896
//------------------------------schedule_local---------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   897
// Topological sort within a block.  Someday become a real scheduler.
33065
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   898
bool PhaseCFG::schedule_local(Block* block, GrowableArray<int>& ready_cnt, VectorSet& next_call, intptr_t *recalc_pressure_nodes) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   899
  // Already "sorted" are the block start Node (as the first entry), and
489c9b5090e2 Initial load
duke
parents:
diff changeset
   900
  // the block-ending Node and any trailing control projections.  We leave
489c9b5090e2 Initial load
duke
parents:
diff changeset
   901
  // these alone.  PhiNodes and ParmNodes are made to follow the block start
489c9b5090e2 Initial load
duke
parents:
diff changeset
   902
  // Node.  Everything else gets topo-sorted.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   903
489c9b5090e2 Initial load
duke
parents:
diff changeset
   904
#ifndef PRODUCT
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   905
    if (trace_opto_pipelining()) {
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   906
      tty->print_cr("# --- schedule_local B%d, before: ---", block->_pre_order);
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   907
      for (uint i = 0;i < block->number_of_nodes(); i++) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   908
        tty->print("# ");
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   909
        block->get_node(i)->fast_dump();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   910
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   911
      tty->print_cr("#");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   912
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   913
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   914
489c9b5090e2 Initial load
duke
parents:
diff changeset
   915
  // RootNode is already sorted
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   916
  if (block->number_of_nodes() == 1) {
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   917
    return true;
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   918
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   919
33065
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   920
  bool block_size_threshold_ok = (block->number_of_nodes() > 10) ? true : false;
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   921
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   922
  // We track the uses of local definitions as input dependences so that
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   923
  // we know when a given instruction is avialable to be scheduled.
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   924
  uint i;
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   925
  if (OptoRegScheduling && block_size_threshold_ok) {
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   926
    for (i = 1; i < block->number_of_nodes(); i++) { // setup nodes for pressure calc
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   927
      Node *n = block->get_node(i);
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   928
      n->remove_flag(Node::Flag_is_scheduled);
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   929
      if (!n->is_Phi()) {
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   930
        recalc_pressure_nodes[n->_idx] = 0x7fff7fff;
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   931
      }
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   932
    }
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   933
  }
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   934
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   935
  // Move PhiNodes and ParmNodes from 1 to cnt up to the start
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   936
  uint node_cnt = block->end_idx();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   937
  uint phi_cnt = 1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   938
  for( i = 1; i<node_cnt; i++ ) { // Scan for Phi
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   939
    Node *n = block->get_node(i);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   940
    if( n->is_Phi() ||          // Found a PhiNode or ParmNode
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   941
        (n->is_Proj()  && n->in(0) == block->head()) ) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   942
      // Move guy at 'phi_cnt' to the end; makes a hole at phi_cnt
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   943
      block->map_node(block->get_node(phi_cnt), i);
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   944
      block->map_node(n, phi_cnt++);  // swap Phi/Parm up front
33065
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   945
      if (OptoRegScheduling && block_size_threshold_ok) {
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   946
        // mark n as scheduled
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   947
        n->add_flag(Node::Flag_is_scheduled);
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
   948
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   949
    } else {                    // All others
489c9b5090e2 Initial load
duke
parents:
diff changeset
   950
      // Count block-local inputs to 'n'
489c9b5090e2 Initial load
duke
parents:
diff changeset
   951
      uint cnt = n->len();      // Input count
489c9b5090e2 Initial load
duke
parents:
diff changeset
   952
      uint local = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   953
      for( uint j=0; j<cnt; j++ ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   954
        Node *m = n->in(j);
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   955
        if( m && get_block_for_node(m) == block && !m->is_top() )
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   956
          local++;              // One more block-local input
489c9b5090e2 Initial load
duke
parents:
diff changeset
   957
      }
11567
512b2c76e3b7 7116050: C2/ARM: memory stomping error with DivideMcTests
roland
parents: 11196
diff changeset
   958
      ready_cnt.at_put(n->_idx, local); // Count em up
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   959
9175
80221fe09772 7032963: StoreCM shouldn't participate in store elimination
never
parents: 7397
diff changeset
   960
#ifdef ASSERT
59053
ba6c248cae19 8232365: Implementation for JEP 363: Remove the Concurrent Mark Sweep (CMS) Garbage Collector
lkorinth
parents: 55563
diff changeset
   961
      if (UseG1GC) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   962
        if( n->is_Mach() && n->as_Mach()->ideal_Opcode() == Op_StoreCM ) {
9175
80221fe09772 7032963: StoreCM shouldn't participate in store elimination
never
parents: 7397
diff changeset
   963
          // Check the precedence edges
80221fe09772 7032963: StoreCM shouldn't participate in store elimination
never
parents: 7397
diff changeset
   964
          for (uint prec = n->req(); prec < n->len(); prec++) {
80221fe09772 7032963: StoreCM shouldn't participate in store elimination
never
parents: 7397
diff changeset
   965
            Node* oop_store = n->in(prec);
80221fe09772 7032963: StoreCM shouldn't participate in store elimination
never
parents: 7397
diff changeset
   966
            if (oop_store != NULL) {
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   967
              assert(get_block_for_node(oop_store)->_dom_depth <= block->_dom_depth, "oop_store must dominate card-mark");
9175
80221fe09772 7032963: StoreCM shouldn't participate in store elimination
never
parents: 7397
diff changeset
   968
            }
80221fe09772 7032963: StoreCM shouldn't participate in store elimination
never
parents: 7397
diff changeset
   969
          }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   970
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   971
      }
9175
80221fe09772 7032963: StoreCM shouldn't participate in store elimination
never
parents: 7397
diff changeset
   972
#endif
80221fe09772 7032963: StoreCM shouldn't participate in store elimination
never
parents: 7397
diff changeset
   973
80221fe09772 7032963: StoreCM shouldn't participate in store elimination
never
parents: 7397
diff changeset
   974
      // A few node types require changing a required edge to a precedence edge
80221fe09772 7032963: StoreCM shouldn't participate in store elimination
never
parents: 7397
diff changeset
   975
      // before allocation.
4470
1e6edcab3109 6895383: JCK test throws NPE for method compiled with Escape Analysis
kvn
parents: 2348
diff changeset
   976
      if( n->is_Mach() && n->req() > TypeFunc::Parms &&
1e6edcab3109 6895383: JCK test throws NPE for method compiled with Escape Analysis
kvn
parents: 2348
diff changeset
   977
          (n->as_Mach()->ideal_Opcode() == Op_MemBarAcquire ||
1e6edcab3109 6895383: JCK test throws NPE for method compiled with Escape Analysis
kvn
parents: 2348
diff changeset
   978
           n->as_Mach()->ideal_Opcode() == Op_MemBarVolatile) ) {
961
7fb3b13d4205 6726999: nsk/stress/jck12a/jck12a010 assert(n != null,"Bad immediate dominator info.")
kvn
parents: 670
diff changeset
   979
        // MemBarAcquire could be created without Precedent edge.
7fb3b13d4205 6726999: nsk/stress/jck12a/jck12a010 assert(n != null,"Bad immediate dominator info.")
kvn
parents: 670
diff changeset
   980
        // del_req() replaces the specified edge with the last input edge
7fb3b13d4205 6726999: nsk/stress/jck12a/jck12a010 assert(n != null,"Bad immediate dominator info.")
kvn
parents: 670
diff changeset
   981
        // and then removes the last edge. If the specified edge > number of
7fb3b13d4205 6726999: nsk/stress/jck12a/jck12a010 assert(n != null,"Bad immediate dominator info.")
kvn
parents: 670
diff changeset
   982
        // edges the last edge will be moved outside of the input edges array
7fb3b13d4205 6726999: nsk/stress/jck12a/jck12a010 assert(n != null,"Bad immediate dominator info.")
kvn
parents: 670
diff changeset
   983
        // and the edge will be lost. This is why this code should be
7fb3b13d4205 6726999: nsk/stress/jck12a/jck12a010 assert(n != null,"Bad immediate dominator info.")
kvn
parents: 670
diff changeset
   984
        // executed only when Precedent (== TypeFunc::Parms) edge is present.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   985
        Node *x = n->in(TypeFunc::Parms);
35539
9a687b686976 8146612: C2: Precedence edges specification violated
mdoerr
parents: 34210
diff changeset
   986
        if (x != NULL && get_block_for_node(x) == block && n->find_prec_edge(x) != -1) {
9a687b686976 8146612: C2: Precedence edges specification violated
mdoerr
parents: 34210
diff changeset
   987
          // Old edge to node within same block will get removed, but no precedence
9a687b686976 8146612: C2: Precedence edges specification violated
mdoerr
parents: 34210
diff changeset
   988
          // edge will get added because it already exists. Update ready count.
9a687b686976 8146612: C2: Precedence edges specification violated
mdoerr
parents: 34210
diff changeset
   989
          int cnt = ready_cnt.at(n->_idx);
9a687b686976 8146612: C2: Precedence edges specification violated
mdoerr
parents: 34210
diff changeset
   990
          assert(cnt > 1, "MemBar node %d must not get ready here", n->_idx);
9a687b686976 8146612: C2: Precedence edges specification violated
mdoerr
parents: 34210
diff changeset
   991
          ready_cnt.at_put(n->_idx, cnt-1);
9a687b686976 8146612: C2: Precedence edges specification violated
mdoerr
parents: 34210
diff changeset
   992
        }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   993
        n->del_req(TypeFunc::Parms);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   994
        n->add_prec(x);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   995
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   996
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   997
  }
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   998
  for(uint i2=i; i2< block->number_of_nodes(); i2++ ) // Trailing guys get zapped count
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
   999
    ready_cnt.at_put(block->get_node(i2)->_idx, 0);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1000
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1001
  // All the prescheduled guys do not hold back internal nodes
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1002
  uint i3;
33065
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1003
  for (i3 = 0; i3 < phi_cnt; i3++) {  // For all pre-scheduled
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1004
    Node *n = block->get_node(i3);       // Get pre-scheduled
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1005
    for (DUIterator_Fast jmax, j = n->fast_outs(jmax); j < jmax; j++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1006
      Node* m = n->fast_out(j);
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1007
      if (get_block_for_node(m) == block) { // Local-block user
11567
512b2c76e3b7 7116050: C2/ARM: memory stomping error with DivideMcTests
roland
parents: 11196
diff changeset
  1008
        int m_cnt = ready_cnt.at(m->_idx)-1;
33065
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1009
        if (OptoRegScheduling && block_size_threshold_ok) {
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1010
          // mark m as scheduled
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1011
          if (m_cnt < 0) {
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1012
            m->add_flag(Node::Flag_is_scheduled);
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1013
          }
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1014
        }
11567
512b2c76e3b7 7116050: C2/ARM: memory stomping error with DivideMcTests
roland
parents: 11196
diff changeset
  1015
        ready_cnt.at_put(m->_idx, m_cnt);   // Fix ready count
512b2c76e3b7 7116050: C2/ARM: memory stomping error with DivideMcTests
roland
parents: 11196
diff changeset
  1016
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1017
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1018
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1019
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1020
  Node_List delay;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1021
  // Make a worklist
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1022
  Node_List worklist;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1023
  for(uint i4=i3; i4<node_cnt; i4++ ) {    // Put ready guys on worklist
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1024
    Node *m = block->get_node(i4);
11567
512b2c76e3b7 7116050: C2/ARM: memory stomping error with DivideMcTests
roland
parents: 11196
diff changeset
  1025
    if( !ready_cnt.at(m->_idx) ) {   // Zero ready count?
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1026
      if (m->is_iteratively_computed()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1027
        // Push induction variable increments last to allow other uses
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1028
        // of the phi to be scheduled first. The select() method breaks
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1029
        // ties in scheduling by worklist order.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1030
        delay.push(m);
372
b0d8a9fd3646 6601321: Assert(j == 1 || b->_nodes[j-1]->is_Phi(),"CreateEx must be first instruction in block")
never
parents: 360
diff changeset
  1031
      } else if (m->is_Mach() && m->as_Mach()->ideal_Opcode() == Op_CreateEx) {
b0d8a9fd3646 6601321: Assert(j == 1 || b->_nodes[j-1]->is_Phi(),"CreateEx must be first instruction in block")
never
parents: 360
diff changeset
  1032
        // Force the CreateEx to the top of the list so it's processed
b0d8a9fd3646 6601321: Assert(j == 1 || b->_nodes[j-1]->is_Phi(),"CreateEx must be first instruction in block")
never
parents: 360
diff changeset
  1033
        // first and ends up at the start of the block.
b0d8a9fd3646 6601321: Assert(j == 1 || b->_nodes[j-1]->is_Phi(),"CreateEx must be first instruction in block")
never
parents: 360
diff changeset
  1034
        worklist.insert(0, m);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1035
      } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1036
        worklist.push(m);         // Then on to worklist!
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1037
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1038
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1039
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1040
  while (delay.size()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1041
    Node* d = delay.pop();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1042
    worklist.push(d);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1043
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1044
33065
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1045
  if (OptoRegScheduling && block_size_threshold_ok) {
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1046
    // To stage register pressure calculations we need to examine the live set variables
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1047
    // breaking them up by register class to compartmentalize the calculations.
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1048
    uint float_pressure = Matcher::float_pressure(FLOATPRESSURE);
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1049
    _regalloc->_sched_int_pressure.init(INTPRESSURE);
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1050
    _regalloc->_sched_float_pressure.init(float_pressure);
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1051
    _regalloc->_scratch_int_pressure.init(INTPRESSURE);
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1052
    _regalloc->_scratch_float_pressure.init(float_pressure);
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1053
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1054
    _regalloc->compute_entry_block_pressure(block);
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1055
  }
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1056
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1057
  // Warm up the 'next_call' heuristic bits
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1058
  needed_for_next_call(block, block->head(), next_call);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1059
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1060
#ifndef PRODUCT
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1061
    if (trace_opto_pipelining()) {
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1062
      for (uint j=0; j< block->number_of_nodes(); j++) {
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1063
        Node     *n = block->get_node(j);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1064
        int     idx = n->_idx;
11567
512b2c76e3b7 7116050: C2/ARM: memory stomping error with DivideMcTests
roland
parents: 11196
diff changeset
  1065
        tty->print("#   ready cnt:%3d  ", ready_cnt.at(idx));
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1066
        tty->print("latency:%3d  ", get_latency_for_node(n));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1067
        tty->print("%4d: %s\n", idx, n->Name());
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1068
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1069
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1070
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1071
11567
512b2c76e3b7 7116050: C2/ARM: memory stomping error with DivideMcTests
roland
parents: 11196
diff changeset
  1072
  uint max_idx = (uint)ready_cnt.length();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1073
  // Pull from worklist and schedule
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1074
  while( worklist.size() ) {    // Worklist is not ready
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1075
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1076
#ifndef PRODUCT
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1077
    if (trace_opto_pipelining()) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1078
      tty->print("#   ready list:");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1079
      for( uint i=0; i<worklist.size(); i++ ) { // Inspect entire worklist
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1080
        Node *n = worklist[i];      // Get Node on worklist
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1081
        tty->print(" %d", n->_idx);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1082
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1083
      tty->cr();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1084
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1085
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1086
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1087
    // Select and pop a ready guy from worklist
33065
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1088
    Node* n = select(block, worklist, ready_cnt, next_call, phi_cnt, recalc_pressure_nodes);
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1089
    block->map_node(n, phi_cnt++);    // Schedule him next
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1090
33065
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1091
    if (OptoRegScheduling && block_size_threshold_ok) {
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1092
      n->add_flag(Node::Flag_is_scheduled);
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1093
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1094
      // Now adjust the resister pressure with the node we selected
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1095
      if (!n->is_Phi()) {
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1096
        adjust_register_pressure(n, block, recalc_pressure_nodes, true);
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1097
      }
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1098
    }
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1099
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1100
#ifndef PRODUCT
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1101
    if (trace_opto_pipelining()) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1102
      tty->print("#    select %d: %s", n->_idx, n->Name());
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1103
      tty->print(", latency:%d", get_latency_for_node(n));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1104
      n->dump();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1105
      if (Verbose) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1106
        tty->print("#   ready list:");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1107
        for( uint i=0; i<worklist.size(); i++ ) { // Inspect entire worklist
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1108
          Node *n = worklist[i];      // Get Node on worklist
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1109
          tty->print(" %d", n->_idx);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1110
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1111
        tty->cr();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1112
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1113
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1114
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1115
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1116
    if( n->is_MachCall() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1117
      MachCallNode *mcall = n->as_MachCall();
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1118
      phi_cnt = sched_call(block, phi_cnt, worklist, ready_cnt, mcall, next_call);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1119
      continue;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1120
    }
11196
a310a659c580 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 10255
diff changeset
  1121
a310a659c580 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 10255
diff changeset
  1122
    if (n->is_Mach() && n->as_Mach()->has_call()) {
a310a659c580 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 10255
diff changeset
  1123
      RegMask regs;
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1124
      regs.Insert(_matcher.c_frame_pointer());
11196
a310a659c580 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 10255
diff changeset
  1125
      regs.OR(n->out_RegMask());
a310a659c580 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 10255
diff changeset
  1126
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 22911
diff changeset
  1127
      MachProjNode *proj = new MachProjNode( n, 1, RegMask::Empty, MachProjNode::fat_proj );
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1128
      map_node_to_block(proj, block);
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1129
      block->insert_node(proj, phi_cnt++);
11196
a310a659c580 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 10255
diff changeset
  1130
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1131
      add_call_kills(proj, regs, _matcher._c_reg_save_policy, false);
11196
a310a659c580 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 10255
diff changeset
  1132
    }
a310a659c580 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 10255
diff changeset
  1133
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1134
    // Children are now all ready
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1135
    for (DUIterator_Fast i5max, i5 = n->fast_outs(i5max); i5 < i5max; i5++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1136
      Node* m = n->fast_out(i5); // Get user
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1137
      if (get_block_for_node(m) != block) {
19279
4be3c2e6663c 8022284: Hide internal data structure in PhaseCFG
adlertz
parents: 17384
diff changeset
  1138
        continue;
4be3c2e6663c 8022284: Hide internal data structure in PhaseCFG
adlertz
parents: 17384
diff changeset
  1139
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1140
      if( m->is_Phi() ) continue;
11567
512b2c76e3b7 7116050: C2/ARM: memory stomping error with DivideMcTests
roland
parents: 11196
diff changeset
  1141
      if (m->_idx >= max_idx) { // new node, skip it
11196
a310a659c580 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 10255
diff changeset
  1142
        assert(m->is_MachProj() && n->is_Mach() && n->as_Mach()->has_call(), "unexpected node types");
a310a659c580 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 10255
diff changeset
  1143
        continue;
a310a659c580 7077312: Provide a CALL effect for instruct declaration in the ad file
roland
parents: 10255
diff changeset
  1144
      }
33065
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1145
      int m_cnt = ready_cnt.at(m->_idx) - 1;
11567
512b2c76e3b7 7116050: C2/ARM: memory stomping error with DivideMcTests
roland
parents: 11196
diff changeset
  1146
      ready_cnt.at_put(m->_idx, m_cnt);
512b2c76e3b7 7116050: C2/ARM: memory stomping error with DivideMcTests
roland
parents: 11196
diff changeset
  1147
      if( m_cnt == 0 )
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1148
        worklist.push(m);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1149
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1150
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1151
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1152
  if( phi_cnt != block->end_idx() ) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1153
    // did not schedule all.  Retry, Bailout, or Die
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1154
    if (C->subsume_loads() == true && !C->failing()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1155
      // Retry with subsume_loads == false
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1156
      // If this is the first failure, the sentinel string will "stick"
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1157
      // to the Compile object, and the C2Compiler will see it and retry.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1158
      C->record_failure(C2Compiler::retry_no_subsuming_loads());
42081
30a0176b8af3 8167300: Scheduling failures during gcm should be fatal
roland
parents: 40862
diff changeset
  1159
    } else {
30a0176b8af3 8167300: Scheduling failures during gcm should be fatal
roland
parents: 40862
diff changeset
  1160
      assert(false, "graph should be schedulable");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1161
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1162
    // assert( phi_cnt == end_idx(), "did not schedule all" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1163
    return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1164
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1165
33065
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1166
  if (OptoRegScheduling && block_size_threshold_ok) {
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1167
    _regalloc->compute_exit_block_pressure(block);
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1168
    block->_reg_pressure = _regalloc->_sched_int_pressure.final_pressure();
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1169
    block->_freg_pressure = _regalloc->_sched_float_pressure.final_pressure();
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1170
  }
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1171
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1172
#ifndef PRODUCT
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1173
  if (trace_opto_pipelining()) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1174
    tty->print_cr("#");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1175
    tty->print_cr("# after schedule_local");
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1176
    for (uint i = 0;i < block->number_of_nodes();i++) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1177
      tty->print("# ");
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1178
      block->get_node(i)->fast_dump();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1179
    }
33065
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1180
    tty->print_cr("# ");
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1181
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1182
    if (OptoRegScheduling && block_size_threshold_ok) {
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1183
      tty->print_cr("# pressure info : %d", block->_pre_order);
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1184
      _regalloc->print_pressure_info(_regalloc->_sched_int_pressure, "int register info");
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1185
      _regalloc->print_pressure_info(_regalloc->_sched_float_pressure, "float register info");
55892792936f 8134802: LCM register pressure scheduling
mcberg
parents: 29081
diff changeset
  1186
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1187
    tty->cr();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1188
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1189
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1190
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1191
  return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1192
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1193
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1194
//--------------------------catch_cleanup_fix_all_inputs-----------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1195
static void catch_cleanup_fix_all_inputs(Node *use, Node *old_def, Node *new_def) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1196
  for (uint l = 0; l < use->len(); l++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1197
    if (use->in(l) == old_def) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1198
      if (l < use->req()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1199
        use->set_req(l, new_def);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1200
      } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1201
        use->rm_prec(l);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1202
        use->add_prec(new_def);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1203
        l--;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1204
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1205
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1206
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1207
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1208
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1209
//------------------------------catch_cleanup_find_cloned_def------------------
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1210
Node* PhaseCFG::catch_cleanup_find_cloned_def(Block *use_blk, Node *def, Block *def_blk, int n_clone_idx) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1211
  assert( use_blk != def_blk, "Inter-block cleanup only");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1212
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1213
  // The use is some block below the Catch.  Find and return the clone of the def
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1214
  // that dominates the use. If there is no clone in a dominating block, then
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1215
  // create a phi for the def in a dominating block.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1216
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1217
  // Find which successor block dominates this use.  The successor
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1218
  // blocks must all be single-entry (from the Catch only; I will have
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1219
  // split blocks to make this so), hence they all dominate.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1220
  while( use_blk->_dom_depth > def_blk->_dom_depth+1 )
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1221
    use_blk = use_blk->_idom;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1222
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1223
  // Find the successor
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1224
  Node *fixup = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1225
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1226
  uint j;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1227
  for( j = 0; j < def_blk->_num_succs; j++ )
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1228
    if( use_blk == def_blk->_succs[j] )
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1229
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1230
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1231
  if( j == def_blk->_num_succs ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1232
    // Block at same level in dom-tree is not a successor.  It needs a
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1233
    // PhiNode, the PhiNode uses from the def and IT's uses need fixup.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1234
    Node_Array inputs = new Node_List(Thread::current()->resource_area());
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1235
    for(uint k = 1; k < use_blk->num_preds(); k++) {
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1236
      Block* block = get_block_for_node(use_blk->pred(k));
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1237
      inputs.map(k, catch_cleanup_find_cloned_def(block, def, def_blk, n_clone_idx));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1238
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1239
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1240
    // Check to see if the use_blk already has an identical phi inserted.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1241
    // If it exists, it will be at the first position since all uses of a
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1242
    // def are processed together.
19717
7819ffdaf0ff 8023691: Create interface for nodes in class Block
adlertz
parents: 19330
diff changeset
  1243
    Node *phi = use_blk->get_node(1);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1244
    if( phi->is_Phi() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1245
      fixup = phi;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1246
      for (uint k = 1; k < use_blk->num_preds(); k++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1247
        if (phi->in(k) != inputs[k]) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1248
          // Not a match
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1249
          fixup = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1250
          break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1251
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1252
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1253
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1254
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1255
    // If an existing PhiNode was not found, make a new one.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1256
    if (fixup == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1257
      Node *new_phi = PhiNode::make(use_blk->head(), def);
19717
7819ffdaf0ff 8023691: Create interface for nodes in class Block
adlertz
parents: 19330
diff changeset
  1258
      use_blk->insert_node(new_phi, 1);
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1259
      map_node_to_block(new_phi, use_blk);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1260
      for (uint k = 1; k < use_blk->num_preds(); k++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1261
        new_phi->set_req(k, inputs[k]);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1262
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1263
      fixup = new_phi;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1264
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1265
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1266
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1267
    // Found the use just below the Catch.  Make it use the clone.
19717
7819ffdaf0ff 8023691: Create interface for nodes in class Block
adlertz
parents: 19330
diff changeset
  1268
    fixup = use_blk->get_node(n_clone_idx);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1269
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1270
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1271
  return fixup;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1272
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1273
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1274
//--------------------------catch_cleanup_intra_block--------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1275
// Fix all input edges in use that reference "def".  The use is in the same
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1276
// block as the def and both have been cloned in each successor block.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1277
static void catch_cleanup_intra_block(Node *use, Node *def, Block *blk, int beg, int n_clone_idx) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1278
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1279
  // Both the use and def have been cloned. For each successor block,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1280
  // get the clone of the use, and make its input the clone of the def
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1281
  // found in that block.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1282
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1283
  uint use_idx = blk->find_node(use);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1284
  uint offset_idx = use_idx - beg;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1285
  for( uint k = 0; k < blk->_num_succs; k++ ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1286
    // Get clone in each successor block
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1287
    Block *sb = blk->_succs[k];
19717
7819ffdaf0ff 8023691: Create interface for nodes in class Block
adlertz
parents: 19330
diff changeset
  1288
    Node *clone = sb->get_node(offset_idx+1);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1289
    assert( clone->Opcode() == use->Opcode(), "" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1290
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1291
    // Make use-clone reference the def-clone
19717
7819ffdaf0ff 8023691: Create interface for nodes in class Block
adlertz
parents: 19330
diff changeset
  1292
    catch_cleanup_fix_all_inputs(clone, def, sb->get_node(n_clone_idx));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1293
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1294
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1295
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1296
//------------------------------catch_cleanup_inter_block---------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1297
// Fix all input edges in use that reference "def".  The use is in a different
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1298
// block than the def.
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1299
void PhaseCFG::catch_cleanup_inter_block(Node *use, Block *use_blk, Node *def, Block *def_blk, int n_clone_idx) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1300
  if( !use_blk ) return;        // Can happen if the use is a precedence edge
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1301
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1302
  Node *new_def = catch_cleanup_find_cloned_def(use_blk, def, def_blk, n_clone_idx);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1303
  catch_cleanup_fix_all_inputs(use, def, new_def);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1304
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1305
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1306
//------------------------------call_catch_cleanup-----------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1307
// If we inserted any instructions between a Call and his CatchNode,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1308
// clone the instructions on all paths below the Catch.
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1309
void PhaseCFG::call_catch_cleanup(Block* block) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1310
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1311
  // End of region to clone
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1312
  uint end = block->end_idx();
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1313
  if( !block->get_node(end)->is_Catch() ) return;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1314
  // Start of region to clone
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1315
  uint beg = end;
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1316
  while(!block->get_node(beg-1)->is_MachProj() ||
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1317
        !block->get_node(beg-1)->in(0)->is_MachCall() ) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1318
    beg--;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1319
    assert(beg > 0,"Catch cleanup walking beyond block boundary");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1320
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1321
  // Range of inserted instructions is [beg, end)
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1322
  if( beg == end ) return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1323
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1324
  // Clone along all Catch output paths.  Clone area between the 'beg' and
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1325
  // 'end' indices.
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1326
  for( uint i = 0; i < block->_num_succs; i++ ) {
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1327
    Block *sb = block->_succs[i];
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1328
    // Clone the entire area; ignoring the edge fixup for now.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1329
    for( uint j = end; j > beg; j-- ) {
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1330
      Node *clone = block->get_node(j-1)->clone();
19717
7819ffdaf0ff 8023691: Create interface for nodes in class Block
adlertz
parents: 19330
diff changeset
  1331
      sb->insert_node(clone, 1);
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1332
      map_node_to_block(clone, sb);
40086
82b9d0ae29b7 8157306: Random infrequent null pointer exceptions in javac
aph
parents: 36336
diff changeset
  1333
      if (clone->needs_anti_dependence_check()) {
82b9d0ae29b7 8157306: Random infrequent null pointer exceptions in javac
aph
parents: 36336
diff changeset
  1334
        insert_anti_dependences(sb, clone);
82b9d0ae29b7 8157306: Random infrequent null pointer exceptions in javac
aph
parents: 36336
diff changeset
  1335
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1336
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1337
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1338
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1339
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1340
  // Fixup edges.  Check the def-use info per cloned Node
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1341
  for(uint i2 = beg; i2 < end; i2++ ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1342
    uint n_clone_idx = i2-beg+1; // Index of clone of n in each successor block
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1343
    Node *n = block->get_node(i2);        // Node that got cloned
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1344
    // Need DU safe iterator because of edge manipulation in calls.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1345
    Unique_Node_List *out = new Unique_Node_List(Thread::current()->resource_area());
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1346
    for (DUIterator_Fast j1max, j1 = n->fast_outs(j1max); j1 < j1max; j1++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1347
      out->push(n->fast_out(j1));
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1348
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1349
    uint max = out->size();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1350
    for (uint j = 0; j < max; j++) {// For all users
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1351
      Node *use = out->pop();
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1352
      Block *buse = get_block_for_node(use);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1353
      if( use->is_Phi() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1354
        for( uint k = 1; k < use->req(); k++ )
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1355
          if( use->in(k) == n ) {
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1356
            Block* b = get_block_for_node(buse->pred(k));
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1357
            Node *fixup = catch_cleanup_find_cloned_def(b, n, block, n_clone_idx);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1358
            use->set_req(k, fixup);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1359
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1360
      } else {
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1361
        if (block == buse) {
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1362
          catch_cleanup_intra_block(use, n, block, beg, n_clone_idx);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1363
        } else {
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1364
          catch_cleanup_inter_block(use, buse, n, block, n_clone_idx);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1365
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1366
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1367
    } // End for all users
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1368
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1369
  } // End of for all Nodes in cloned area
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1370
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1371
  // Remove the now-dead cloned ops
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1372
  for(uint i3 = beg; i3 < end; i3++ ) {
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1373
    block->get_node(beg)->disconnect_inputs(NULL, C);
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1374
    block->remove_node(beg);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1375
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1376
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1377
  // If the successor blocks have a CreateEx node, move it back to the top
19721
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1378
  for(uint i4 = 0; i4 < block->_num_succs; i4++ ) {
8ecbb2cdc965 8023988: Move local scheduling of nodes to the CFG creation and code motion phase (PhaseCFG)
adlertz
parents: 19717
diff changeset
  1379
    Block *sb = block->_succs[i4];
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1380
    uint new_cnt = end - beg;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1381
    // Remove any newly created, but dead, nodes.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1382
    for( uint j = new_cnt; j > 0; j-- ) {
19717
7819ffdaf0ff 8023691: Create interface for nodes in class Block
adlertz
parents: 19330
diff changeset
  1383
      Node *n = sb->get_node(j);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1384
      if (n->outcnt() == 0 &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1385
          (!n->is_Proj() || n->as_Proj()->in(0)->outcnt() == 1) ){
14623
70c4c1be0a14 7092905: C2: Keep track of the number of dead nodes
bharadwaj
parents: 13969
diff changeset
  1386
        n->disconnect_inputs(NULL, C);
19717
7819ffdaf0ff 8023691: Create interface for nodes in class Block
adlertz
parents: 19330
diff changeset
  1387
        sb->remove_node(j);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1388
        new_cnt--;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1389
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1390
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1391
    // If any newly created nodes remain, move the CreateEx node to the top
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1392
    if (new_cnt > 0) {
19717
7819ffdaf0ff 8023691: Create interface for nodes in class Block
adlertz
parents: 19330
diff changeset
  1393
      Node *cex = sb->get_node(1+new_cnt);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1394
      if( cex->is_Mach() && cex->as_Mach()->ideal_Opcode() == Op_CreateEx ) {
19717
7819ffdaf0ff 8023691: Create interface for nodes in class Block
adlertz
parents: 19330
diff changeset
  1395
        sb->remove_node(1+new_cnt);
7819ffdaf0ff 8023691: Create interface for nodes in class Block
adlertz
parents: 19330
diff changeset
  1396
        sb->insert_node(cex, 1);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1397
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1398
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1399
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1400
}