hotspot/src/jdk.internal.vm.compiler/share/classes/org.graalvm.compiler.phases.common/src/org/graalvm/compiler/phases/common/ExpandLogicPhase.java
author iveresov
Fri, 11 Aug 2017 17:36:26 -0700
changeset 46807 8b2c620d7092
parent 46762 f7defa99f173
permissions -rw-r--r--
8186158: 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;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    38
import org.graalvm.compiler.nodes.ShortCircuitOrNode;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    39
import org.graalvm.compiler.nodes.StructuredGraph;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    40
import org.graalvm.compiler.nodes.ValueNode;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    41
import org.graalvm.compiler.nodes.calc.ConditionalNode;
46459
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    42
import org.graalvm.compiler.nodes.calc.FloatEqualsNode;
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    43
import org.graalvm.compiler.nodes.calc.FloatLessThanNode;
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    44
import org.graalvm.compiler.nodes.calc.IntegerEqualsNode;
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    45
import org.graalvm.compiler.nodes.calc.IntegerLessThanNode;
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    46
import org.graalvm.compiler.nodes.calc.NormalizeCompareNode;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    47
import org.graalvm.compiler.phases.Phase;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    48
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    49
public class ExpandLogicPhase extends Phase {
46807
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
    50
    private static final double EPSILON = 1E-6;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    51
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    52
    @Override
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
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    59
        for (NormalizeCompareNode logic : graph.getNodes(NormalizeCompareNode.TYPE)) {
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    60
            processNormalizeCompareNode(logic);
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    61
        }
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    62
        graph.setAfterExpandLogic();
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    63
    }
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    64
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    65
    private static void processNormalizeCompareNode(NormalizeCompareNode normalize) {
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    66
        LogicNode equalComp;
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    67
        LogicNode lessComp;
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    68
        StructuredGraph graph = normalize.graph();
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    69
        ValueNode x = normalize.getX();
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    70
        ValueNode y = normalize.getY();
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    71
        if (x.stamp() instanceof FloatStamp) {
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    72
            equalComp = graph.addOrUniqueWithInputs(FloatEqualsNode.create(x, y));
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    73
            lessComp = graph.addOrUniqueWithInputs(FloatLessThanNode.create(x, y, normalize.isUnorderedLess()));
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    74
        } else {
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    75
            equalComp = graph.addOrUniqueWithInputs(IntegerEqualsNode.create(x, y));
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    76
            lessComp = graph.addOrUniqueWithInputs(IntegerLessThanNode.create(x, y));
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    77
        }
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    78
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    79
        Stamp stamp = normalize.stamp();
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    80
        ConditionalNode equalValue = graph.unique(
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    81
                        new ConditionalNode(equalComp, ConstantNode.forIntegerStamp(stamp, 0, graph), ConstantNode.forIntegerStamp(stamp, 1, graph)));
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    82
        ConditionalNode value = graph.unique(new ConditionalNode(lessComp, ConstantNode.forIntegerStamp(stamp, -1, graph), equalValue));
7d4e637d3f21 8180267: Update Graal
kvn
parents: 46344
diff changeset
    83
        normalize.replaceAtUsagesAndDelete(value);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    84
    }
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
    private static void processBinary(ShortCircuitOrNode binary) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    87
        while (binary.usages().isNotEmpty()) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    88
            Node usage = binary.usages().first();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    89
            if (usage instanceof ShortCircuitOrNode) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    90
                processBinary((ShortCircuitOrNode) usage);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    91
            } else if (usage instanceof IfNode) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    92
                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
    93
            } else if (usage instanceof ConditionalNode) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    94
                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
    95
            } else {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    96
                throw GraalError.shouldNotReachHere();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    97
            }
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
        binary.safeDelete();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   100
    }
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
    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
   103
        /*
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   104
         * this method splits an IfNode, which has a ShortCircuitOrNode as its condition, into two
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   105
         * separate IfNodes: if(X) and if(Y)
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   106
         *
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   107
         * 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
   108
         * one assumes that the shortCircuitProbability and the probability on the IfNode were
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   109
         * 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
   110
         * another mechanism for computing the probabilities.
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   111
         */
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   112
        AbstractBeginNode trueTarget = ifNode.trueSuccessor();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   113
        AbstractBeginNode falseTarget = ifNode.falseSuccessor();
46807
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   114
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   115
        // 1st approach
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   116
        // assumption: P(originalIf.trueSuccessor) == P(X) + ((1 - P(X)) * P(Y))
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   117
        double firstIfTrueProbability = shortCircuitProbability;
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   118
        double secondIfTrueProbability = sanitizeProbability((ifNode.getTrueSuccessorProbability() - shortCircuitProbability) / (1 - shortCircuitProbability));
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   119
        double expectedOriginalIfTrueProbability = firstIfTrueProbability + (1 - firstIfTrueProbability) * secondIfTrueProbability;
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   120
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   121
        if (!doubleEquals(ifNode.getTrueSuccessorProbability(), expectedOriginalIfTrueProbability)) {
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   122
            /*
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   123
             * 2nd approach
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   124
             *
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   125
             * the assumption above did not hold, so we either used an artificial probability as
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   126
             * shortCircuitProbability or the ShortCircuitOrNode was moved to some other IfNode.
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   127
             *
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   128
             * so, we distribute the if's trueSuccessorProbability between the newly generated if
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   129
             * nodes according to the shortCircuitProbability. the following invariant is always
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   130
             * true in this case: P(originalIf.trueSuccessor) == P(X) + ((1 - P(X)) * P(Y))
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   131
             */
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   132
            firstIfTrueProbability = ifNode.getTrueSuccessorProbability() * shortCircuitProbability;
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   133
            secondIfTrueProbability = sanitizeProbability(1 - (ifNode.probability(falseTarget) / (1 - firstIfTrueProbability)));
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   134
        }
46807
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   135
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   136
        ifNode.clearSuccessors();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   137
        Graph graph = ifNode.graph();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   138
        AbstractMergeNode trueTargetMerge = graph.add(new MergeNode());
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   139
        trueTargetMerge.setNext(trueTarget);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   140
        EndNode firstTrueEnd = graph.add(new EndNode());
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   141
        EndNode secondTrueEnd = graph.add(new EndNode());
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   142
        trueTargetMerge.addForwardEnd(firstTrueEnd);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   143
        trueTargetMerge.addForwardEnd(secondTrueEnd);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   144
        AbstractBeginNode firstTrueTarget = BeginNode.begin(firstTrueEnd);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   145
        AbstractBeginNode secondTrueTarget = BeginNode.begin(secondTrueEnd);
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   146
        if (yNegated) {
46807
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   147
            secondIfTrueProbability = 1.0 - secondIfTrueProbability;
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   148
        }
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   149
        if (xNegated) {
46807
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   150
            firstIfTrueProbability = 1.0 - firstIfTrueProbability;
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   151
        }
46807
8b2c620d7092 8186158: Update Graal
iveresov
parents: 46762
diff changeset
   152
        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
   153
        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
   154
        ifNode.replaceAtPredecessor(firstIf);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   155
        ifNode.safeDelete();
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
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   171
    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
   172
        ValueNode trueTarget = conditional.trueValue();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   173
        ValueNode falseTarget = conditional.falseValue();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   174
        Graph graph = conditional.graph();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   175
        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
   176
        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
   177
        conditional.replaceAndDelete(firstConditional);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   178
    }
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
    @Override
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   181
    public boolean checkContract() {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   182
        return false;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   183
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   184
}