src/jdk.internal.vm.compiler/share/classes/org.graalvm.compiler.loop/src/org/graalvm/compiler/loop/DefaultLoopPolicies.java
author dlong
Sat, 08 Dec 2018 00:56:10 -0800
changeset 52910 583fd71c47d6
parent 52578 7dd81e82d083
child 54084 84f10bbf993f
permissions -rw-r--r--
8214023: Update Graal Reviewed-by: kvn
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
     1
/*
52910
583fd71c47d6 8214023: Update Graal
dlong
parents: 52578
diff changeset
     2
 * Copyright (c) 2012, 2018, Oracle and/or its affiliates. All rights reserved.
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
     4
 *
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
     7
 * published by the Free Software Foundation.
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
     8
 *
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    13
 * accompanied this code).
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    14
 *
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    18
 *
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    21
 * questions.
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    22
 */
50858
2d3e99a72541 8205824: Update Graal
never
parents: 49451
diff changeset
    23
2d3e99a72541 8205824: Update Graal
never
parents: 49451
diff changeset
    24
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    25
package org.graalvm.compiler.loop;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    26
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    27
import static org.graalvm.compiler.core.common.GraalOptions.LoopMaxUnswitch;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    28
import static org.graalvm.compiler.core.common.GraalOptions.MaximumDesiredSize;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    29
import static org.graalvm.compiler.core.common.GraalOptions.MinimumPeelProbability;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    30
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    31
import java.util.List;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    32
49451
e06f9607f370 8198969: Update Graal
iveresov
parents: 47798
diff changeset
    33
import org.graalvm.compiler.core.common.util.UnsignedLong;
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
    34
import org.graalvm.compiler.debug.CounterKey;
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
    35
import org.graalvm.compiler.debug.DebugContext;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    36
import org.graalvm.compiler.graph.Node;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    37
import org.graalvm.compiler.graph.NodeBitMap;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    38
import org.graalvm.compiler.nodes.AbstractBeginNode;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    39
import org.graalvm.compiler.nodes.ControlSplitNode;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    40
import org.graalvm.compiler.nodes.DeoptimizeNode;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    41
import org.graalvm.compiler.nodes.FixedNode;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    42
import org.graalvm.compiler.nodes.FixedWithNextNode;
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
    43
import org.graalvm.compiler.nodes.InvokeNode;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    44
import org.graalvm.compiler.nodes.LoopBeginNode;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    45
import org.graalvm.compiler.nodes.MergeNode;
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
    46
import org.graalvm.compiler.nodes.StructuredGraph;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    47
import org.graalvm.compiler.nodes.VirtualState;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    48
import org.graalvm.compiler.nodes.VirtualState.VirtualClosure;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    49
import org.graalvm.compiler.nodes.cfg.Block;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    50
import org.graalvm.compiler.nodes.cfg.ControlFlowGraph;
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
    51
import org.graalvm.compiler.nodes.debug.ControlFlowAnchorNode;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    52
import org.graalvm.compiler.nodes.java.TypeSwitchNode;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    53
import org.graalvm.compiler.options.Option;
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
    54
import org.graalvm.compiler.options.OptionKey;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    55
import org.graalvm.compiler.options.OptionType;
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
    56
