src/jdk.internal.vm.compiler/share/classes/org.graalvm.compiler.nodes/src/org/graalvm/compiler/nodes/calc/IntegerLessThanNode.java
author dlong
Tue, 24 Sep 2019 12:47:15 -0400
changeset 58299 6df94ce3ab2f
parent 54084 84f10bbf993f
permissions -rw-r--r--
8229201: Update Graal Reviewed-by: kvn
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
     1
/*
52578
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
     2
 * Copyright (c) 2011, 2018, Oracle and/or its affiliates. All rights reserved.
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
     4
 *
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
     7
 * published by the Free Software Foundation.
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
     8
 *
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    13
 * accompanied this code).
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    14
 *
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    18
 *
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    21
 * questions.
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    22
 */
50858
2d3e99a72541 8205824: Update Graal
never
parents: 48861
diff changeset
    23
2d3e99a72541 8205824: Update Graal
never
parents: 48861
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
48861
47f19ff9903c 8194819: Update Graal
iveresov
parents: 48190
diff changeset
    27
import static org.graalvm.compiler.core.common.calc.CanonicalCondition.LT;
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
    28
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
    29
import org.graalvm.compiler.core.common.NumUtil;
48861
47f19ff9903c 8194819: Update Graal
iveresov
parents: 48190
diff changeset
    30
import org.graalvm.compiler.core.common.calc.CanonicalCondition;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    31
import org.graalvm.compiler.core.common.type.FloatStamp;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    32
import org.graalvm.compiler.core.common.type.IntegerStamp;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    33
import org.graalvm.compiler.core.common.type.StampFactory;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    34
import org.graalvm.compiler.debug.GraalError;
46393
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
    35
import org.graalvm.compiler.graph.Node;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    36
import org.graalvm.compiler.graph.NodeClass;
46393
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
    37
import org.graalvm.compiler.graph.spi.CanonicalizerTool;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    38
import org.graalvm.compiler.nodeinfo.NodeInfo;
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
    39
import org.graalvm.compiler.nodes.ConstantNode;
46393
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
    40
import org.graalvm.compiler.nodes.LogicConstantNode;
46371
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
    41
import org.graalvm.compiler.nodes.LogicNegationNode;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    42
import org.graalvm.compiler.nodes.LogicNode;
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
    43
import org.graalvm.compiler.nodes.NodeView;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    44
import org.graalvm.compiler.nodes.ValueNode;
48861
47f19ff9903c 8194819: Update Graal
iveresov
parents: 48190
diff changeset
    45
import org.graalvm.compiler.options.OptionValues;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    46
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    47
import jdk.vm.ci.code.CodeUtil;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    48
import jdk.vm.ci.meta.Constant;
48861
47f19ff9903c 8194819: Update Graal
iveresov
parents: 48190
diff changeset
    49
import jdk.vm.ci.meta.ConstantReflectionProvider;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    50
import jdk.vm.ci.meta.JavaKind;
48861
47f19ff9903c 8194819: Update Graal
iveresov
parents: 48190
diff changeset
    51
import jdk.vm.ci.meta.MetaAccessProvider;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    52
import jdk.vm.ci.meta.PrimitiveConstant;
54084
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52578
diff changeset
    53
import jdk.vm.ci.meta.TriState;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    54
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    55
@NodeInfo(shortName = "<")
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
    56
