src/jdk.internal.vm.compiler/share/classes/org.graalvm.compiler.phases.common/src/org/graalvm/compiler/phases/common/ExpandLogicPhase.java
author dlong
Tue, 24 Sep 2019 12:47:15 -0400
changeset 58299 6df94ce3ab2f
parent 52910 583fd71c47d6
permissions -rw-r--r--
8229201: 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: 50858
diff changeset
     2
 * Copyright (c) 2013, 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: 50330
diff changeset
    23
2d3e99a72541 8205824: Update Graal
never
parents: 50330
diff changeset
    24
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    25
package org.graalvm.compiler.phases.common;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    26
46459
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    27
import org.graalvm.compiler.core.common.type.Stamp;
49873
26ebfe8ce852 8199755: Update Graal
dlong
parents: 48190
diff changeset
    28
import org.graalvm.compiler.debug.DebugCloseable;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    29
import org.graalvm.compiler.debug.GraalError;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    30
import org.graalvm.compiler.graph.Graph;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    31
import org.graalvm.compiler.graph.Node;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    32
import org.graalvm.compiler.nodes.AbstractBeginNode;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    33
import org.graalvm.compiler.nodes.AbstractMergeNode;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    34
import org.graalvm.compiler.nodes.BeginNode;
46459
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    35
import org.graalvm.compiler.nodes.ConstantNode;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    36
import org.graalvm.compiler.nodes.EndNode;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    37
import org.graalvm.compiler.nodes.IfNode;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    38
import org.graalvm.compiler.nodes.LogicNode;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    39
import org.graalvm.compiler.nodes.MergeNode;
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47216
diff changeset
    40
import org.graalvm.compiler.nodes.NodeView;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    41
import org.graalvm.compiler.nodes.ShortCircuitOrNode;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    42
import org.graalvm.compiler.nodes.StructuredGraph;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    43
import org.graalvm.compiler.nodes.ValueNode;
58299
6df94ce3ab2f 8229201: Update Graal
dlong
parents: 52910
diff changeset
    44
