src/jdk.internal.vm.compiler/share/classes/org.graalvm.compiler.nodes/src/org/graalvm/compiler/nodes/calc/BinaryArithmeticNode.java
author dlong
Wed, 09 Oct 2019 21:53:48 -0700
changeset 58533 46b0b7fe255c
parent 54084 84f10bbf993f
child 58877 aec7bf35d6f5
permissions -rw-r--r--
8231973: 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
/*
58533
46b0b7fe255c 8231973: Update Graal
dlong
parents: 54084
diff changeset
     2
 * Copyright (c) 2009, 2019, Oracle and/or its affiliates. All rights reserved.
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
     4
 *
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
     7
 * published by the Free Software Foundation.
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
     8
 *
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    13
 * accompanied this code).
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    14
 *
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    18
 *
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    21
 * questions.
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    22
 */
50858
2d3e99a72541 8205824: Update Graal
never
parents: 48190
diff changeset
    23
2d3e99a72541 8205824: Update Graal
never
parents: 48190
diff changeset
    24
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    25
package org.graalvm.compiler.nodes.calc;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    26
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    27
import static org.graalvm.compiler.nodeinfo.NodeCycles.CYCLES_1;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    28
import static org.graalvm.compiler.nodeinfo.NodeSize.SIZE_1;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    29
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    30
import org.graalvm.compiler.core.common.type.ArithmeticOpTable;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    31
import org.graalvm.compiler.core.common.type.ArithmeticOpTable.BinaryOp;
52578
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
    32
import org.graalvm.compiler.core.common.type.IntegerStamp;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    33
import org.graalvm.compiler.core.common.type.Stamp;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    34
import org.graalvm.compiler.debug.GraalError;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    35
import org.graalvm.compiler.graph.Graph;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    36
import org.graalvm.compiler.graph.Node;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    37
import org.graalvm.compiler.graph.NodeClass;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    38
import org.graalvm.compiler.graph.iterators.NodePredicate;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    39
import org.graalvm.compiler.graph.spi.Canonicalizable;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    40
import org.graalvm.compiler.graph.spi.CanonicalizerTool;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    41
import org.graalvm.compiler.nodeinfo.NodeInfo;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    42
import org.graalvm.compiler.nodes.ArithmeticOperation;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    43
import org.graalvm.compiler.nodes.ConstantNode;
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
    44
