hotspot/src/share/vm/opto/loopPredicate.cpp
author jwilhelm
Thu, 06 Jul 2017 01:50:26 +0200
changeset 46630 75aa3e39d02c
parent 37287 c2660335bf81
child 46735 219c4312853e
permissions -rw-r--r--
8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8 8182656: Make the required changes in GC code to build on OSX 10 + Xcode 8 8182657: Make the required changes in Runtime code to build on OSX 10 + Xcode 8 8182658: Make the required changes in Compiler code to build on OSX 10 + Xcode 8 Reviewed-by: jwilhelm, ehelin, phh Contributed-by: phh <hohensee@amazon.com>, jwilhelm <jesper.wilhelmsson@oracle.com>
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
     1
/*
46630
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 37287
diff changeset
     2
 * Copyright (c) 2011, 2017, Oracle and/or its affiliates. All rights reserved.
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
     4
 *
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
     7
 * published by the Free Software Foundation.
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
     8
 *
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    13
 * accompanied this code).
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    14
 *
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    18
 *
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    21
 * questions.
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    22
 *
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    23
 */
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    24
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    25
#include "precompiled.hpp"
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    26
#include "opto/loopnode.hpp"
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    27
#include "opto/addnode.hpp"
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    28
#include "opto/callnode.hpp"
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    29
#include "opto/connode.hpp"
23528
8f1a7f5e8066 8001532: C2 node files refactoring
morris
parents: 22234
diff changeset
    30
#include "opto/convertnode.hpp"
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    31
#include "opto/loopnode.hpp"
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    32
#include "opto/mulnode.hpp"
23528
8f1a7f5e8066 8001532: C2 node files refactoring
morris
parents: 22234
diff changeset
    33
#include "opto/opaquenode.hpp"
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    34
#include "opto/rootnode.hpp"
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    35
#include "opto/subnode.hpp"
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    36
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    37
/*
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    38
 * The general idea of Loop Predication is to insert a predicate on the entry
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    39
 * path to a loop, and raise a uncommon trap if the check of the condition fails.
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    40
 * The condition checks are promoted from inside the loop body, and thus
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    41
 * the checks inside the loop could be eliminated. Currently, loop predication
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    42
 * optimization has been applied to remove array range check and loop invariant
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    43
 * checks (such as null checks).
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    44
*/
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    45
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    46
//-------------------------------register_control-------------------------
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    47
void PhaseIdealLoop::register_control(Node* n, IdealLoopTree *loop, Node* pred) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    48
  assert(n->is_CFG(), "must be control node");
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    49
  _igvn.register_new_node_with_optimizer(n);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    50
  loop->_body.push(n);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    51
  set_loop(n, loop);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    52
  // When called from beautify_loops() idom is not constructed yet.
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    53
  if (_idom != NULL) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    54
    set_idom(n, pred, dom_depth(pred));
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    55
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    56
}
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    57
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    58
//------------------------------create_new_if_for_predicate------------------------
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    59
// create a new if above the uct_if_pattern for the predicate to be promoted.
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    60
//
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    61
//          before                                after
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    62
//        ----------                           ----------
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    63
//           ctrl                                 ctrl
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    64
//            |                                     |
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    65
//            |                                     |
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    66
//            v                                     v
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    67
//           iff                                 new_iff
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    68
//          /    \                                /      \
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    69
//         /      \                              /        \
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    70
//        v        v                            v          v
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    71
//  uncommon_proj cont_proj                   if_uct     if_cont
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    72
// \      |        |                           |          |
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    73
//  \     |        |                           |          |
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    74
//   v    v        v                           |          v
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    75
//     rgn       loop                          |         iff
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    76
//      |                                      |        /     \
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    77
//      |                                      |       /       \
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    78
//      v                                      |      v         v
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    79
// uncommon_trap                               | uncommon_proj cont_proj
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    80
//                                           \  \    |           |
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    81
//                                            \  \   |           |
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    82
//                                             v  v  v           v
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    83
//                                               rgn           loop
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    84
//                                                |
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    85
//                                                |
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    86
//                                                v
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    87
//                                           uncommon_trap
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    88
//
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    89
//
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    90
// We will create a region to guard the uct call if there is no one there.
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    91
// The true projecttion (if_cont) of the new_iff is returned.
30309
da3efc8ed2cb 8078426: mb/jvm/compiler/InterfaceCalls/testAC2 - assert(predicate_proj == 0L) failed: only one predicate entry expected
roland
parents: 26166
diff changeset
    92
// This code is also used to clone predicates to cloned loops.
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    93
ProjNode* PhaseIdealLoop::create_new_if_for_predicate(ProjNode* cont_proj, Node* new_entry,
34164
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 32733
diff changeset
    94
                                                      Deoptimization::DeoptReason reason,
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 32733
diff changeset
    95
                                                      int opcode) {
21089
e1986ff6fe2e 8024069: replace_in_map() should operate on parent maps
roland
parents: 17383
diff changeset
    96
  assert(cont_proj->is_uncommon_trap_if_pattern(reason), "must be a uct if pattern!");
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    97
  IfNode* iff = cont_proj->in(0)->as_If();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    98
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
    99
  ProjNode *uncommon_proj = iff->proj_out(1 - cont_proj->_con);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   100
  Node     *rgn   = uncommon_proj->unique_ctrl_out();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   101
  assert(rgn->is_Region() || rgn->is_Call(), "must be a region or call uct");
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   102
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   103
  uint proj_index = 1; // region's edge corresponding to uncommon_proj
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   104
  if (!rgn->is_Region()) { // create a region to guard the call
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   105
    assert(rgn->is_Call(), "must be call uct");
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   106
    CallNode* call = rgn->as_Call();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   107
    IdealLoopTree* loop = get_loop(call);
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24424
diff changeset
   108
    rgn = new RegionNode(1);
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   109
    rgn->add_req(uncommon_proj);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   110
    register_control(rgn, loop, uncommon_proj);
25913
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 24923
diff changeset
   111
    _igvn.replace_input_of(call, 0, rgn);
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   112
    // When called from beautify_loops() idom is not constructed yet.
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   113
    if (_idom != NULL) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   114
      set_idom(call, rgn, dom_depth(rgn));
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   115
    }
32733
0982a7e7eb15 8134974: 8130847 broken with loop predicates
roland
parents: 31035
diff changeset
   116
    for (DUIterator_Fast imax, i = uncommon_proj->fast_outs(imax); i < imax; i++) {
0982a7e7eb15 8134974: 8130847 broken with loop predicates
roland
parents: 31035
diff changeset
   117
      Node* n = uncommon_proj->fast_out(i);
0982a7e7eb15 8134974: 8130847 broken with loop predicates
roland
parents: 31035
diff changeset
   118
      if (n->is_Load() || n->is_Store()) {
0982a7e7eb15 8134974: 8130847 broken with loop predicates
roland
parents: 31035
diff changeset
   119
        _igvn.replace_input_of(n, 0, rgn);
0982a7e7eb15 8134974: 8130847 broken with loop predicates
roland
parents: 31035
diff changeset
   120
        --i; --imax;
0982a7e7eb15 8134974: 8130847 broken with loop predicates
roland
parents: 31035
diff changeset
   121
      }
0982a7e7eb15 8134974: 8130847 broken with loop predicates
roland
parents: 31035
diff changeset
   122
    }
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   123
  } else {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   124
    // Find region's edge corresponding to uncommon_proj
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   125
    for (; proj_index < rgn->req(); proj_index++)
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   126
      if (rgn->in(proj_index) == uncommon_proj) break;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   127
    assert(proj_index < rgn->req(), "sanity");
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   128
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   129
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   130
  Node* entry = iff->in(0);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   131
  if (new_entry != NULL) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   132
    // Clonning the predicate to new location.
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   133
    entry = new_entry;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   134
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   135
  // Create new_iff
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   136
  IdealLoopTree* lp = get_loop(entry);
34164
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 32733
diff changeset
   137
  IfNode* new_iff = NULL;
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 32733
diff changeset
   138
  if (opcode == Op_If) {
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 32733
diff changeset
   139
    new_iff = new IfNode(entry, iff->in(1), iff->_prob, iff->_fcnt);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 32733
diff changeset
   140
  } else {
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 32733
diff changeset
   141
    assert(opcode == Op_RangeCheck, "no other if variant here");
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 32733
diff changeset
   142
    new_iff = new RangeCheckNode(entry, iff->in(1), iff->_prob, iff->_fcnt);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 32733
diff changeset
   143
  }
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   144
  register_control(new_iff, lp, entry);
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24424
diff changeset
   145
  Node *if_cont = new IfTrueNode(new_iff);
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24424
diff changeset
   146
  Node *if_uct  = new IfFalseNode(new_iff);
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   147
  if (cont_proj->is_IfFalse()) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   148
    // Swap
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   149
    Node* tmp = if_uct; if_uct = if_cont; if_cont = tmp;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   150
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   151
  register_control(if_cont, lp, new_iff);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   152
  register_control(if_uct, get_loop(rgn), new_iff);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   153
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   154
  // if_uct to rgn
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   155
  _igvn.hash_delete(rgn);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   156
  rgn->add_req(if_uct);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   157
  // When called from beautify_loops() idom is not constructed yet.
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   158
  if (_idom != NULL) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   159
    Node* ridom = idom(rgn);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   160
    Node* nrdom = dom_lca(ridom, new_iff);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   161
    set_idom(rgn, nrdom, dom_depth(rgn));
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   162
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   163
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   164
  // If rgn has phis add new edges which has the same
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   165
  // value as on original uncommon_proj pass.
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   166
  assert(rgn->in(rgn->req() -1) == if_uct, "new edge should be last");
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   167
  bool has_phi = false;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   168
  for (DUIterator_Fast imax, i = rgn->fast_outs(imax); i < imax; i++) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   169
    Node* use = rgn->fast_out(i);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   170
    if (use->is_Phi() && use->outcnt() > 0) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   171
      assert(use->in(0) == rgn, "");
12958
009b6c9586d8 7173340: C2: code cleanup: use PhaseIterGVN::replace_edge(Node*, int, Node*) where applicable
kvn
parents: 10263
diff changeset
   172
      _igvn.rehash_node_delayed(use);
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   173
      use->add_req(use->in(proj_index));
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   174
      has_phi = true;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   175
    }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   176
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   177
  assert(!has_phi || rgn->req() > 3, "no phis when region is created");
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   178
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   179
  if (new_entry == NULL) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   180
    // Attach if_cont to iff
