src/jdk.internal.vm.compiler/share/classes/org.graalvm.compiler.loop/src/org/graalvm/compiler/loop/DefaultLoopPolicies.java
author dlong
Thu, 31 Oct 2019 16:54:16 -0700
changeset 58877 aec7bf35d6f5
parent 58299 6df94ce3ab2f
permissions -rw-r--r--
8233273: 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
/*
58299
6df94ce3ab2f 8229201: Update Graal
dlong
parents: 54084
diff changeset
     2
 * Copyright (c) 2012, 2019, 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;
58877
aec7bf35d6f5 8233273: Update Graal
dlong
parents: 58299
diff changeset
    29
import static org.graalvm.compiler.core.common.GraalOptions.MinimumPeelFrequency;
43972
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;
54084
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
    36
import org.graalvm.compiler.debug.GraalError;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    37
import org.graalvm.compiler.graph.Node;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    38
import org.graalvm.compiler.graph.NodeBitMap;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    39
import org.graalvm.compiler.nodes.AbstractBeginNode;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    40
import org.graalvm.compiler.nodes.ControlSplitNode;
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
    41
import org.graalvm.compiler.nodes.InvokeNode;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    42
import org.graalvm.compiler.nodes.LoopBeginNode;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    43
import org.graalvm.compiler.nodes.MergeNode;
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
    44
import org.graalvm.compiler.nodes.StructuredGraph;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    45
import org.graalvm.compiler.nodes.VirtualState;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    46
import org.graalvm.compiler.nodes.VirtualState.VirtualClosure;
54084
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
    47
import org.graalvm.compiler.nodes.calc.CompareNode;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    48
import org.graalvm.compiler.nodes.cfg.Block;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    49
import org.graalvm.compiler.nodes.cfg.ControlFlowGraph;
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
    50
import org.graalvm.compiler.nodes.debug.ControlFlowAnchorNode;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    51
import org.graalvm.compiler.options.Option;
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
    52
import org.graalvm.compiler.options.OptionKey;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    53
import org.graalvm.compiler.options.OptionType;
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
    54
import org.graalvm.compiler.options.OptionValues;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    55
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    56
import jdk.vm.ci.meta.MetaAccessProvider;
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
public class DefaultLoopPolicies implements LoopPolicies {
47798
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
    59
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
    60
    public static class Options {
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
    61
        @Option(help = "", type = OptionType.Expert) public static final OptionKey<Integer> LoopUnswitchMaxIncrease = new OptionKey<>(500);
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
    62
        @Option(help = "", type = OptionType.Expert) public static final OptionKey<Integer> LoopUnswitchTrivial = new OptionKey<>(10);
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
    63
        @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
    64
54084
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
    65
        @Option(help = "", type = OptionType.Expert) public static final OptionKey<Integer> FullUnrollMaxNodes = new OptionKey<>(400);
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
    66
        @Option(help = "", type = OptionType.Expert) public static final OptionKey<Integer> FullUnrollConstantCompareBoost = new OptionKey<>(15);
47798
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
    67
        @Option(help = "", type = OptionType.Expert) public static final OptionKey<Integer> FullUnrollMaxIterations = new OptionKey<>(600);
54084
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
    68
        @Option(help = "", type = OptionType.Expert) public static final OptionKey<Integer> ExactFullUnrollMaxNodes = new OptionKey<>(800);
47798
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
    69
        @Option(help = "", type = OptionType.Expert) public static final OptionKey<Integer> ExactPartialUnrollMaxNodes = new OptionKey<>(200);
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
    70
47798
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
    71
        @Option(help = "", type = OptionType.Expert) public static final OptionKey<Integer> UnrollMaxIterations = new OptionKey<>(16);
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
    72
    }
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    73
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    74
    @Override
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    75
    public boolean shouldPeel(LoopEx loop, ControlFlowGraph cfg, MetaAccessProvider metaAccess) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    76
        LoopBeginNode loopBegin = loop.loopBegin();
52578
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
    77
        double entryProbability = cfg.blockFor(loopBegin.forwardEnd()).getRelativeFrequency();
58877
aec7bf35d6f5 8233273: Update Graal
dlong
parents: 58299
diff changeset
    78
        StructuredGraph graph = cfg.graph;
aec7bf35d6f5 8233273: Update Graal
dlong
parents: 58299
diff changeset
    79
        OptionValues options = graph.getOptions();
aec7bf35d6f5 8233273: Update Graal
dlong
parents: 58299
diff changeset
    80
aec7bf35d6f5 8233273: Update Graal
dlong
parents: 58299
diff changeset
    81
        if (entryProbability < MinimumPeelFrequency.getValue(options)) {
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    82
            return false;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    83
        }
58877
aec7bf35d6f5 8233273: Update Graal
dlong
parents: 58299
diff changeset
    84
aec7bf35d6f5 8233273: Update Graal
dlong
parents: 58299
diff changeset
    85
        if (loop.parent() != null) {
aec7bf35d6f5 8233273: Update Graal
dlong
parents: 58299
diff changeset
    86
            if (loop.size() > loop.parent().size() >> 1) {
aec7bf35d6f5 8233273: Update Graal
dlong
parents: 58299
diff changeset
    87
                // This loops make up more than half of the parent loop in terms of number of nodes.
aec7bf35d6f5 8233273: Update Graal
dlong
parents: 58299
diff changeset
    88
                // There is a risk that this loop unproportionally increases parent loop body size.
aec7bf35d6f5 8233273: Update Graal
dlong
parents: 58299
diff changeset
    89
                return false;
aec7bf35d6f5 8233273: Update Graal
dlong
parents: 58299
diff changeset
    90
            }
aec7bf35d6f5 8233273: Update Graal
dlong
parents: 58299
diff changeset
    91
        }
aec7bf35d6f5 8233273: Update Graal
dlong
parents: 58299
diff changeset
    92
aec7bf35d6f5 8233273: Update Graal
dlong
parents: 58299
diff changeset
    93
        if (loop.loop().getChildren().size() > 0) {
aec7bf35d6f5 8233273: Update Graal
dlong
parents: 58299
diff changeset
    94
            // This loop has child loops. Loop peeling could explode graph size.
aec7bf35d6f5 8233273: Update Graal
dlong
parents: 58299
diff changeset
    95
            return false;
aec7bf35d6f5 8233273: Update Graal
dlong
parents: 58299
diff changeset
    96
        }
aec7bf35d6f5 8233273: Update Graal
dlong
parents: 58299
diff changeset
    97
aec7bf35d6f5 8233273: Update Graal
dlong
parents: 58299
diff changeset
    98
        if (loop.size() + graph.getNodeCount() > MaximumDesiredSize.getValue(options)) {
aec7bf35d6f5 8233273: Update Graal
dlong
parents: 58299
diff changeset
    99
            // We are out of budget for peeling.
aec7bf35d6f5 8233273: Update Graal
dlong
parents: 58299
diff changeset
   100
            return false;
aec7bf35d6f5 8233273: Update Graal
dlong
parents: 58299
diff changeset
   101
        }
aec7bf35d6f5 8233273: Update Graal
dlong
parents: 58299
diff changeset
   102
aec7bf35d6f5 8233273: Update Graal
dlong
parents: 58299
diff changeset
   103
        return true;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   104
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   105
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   106
    @Override
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   107
    public boolean shouldFullUnroll(LoopEx loop) {
54084
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   108
        if (!loop.isCounted() || !loop.counted().isConstantMaxTripCount() || !loop.counted().counterNeverOverflows()) {
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   109
            return false;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   110
        }
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   111
        OptionValues options = loop.entryPoint().getOptions();
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   112
        CountedLoopInfo counted = loop.counted();
49451
e06f9607f370 8198969: Update Graal
iveresov
parents: 47798
diff changeset
   113
        UnsignedLong maxTrips = counted.constantMaxTripCount();
e06f9607f370 8198969: Update Graal
iveresov
parents: 47798
diff changeset
   114
        if (maxTrips.equals(0)) {
e06f9607f370 8198969: Update Graal
iveresov
parents: 47798
diff changeset
   115
            return loop.canDuplicateLoop();
e06f9607f370 8198969: Update Graal
iveresov
parents: 47798
diff changeset
   116
        }
54084
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   117
        if (maxTrips.isGreaterThan(Options.FullUnrollMaxIterations.getValue(options))) {
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   118
            return false;
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   119
        }
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   120
        int globalMax = MaximumDesiredSize.getValue(options) - loop.loopBegin().graph().getNodeCount();
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   121
        if (globalMax <= 0) {
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   122
            return false;
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   123
        }
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   124
        int maxNodes = counted.isExactTripCount() ? Options.ExactFullUnrollMaxNodes.getValue(options) : Options.FullUnrollMaxNodes.getValue(options);
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   125
        for (Node usage : counted.getCounter().valueNode().usages()) {
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   126
            if (usage instanceof CompareNode) {
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   127
                CompareNode compare = (CompareNode) usage;
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   128
                if (compare.getY().isConstant()) {
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   129
                    maxNodes += Options.FullUnrollConstantCompareBoost.getValue(options);
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   130
                }
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   131
            }
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   132
        }
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   133
        maxNodes = Math.min(maxNodes, globalMax);
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   134
        int size = loop.inside().nodes().count();
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   135
        size -= 2; // remove the counted if and its non-exit begin
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   136
        size -= loop.loopBegin().loopEnds().count();
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   137
        GraalError.guarantee(size >= 0, "Wrong size");
49451
e06f9607f370 8198969: Update Graal
iveresov
parents: 47798
diff changeset
   138
        /* @formatter:off
e06f9607f370 8198969: Update Graal
iveresov
parents: 47798
diff changeset
   139
         * The check below should not throw ArithmeticException because:
e06f9607f370 8198969: Update Graal
iveresov
parents: 47798
diff changeset
   140
         * maxTrips is guaranteed to be >= 1 by the check above
e06f9607f370 8198969: Update Graal
iveresov
parents: 47798
diff changeset
   141
         * - maxTrips * size can not overfow because:
e06f9607f370 8198969: Update Graal
iveresov
parents: 47798
diff changeset
   142
         *   - maxTrips <= FullUnrollMaxIterations <= Integer.MAX_VALUE
e06f9607f370 8198969: Update Graal
iveresov
parents: 47798
diff changeset
   143
         *   - 1 <= size <= Integer.MAX_VALUE
e06f9607f370 8198969: Update Graal
iveresov
parents: 47798
diff changeset
   144
         * @formatter:on
e06f9607f370 8198969: Update Graal
iveresov
parents: 47798
diff changeset
   145
         */