import org.graalvm.compiler.nodes.NodeView;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    45
import org.graalvm.compiler.nodes.StructuredGraph;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    46
import org.graalvm.compiler.nodes.ValueNode;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    47
import org.graalvm.compiler.nodes.ValuePhiNode;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    48
import org.graalvm.compiler.nodes.spi.ArithmeticLIRLowerable;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    49
import org.graalvm.compiler.nodes.spi.NodeValueMap;
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
import jdk.vm.ci.meta.Constant;
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
@NodeInfo(cycles = CYCLES_1, size = SIZE_1)
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    54
public abstract class BinaryArithmeticNode<OP> extends BinaryNode implements ArithmeticOperation, ArithmeticLIRLowerable, Canonicalizable.Binary<ValueNode> {
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
    @SuppressWarnings("rawtypes") public static final NodeClass<BinaryArithmeticNode> TYPE = NodeClass.create(BinaryArithmeticNode.class);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    57
58533
46b0b7fe255c 8231973: Update Graal
dlong
parents: 54084
diff changeset
    58
    protected BinaryArithmeticNode(NodeClass<? extends BinaryArithmeticNode<OP>> c, BinaryOp<OP> opForStampComputation, ValueNode x, ValueNode y) {
46b0b7fe255c 8231973: Update Graal
dlong
parents: 54084
diff changeset
    59
        super(c, opForStampComputation.foldStamp(x.stamp(NodeView.DEFAULT), y.stamp(NodeView.DEFAULT)), x, y);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    60
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    61
58533
46b0b7fe255c 8231973: Update Graal
dlong
parents: 54084
diff changeset
    62
    public static ArithmeticOpTable getArithmeticOpTable(ValueNode forValue) {
46b0b7fe255c 8231973: Update Graal
dlong
parents: 54084
diff changeset
    63
        return ArithmeticOpTable.forStamp(forValue.stamp(NodeView.DEFAULT));
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    64
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    65
58533
46b0b7fe255c 8231973: Update Graal
dlong
parents: 54084
diff changeset
    66
    protected abstract BinaryOp<OP> getOp(ArithmeticOpTable table);
46b0b7fe255c 8231973: Update Graal
dlong
parents: 54084
diff changeset
    67
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    68
    protected final BinaryOp<OP> getOp(ValueNode forX, ValueNode forY) {
58533
46b0b7fe255c 8231973: Update Graal
dlong
parents: 54084
diff changeset
    69
        ArithmeticOpTable table = getArithmeticOpTable(forX);
46b0b7fe255c 8231973: Update Graal
dlong
parents: 54084
diff changeset
    70
        assert table.equals(getArithmeticOpTable(forY));
46b0b7fe255c 8231973: Update Graal
dlong
parents: 54084
diff changeset
    71
        return getOp(table);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    72
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    73
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    74
    @Override
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    75
    public final BinaryOp<OP> getArithmeticOp() {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    76
        return getOp(getX(), getY());
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    77
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    78
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    79
    public boolean isAssociative() {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    80
        return getArithmeticOp().isAssociative();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    81
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    82
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    83
    @Override
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    84
    public ValueNode canonical(CanonicalizerTool tool, ValueNode forX, ValueNode forY) {
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
    85
        NodeView view = NodeView.from(tool);
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
    86
        ValueNode result = tryConstantFold(getOp(forX, forY), forX, forY, stamp(view), view);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    87
        if (result != null) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    88
            return result;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    89
        }
54084
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
    90
        if (forX instanceof ConditionalNode && forY.isConstant() && forX.hasExactlyOneUsage()) {
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
    91
            ConditionalNode conditionalNode = (ConditionalNode) forX;
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
    92
            BinaryOp<OP> arithmeticOp = getArithmeticOp();
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
    93
            ConstantNode trueConstant = tryConstantFold(arithmeticOp, conditionalNode.trueValue(), forY, this.stamp(view), view);
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
    94
            if (trueConstant != null) {
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
    95
                ConstantNode falseConstant = tryConstantFold(arithmeticOp, conditionalNode.falseValue(), forY, this.stamp(view), view);
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
    96
                if (falseConstant != null) {
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
    97
                    // @formatter:off
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
    98
                    /* The arithmetic is folded into a constant on both sides of the conditional.
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
    99
                     * Example:
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   100
                     *            (cond ? -5 : 5) + 100
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   101
                     * canonicalizes to:
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   102
                     *            (cond ? 95 : 105)
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   103
                     */
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   104
                    // @formatter:on
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   105
                    return ConditionalNode.create(conditionalNode.condition, trueConstant,
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   106
                                    falseConstant, view);
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   107
                }
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   108
            }
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   109
        }
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   110
        return this;
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
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   113
    @SuppressWarnings("unused")
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   114
    public static <OP> ConstantNode tryConstantFold(BinaryOp<OP> op, ValueNode forX, ValueNode forY, Stamp stamp, NodeView view) {
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   115
        if (forX.isConstant() && forY.isConstant()) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   116
            Constant ret = op.foldConstant(forX.asConstant(), forY.asConstant());
47798
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
   117
            if (ret != null) {
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
   118
                return ConstantNode.forPrimitive(stamp, ret);
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
   119
            }
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   120
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   121
        return null;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   122
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   123
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   124
    @Override
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   125
    public Stamp foldStamp(Stamp stampX, Stamp stampY) {
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   126
        assert stampX.isCompatible(x.stamp(NodeView.DEFAULT)) && stampY.isCompatible(y.stamp(NodeView.DEFAULT));
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   127
        return getArithmeticOp().foldStamp(stampX, stampY);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   128
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   129
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   130
    public static ValueNode add(StructuredGraph graph, ValueNode v1, ValueNode v2, NodeView view) {
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   131
        return graph.addOrUniqueWithInputs(AddNode.create(v1, v2, view));
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   132
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   133
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   134
    public static ValueNode add(ValueNode v1, ValueNode v2, NodeView view) {
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   135
        return AddNode.create(v1, v2, view);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   136
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   137
54084
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   138
    public static ValueNode add(ValueNode v1, ValueNode v2) {
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   139
        return add(v1, v2, NodeView.DEFAULT);
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   140
    }
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   141
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   142
    public static ValueNode mul(StructuredGraph graph, ValueNode v1, ValueNode v2, NodeView view) {
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   143
        return graph.addOrUniqueWithInputs(MulNode.create(v1, v2, view));
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   144
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   145
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   146
    public static ValueNode mul(ValueNode v1, ValueNode v2, NodeView view) {
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   147
        return MulNode.create(v1, v2, view);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   148
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   149
54084
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   150
    public static ValueNode mul(ValueNode v1, ValueNode v2) {
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   151
        return mul(v1, v2, NodeView.DEFAULT);
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   152
    }
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   153
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   154
    public static ValueNode sub(StructuredGraph graph, ValueNode v1, ValueNode v2, NodeView view) {
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   155
        return graph.addOrUniqueWithInputs(SubNode.create(v1, v2, view));
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
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   158
    public static ValueNode sub(ValueNode v1, ValueNode v2, NodeView view) {
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   159
        return SubNode.create(v1, v2, view);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   160
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   161
54084
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   162
    public static ValueNode sub(ValueNode v1, ValueNode v2) {
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   163
        return sub(v1, v2, NodeView.DEFAULT);
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   164
    }
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   165
52578
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   166
    public static ValueNode branchlessMin(ValueNode v1, ValueNode v2, NodeView view) {
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   167
        if (v1.isDefaultConstant() && !v2.isDefaultConstant()) {
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   168
            return branchlessMin(v2, v1, view);
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   169
        }
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   170
        int bits = ((IntegerStamp) v1.stamp(view)).getBits();
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   171
        assert ((IntegerStamp) v2.stamp(view)).getBits() == bits;
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   172
        ValueNode t1 = sub(v1, v2, view);
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   173
        ValueNode t2 = RightShiftNode.create(t1, bits - 1, view);
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   174
        ValueNode t3 = AndNode.create(t1, t2, view);
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   175
        return add(v2, t3, view);
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   176
    }
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   177
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   178
    public static ValueNode branchlessMax(ValueNode v1, ValueNode v2, NodeView view) {
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   179
        if (v1.isDefaultConstant() && !v2.isDefaultConstant()) {
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   180
            return branchlessMax(v2, v1, view);
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   181
        }
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   182
        int bits = ((IntegerStamp) v1.stamp(view)).getBits();
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   183
        assert ((IntegerStamp) v2.stamp(view)).getBits() == bits;
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   184
        if (v2.isDefaultConstant()) {
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   185
            // prefer a & ~(a>>31) to a - (a & (a>>31))
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   186
            return AndNode.create(v1, NotNode.create(RightShiftNode.create(v1, bits - 1, view)), view);
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   187
        } else {
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   188
            ValueNode t1 = sub(v1, v2, view);
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   189
            ValueNode t2 = RightShiftNode.create(t1, bits - 1, view);
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   190
            ValueNode t3 = AndNode.create(t1, t2, view);
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   191
            return sub(v1, t3, view);
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   192
        }
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   193
    }
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   194
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   195
    private enum ReassociateMatch {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   196
        x,
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   197
        y;
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
        public ValueNode getValue(BinaryNode binary) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   200
            switch (this) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   201
                case x:
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   202
                    return binary.getX();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   203
                case y:
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   204
                    return binary.getY();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   205
                default:
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   206
                    throw GraalError.shouldNotReachHere();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   207
            }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   208
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   209
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   210
        public ValueNode getOtherValue(BinaryNode binary) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   211
            switch (this) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   212
                case x:
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   213
                    return binary.getY();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   214
                case y:
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   215
                    return binary.getX();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   216
                default:
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   217
                    throw GraalError.shouldNotReachHere();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   218
            }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   219
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   220
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   221
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   222
    private static ReassociateMatch findReassociate(BinaryNode binary, NodePredicate criterion) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   223
        boolean resultX = criterion.apply(binary.getX());
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   224
        boolean resultY = criterion.apply(binary.getY());
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   225
        if (resultX && !resultY) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   226
            return ReassociateMatch.x;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   227
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   228
        if (!resultX && resultY) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   229
            return ReassociateMatch.y;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   230
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   231
        return null;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   232
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   233
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   234
    //@formatter:off
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   235
    /*
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   236
     * In reassociate, complexity comes from the handling of IntegerSub (non commutative) which can
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   237
     * be mixed with IntegerAdd. It first tries to find m1, m2 which match the criterion :
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   238
     * (a o m2) o m1
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   239
     * (m2 o a) o m1
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   240
     * m1 o (a o m2)
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   241
     * m1 o (m2 o a)
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   242
     * It then produces 4 boolean for the -/+ cases:
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   243
     * invertA : should the final expression be like *-a (rather than a+*)
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   244
     * aSub : should the final expression be like a-* (rather than a+*)
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   245
     * invertM1 : should the final expression contain -m1
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   246
     * invertM2 : should the final expression contain -m2
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   247
     *
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   248
     */
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   249
    //@formatter:on
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   250
    /**
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   251
     * Tries to re-associate values which satisfy the criterion. For example with a constantness
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   252
     * criterion: {@code (a + 2) + 1 => a + (1 + 2)}
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   253
     * <p>
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   254
     * This method accepts only {@linkplain BinaryOp#isAssociative() associative} operations such as
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   255
     * +, -, *, &amp;, | and ^
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   256
     *
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   257
     * @param forY
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   258
     * @param forX
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   259
     */
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   260
    public static ValueNode reassociate(BinaryArithmeticNode<?> node, NodePredicate criterion, ValueNode forX, ValueNode forY, NodeView view) {
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   261
        assert node.getOp(forX, forY).isAssociative();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   262
        ReassociateMatch match1 = findReassociate(node, criterion);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   263
        if (match1 == null) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   264
            return node;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   265
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   266
        ValueNode otherValue = match1.getOtherValue(node);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   267
        boolean addSub = false;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   268
        boolean subAdd = false;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   269
        if (otherValue.getClass() != node.getClass()) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   270
            if (node instanceof AddNode && otherValue instanceof SubNode) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   271
                addSub = true;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   272
            } else if (node instanceof SubNode && otherValue instanceof AddNode) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   273
                subAdd = true;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   274
            } else {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   275
                return node;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   276
            }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   277
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   278
        BinaryNode other = (BinaryNode) otherValue;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   279
        ReassociateMatch match2 = findReassociate(other, criterion);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   280
        if (match2 == null) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   281
            return node;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   282
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   283
        boolean invertA = false;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   284
        boolean aSub = false;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   285
        boolean invertM1 = false;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   286
        boolean invertM2 = false;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   287
        if (addSub) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   288
            invertM2 = match2 == ReassociateMatch.y;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   289
            invertA = !invertM2;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   290
        } else if (subAdd) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   291
            invertA = invertM2 = match1 == ReassociateMatch.x;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   292
            invertM1 = !invertM2;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   293
        } else if (node instanceof SubNode && other instanceof SubNode) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   294
            invertA = match1 == ReassociateMatch.x ^ match2 == ReassociateMatch.x;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   295
            aSub = match1 == ReassociateMatch.y && match2 == ReassociateMatch.y;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   296
            invertM1 = match1 == ReassociateMatch.y && match2 == ReassociateMatch.x;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   297
            invertM2 = match1 == ReassociateMatch.x && match2 == ReassociateMatch.x;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   298
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   299
        assert !(invertM1 && invertM2) && !(invertA && aSub);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   300
        ValueNode m1 = match1.getValue(node);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   301
        ValueNode m2 = match2.getValue(other);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   302
        ValueNode a = match2.getOtherValue(other);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   303
        if (node instanceof AddNode || node instanceof SubNode) {
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   304
            ValueNode associated;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   305
            if (invertM1) {
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   306
                associated = BinaryArithmeticNode.sub(m2, m1, view);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   307
            } else if (invertM2) {
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   308
                associated = BinaryArithmeticNode.sub(m1, m2, view);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   309
            } else {
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   310
                associated = BinaryArithmeticNode.add(m1, m2, view);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   311
            }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   312
            if (invertA) {
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   313
                return BinaryArithmeticNode.sub(associated, a, view);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   314
            }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   315
            if (aSub) {
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   316
                return BinaryArithmeticNode.sub(a, associated, view);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   317
            }
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   318
            return BinaryArithmeticNode.add(a, associated, view);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   319
        } else if (node instanceof MulNode) {
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   320
            return BinaryArithmeticNode.mul(a, AddNode.mul(m1, m2, view), view);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   321
        } else if (node instanceof AndNode) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   322
            return new AndNode(a, new AndNode(m1, m2));
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   323
        } else if (node instanceof OrNode) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   324
            return new OrNode(a, new OrNode(m1, m2));
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   325
        } else if (node instanceof XorNode) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   326
            return new XorNode(a, new XorNode(m1, m2));
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   327
        } else {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   328
            throw GraalError.shouldNotReachHere();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   329
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   330
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   331
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   332
    /**
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   333
     * Ensure a canonical ordering of inputs for commutative nodes to improve GVN results. Order the
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   334
     * inputs by increasing {@link Node#id} and call {@link Graph#findDuplicate(Node)} on the node
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   335
     * if it's currently in a graph. It's assumed that if there was a constant on the left it's been
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   336
     * moved to the right by other code and that ordering is left alone.
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   337
     *
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   338
     * @return the original node or another node with the same input ordering
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   339
     */
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   340
    @SuppressWarnings("deprecation")
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   341
    public BinaryNode maybeCommuteInputs() {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   342
        assert this instanceof BinaryCommutative;
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   343
        if (!y.isConstant() && (x.isConstant() || x.getId() > y.getId())) {
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   344
            ValueNode tmp = x;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   345
            x = y;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   346
            y = tmp;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   347
            if (graph() != null) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   348
                // See if this node already exists
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   349
                BinaryNode duplicate = graph().findDuplicate(this);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   350
                if (duplicate != null) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   351
                    return duplicate;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   352
                }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   353
            }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   354
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   355
        return this;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   356
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   357
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   358
    /**
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   359
     * Determines if it would be better to swap the inputs in order to produce better assembly code.
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   360
     * First we try to pick a value which is dead after this use. If both values are dead at this
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   361
     * use then we try pick an induction variable phi to encourage the phi to live in a single
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   362
     * register.
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   363
     *
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   364
     * @param nodeValueMap
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   365
     * @return true if inputs should be swapped, false otherwise
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   366
     */
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   367
    protected boolean shouldSwapInputs(NodeValueMap nodeValueMap) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   368
        final boolean xHasOtherUsages = getX().hasUsagesOtherThan(this, nodeValueMap);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   369
        final boolean yHasOtherUsages = getY().hasUsagesOtherThan(this, nodeValueMap);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   370
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   371
        if (!getY().isConstant() && !yHasOtherUsages) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   372
            if (xHasOtherUsages == yHasOtherUsages) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   373
                return getY() instanceof ValuePhiNode && getY().inputs().contains(this);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   374
            } else {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   375
                return true;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   376
            }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   377
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   378
        return false;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   379
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   380
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   381
}