25913
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 24923
diff changeset
   181
    _igvn.replace_input_of(iff, 0, if_cont);
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   182
    if (_idom != NULL) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   183
      set_idom(iff, if_cont, dom_depth(iff));
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   184
    }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   185
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   186
  return if_cont->as_Proj();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   187
}
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   188
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   189
//------------------------------create_new_if_for_predicate------------------------
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   190
// Create a new if below new_entry for the predicate to be cloned (IGVN optimization)
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   191
ProjNode* PhaseIterGVN::create_new_if_for_predicate(ProjNode* cont_proj, Node* new_entry,
34164
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 32733
diff changeset
   192
                                                    Deoptimization::DeoptReason reason,
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 32733
diff changeset
   193
                                                    int opcode) {
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   194
  assert(new_entry != 0, "only used for clone predicate");
21089
e1986ff6fe2e 8024069: replace_in_map() should operate on parent maps
roland
parents: 17383
diff changeset
   195
  assert(cont_proj->is_uncommon_trap_if_pattern(reason), "must be a uct if pattern!");
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   196
  IfNode* iff = cont_proj->in(0)->as_If();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   197
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   198
  ProjNode *uncommon_proj = iff->proj_out(1 - cont_proj->_con);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   199
  Node     *rgn   = uncommon_proj->unique_ctrl_out();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   200
  assert(rgn->is_Region() || rgn->is_Call(), "must be a region or call uct");
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   201
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   202
  uint proj_index = 1; // region's edge corresponding to uncommon_proj
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   203
  if (!rgn->is_Region()) { // create a region to guard the call
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   204
    assert(rgn->is_Call(), "must be call uct");
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   205
    CallNode* call = rgn->as_Call();
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24424
diff changeset
   206
    rgn = new RegionNode(1);
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   207
    register_new_node_with_optimizer(rgn);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   208
    rgn->add_req(uncommon_proj);
25913
81dbc151e91c 8040213: C2 does not put all modified nodes on IGVN worklist
thartmann
parents: 24923
diff changeset
   209
    replace_input_of(call, 0, rgn);
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   210
  } else {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   211
    // Find region's edge corresponding to uncommon_proj
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   212
    for (; proj_index < rgn->req(); proj_index++)
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   213
      if (rgn->in(proj_index) == uncommon_proj) break;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   214
    assert(proj_index < rgn->req(), "sanity");
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   215
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   216
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   217
  // Create new_iff in new location.
34164
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 32733
diff changeset
   218
  IfNode* new_iff = NULL;
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 32733
diff changeset
   219
  if (opcode == Op_If) {
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 32733
diff changeset
   220
    new_iff = new IfNode(new_entry, iff->in(1), iff->_prob, iff->_fcnt);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 32733
diff changeset
   221
  } else {
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 32733
diff changeset
   222
    assert(opcode == Op_RangeCheck, "no other if variant here");
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 32733
diff changeset
   223
    new_iff = new RangeCheckNode(new_entry, iff->in(1), iff->_prob, iff->_fcnt);
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 32733
diff changeset
   224
  }
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   225
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   226
  register_new_node_with_optimizer(new_iff);
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24424
diff changeset
   227
  Node *if_cont = new IfTrueNode(new_iff);
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24424
diff changeset
   228
  Node *if_uct  = new IfFalseNode(new_iff);
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   229
  if (cont_proj->is_IfFalse()) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   230
    // Swap
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   231
    Node* tmp = if_uct; if_uct = if_cont; if_cont = tmp;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   232
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   233
  register_new_node_with_optimizer(if_cont);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   234
  register_new_node_with_optimizer(if_uct);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   235
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   236
  // if_uct to rgn
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   237
  hash_delete(rgn);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   238
  rgn->add_req(if_uct);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   239
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   240
  // If rgn has phis add corresponding new edges which has the same
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   241
  // value as on original uncommon_proj pass.
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   242
  assert(rgn->in(rgn->req() -1) == if_uct, "new edge should be last");
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   243
  bool has_phi = false;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   244
  for (DUIterator_Fast imax, i = rgn->fast_outs(imax); i < imax; i++) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   245
    Node* use = rgn->fast_out(i);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   246
    if (use->is_Phi() && use->outcnt() > 0) {
12958
009b6c9586d8 7173340: C2: code cleanup: use PhaseIterGVN::replace_edge(Node*, int, Node*) where applicable
kvn
parents: 10263
diff changeset
   247
      rehash_node_delayed(use);
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   248
      use->add_req(use->in(proj_index));
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   249
      has_phi = true;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   250
    }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   251
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   252
  assert(!has_phi || rgn->req() > 3, "no phis when region is created");
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   253
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   254
  return if_cont->as_Proj();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   255
}
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   256
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   257
//--------------------------clone_predicate-----------------------
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   258
ProjNode* PhaseIdealLoop::clone_predicate(ProjNode* predicate_proj, Node* new_entry,
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   259
                                          Deoptimization::DeoptReason reason,
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   260
                                          PhaseIdealLoop* loop_phase,
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   261
                                          PhaseIterGVN* igvn) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   262
  ProjNode* new_predicate_proj;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   263
  if (loop_phase != NULL) {
34164
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 32733
diff changeset
   264
    new_predicate_proj = loop_phase->create_new_if_for_predicate(predicate_proj, new_entry, reason, Op_If);
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   265
  } else {
34164
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 32733
diff changeset
   266
    new_predicate_proj =       igvn->create_new_if_for_predicate(predicate_proj, new_entry, reason, Op_If);
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   267
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   268
  IfNode* iff = new_predicate_proj->in(0)->as_If();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   269
  Node* ctrl  = iff->in(0);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   270
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   271
  // Match original condition since predicate's projections could be swapped.
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   272
  assert(predicate_proj->in(0)->in(1)->in(1)->Opcode()==Op_Opaque1, "must be");
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24424
diff changeset
   273
  Node* opq = new Opaque1Node(igvn->C, predicate_proj->in(0)->in(1)->in(1)->in(1));
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   274
  igvn->C->add_predicate_opaq(opq);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   275
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24424
diff changeset
   276
  Node* bol = new Conv2BNode(opq);
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   277
  if (loop_phase != NULL) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   278
    loop_phase->register_new_node(opq, ctrl);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   279
    loop_phase->register_new_node(bol, ctrl);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   280
  } else {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   281
    igvn->register_new_node_with_optimizer(opq);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   282
    igvn->register_new_node_with_optimizer(bol);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   283
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   284
  igvn->hash_delete(iff);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   285
  iff->set_req(1, bol);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   286
  return new_predicate_proj;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   287
}
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   288
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   289
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   290
//--------------------------clone_loop_predicates-----------------------
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   291
// Interface from IGVN
9446
748a37b25d10 5091921: Sign flip issues in loop optimizer
kvn
parents: 9437
diff changeset
   292
Node* PhaseIterGVN::clone_loop_predicates(Node* old_entry, Node* new_entry, bool clone_limit_check) {
10258
10c77b8c8d3e 7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents: 10253
diff changeset
   293
  return PhaseIdealLoop::clone_loop_predicates(old_entry, new_entry, clone_limit_check, NULL, this);
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   294
}
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   295
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   296
// Interface from PhaseIdealLoop
9446
748a37b25d10 5091921: Sign flip issues in loop optimizer
kvn
parents: 9437
diff changeset
   297
Node* PhaseIdealLoop::clone_loop_predicates(Node* old_entry, Node* new_entry, bool clone_limit_check) {
10258
10c77b8c8d3e 7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents: 10253
diff changeset
   298
  return clone_loop_predicates(old_entry, new_entry, clone_limit_check, this, &this->_igvn);
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   299
}
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   300
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   301
// Clone loop predicates to cloned loops (peeled, unswitched, split_if).
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   302
Node* PhaseIdealLoop::clone_loop_predicates(Node* old_entry, Node* new_entry,
9446
748a37b25d10 5091921: Sign flip issues in loop optimizer
kvn
parents: 9437
diff changeset
   303
                                                bool clone_limit_check,
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   304
                                                PhaseIdealLoop* loop_phase,
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   305
                                                PhaseIterGVN* igvn) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   306
#ifdef ASSERT
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   307
  if (new_entry == NULL || !(new_entry->is_Proj() || new_entry->is_Region() || new_entry->is_SafePoint())) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   308
    if (new_entry != NULL)
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   309
      new_entry->dump();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   310
    assert(false, "not IfTrue, IfFalse, Region or SafePoint");
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   311
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   312
#endif
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   313
  // Search original predicates
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   314
  Node* entry = old_entry;
9446
748a37b25d10 5091921: Sign flip issues in loop optimizer
kvn
parents: 9437
diff changeset
   315
  ProjNode* limit_check_proj = NULL;
37287
c2660335bf81 8072422: Cleanup: Remove some unused flags/code in loop optimizations
zmajo
parents: 35558
diff changeset
   316
  limit_check_proj = find_predicate_insertion_point(entry, Deoptimization::Reason_loop_limit_check);
c2660335bf81 8072422: Cleanup: Remove some unused flags/code in loop optimizations
zmajo
parents: 35558
diff changeset
   317
  if (limit_check_proj != NULL) {
c2660335bf81 8072422: Cleanup: Remove some unused flags/code in loop optimizations
zmajo
parents: 35558
diff changeset
   318
    entry = entry->in(0)->in(0);
9446
748a37b25d10 5091921: Sign flip issues in loop optimizer
kvn
parents: 9437
diff changeset
   319
  }
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   320
  if (UseLoopPredicate) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   321
    ProjNode* predicate_proj = find_predicate_insertion_point(entry, Deoptimization::Reason_predicate);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   322
    if (predicate_proj != NULL) { // right pattern that can be used by loop predication
10258
10c77b8c8d3e 7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents: 10253
diff changeset
   323
      // clone predicate
10c77b8c8d3e 7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents: 10253
diff changeset
   324
      new_entry = clone_predicate(predicate_proj, new_entry,
10c77b8c8d3e 7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents: 10253
diff changeset
   325
                                  Deoptimization::Reason_predicate,
10c77b8c8d3e 7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents: 10253
diff changeset
   326
                                  loop_phase, igvn);
10c77b8c8d3e 7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents: 10253
diff changeset
   327
      assert(new_entry != NULL && new_entry->is_Proj(), "IfTrue or IfFalse after clone predicate");
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   328
      if (TraceLoopPredicate) {
10258
10c77b8c8d3e 7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents: 10253
diff changeset
   329
        tty->print("Loop Predicate cloned: ");
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   330
        debug_only( new_entry->in(0)->dump(); )
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   331
      }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   332
    }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   333
  }