public final class IntegerLessThanNode extends IntegerLowerThanNode {
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    57
    public static final NodeClass<IntegerLessThanNode> TYPE = NodeClass.create(IntegerLessThanNode.class);
47798
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
    58
    private static final LessThanOp OP = new LessThanOp();
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    59
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    60
    public IntegerLessThanNode(ValueNode x, ValueNode y) {
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
    61
        super(TYPE, x, y, OP);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    62
        assert !x.getStackKind().isNumericFloat() && x.getStackKind() != JavaKind.Object;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    63
        assert !y.getStackKind().isNumericFloat() && y.getStackKind() != JavaKind.Object;
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
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
    66
    public static LogicNode create(ValueNode x, ValueNode y, NodeView view) {
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
    67
        return OP.create(x, y, view);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    68
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    69
46393
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
    70
    public static LogicNode create(ConstantReflectionProvider constantReflection, MetaAccessProvider metaAccess, OptionValues options, Integer smallestCompareWidth,
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
    71
                    ValueNode x, ValueNode y, NodeView view) {
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
    72
        LogicNode value = OP.canonical(constantReflection, metaAccess, options, smallestCompareWidth, OP.getCondition(), false, x, y, view);
46393
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
    73
        if (value != null) {
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
    74
            return value;
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
    75
        }
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
    76
        return create(x, y, view);
46393
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
    77
    }
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
    78
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    79
    @Override
46393
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
    80
    public Node canonical(CanonicalizerTool tool, ValueNode forX, ValueNode forY) {
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
    81
        NodeView view = NodeView.from(tool);
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
    82
        ValueNode value = OP.canonical(tool.getConstantReflection(), tool.getMetaAccess(), tool.getOptions(), tool.smallestCompareWidth(), OP.getCondition(), false, forX, forY, view);
46393
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
    83
        if (value != null) {
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
    84
            return 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
        return this;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    87
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    88
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    89
    public static boolean subtractMayUnderflow(long x, long y, long minValue) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    90
        long r = x - y;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    91
        // HD 2-12 Overflow iff the arguments have different signs and
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    92
        // the sign of the result is different than the sign of x
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    93
        return (((x ^ y) & (x ^ r)) < 0) || r <= minValue;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    94
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    95
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    96
    public static boolean subtractMayOverflow(long x, long y, long maxValue) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    97
        long r = x - y;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    98
        // HD 2-12 Overflow iff the arguments have different signs and
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    99
        // the sign of the result is different than the sign of x
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   100
        return (((x ^ y) & (x ^ r)) < 0) || r > maxValue;
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
46393
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   103
    public static class LessThanOp extends LowerOp {
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   104
        @Override
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   105
        protected CompareNode duplicateModified(ValueNode newX, ValueNode newY, boolean unorderedIsTrue, NodeView view) {
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   106
            if (newX.stamp(view) instanceof FloatStamp && newY.stamp(view) instanceof FloatStamp) {
46393
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   107
                return new FloatLessThanNode(newX, newY, unorderedIsTrue); // TODO: Is the last arg
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   108
                                                                           // supposed to be true?
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   109
            } else if (newX.stamp(view) instanceof IntegerStamp && newY.stamp(view) instanceof IntegerStamp) {
46393
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   110
                return new IntegerLessThanNode(newX, newY);
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   111
            }
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   112
            throw GraalError.shouldNotReachHere();
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   113
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   114
46393
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   115
        @Override
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   116
        protected LogicNode optimizeNormalizeCompare(ConstantReflectionProvider constantReflection, MetaAccessProvider metaAccess, OptionValues options, Integer smallestCompareWidth,
58299
6df94ce3ab2f 8229201: Update Graal
dlong
parents: 54084
diff changeset
   117
                        Constant constant, AbstractNormalizeCompareNode normalizeNode, boolean mirrored, NodeView view) {
46393
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   118
            PrimitiveConstant primitive = (PrimitiveConstant) constant;
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   119
            /* @formatter:off
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   120
             * a NC b < c  (not mirrored)
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   121
             * cases for c:
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   122
             *  0         -> a < b
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   123
             *  [MIN, -1] -> false
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   124
             *  1         -> a <= b
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   125
             *  [2, MAX]  -> true
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   126
             * unordered-is-less means unordered-is-true.
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   127
             *
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   128
             * c < a NC b  (mirrored)
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   129
             * cases for c:
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   130
             *  0         -> a > b
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   131
             *  [1, MAX]  -> false
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   132
             *  -1        -> a >= b
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   133
             *  [MIN, -2] -> true
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   134
             * unordered-is-less means unordered-is-false.
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   135
             *
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   136
             *  We can handle mirroring by swapping a & b and negating the constant.
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   137
             *  @formatter:on
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   138
             */
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   139
            long cst = mirrored ? -primitive.asLong() : primitive.asLong();
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   140
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   141
            if (cst == 0) {
58299
6df94ce3ab2f 8229201: Update Graal
dlong
parents: 54084
diff changeset
   142
                return normalizeNode.createLowerComparison(mirrored, constantReflection, metaAccess, options, smallestCompareWidth, view);
46393
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   143
            } else if (cst == 1) {
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   144
                // a <= b <=> !(a > b)
58299
6df94ce3ab2f 8229201: Update Graal
dlong
parents: 54084
diff changeset
   145
                // since we negate, we have to reverse the unordered result
6df94ce3ab2f 8229201: Update Graal
dlong
parents: 54084
diff changeset
   146
                LogicNode compare = normalizeNode.createLowerComparison(!mirrored, constantReflection, metaAccess, options, smallestCompareWidth, view);
46393
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   147
                return LogicNegationNode.create(compare);
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   148
            } else if (cst <= -1) {
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   149
                return LogicConstantNode.contradiction();
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   150
            } else {
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   151
                assert cst >= 2;
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   152
                return LogicConstantNode.tautology();
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   153
            }
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   154
        }
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   155
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   156
        @Override
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   157
        protected LogicNode findSynonym(ValueNode forX, ValueNode forY, NodeView view) {
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   158
            LogicNode result = super.findSynonym(forX, forY, view);
46371
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
   159
            if (result != null) {
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
   160
                return result;
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
   161
            }
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   162
            if (forX.stamp(view) instanceof IntegerStamp && forY.stamp(view) instanceof IntegerStamp) {
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   163
                if (IntegerStamp.sameSign((IntegerStamp) forX.stamp(view), (IntegerStamp) forY.stamp(view))) {
46371
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
   164
                    return new IntegerBelowNode(forX, forY);
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
   165
                }
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
   166
            }
52578
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   167
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   168
            // Attempt to optimize the case where we can fold a constant from the left side (either
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   169
            // from an add or sub) into the constant on the right side.
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   170
            if (forY.isConstant()) {
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   171
                if (forX instanceof SubNode) {
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   172
                    SubNode sub = (SubNode) forX;
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   173
                    ValueNode xx = null;
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   174
                    ValueNode yy = null;
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   175
                    boolean negate = false;
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   176
                    if (forY.asConstant().isDefaultForKind()) {
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   177
                        // (x - y) < 0 when x - y is known not to underflow <=> x < y
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   178
                        xx = sub.getX();
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   179
                        yy = sub.getY();
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   180
                    } else if (forY.isJavaConstant() && forY.asJavaConstant().asLong() == 1) {
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   181
                        // (x - y) < 1 when x - y is known not to underflow <=> !(y < x)
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   182
                        xx = sub.getY();
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   183
                        yy = sub.getX();
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   184
                        negate = true;
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   185
                    }
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   186
                    if (xx != null) {
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   187
                        assert yy != null;
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   188
                        IntegerStamp xStamp = (IntegerStamp) sub.getX().stamp(view);
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   189
                        IntegerStamp yStamp = (IntegerStamp) sub.getY().stamp(view);
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   190
                        long minValue = CodeUtil.minValue(xStamp.getBits());
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   191
                        long maxValue = CodeUtil.maxValue(xStamp.getBits());
46371
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
   192
52578
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   193
                        if (!subtractMayUnderflow(xStamp.lowerBound(), yStamp.upperBound(), minValue) && !subtractMayOverflow(xStamp.upperBound(), yStamp.lowerBound(), maxValue)) {
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   194
                            LogicNode logic = new IntegerLessThanNode(xx, yy);
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   195
                            if (negate) {
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   196
                                logic = LogicNegationNode.create(logic);
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   197
                            }
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   198
                            return logic;
46371
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
   199
                        }
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
   200
                    }
52578
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   201
                } else if (forX instanceof AddNode) {
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   202
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   203
                    // (x + xConstant) < forY => x < (forY - xConstant)
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   204
                    AddNode addNode = (AddNode) forX;
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   205
                    if (addNode.getY().isJavaConstant()) {
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   206
                        IntegerStamp xStamp = (IntegerStamp) addNode.getX().stamp(view);
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   207
                        if (!IntegerStamp.addCanOverflow(xStamp, (IntegerStamp) addNode.getY().stamp(view))) {
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   208
                            long minValue = CodeUtil.minValue(xStamp.getBits());
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   209
                            long maxValue = CodeUtil.maxValue(xStamp.getBits());
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   210
                            long yConstant = forY.asJavaConstant().asLong();
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   211
                            long xConstant = addNode.getY().asJavaConstant().asLong();
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   212
                            if (!subtractMayUnderflow(yConstant, xConstant, minValue) && !subtractMayOverflow(yConstant, xConstant, maxValue)) {
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   213
                                long newConstant = yConstant - xConstant;
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   214
                                return IntegerLessThanNode.create(addNode.getX(), ConstantNode.forIntegerStamp(xStamp, newConstant), view);
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   215
                            }
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   216
                        }
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   217
                    }
46371
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
   218
                }
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
   219
            }
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
   220
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   221
            if (forX.stamp(view) instanceof IntegerStamp) {
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   222
                assert forY.stamp(view) instanceof IntegerStamp;
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   223
                int bits = ((IntegerStamp) forX.stamp(view)).getBits();
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   224
                assert ((IntegerStamp) forY.stamp(view)).getBits() == bits;
52578
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   225
                LogicNode logic = canonicalizeRangeFlip(forX, forY, bits, true, view);
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   226
                if (logic != null) {
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   227
                    return logic;
46371
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
   228
                }
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
   229
            }
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
   230
            return null;
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
   231
        }
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
   232
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
   233
        @Override
48861
47f19ff9903c 8194819: Update Graal
iveresov
parents: 48190
diff changeset
   234
        protected CanonicalCondition getCondition() {
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   235
            return LT;
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   236
        }
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   237
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   238
        @Override
46371
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
   239
        protected IntegerLowerThanNode createNode(ValueNode x, ValueNode y) {
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   240
            return new IntegerLessThanNode(x, y);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   241
        }
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   242
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   243
        @Override
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   244
        protected long upperBound(IntegerStamp stamp) {
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   245
            return stamp.upperBound();
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   246
        }
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   247
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   248
        @Override
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   249
        protected long lowerBound(IntegerStamp stamp) {
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   250
            return stamp.lowerBound();
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   251
        }
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   252
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   253
        @Override
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   254
        protected int compare(long a, long b) {
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   255
            return Long.compare(a, b);
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   256
        }
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   257
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   258
        @Override
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   259
        protected long min(long a, long b) {
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   260
            return Math.min(a, b);
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   261
        }
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   262
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   263
        @Override
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   264
        protected long max(long a, long b) {
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   265
            return Math.max(a, b);
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   266
        }
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   267
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   268
        @Override
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   269
        protected long cast(long a, int bits) {
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   270
            return CodeUtil.signExtend(a, bits);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   271
        }
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   272
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   273
        @Override
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   274
        protected long minValue(int bits) {
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   275
            return NumUtil.minValue(bits);
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   276
        }
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   277
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   278
        @Override
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   279
        protected long maxValue(int bits) {
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   280
            return NumUtil.maxValue(bits);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   281
        }
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   282
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   283
        @Override
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   284
        protected IntegerStamp forInteger(int bits, long min, long max) {
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   285
            return StampFactory.forInteger(bits, cast(min, bits), cast(max, bits));
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   286
        }
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   287
    }
54084
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52578
diff changeset
   288
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52578
diff changeset
   289
    @Override
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52578
diff changeset
   290
    public TriState implies(boolean thisNegated, LogicNode other) {
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52578
diff changeset
   291
        if (!thisNegated) {
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52578
diff changeset
   292
            if (other instanceof IntegerLessThanNode) {
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52578
diff changeset
   293
                ValueNode otherX = ((IntegerLessThanNode) other).getX();
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52578
diff changeset
   294
                ValueNode otherY = ((IntegerLessThanNode) other).getY();
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52578
diff changeset
   295
                // x < y => !y < x
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52578
diff changeset
   296
                if (getX() == otherY && getY() == otherX) {
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52578
diff changeset
   297
                    return TriState.FALSE;
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52578
diff changeset
   298
                }
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52578
diff changeset
   299
            }
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52578
diff changeset
   300
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52578
diff changeset
   301
            // x < y => !x == y
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52578
diff changeset
   302
            // x < y => !y == x
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52578
diff changeset
   303
            if (other instanceof IntegerEqualsNode) {
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52578
diff changeset
   304
                ValueNode otherX = ((IntegerEqualsNode) other).getX();
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52578
diff changeset
   305
                ValueNode otherY = ((IntegerEqualsNode) other).getY();
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52578
diff changeset
   306
                if ((getX() == otherX && getY() == otherY) || (getX() == otherY && getY() == otherX)) {
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52578
diff changeset
   307
                    return TriState.FALSE;
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52578
diff changeset
   308
                }
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52578
diff changeset
   309
            }
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52578
diff changeset
   310
        }
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52578
diff changeset
   311
        return super.implies(thisNegated, other);
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52578
diff changeset
   312
    }
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   313
}