import org.graalvm.compiler.nodes.calc.AbstractNormalizeCompareNode;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    45
import org.graalvm.compiler.nodes.calc.ConditionalNode;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    46
import org.graalvm.compiler.phases.Phase;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    47
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    48
public class ExpandLogicPhase extends Phase {
46807
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
    49
    private static final double EPSILON = 1E-6;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    50
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    51
    @Override
49873
26ebfe8ce852 8199755: Update Graal
dlong
parents: 48190
diff changeset
    52
    @SuppressWarnings("try")
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    53
    protected void run(StructuredGraph graph) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    54
        for (ShortCircuitOrNode logic : graph.getNodes(ShortCircuitOrNode.TYPE)) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    55
            processBinary(logic);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    56
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    57
        assert graph.getNodes(ShortCircuitOrNode.TYPE).isEmpty();
46459
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    58
58299
6df94ce3ab2f 8229201: Update Graal
dlong
parents: 52910
diff changeset
    59
        for (AbstractNormalizeCompareNode logic : graph.getNodes(AbstractNormalizeCompareNode.TYPE)) {
49873
26ebfe8ce852 8199755: Update Graal
dlong
parents: 48190
diff changeset
    60
            try (DebugCloseable context = logic.withNodeSourcePosition()) {
26ebfe8ce852 8199755: Update Graal
dlong
parents: 48190
diff changeset
    61
                processNormalizeCompareNode(logic);
26ebfe8ce852 8199755: Update Graal
dlong
parents: 48190
diff changeset
    62
            }
46459
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    63
        }
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    64
        graph.setAfterExpandLogic();
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    65
    }
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    66
58299
6df94ce3ab2f 8229201: Update Graal
dlong
parents: 52910
diff changeset
    67
    private static void processNormalizeCompareNode(AbstractNormalizeCompareNode normalize) {
46459
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    68
        StructuredGraph graph = normalize.graph();
58299
6df94ce3ab2f 8229201: Update Graal
dlong
parents: 52910
diff changeset
    69
        LogicNode equalComp = graph.addOrUniqueWithInputs(normalize.createEqualComparison());
6df94ce3ab2f 8229201: Update Graal
dlong
parents: 52910
diff changeset
    70
        LogicNode lessComp = graph.addOrUniqueWithInputs(normalize.createLowerComparison());
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47216
diff changeset
    71
        Stamp stamp = normalize.stamp(NodeView.DEFAULT);
58299
6df94ce3ab2f 8229201: Update Graal
dlong
parents: 52910
diff changeset
    72
        ConditionalNode equalValue = graph.unique(new ConditionalNode(equalComp, ConstantNode.forIntegerStamp(stamp, 0, graph), ConstantNode.forIntegerStamp(stamp, 1, graph)));
46459
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    73
        ConditionalNode value = graph.unique(new ConditionalNode(lessComp, ConstantNode.forIntegerStamp(stamp, -1, graph), equalValue));
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    74
        normalize.replaceAtUsagesAndDelete(value);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    75
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    76
49873
26ebfe8ce852 8199755: Update Graal
dlong
parents: 48190
diff changeset
    77
    @SuppressWarnings("try")
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    78
    private static void processBinary(ShortCircuitOrNode binary) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    79
        while (binary.usages().isNotEmpty()) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    80
            Node usage = binary.usages().first();
49873
26ebfe8ce852 8199755: Update Graal
dlong
parents: 48190
diff changeset
    81
            try (DebugCloseable nsp = usage.withNodeSourcePosition()) {
26ebfe8ce852 8199755: Update Graal
dlong
parents: 48190
diff changeset
    82
                if (usage instanceof ShortCircuitOrNode) {
26ebfe8ce852 8199755: Update Graal
dlong
parents: 48190
diff changeset
    83
                    processBinary((ShortCircuitOrNode) usage);
26ebfe8ce852 8199755: Update Graal
dlong
parents: 48190
diff changeset
    84
                } else if (usage instanceof IfNode) {
26ebfe8ce852 8199755: Update Graal
dlong
parents: 48190
diff changeset
    85
                    processIf(binary.getX(), binary.isXNegated(), binary.getY(), binary.isYNegated(), (IfNode) usage, binary.getShortCircuitProbability());
26ebfe8ce852 8199755: Update Graal
dlong
parents: 48190
diff changeset
    86
                } else if (usage instanceof ConditionalNode) {
26ebfe8ce852 8199755: Update Graal
dlong
parents: 48190
diff changeset
    87
                    processConditional(binary.getX(), binary.isXNegated(), binary.getY(), binary.isYNegated(), (ConditionalNode) usage);
26ebfe8ce852 8199755: Update Graal
dlong
parents: 48190
diff changeset
    88
                } else {
26ebfe8ce852 8199755: Update Graal
dlong
parents: 48190
diff changeset
    89
                    throw GraalError.shouldNotReachHere();
26ebfe8ce852 8199755: Update Graal
dlong
parents: 48190
diff changeset
    90
                }
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    91
            }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    92
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    93
        binary.safeDelete();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    94
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    95
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    96
    private static void processIf(LogicNode x, boolean xNegated, LogicNode y, boolean yNegated, IfNode ifNode, double shortCircuitProbability) {
50330
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
    97
        /*
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
    98
         * this method splits an IfNode, which has a ShortCircuitOrNode as its condition, into two
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
    99
         * separate IfNodes: if(X) and if(Y)
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   100
         *
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   101
         * for computing the probabilities P(X) and P(Y), we use two different approaches. The first
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   102
         * one assumes that the shortCircuitProbability and the probability on the IfNode were
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   103
         * created with each other in mind. If this assumption does not hold, we fall back to
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   104
         * another mechanism for computing the probabilities.
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   105
         */
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   106
        AbstractBeginNode trueTarget = ifNode.trueSuccessor();
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   107
        AbstractBeginNode falseTarget = ifNode.falseSuccessor();
49873
26ebfe8ce852 8199755: Update Graal
dlong
parents: 48190
diff changeset
   108
50330
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   109
        // 1st approach
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   110
        // assumption: P(originalIf.trueSuccessor) == P(X) + ((1 - P(X)) * P(Y))
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   111
        double firstIfTrueProbability = shortCircuitProbability;
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   112
        double secondIfTrueProbability = sanitizeProbability((ifNode.getTrueSuccessorProbability() - shortCircuitProbability) / (1 - shortCircuitProbability));
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   113
        double expectedOriginalIfTrueProbability = firstIfTrueProbability + (1 - firstIfTrueProbability) * secondIfTrueProbability;
46807
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   114
50330
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   115
        if (!doubleEquals(ifNode.getTrueSuccessorProbability(), expectedOriginalIfTrueProbability)) {
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   116
            /*
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   117
             * 2nd approach
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   118
             *
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   119
             * the assumption above did not hold, so we either used an artificial probability as
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   120
             * shortCircuitProbability or the ShortCircuitOrNode was moved to some other IfNode.
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   121
             *
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   122
             * so, we distribute the if's trueSuccessorProbability between the newly generated if
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   123
             * nodes according to the shortCircuitProbability. the following invariant is always
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   124
             * true in this case: P(originalIf.trueSuccessor) == P(X) + ((1 - P(X)) * P(Y))
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   125
             */
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   126
            firstIfTrueProbability = ifNode.getTrueSuccessorProbability() * shortCircuitProbability;
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   127
            secondIfTrueProbability = sanitizeProbability(1 - (ifNode.probability(falseTarget) / (1 - firstIfTrueProbability)));
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   128
        }
49873
26ebfe8ce852 8199755: Update Graal
dlong
parents: 48190
diff changeset
   129
50330
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   130
        ifNode.clearSuccessors();
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   131
        Graph graph = ifNode.graph();
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   132
        AbstractMergeNode trueTargetMerge = graph.add(new MergeNode());
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   133
        trueTargetMerge.setNext(trueTarget);
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   134
        EndNode firstTrueEnd = graph.add(new EndNode());
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   135
        EndNode secondTrueEnd = graph.add(new EndNode());
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   136
        trueTargetMerge.addForwardEnd(firstTrueEnd);
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   137
        trueTargetMerge.addForwardEnd(secondTrueEnd);
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   138
        AbstractBeginNode firstTrueTarget = BeginNode.begin(firstTrueEnd);
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   139
        firstTrueTarget.setNodeSourcePosition(trueTarget.getNodeSourcePosition());
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   140
        AbstractBeginNode secondTrueTarget = BeginNode.begin(secondTrueEnd);
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   141
        secondTrueTarget.setNodeSourcePosition(trueTarget.getNodeSourcePosition());
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   142
        if (yNegated) {
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   143
            secondIfTrueProbability = 1.0 - secondIfTrueProbability;
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   144
        }
50330
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   145
        if (xNegated) {
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   146
            firstIfTrueProbability = 1.0 - firstIfTrueProbability;
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   147
        }
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   148
        IfNode secondIf = new IfNode(y, yNegated ? falseTarget : secondTrueTarget, yNegated ? secondTrueTarget : falseTarget, secondIfTrueProbability);
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   149
        secondIf.setNodeSourcePosition(ifNode.getNodeSourcePosition());
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   150
        AbstractBeginNode secondIfBegin = BeginNode.begin(graph.add(secondIf));
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   151
        secondIfBegin.setNodeSourcePosition(falseTarget.getNodeSourcePosition());
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   152
        IfNode firstIf = graph.add(new IfNode(x, xNegated ? secondIfBegin : firstTrueTarget, xNegated ? firstTrueTarget : secondIfBegin, firstIfTrueProbability));
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   153
        firstIf.setNodeSourcePosition(ifNode.getNodeSourcePosition());
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   154
        ifNode.replaceAtPredecessor(firstIf);
2cbc42a5764b 8202670: Update Graal
dlong
parents: 49873
diff changeset
   155
        ifNode.safeDelete();
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   156
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   157
46807
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   158
    private static boolean doubleEquals(double a, double b) {
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   159
        assert !Double.isNaN(a) && !Double.isNaN(b) && !Double.isInfinite(a) && !Double.isInfinite(b);
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   160
        return a - EPSILON < b && a + EPSILON > b;
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   161
    }
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   162
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   163
    private static double sanitizeProbability(double value) {
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   164
        double newValue = Math.min(1.0, Math.max(0.0, value));
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   165
        if (Double.isNaN(newValue)) {
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   166
            newValue = 0.5;
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   167
        }
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   168
        return newValue;
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   169
    }
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   170
49873
26ebfe8ce852 8199755: Update Graal
dlong
parents: 48190
diff changeset
   171
    @SuppressWarnings("try")
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   172
    private static void processConditional(LogicNode x, boolean xNegated, LogicNode y, boolean yNegated, ConditionalNode conditional) {
49873
26ebfe8ce852 8199755: Update Graal
dlong
parents: 48190
diff changeset
   173
        try (DebugCloseable context = conditional.withNodeSourcePosition()) {
26ebfe8ce852 8199755: Update Graal
dlong
parents: 48190
diff changeset
   174
            ValueNode trueTarget = conditional.trueValue();
26ebfe8ce852 8199755: Update Graal
dlong
parents: 48190
diff changeset
   175
            ValueNode falseTarget = conditional.falseValue();
26ebfe8ce852 8199755: Update Graal
dlong
parents: 48190
diff changeset
   176
            Graph graph = conditional.graph();
26ebfe8ce852 8199755: Update Graal
dlong
parents: 48190
diff changeset
   177
            ConditionalNode secondConditional = graph.unique(new ConditionalNode(y, yNegated ? falseTarget : trueTarget, yNegated ? trueTarget : falseTarget));
26ebfe8ce852 8199755: Update Graal
dlong
parents: 48190
diff changeset
   178
            ConditionalNode firstConditional = graph.unique(new ConditionalNode(x, xNegated ? secondConditional : trueTarget, xNegated ? trueTarget : secondConditional));
26ebfe8ce852 8199755: Update Graal
dlong
parents: 48190
diff changeset
   179
            conditional.replaceAndDelete(firstConditional);
26ebfe8ce852 8199755: Update Graal
dlong
parents: 48190
diff changeset
   180
        }
43972
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
    @Override
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   184
    public boolean checkContract() {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   185
        return false;
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
}