9446
748a37b25d10 5091921: Sign flip issues in loop optimizer
kvn
parents: 9437
diff changeset
   334
  if (limit_check_proj != NULL && clone_limit_check) {
748a37b25d10 5091921: Sign flip issues in loop optimizer
kvn
parents: 9437
diff changeset
   335
    // Clone loop limit check last to insert it before loop.
748a37b25d10 5091921: Sign flip issues in loop optimizer
kvn
parents: 9437
diff changeset
   336
    // Don't clone a limit check which was already finalized
748a37b25d10 5091921: Sign flip issues in loop optimizer
kvn
parents: 9437
diff changeset
   337
    // for this counted loop (only one limit check is needed).
10258
10c77b8c8d3e 7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents: 10253
diff changeset
   338
    new_entry = clone_predicate(limit_check_proj, new_entry,
10c77b8c8d3e 7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents: 10253
diff changeset
   339
                                Deoptimization::Reason_loop_limit_check,
10c77b8c8d3e 7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents: 10253
diff changeset
   340
                                loop_phase, igvn);
10c77b8c8d3e 7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents: 10253
diff changeset
   341
    assert(new_entry != NULL && new_entry->is_Proj(), "IfTrue or IfFalse after clone limit check");
9446
748a37b25d10 5091921: Sign flip issues in loop optimizer
kvn
parents: 9437
diff changeset
   342
    if (TraceLoopLimitCheck) {
10258
10c77b8c8d3e 7068051: SIGSEGV in PhaseIdealLoop::build_loop_late_post
kvn
parents: 10253
diff changeset
   343
      tty->print("Loop Limit Check cloned: ");
9446
748a37b25d10 5091921: Sign flip issues in loop optimizer
kvn
parents: 9437
diff changeset
   344
      debug_only( new_entry->in(0)->dump(); )
748a37b25d10 5091921: Sign flip issues in loop optimizer
kvn
parents: 9437
diff changeset
   345
    }
748a37b25d10 5091921: Sign flip issues in loop optimizer
kvn
parents: 9437
diff changeset
   346
  }
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   347
  return new_entry;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   348
}
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   349
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   350
//--------------------------skip_loop_predicates------------------------------
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   351
// Skip related predicates.
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   352
Node* PhaseIdealLoop::skip_loop_predicates(Node* entry) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   353
  Node* predicate = NULL;
