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