54084
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   146
        if (maxTrips.minus(1).times(size).isLessOrEqualTo(maxNodes)) {
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   147
            // check whether we're allowed to unroll this loop
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   148
            return loop.canDuplicateLoop();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   149
        } else {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   150
            return false;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   151
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   152
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   153
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   154
    @Override
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   155
    public boolean shouldPartiallyUnroll(LoopEx loop) {
46963
089674d9949b 8186681: Update Graal
iveresov
parents: 46640
diff changeset
   156
        LoopBeginNode loopBegin = loop.loopBegin();
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   157
        if (!loop.isCounted()) {
46963
089674d9949b 8186681: Update Graal
iveresov
parents: 46640
diff changeset
   158
            loopBegin.getDebug().log(DebugContext.VERBOSE_LEVEL, "shouldPartiallyUnroll %s isn't counted", loopBegin);
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
        OptionValues options = loop.entryPoint().getOptions();
47798
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
   162
        int maxNodes = Options.ExactPartialUnrollMaxNodes.getValue(options);
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   163
        maxNodes = Math.min(maxNodes, Math.max(0, MaximumDesiredSize.getValue(options) - loop.loopBegin().graph().getNodeCount()));
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   164
        int size = Math.max(1, loop.size() - 1 - loop.loopBegin().phis().count());
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   165
        int unrollFactor = loopBegin.getUnrollFactor();
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   166
        if (unrollFactor == 1) {
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   167
            double loopFrequency = loopBegin.loopFrequency();
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   168
            if (loopBegin.isSimpleLoop() && loopFrequency < 5.0) {
46963
089674d9949b 8186681: Update Graal
iveresov
parents: 46640
diff changeset
   169
                loopBegin.getDebug().log(DebugContext.VERBOSE_LEVEL, "shouldPartiallyUnroll %s frequency too low %s ", loopBegin, loopFrequency);
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   170
                return false;
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   171
            }
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   172
            loopBegin.setLoopOrigFrequency(loopFrequency);
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   173
        }
47798
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
   174
        int maxUnroll = Options.UnrollMaxIterations.getValue(options);
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   175
        // Now correct size for the next unroll. UnrollMaxIterations == 1 means perform the
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   176
        // pre/main/post transformation but don't actually unroll the main loop.
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   177
        size += size;
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   178
        if (maxUnroll == 1 && loopBegin.isSimpleLoop() || size <= maxNodes && unrollFactor < maxUnroll) {
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   179
            // Will the next unroll fit?
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   180
            if ((int) loopBegin.loopOrigFrequency() < (unrollFactor * 2)) {
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   181
                return false;
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   182
            }
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   183
            // Check whether we're allowed to unroll this loop
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   184
            for (Node node : loop.inside().nodes()) {
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   185
                if (node instanceof ControlFlowAnchorNode) {
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   186
                    return false;
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   187
                }
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   188
                if (node instanceof InvokeNode) {
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   189
                    return false;
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   190
                }
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   191
            }
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   192
            return true;
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   193
        } else {
46963
089674d9949b 8186681: Update Graal
iveresov
parents: 46640
diff changeset
   194
            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
   195
            return false;
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   196
        }
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   197
    }
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   198
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   199
    @Override
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   200
    public boolean shouldTryUnswitch(LoopEx loop) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   201
        LoopBeginNode loopBegin = loop.loopBegin();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   202
        double loopFrequency = loopBegin.loopFrequency();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   203
        if (loopFrequency <= 1.0) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   204
            return false;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   205
        }
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   206
        OptionValues options = loop.entryPoint().getOptions();
58877
aec7bf35d6f5 8233273: Update Graal
dlong
parents: 58299
diff changeset
   207
        return loopBegin.unswitches() < LoopMaxUnswitch.getValue(options);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   208
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   209
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   210
    private static final class CountingClosure implements VirtualClosure {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   211
        int count;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   212
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   213
        @Override
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   214
        public void apply(VirtualState node) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   215
            count++;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   216
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   217
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   218
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   219
    private static class IsolatedInitialization {
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   220
        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
   221
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   222
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   223
    @Override
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   224
    public boolean shouldUnswitch(LoopEx loop, List<ControlSplitNode> controlSplits) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   225
        int phis = 0;
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   226
        StructuredGraph graph = loop.loopBegin().graph();
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   227
        DebugContext debug = graph.getDebug();
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   228
        NodeBitMap branchNodes = graph.createNodeBitMap();
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   229
        for (ControlSplitNode controlSplit : controlSplits) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   230
            for (Node successor : controlSplit.successors()) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   231
                AbstractBeginNode branch = (AbstractBeginNode) successor;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   232
                // 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
   233
                loop.nodesInLoopBranch(branchNodes, branch);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   234
            }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   235
            Block postDomBlock = loop.loopsData().getCFG().blockFor(controlSplit).getPostdominator();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   236
            if (postDomBlock != null) {
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   237
                IsolatedInitialization.UNSWITCH_SPLIT_WITH_PHIS.increment(debug);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   238
                phis += ((MergeNode) postDomBlock.getBeginNode()).phis().count();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   239
            }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   240
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   241
        int inBranchTotal = branchNodes.count();
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
        CountingClosure stateNodesCount = new CountingClosure();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   244
        double loopFrequency = loop.loopBegin().loopFrequency();
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   245
        OptionValues options = loop.loopBegin().getOptions();
47798
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
   246
        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
   247
47798
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
   248
        maxDiff = Math.min(maxDiff, Options.LoopUnswitchMaxIncrease.getValue(options));
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   249
        int remainingGraphSpace = MaximumDesiredSize.getValue(options) - graph.getNodeCount();
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   250
        maxDiff = Math.min(maxDiff, remainingGraphSpace);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   251
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   252
        loop.loopBegin().stateAfter().applyToVirtual(stateNodesCount);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   253
        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
   254
        int actualDiff = (loopTotal - inBranchTotal);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   255
        ControlSplitNode firstSplit = controlSplits.get(0);
58877
aec7bf35d6f5 8233273: Update Graal
dlong
parents: 58299
diff changeset
   256
aec7bf35d6f5 8233273: Update Graal
dlong
parents: 58299
diff changeset
   257
        int copies = firstSplit.successors().count() - 1;
aec7bf35d6f5 8233273: Update Graal
dlong
parents: 58299
diff changeset
   258
        actualDiff = actualDiff * copies;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   259
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46344
diff changeset
   260
        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
   261
                        loopFrequency, phis, actualDiff <= maxDiff);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   262
        if (actualDiff <= maxDiff) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   263
            // check whether we're allowed to unswitch this loop
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   264
            return loop.canDuplicateLoop();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   265
        } else {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   266
            return false;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   267
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   268
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   269
}