37287
c2660335bf81 8072422: Cleanup: Remove some unused flags/code in loop optimizations
zmajo
parents: 35558
diff changeset
   354
  predicate = find_predicate_insertion_point(entry, Deoptimization::Reason_loop_limit_check);
c2660335bf81 8072422: Cleanup: Remove some unused flags/code in loop optimizations
zmajo
parents: 35558
diff changeset
   355
  if (predicate != NULL) {
c2660335bf81 8072422: Cleanup: Remove some unused flags/code in loop optimizations
zmajo
parents: 35558
diff changeset
   356
    entry = entry->in(0)->in(0);
9446
748a37b25d10 5091921: Sign flip issues in loop optimizer
kvn
parents: 9437
diff changeset
   357
  }
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   358
  if (UseLoopPredicate) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   359
    predicate = find_predicate_insertion_point(entry, Deoptimization::Reason_predicate);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   360
    if (predicate != NULL) { // right pattern that can be used by loop predication
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   361
      IfNode* iff = entry->in(0)->as_If();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   362
      ProjNode* uncommon_proj = iff->proj_out(1 - entry->as_Proj()->_con);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   363
      Node* rgn = uncommon_proj->unique_ctrl_out();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   364
      assert(rgn->is_Region() || rgn->is_Call(), "must be a region or call uct");
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   365
      entry = entry->in(0)->in(0);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   366
      while (entry != NULL && entry->is_Proj() && entry->in(0)->is_If()) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   367
        uncommon_proj = entry->in(0)->as_If()->proj_out(1 - entry->as_Proj()->_con);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   368
        if (uncommon_proj->unique_ctrl_out() != rgn)
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   369
          break;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   370
        entry = entry->in(0)->in(0);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   371
      }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   372
    }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   373
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   374
  return entry;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   375
}
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   376
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   377
//--------------------------find_predicate_insertion_point-------------------
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   378
// Find a good location to insert a predicate
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   379
ProjNode* PhaseIdealLoop::find_predicate_insertion_point(Node* start_c, Deoptimization::DeoptReason reason) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   380
  if (start_c == NULL || !start_c->is_Proj())
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   381
    return NULL;
21089
e1986ff6fe2e 8024069: replace_in_map() should operate on parent maps
roland
parents: 17383
diff changeset
   382
  if (start_c->as_Proj()->is_uncommon_trap_if_pattern(reason)) {
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   383
    return start_c->as_Proj();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   384
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   385
  return NULL;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   386
}
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   387
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   388
//--------------------------find_predicate------------------------------------
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   389
// Find a predicate
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   390
Node* PhaseIdealLoop::find_predicate(Node* entry) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   391
  Node* predicate = NULL;
37287
c2660335bf81 8072422: Cleanup: Remove some unused flags/code in loop optimizations
zmajo
parents: 35558
diff changeset
   392
  predicate = find_predicate_insertion_point(entry, Deoptimization::Reason_loop_limit_check);
c2660335bf81 8072422: Cleanup: Remove some unused flags/code in loop optimizations
zmajo
parents: 35558
diff changeset
   393
  if (predicate != NULL) { // right pattern that can be used by loop predication
c2660335bf81 8072422: Cleanup: Remove some unused flags/code in loop optimizations
zmajo
parents: 35558
diff changeset
   394
    return entry;
9446
748a37b25d10 5091921: Sign flip issues in loop optimizer
kvn
parents: 9437
diff changeset
   395
  }
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   396
  if (UseLoopPredicate) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   397
    predicate = find_predicate_insertion_point(entry, Deoptimization::Reason_predicate);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   398
    if (predicate != NULL) { // right pattern that can be used by loop predication
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   399
      return entry;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   400
    }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   401
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   402
  return NULL;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   403
}
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   404
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   405
//------------------------------Invariance-----------------------------------
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   406
// Helper class for loop_predication_impl to compute invariance on the fly and
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   407
// clone invariants.
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   408
class Invariance : public StackObj {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   409
  VectorSet _visited, _invariant;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   410
  Node_Stack _stack;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   411
  VectorSet _clone_visited;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   412
  Node_List _old_new; // map of old to new (clone)
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   413
  IdealLoopTree* _lpt;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   414
  PhaseIdealLoop* _phase;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   415
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   416
  // Helper function to set up the invariance for invariance computation
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   417
  // If n is a known invariant, set up directly. Otherwise, look up the
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   418
  // the possibility to push n onto the stack for further processing.
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   419
  void visit(Node* use, Node* n) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   420
    if (_lpt->is_invariant(n)) { // known invariant
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   421
      _invariant.set(n->_idx);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   422
    } else if (!n->is_CFG()) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   423
      Node *n_ctrl = _phase->ctrl_or_self(n);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   424
      Node *u_ctrl = _phase->ctrl_or_self(use); // self if use is a CFG
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   425
      if (_phase->is_dominator(n_ctrl, u_ctrl)) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   426
        _stack.push(n, n->in(0) == NULL ? 1 : 0);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   427
      }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   428
    }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   429
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   430
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   431
  // Compute invariance for "the_node" and (possibly) all its inputs recursively
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   432
  // on the fly
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   433
  void compute_invariance(Node* n) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   434
    assert(_visited.test(n->_idx), "must be");
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   435
    visit(n, n);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   436
    while (_stack.is_nonempty()) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   437
      Node*  n = _stack.node();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   438
      uint idx = _stack.index();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   439
      if (idx == n->req()) { // all inputs are processed
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   440
        _stack.pop();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   441
        // n is invariant if it's inputs are all invariant
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   442
        bool all_inputs_invariant = true;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   443
        for (uint i = 0; i < n->req(); i++) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   444
          Node* in = n->in(i);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   445
          if (in == NULL) continue;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   446
          assert(_visited.test(in->_idx), "must have visited input");
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   447
          if (!_invariant.test(in->_idx)) { // bad guy
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   448
            all_inputs_invariant = false;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   449
            break;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   450
          }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   451
        }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   452
        if (all_inputs_invariant) {
31035
0f0743952c41 8077504: Unsafe load can loose control dependency and cause crash
roland
parents: 30309
diff changeset
   453
          // If n's control is a predicate that was moved out of the
0f0743952c41 8077504: Unsafe load can loose control dependency and cause crash
roland
parents: 30309
diff changeset
   454
          // loop, it was marked invariant but n is only invariant if
0f0743952c41 8077504: Unsafe load can loose control dependency and cause crash
roland
parents: 30309
diff changeset
   455
          // it depends only on that test. Otherwise, unless that test
0f0743952c41 8077504: Unsafe load can loose control dependency and cause crash
roland
parents: 30309
diff changeset
   456
          // is out of the loop, it's not invariant.
0f0743952c41 8077504: Unsafe load can loose control dependency and cause crash
roland
parents: 30309
diff changeset
   457
          if (n->is_CFG() || n->depends_only_on_test() || n->in(0) == NULL || !_phase->is_member(_lpt, n->in(0))) {
0f0743952c41 8077504: Unsafe load can loose control dependency and cause crash
roland
parents: 30309
diff changeset
   458
            _invariant.set(n->_idx); // I am a invariant too
0f0743952c41 8077504: Unsafe load can loose control dependency and cause crash
roland
parents: 30309
diff changeset
   459
          }
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   460
        }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   461
      } else { // process next input
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   462
        _stack.set_index(idx + 1);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   463
        Node* m = n->in(idx);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   464
        if (m != NULL && !_visited.test_set(m->_idx)) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   465
          visit(n, m);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   466
        }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   467
      }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   468
    }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   469
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   470
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   471
  // Helper function to set up _old_new map for clone_nodes.
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   472
  // If n is a known invariant, set up directly ("clone" of n == n).
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   473
  // Otherwise, push n onto the stack for real cloning.
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   474
  void clone_visit(Node* n) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   475
    assert(_invariant.test(n->_idx), "must be invariant");
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   476
    if (_lpt->is_invariant(n)) { // known invariant
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   477
      _old_new.map(n->_idx, n);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   478
    } else { // to be cloned
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   479
      assert(!n->is_CFG(), "should not see CFG here");
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   480
      _stack.push(n, n->in(0) == NULL ? 1 : 0);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   481
    }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   482
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   483
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   484
  // Clone "n" and (possibly) all its inputs recursively
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   485
  void clone_nodes(Node* n, Node* ctrl) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   486
    clone_visit(n);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   487
    while (_stack.is_nonempty()) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   488
      Node*  n = _stack.node();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   489
      uint idx = _stack.index();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   490
      if (idx == n->req()) { // all inputs processed, clone n!
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   491
        _stack.pop();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   492
        // clone invariant node
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   493
        Node* n_cl = n->clone();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   494
        _old_new.map(n->_idx, n_cl);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   495
        _phase->register_new_node(n_cl, ctrl);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   496
        for (uint i = 0; i < n->req(); i++) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   497
          Node* in = n_cl->in(i);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   498
          if (in == NULL) continue;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   499
          n_cl->set_req(i, _old_new[in->_idx]);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   500
        }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   501
      } else { // process next input
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   502
        _stack.set_index(idx + 1);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   503
        Node* m = n->in(idx);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   504
        if (m != NULL && !_clone_visited.test_set(m->_idx)) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   505
          clone_visit(m); // visit the input
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   506
        }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   507
      }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   508
    }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   509
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   510
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   511
 public:
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   512
  Invariance(Arena* area, IdealLoopTree* lpt) :
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   513
    _lpt(lpt), _phase(lpt->_phase),
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   514
    _visited(area), _invariant(area), _stack(area, 10 /* guess */),
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   515
    _clone_visited(area), _old_new(area)
