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