import org.graalvm.compiler.options.OptionValues;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    57
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    58
import jdk.vm.ci.meta.MetaAccessProvider;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    59
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    60
public class DefaultLoopPolicies implements LoopPolicies {
47798
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
    61
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
    62
    public static class Options {
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
    63
        @Option(help = "", type = OptionType.Expert) public static final OptionKey<Integer> LoopUnswitchMaxIncrease = new OptionKey<>(500);
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
    64
        @Option(help = "", type = OptionType.Expert) public static final OptionKey<Integer> LoopUnswitchTrivial = new OptionKey<>(10);
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
    65
        @Option(help = "", type = OptionType.Expert) public static final OptionKey<Double> LoopUnswitchFrequencyBoost = new OptionKey<>(10.0);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    66
47798
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
    67
        @Option(help = "", type = OptionType.Expert) public static final OptionKey<Integer> FullUnrollMaxNodes = new OptionKey<>(300);
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
    68
        @Option(help = "", type = OptionType.Expert) public static final OptionKey<Integer> FullUnrollMaxIterations = new OptionKey<>(600);
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
    69
        @Option(help = "", type = OptionType.Expert) public static final OptionKey<Integer> ExactFullUnrollMaxNodes = new OptionKey<>(1200);
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
    70
        @Option(help = "", type = OptionType.Expert) public static final OptionKey<Integer> ExactPartialUnrollMaxNodes = new OptionKey<>(200);
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
    71
47798
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
    72
        @Option(help = "", type = OptionType.Expert) public static final OptionKey<Integer> UnrollMaxIterations = new OptionKey<>(16);
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
    73
    }
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    74
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    75
    @Override
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    76
    public boolean shouldPeel(LoopEx loop, ControlFlowGraph cfg, MetaAccessProvider metaAccess) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    77
        LoopBeginNode loopBegin = loop.loopBegin();
52578
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
    78
        double entryProbability = cfg.blockFor(loopBegin.forwardEnd()).getRelativeFrequency();
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
    79
        OptionValues options = cfg.graph.getOptions();
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
    80
        if (entryProbability > MinimumPeelProbability.getValue(options) && loop.size() + loopBegin.graph().getNodeCount() < MaximumDesiredSize.getValue(options)) {
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    81
            // check whether we're allowed to peel this loop
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    82
            return loop.canDuplicateLoop();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    83
        } else {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    84
            return false;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    85
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    86
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    87
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    88
    @Override
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    89
    public boolean shouldFullUnroll(LoopEx loop) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    90
        if (!loop.isCounted() || !loop.counted().isConstantMaxTripCount()) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    91
            return false;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    92
        }
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
    93
        OptionValues options = loop.entryPoint().getOptions();
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    94
        CountedLoopInfo counted = loop.counted();
49451
e06f9607f370 8198969: Update Graal
iveresov
parents: 47798
diff changeset
    95
        UnsignedLong maxTrips = counted.constantMaxTripCount();
e06f9607f370 8198969: Update Graal
iveresov
parents: 47798
diff changeset
    96
        if (maxTrips.equals(0)) {
e06f9607f370 8198969: Update Graal
iveresov
parents: 47798
diff changeset
    97
            return loop.canDuplicateLoop();
e06f9607f370 8198969: Update Graal
iveresov
parents: 47798
diff changeset
    98
        }
47798
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
    99
        int maxNodes = (counted.isExactTripCount() && counted.isConstantExactTripCount()) ? Options.ExactFullUnrollMaxNodes.getValue(options) : Options.FullUnrollMaxNodes.getValue(options);
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   100
        maxNodes = Math.min(maxNodes, Math.max(0, MaximumDesiredSize.getValue(options) - loop.loopBegin().graph().getNodeCount()));
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   101
        int size = Math.max(1, loop.size() - 1 - loop.loopBegin().phis().count());
49451
e06f9607f370 8198969: Update Graal
iveresov
parents: 47798
diff changeset
   102
        /* @formatter:off
e06f9607f370 8198969: Update Graal
iveresov
parents: 47798
diff changeset
   103
         * The check below should not throw ArithmeticException because:
e06f9607f370 8198969: Update Graal
iveresov
parents: 47798
diff changeset
   104
         * maxTrips is guaranteed to be >= 1 by the check above
e06f9607f370 8198969: Update Graal
iveresov
parents: 47798
diff changeset
   105
         * - maxTrips * size can not overfow because:
e06f9607f370 8198969: Update Graal
iveresov
parents: 47798
diff changeset
   106
         *   - maxTrips <= FullUnrollMaxIterations <= Integer.MAX_VALUE
e06f9607f370 8198969: Update Graal
iveresov
parents: 47798
diff changeset
   107
         *   - 1 <= size <= Integer.MAX_VALUE
e06f9607f370 8198969: Update Graal
iveresov
parents: 47798
diff changeset
   108
         * @formatter:on
e06f9607f370 8198969: Update Graal
iveresov
parents: 47798
diff changeset
   109
         */
e06f9607f370 8198969: Update Graal
iveresov
parents: 47798
diff changeset
   110
        if (maxTrips.isLessOrEqualTo(Options.FullUnrollMaxIterations.getValue(options)) && maxTrips.minus(1).times(size).isLessOrEqualTo(maxNodes)) {
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   111
            // check whether we're allowed to unroll this loop
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   112
            return loop.canDuplicateLoop();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   113
        } else {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   114
            return false;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   115
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   116
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   117
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   118
    @Override
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   119
    public boolean shouldPartiallyUnroll(LoopEx loop) {
46963
089674d9949b 8186681: Update Graal
iveresov
parents: 46640
diff changeset
   120
        LoopBeginNode loopBegin = loop.loopBegin();
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   121
        if (!loop.isCounted()) {
46963
089674d9949b 8186681: Update Graal
iveresov
parents: 46640
diff changeset
   122
            loopBegin.getDebug().log(DebugContext.VERBOSE_LEVEL, "shouldPartiallyUnroll %s isn't counted", loopBegin);
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   123
            return false;
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   124
        }
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   125
        OptionValues options = loop.entryPoint().getOptions();
47798
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
   126
        int maxNodes = Options.ExactPartialUnrollMaxNodes.getValue(options);
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   127
        maxNodes = Math.min(maxNodes, Math.max(0, MaximumDesiredSize.getValue(options) - loop.loopBegin().graph().getNodeCount()));
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   128
        int size = Math.max(1, loop.size() - 1 - loop.loopBegin().phis().count());
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   129
        int unrollFactor = loopBegin.getUnrollFactor();
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   130
        if (unrollFactor == 1) {
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   131
            double loopFrequency = loopBegin.loopFrequency();
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   132
            if (loopBegin.isSimpleLoop() && loopFrequency < 5.0) {
46963
089674d9949b 8186681: Update Graal
iveresov
parents: 46640
diff changeset
   133
                loopBegin.getDebug().log(DebugContext.VERBOSE_LEVEL, "shouldPartiallyUnroll %s frequency too low %s ", loopBegin, loopFrequency);
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   134
                return false;
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   135
            }
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   136
            loopBegin.setLoopOrigFrequency(loopFrequency);
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   137
        }
47798
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
   138
        int maxUnroll = Options.UnrollMaxIterations.getValue(options);
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   139
        // Now correct size for the next unroll. UnrollMaxIterations == 1 means perform the
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   140
        // pre/main/post transformation but don't actually unroll the main loop.
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   141
        size += size;
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   142
        if (maxUnroll == 1 && loopBegin.isSimpleLoop() || size <= maxNodes && unrollFactor < maxUnroll) {
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   143
            // Will the next unroll fit?
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   144
            if ((int) loopBegin.loopOrigFrequency() < (unrollFactor * 2)) {
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   145
                return false;
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   146
            }
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   147
            // Check whether we're allowed to unroll this loop
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   148
            for (Node node : loop.inside().nodes()) {
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   149
                if (node instanceof ControlFlowAnchorNode) {
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   150
                    return false;
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   151
                }
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   152
                if (node instanceof InvokeNode) {
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   153
                    return false;
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   154
                }
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   155
            }
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   156
            return true;
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   157
        } else {
46963
089674d9949b 8186681: Update Graal
iveresov
parents: 46640
diff changeset
   158
            loopBegin.getDebug().log(DebugContext.VERBOSE_LEVEL, "shouldPartiallyUnroll %s unrolled loop is too large %s ", loopBegin, size);
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   159
            return false;
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   160
        }
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   161
    }
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   162
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   163
    @Override
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   164
    public boolean shouldTryUnswitch(LoopEx loop) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   165
        LoopBeginNode loopBegin = loop.loopBegin();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   166
        double loopFrequency = loopBegin.loopFrequency();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   167
        if (loopFrequency <= 1.0) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   168
            return false;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   169
        }
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   170
        OptionValues options = loop.entryPoint().getOptions();
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   171
        return loopBegin.unswitches() <= LoopMaxUnswitch.getValue(options);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   172
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   173
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   174
    private static final class CountingClosure implements VirtualClosure {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   175
        int count;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   176
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   177
        @Override
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   178
        public void apply(VirtualState node) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   179
            count++;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   180
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   181
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   182
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   183
    private static class IsolatedInitialization {
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   184
        static final CounterKey UNSWITCH_SPLIT_WITH_PHIS = DebugContext.counter("UnswitchSplitWithPhis");
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   185
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   186
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   187
    @Override
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   188
    public boolean shouldUnswitch(LoopEx loop, List<ControlSplitNode> controlSplits) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   189
        int phis = 0;
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   190
        StructuredGraph graph = loop.loopBegin().graph();
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   191
        DebugContext debug = graph.getDebug();
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   192
        NodeBitMap branchNodes = graph.createNodeBitMap();
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   193
        for (ControlSplitNode controlSplit : controlSplits) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   194
            for (Node successor : controlSplit.successors()) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   195
                AbstractBeginNode branch = (AbstractBeginNode) successor;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   196
                // this may count twice because of fall-through in switches
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   197
                loop.nodesInLoopBranch(branchNodes, branch);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   198
            }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   199
            Block postDomBlock = loop.loopsData().getCFG().blockFor(controlSplit).getPostdominator();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   200
            if (postDomBlock != null) {
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   201
                IsolatedInitialization.UNSWITCH_SPLIT_WITH_PHIS.increment(debug);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   202
                phis += ((MergeNode) postDomBlock.getBeginNode()).phis().count();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   203
            }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   204
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   205
        int inBranchTotal = branchNodes.count();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   206
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   207
        CountingClosure stateNodesCount = new CountingClosure();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   208
        double loopFrequency = loop.loopBegin().loopFrequency();
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   209
        OptionValues options = loop.loopBegin().getOptions();
47798
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
   210
        int maxDiff = Options.LoopUnswitchTrivial.getValue(options) + (int) (Options.LoopUnswitchFrequencyBoost.getValue(options) * (loopFrequency - 1.0 + phis));
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   211
47798
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
   212
        maxDiff = Math.min(maxDiff, Options.LoopUnswitchMaxIncrease.getValue(options));
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   213
        int remainingGraphSpace = MaximumDesiredSize.getValue(options) - graph.getNodeCount();
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   214
        maxDiff = Math.min(maxDiff, remainingGraphSpace);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   215
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   216
        loop.loopBegin().stateAfter().applyToVirtual(stateNodesCount);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   217
        int loopTotal = loop.size() - loop.loopBegin().phis().count() - stateNodesCount.count - 1;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   218
        int actualDiff = (loopTotal - inBranchTotal);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   219
        ControlSplitNode firstSplit = controlSplits.get(0);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   220
        if (firstSplit instanceof TypeSwitchNode) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   221
            int copies = firstSplit.successors().count() - 1;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   222
            for (Node succ : firstSplit.successors()) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   223
                FixedNode current = (FixedNode) succ;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   224
                while (current instanceof FixedWithNextNode) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   225
                    current = ((FixedWithNextNode) current).next();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   226
                }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   227
                if (current instanceof DeoptimizeNode) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   228
                    copies--;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   229
                }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   230
            }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   231
            actualDiff = actualDiff * copies;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   232
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   233
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   234
        debug.log("shouldUnswitch(%s, %s) : delta=%d (%.2f%% inside of branches), max=%d, f=%.2f, phis=%d -> %b", loop, controlSplits, actualDiff, (double) (inBranchTotal) / loopTotal * 100, maxDiff,
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   235
                        loopFrequency, phis, actualDiff <= maxDiff);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   236
        if (actualDiff <= maxDiff) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   237
            // check whether we're allowed to unswitch this loop
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   238
            return loop.canDuplicateLoop();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   239
        } else {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   240
            return false;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   241
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   242
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   243
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   244
}