35558
9b3b2740c3ec 8146792: Predicate moved after partial peel may lead to broken graph
roland
parents: 34180
diff changeset
   516
  {
9b3b2740c3ec 8146792: Predicate moved after partial peel may lead to broken graph
roland
parents: 34180
diff changeset
   517
    Node* head = _lpt->_head;
9b3b2740c3ec 8146792: Predicate moved after partial peel may lead to broken graph
roland
parents: 34180
diff changeset
   518
    Node* entry = head->in(LoopNode::EntryControl);
9b3b2740c3ec 8146792: Predicate moved after partial peel may lead to broken graph
roland
parents: 34180
diff changeset
   519
    if (entry->outcnt() != 1) {
9b3b2740c3ec 8146792: Predicate moved after partial peel may lead to broken graph
roland
parents: 34180
diff changeset
   520
      // If a node is pinned between the predicates and the loop
9b3b2740c3ec 8146792: Predicate moved after partial peel may lead to broken graph
roland
parents: 34180
diff changeset
   521
      // entry, we won't be able to move any node in the loop that
9b3b2740c3ec 8146792: Predicate moved after partial peel may lead to broken graph
roland
parents: 34180
diff changeset
   522
      // depends on it above it in a predicate. Mark all those nodes
9b3b2740c3ec 8146792: Predicate moved after partial peel may lead to broken graph
roland
parents: 34180
diff changeset
   523
      // as non loop invariatnt.
9b3b2740c3ec 8146792: Predicate moved after partial peel may lead to broken graph
roland
parents: 34180
diff changeset
   524
      Unique_Node_List wq;
9b3b2740c3ec 8146792: Predicate moved after partial peel may lead to broken graph
roland
parents: 34180
diff changeset
   525
      wq.push(entry);
9b3b2740c3ec 8146792: Predicate moved after partial peel may lead to broken graph
roland
parents: 34180
diff changeset
   526
      for (uint next = 0; next < wq.size(); ++next) {
9b3b2740c3ec 8146792: Predicate moved after partial peel may lead to broken graph
roland
parents: 34180
diff changeset
   527
        Node *n = wq.at(next);
9b3b2740c3ec 8146792: Predicate moved after partial peel may lead to broken graph
roland
parents: 34180
diff changeset
   528
        for (DUIterator_Fast imax, i = n->fast_outs(imax); i < imax; i++) {
9b3b2740c3ec 8146792: Predicate moved after partial peel may lead to broken graph
roland
parents: 34180
diff changeset
   529
          Node* u = n->fast_out(i);
9b3b2740c3ec 8146792: Predicate moved after partial peel may lead to broken graph
roland
parents: 34180
diff changeset
   530
          if (!u->is_CFG()) {
9b3b2740c3ec 8146792: Predicate moved after partial peel may lead to broken graph
roland
parents: 34180
diff changeset
   531
            Node* c = _phase->get_ctrl(u);
9b3b2740c3ec 8146792: Predicate moved after partial peel may lead to broken graph
roland
parents: 34180
diff changeset
   532
            if (_lpt->is_member(_phase->get_loop(c)) || _phase->is_dominator(c, head)) {
9b3b2740c3ec 8146792: Predicate moved after partial peel may lead to broken graph
roland
parents: 34180
diff changeset
   533
              _visited.set(u->_idx);
9b3b2740c3ec 8146792: Predicate moved after partial peel may lead to broken graph
roland
parents: 34180
diff changeset
   534
              wq.push(u);
9b3b2740c3ec 8146792: Predicate moved after partial peel may lead to broken graph
roland
parents: 34180
diff changeset
   535
            }
9b3b2740c3ec 8146792: Predicate moved after partial peel may lead to broken graph
roland
parents: 34180
diff changeset
   536
          }
9b3b2740c3ec 8146792: Predicate moved after partial peel may lead to broken graph
roland
parents: 34180
diff changeset
   537
        }
9b3b2740c3ec 8146792: Predicate moved after partial peel may lead to broken graph
roland
parents: 34180
diff changeset
   538
      }
9b3b2740c3ec 8146792: Predicate moved after partial peel may lead to broken graph
roland
parents: 34180
diff changeset
   539
    }
9b3b2740c3ec 8146792: Predicate moved after partial peel may lead to broken graph
roland
parents: 34180
diff changeset
   540
  }
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   541
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   542
  // Map old to n for invariance computation and clone
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   543
  void map_ctrl(Node* old, Node* n) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   544
    assert(old->is_CFG() && n->is_CFG(), "must be");
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   545
    _old_new.map(old->_idx, n); // "clone" of old is n
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   546
    _invariant.set(old->_idx);  // old is invariant
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   547
    _clone_visited.set(old->_idx);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   548
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   549
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   550
  // Driver function to compute invariance
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   551
  bool is_invariant(Node* n) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   552
    if (!_visited.test_set(n->_idx))
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   553
      compute_invariance(n);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   554
    return (_invariant.test(n->_idx) != 0);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   555
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   556
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   557
  // Driver function to clone invariant
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   558
  Node* clone(Node* n, Node* ctrl) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   559
    assert(ctrl->is_CFG(), "must be");
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   560
    assert(_invariant.test(n->_idx), "must be an invariant");
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   561
    if (!_clone_visited.test(n->_idx))
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   562
      clone_nodes(n, ctrl);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   563
    return _old_new[n->_idx];
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   564
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   565
};
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   566
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   567
//------------------------------is_range_check_if -----------------------------------
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   568
// Returns true if the predicate of iff is in "scale*iv + offset u< load_range(ptr)" format
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   569
// Note: this function is particularly designed for loop predication. We require load_range
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   570
//       and offset to be loop invariant computed on the fly by "invar"
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   571
bool IdealLoopTree::is_range_check_if(IfNode *iff, PhaseIdealLoop *phase, Invariance& invar) const {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   572
  if (!is_loop_exit(iff)) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   573
    return false;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   574
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   575
  if (!iff->in(1)->is_Bool()) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   576
    return false;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   577
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   578
  const BoolNode *bol = iff->in(1)->as_Bool();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   579
  if (bol->_test._test != BoolTest::lt) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   580
    return false;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   581
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   582
  if (!bol->in(1)->is_Cmp()) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   583
    return false;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   584
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   585
  const CmpNode *cmp = bol->in(1)->as_Cmp();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   586
  if (cmp->Opcode() != Op_CmpU) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   587
    return false;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   588
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   589
  Node* range = cmp->in(2);
34180
f0ec91019db2 8042997: Make intrinsic some or all check index/range methods
roland
parents: 34164
diff changeset
   590
  if (range->Opcode() != Op_LoadRange && !iff->is_RangeCheck()) {
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   591
    const TypeInt* tint = phase->_igvn.type(range)->isa_int();
9446
748a37b25d10 5091921: Sign flip issues in loop optimizer
kvn
parents: 9437
diff changeset
   592
    if (tint == NULL || tint->empty() || tint->_lo < 0) {
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   593
      // Allow predication on positive values that aren't LoadRanges.
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   594
      // This allows optimization of loops where the length of the
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   595
      // array is a known value and doesn't need to be loaded back
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   596
      // from the array.
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   597
      return false;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   598
    }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   599
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   600
  if (!invar.is_invariant(range)) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   601
    return false;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   602
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   603
  Node *iv     = _head->as_CountedLoop()->phi();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   604
  int   scale  = 0;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   605
  Node *offset = NULL;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   606
  if (!phase->is_scaled_iv_plus_offset(cmp->in(1), iv, &scale, &offset)) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   607
    return false;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   608
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   609
  if (offset && !invar.is_invariant(offset)) { // offset must be invariant
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   610
    return false;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   611
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   612
  return true;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   613
}
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   614
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   615
//------------------------------rc_predicate-----------------------------------
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   616
// Create a range check predicate
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   617
//
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   618
// for (i = init; i < limit; i += stride) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   619
//    a[scale*i+offset]
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   620
// }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   621
//
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   622
// Compute max(scale*i + offset) for init <= i < limit and build the predicate
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   623
// as "max(scale*i + offset) u< a.length".
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   624
//
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   625
// There are two cases for max(scale*i + offset):
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   626
// (1) stride*scale > 0
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   627
//   max(scale*i + offset) = scale*(limit-stride) + offset
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   628
// (2) stride*scale < 0
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   629
//   max(scale*i + offset) = scale*init + offset
9446
748a37b25d10 5091921: Sign flip issues in loop optimizer
kvn
parents: 9437
diff changeset
   630
BoolNode* PhaseIdealLoop::rc_predicate(IdealLoopTree *loop, Node* ctrl,
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   631
                                       int scale, Node* offset,
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   632
                                       Node* init, Node* limit, Node* stride,
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   633
                                       Node* range, bool upper) {
9437
9981851b4b8c 7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents: 9101
diff changeset
   634
  stringStream* predString = NULL;
9981851b4b8c 7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents: 9101
diff changeset
   635
  if (TraceLoopPredicate) {
9981851b4b8c 7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents: 9101
diff changeset
   636
    predString = new stringStream();
9981851b4b8c 7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents: 9101
diff changeset
   637
    predString->print("rc_predicate ");
9981851b4b8c 7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents: 9101
diff changeset
   638
  }
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   639
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   640
  Node* max_idx_expr  = init;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   641
  int stride_con = stride->get_int();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   642
  if ((stride_con > 0) == (scale > 0) == upper) {
37287
c2660335bf81 8072422: Cleanup: Remove some unused flags/code in loop optimizations
zmajo
parents: 35558
diff changeset
   643
    // Limit is not exact.
c2660335bf81 8072422: Cleanup: Remove some unused flags/code in loop optimizations
zmajo
parents: 35558
diff changeset
   644
    // Calculate exact limit here.
c2660335bf81 8072422: Cleanup: Remove some unused flags/code in loop optimizations
zmajo
parents: 35558
diff changeset
   645
    // Note, counted loop's test is '<' or '>'.
c2660335bf81 8072422: Cleanup: Remove some unused flags/code in loop optimizations
zmajo
parents: 35558
diff changeset
   646
    limit = exact_limit(loop);
c2660335bf81 8072422: Cleanup: Remove some unused flags/code in loop optimizations
zmajo
parents: 35558
diff changeset
   647
    max_idx_expr = new SubINode(limit, stride);
c2660335bf81 8072422: Cleanup: Remove some unused flags/code in loop optimizations
zmajo
parents: 35558
diff changeset
   648
    register_new_node(max_idx_expr, ctrl);
c2660335bf81 8072422: Cleanup: Remove some unused flags/code in loop optimizations
zmajo
parents: 35558
diff changeset
   649
    if (TraceLoopPredicate) predString->print("(limit - stride) ");
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   650
  } else {
9437
9981851b4b8c 7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents: 9101
diff changeset
   651
    if (TraceLoopPredicate) predString->print("init ");
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   652
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   653
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   654
  if (scale != 1) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   655
    ConNode* con_scale = _igvn.intcon(scale);
35558
9b3b2740c3ec 8146792: Predicate moved after partial peel may lead to broken graph
roland
parents: 34180
diff changeset
   656
    set_ctrl(con_scale, C->root());
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24424
diff changeset
   657
    max_idx_expr = new MulINode(max_idx_expr, con_scale);
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   658
    register_new_node(max_idx_expr, ctrl);
9437
9981851b4b8c 7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents: 9101
diff changeset
   659
    if (TraceLoopPredicate) predString->print("* %d ", scale);
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   660
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   661
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   662
  if (offset && (!offset->is_Con() || offset->get_int() != 0)){
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24424
diff changeset
   663
    max_idx_expr = new AddINode(max_idx_expr, offset);
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   664
    register_new_node(max_idx_expr, ctrl);
46630
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 37287
diff changeset
   665
    if (TraceLoopPredicate) {
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 37287
diff changeset
   666
      if (offset->is_Con()) {
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 37287
diff changeset
   667
        predString->print("+ %d ", offset->get_int());
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 37287
diff changeset
   668
      } else {
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 37287
diff changeset
   669
        predString->print("+ offset ");
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 37287
diff changeset
   670
      }
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 37287
diff changeset
   671
    }
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   672
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   673
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24424
diff changeset
   674
  CmpUNode* cmp = new CmpUNode(max_idx_expr, range);
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   675
  register_new_node(cmp, ctrl);
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24424
diff changeset
   676
  BoolNode* bol = new BoolNode(cmp, BoolTest::lt);
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   677
  register_new_node(bol, ctrl);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   678
9437
9981851b4b8c 7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents: 9101
diff changeset
   679
  if (TraceLoopPredicate) {
9981851b4b8c 7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents: 9101
diff changeset
   680
    predString->print_cr("<u range");
24424
2658d7834c6e 8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents: 23528
diff changeset
   681
    tty->print("%s", predString->as_string());
9437
9981851b4b8c 7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents: 9101
diff changeset
   682
  }
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   683
  return bol;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   684
}
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   685
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   686
//------------------------------ loop_predication_impl--------------------------
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   687
// Insert loop predicates for null checks and range checks
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   688
bool PhaseIdealLoop::loop_predication_impl(IdealLoopTree *loop) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   689
  if (!UseLoopPredicate) return false;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   690
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   691
  if (!loop->_head->is_Loop()) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   692
    // Could be a simple region when irreducible loops are present.
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   693
    return false;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   694
  }
9446
748a37b25d10 5091921: Sign flip issues in loop optimizer
kvn
parents: 9437
diff changeset
   695
  LoopNode* head = loop->_head->as_Loop();
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   696
9446
748a37b25d10 5091921: Sign flip issues in loop optimizer
kvn
parents: 9437
diff changeset
   697
  if (head->unique_ctrl_out()->Opcode() == Op_NeverBranch) {
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   698
    // do nothing for infinite loops
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   699
    return false;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   700
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   701
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   702
  CountedLoopNode *cl = NULL;
10263
fa58671dde31 7077439: Possible reference through NULL in loopPredicate.cpp:726
kvn
parents: 10258
diff changeset
   703
  if (head->is_valid_counted_loop()) {
9446
748a37b25d10 5091921: Sign flip issues in loop optimizer
kvn
parents: 9437
diff changeset
   704
    cl = head->as_CountedLoop();
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   705
    // do nothing for iteration-splitted loops
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   706
    if (!cl->is_normal_loop()) return false;
10253
35b975b1e8f3 7070134: Hotspot crashes with sigsegv from PorterStemmer
kvn
parents: 9446
diff changeset
   707
    // Avoid RCE if Counted loop's test is '!='.
35b975b1e8f3 7070134: Hotspot crashes with sigsegv from PorterStemmer
kvn
parents: 9446
diff changeset
   708
    BoolTest::mask bt = cl->loopexit()->test_trip();
35b975b1e8f3 7070134: Hotspot crashes with sigsegv from PorterStemmer
kvn
parents: 9446
diff changeset
   709
    if (bt != BoolTest::lt && bt != BoolTest::gt)
35b975b1e8f3 7070134: Hotspot crashes with sigsegv from PorterStemmer
kvn
parents: 9446
diff changeset
   710
      cl = NULL;
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   711
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   712
9446
748a37b25d10 5091921: Sign flip issues in loop optimizer
kvn
parents: 9437
diff changeset
   713
  Node* entry = head->in(LoopNode::EntryControl);
748a37b25d10 5091921: Sign flip issues in loop optimizer
kvn
parents: 9437
diff changeset
   714
  ProjNode *predicate_proj = NULL;
748a37b25d10 5091921: Sign flip issues in loop optimizer
kvn
parents: 9437
diff changeset
   715
  // Loop limit check predicate should be near the loop.
37287
c2660335bf81 8072422: Cleanup: Remove some unused flags/code in loop optimizations
zmajo
parents: 35558
diff changeset
   716
  predicate_proj = find_predicate_insertion_point(entry, Deoptimization::Reason_loop_limit_check);
c2660335bf81 8072422: Cleanup: Remove some unused flags/code in loop optimizations
zmajo
parents: 35558
diff changeset
   717
  if (predicate_proj != NULL)
c2660335bf81 8072422: Cleanup: Remove some unused flags/code in loop optimizations
zmajo
parents: 35558
diff changeset
   718
    entry = predicate_proj->in(0)->in(0);
9446
748a37b25d10 5091921: Sign flip issues in loop optimizer
kvn
parents: 9437
diff changeset
   719
  predicate_proj = find_predicate_insertion_point(entry, Deoptimization::Reason_predicate);
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   720
  if (!predicate_proj) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   721
#ifndef PRODUCT
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   722
    if (TraceLoopPredicate) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   723
      tty->print("missing predicate:");
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   724
      loop->dump_head();
9446
748a37b25d10 5091921: Sign flip issues in loop optimizer
kvn
parents: 9437
diff changeset
   725
      head->dump(1);
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   726
    }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   727
#endif
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   728
    return false;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   729
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   730
  ConNode* zero = _igvn.intcon(0);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   731
  set_ctrl(zero, C->root());
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   732
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   733
  ResourceArea *area = Thread::current()->resource_area();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   734
  Invariance invar(area, loop);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   735
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   736
  // Create list of if-projs such that a newer proj dominates all older
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   737
  // projs in the list, and they all dominate loop->tail()
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   738
  Node_List if_proj_list(area);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   739
  Node *current_proj = loop->tail(); //start from tail
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   740
  while (current_proj != head) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   741
    if (loop == get_loop(current_proj) && // still in the loop ?
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   742
        current_proj->is_Proj()        && // is a projection  ?
34164
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 32733
diff changeset
   743
        (current_proj->in(0)->Opcode() == Op_If ||
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 32733
diff changeset
   744
         current_proj->in(0)->Opcode() == Op_RangeCheck)) { // is a if projection ?
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   745
      if_proj_list.push(current_proj);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   746
    }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   747
    current_proj = idom(current_proj);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   748
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   749
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   750
  bool hoisted = false; // true if at least one proj is promoted
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   751
  while (if_proj_list.size() > 0) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   752
    // Following are changed to nonnull when a predicate can be hoisted
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   753
    ProjNode* new_predicate_proj = NULL;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   754
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   755
    ProjNode* proj = if_proj_list.pop()->as_Proj();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   756
    IfNode*   iff  = proj->in(0)->as_If();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   757
21089
e1986ff6fe2e 8024069: replace_in_map() should operate on parent maps
roland
parents: 17383
diff changeset
   758
    if (!proj->is_uncommon_trap_if_pattern(Deoptimization::Reason_none)) {
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   759
      if (loop->is_loop_exit(iff)) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   760
        // stop processing the remaining projs in the list because the execution of them
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   761
        // depends on the condition of "iff" (iff->in(1)).
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   762
        break;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   763
      } else {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   764
        // Both arms are inside the loop. There are two cases:
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   765
        // (1) there is one backward branch. In this case, any remaining proj
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   766
        //     in the if_proj list post-dominates "iff". So, the condition of "iff"
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   767
        //     does not determine the execution the remining projs directly, and we
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   768
        //     can safely continue.
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   769
        // (2) both arms are forwarded, i.e. a diamond shape. In this case, "proj"
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   770
        //     does not dominate loop->tail(), so it can not be in the if_proj list.
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   771
        continue;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   772
      }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   773
    }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   774
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   775
    Node*     test = iff->in(1);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   776
    if (!test->is_Bool()){ //Conv2B, ...
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   777
      continue;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   778
    }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   779
    BoolNode* bol = test->as_Bool();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   780
    if (invar.is_invariant(bol)) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   781
      // Invariant test
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   782
      new_predicate_proj = create_new_if_for_predicate(predicate_proj, NULL,
34164
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 32733
diff changeset
   783
                                                       Deoptimization::Reason_predicate,
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 32733
diff changeset
   784
                                                       iff->Opcode());
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   785
      Node* ctrl = new_predicate_proj->in(0)->as_If()->in(0);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   786
      BoolNode* new_predicate_bol = invar.clone(bol, ctrl)->as_Bool();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   787
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   788
      // Negate test if necessary
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   789
      bool negated = false;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   790
      if (proj->_con != predicate_proj->_con) {
24923
9631f7d691dc 8034812: remove IDX_INIT macro hack in Node class
thartmann
parents: 24424
diff changeset
   791
        new_predicate_bol = new BoolNode(new_predicate_bol->in(1), new_predicate_bol->_test.negate());
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   792
        register_new_node(new_predicate_bol, ctrl);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   793
        negated = true;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   794
      }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   795
      IfNode* new_predicate_iff = new_predicate_proj->in(0)->as_If();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   796
      _igvn.hash_delete(new_predicate_iff);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   797
      new_predicate_iff->set_req(1, new_predicate_bol);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   798
#ifndef PRODUCT
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   799
      if (TraceLoopPredicate) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   800
        tty->print("Predicate invariant if%s: %d ", negated ? " negated" : "", new_predicate_iff->_idx);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   801
        loop->dump_head();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   802
      } else if (TraceLoopOpts) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   803
        tty->print("Predicate IC ");
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   804
        loop->dump_head();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   805
      }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   806
#endif
26166
4b49fd58bbd9 7173584: Implement arraycopy as a macro node
roland
parents: 25913
diff changeset
   807
    } else if (cl != NULL && loop->is_range_check_if(iff, this, invar)) {
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   808
      // Range check for counted loops
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   809
      const Node*    cmp    = bol->in(1)->as_Cmp();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   810
      Node*          idx    = cmp->in(1);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   811
      assert(!invar.is_invariant(idx), "index is variant");
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   812
      Node* rng = cmp->in(2);
9446
748a37b25d10 5091921: Sign flip issues in loop optimizer
kvn
parents: 9437
diff changeset
   813
      assert(rng->Opcode() == Op_LoadRange || _igvn.type(rng)->is_int() >= 0, "must be");
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   814
      assert(invar.is_invariant(rng), "range must be invariant");
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   815
      int scale    = 1;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   816
      Node* offset = zero;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   817
      bool ok = is_scaled_iv_plus_offset(idx, cl->phi(), &scale, &offset);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   818
      assert(ok, "must be index expression");
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   819
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   820
      Node* init    = cl->init_trip();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   821
      Node* limit   = cl->limit();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   822
      Node* stride  = cl->stride();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   823
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   824
      // Build if's for the upper and lower bound tests.  The
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   825
      // lower_bound test will dominate the upper bound test and all
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   826
      // cloned or created nodes will use the lower bound test as
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   827
      // their declared control.
34164
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 32733
diff changeset
   828
      ProjNode* lower_bound_proj = create_new_if_for_predicate(predicate_proj, NULL, Deoptimization::Reason_predicate, iff->Opcode());
a9e6034d7707 8137168: Replace IfNode with a new RangeCheckNode for range checks
roland
parents: 32733
diff changeset
   829
      ProjNode* upper_bound_proj = create_new_if_for_predicate(predicate_proj, NULL, Deoptimization::Reason_predicate, iff->Opcode());
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   830
      assert(upper_bound_proj->in(0)->as_If()->in(0) == lower_bound_proj, "should dominate");
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   831
      Node *ctrl = lower_bound_proj->in(0)->as_If()->in(0);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   832
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   833
      // Perform cloning to keep Invariance state correct since the
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   834
      // late schedule will place invariant things in the loop.
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   835
      rng = invar.clone(rng, ctrl);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   836
      if (offset && offset != zero) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   837
        assert(invar.is_invariant(offset), "offset must be loop invariant");
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   838
        offset = invar.clone(offset, ctrl);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   839
      }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   840
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   841
      // Test the lower bound
26166
4b49fd58bbd9 7173584: Implement arraycopy as a macro node
roland
parents: 25913
diff changeset
   842
      BoolNode*  lower_bound_bol = rc_predicate(loop, ctrl, scale, offset, init, limit, stride, rng, false);
4b49fd58bbd9 7173584: Implement arraycopy as a macro node
roland
parents: 25913
diff changeset
   843
      // Negate test if necessary
4b49fd58bbd9 7173584: Implement arraycopy as a macro node
roland
parents: 25913
diff changeset
   844
      bool negated = false;
4b49fd58bbd9 7173584: Implement arraycopy as a macro node
roland
parents: 25913
diff changeset
   845
      if (proj->_con != predicate_proj->_con) {
4b49fd58bbd9 7173584: Implement arraycopy as a macro node
roland
parents: 25913
diff changeset
   846
        lower_bound_bol = new BoolNode(lower_bound_bol->in(1), lower_bound_bol->_test.negate());
4b49fd58bbd9 7173584: Implement arraycopy as a macro node
roland
parents: 25913
diff changeset
   847
        register_new_node(lower_bound_bol, ctrl);
4b49fd58bbd9 7173584: Implement arraycopy as a macro node
roland
parents: 25913
diff changeset
   848
        negated = true;
4b49fd58bbd9 7173584: Implement arraycopy as a macro node
roland
parents: 25913
diff changeset
   849
      }
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   850
      IfNode* lower_bound_iff = lower_bound_proj->in(0)->as_If();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   851
      _igvn.hash_delete(lower_bound_iff);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   852
      lower_bound_iff->set_req(1, lower_bound_bol);
26166
4b49fd58bbd9 7173584: Implement arraycopy as a macro node
roland
parents: 25913
diff changeset
   853
      if (TraceLoopPredicate) tty->print_cr("lower bound check if: %s %d ", negated ? " negated" : "", lower_bound_iff->_idx);
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   854
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   855
      // Test the upper bound
26166
4b49fd58bbd9 7173584: Implement arraycopy as a macro node
roland
parents: 25913
diff changeset
   856
      BoolNode* upper_bound_bol = rc_predicate(loop, lower_bound_proj, scale, offset, init, limit, stride, rng, true);
4b49fd58bbd9 7173584: Implement arraycopy as a macro node
roland
parents: 25913
diff changeset
   857
      negated = false;
4b49fd58bbd9 7173584: Implement arraycopy as a macro node
roland
parents: 25913
diff changeset
   858
      if (proj->_con != predicate_proj->_con) {
4b49fd58bbd9 7173584: Implement arraycopy as a macro node
roland
parents: 25913
diff changeset
   859
        upper_bound_bol = new BoolNode(upper_bound_bol->in(1), upper_bound_bol->_test.negate());
4b49fd58bbd9 7173584: Implement arraycopy as a macro node
roland
parents: 25913
diff changeset
   860
        register_new_node(upper_bound_bol, ctrl);
4b49fd58bbd9 7173584: Implement arraycopy as a macro node
roland
parents: 25913
diff changeset
   861
        negated = true;
4b49fd58bbd9 7173584: Implement arraycopy as a macro node
roland
parents: 25913
diff changeset
   862
      }
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   863
      IfNode* upper_bound_iff = upper_bound_proj->in(0)->as_If();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   864
      _igvn.hash_delete(upper_bound_iff);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   865
      upper_bound_iff->set_req(1, upper_bound_bol);
26166
4b49fd58bbd9 7173584: Implement arraycopy as a macro node
roland
parents: 25913
diff changeset
   866
      if (TraceLoopPredicate) tty->print_cr("upper bound check if: %s %d ", negated ? " negated" : "", lower_bound_iff->_idx);
9101
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   867
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   868
      // Fall through into rest of the clean up code which will move
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   869
      // any dependent nodes onto the upper bound test.
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   870
      new_predicate_proj = upper_bound_proj;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   871
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   872
#ifndef PRODUCT
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   873
      if (TraceLoopOpts && !TraceLoopPredicate) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   874
        tty->print("Predicate RC ");
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   875
        loop->dump_head();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   876
      }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   877
#endif
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   878
    } else {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   879
      // Loop variant check (for example, range check in non-counted loop)
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   880
      // with uncommon trap.
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   881
      continue;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   882
    }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   883
    assert(new_predicate_proj != NULL, "sanity");
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   884
    // Success - attach condition (new_predicate_bol) to predicate if
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   885
    invar.map_ctrl(proj, new_predicate_proj); // so that invariance test can be appropriate
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   886
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   887
    // Eliminate the old If in the loop body
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   888
    dominated_by( new_predicate_proj, iff, proj->_con != new_predicate_proj->_con );
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   889
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   890
    hoisted = true;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   891
    C->set_major_progress();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   892
  } // end while
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   893
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   894
#ifndef PRODUCT
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   895
  // report that the loop predication has been actually performed
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   896
  // for this loop
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   897
  if (TraceLoopPredicate && hoisted) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   898
    tty->print("Loop Predication Performed:");
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   899
    loop->dump_head();
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   900
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   901
#endif
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   902
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   903
  return hoisted;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   904
}
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   905
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   906
//------------------------------loop_predication--------------------------------
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   907
// driver routine for loop predication optimization
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   908
bool IdealLoopTree::loop_predication( PhaseIdealLoop *phase) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   909
  bool hoisted = false;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   910
  // Recursively promote predicates
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   911
  if (_child) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   912
    hoisted = _child->loop_predication( phase);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   913
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   914
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   915
  // self
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   916
  if (!_irreducible && !tail()->is_top()) {
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   917
    hoisted |= phase->loop_predication_impl(this);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   918
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   919
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   920
  if (_next) { //sibling
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   921
    hoisted |= _next->loop_predication( phase);
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   922
  }
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   923
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   924
  return hoisted;
ff58f9a8e31c 7004535: Clone loop predicate during loop unswitch
kvn
parents:
diff changeset
   925
}