src/jdk.internal.vm.compiler/share/classes/org.graalvm.compiler.nodes/src/org/graalvm/compiler/nodes/calc/IntegerLessThanNode.java
author dlong
Thu, 15 Nov 2018 09:04:07 -0800
changeset 52578 7dd81e82d083
parent 50858 2d3e99a72541
child 54084 84f10bbf993f
permissions -rw-r--r--
8210777: 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;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    53
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    54
@NodeInfo(shortName = "<")
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
    55
public final class IntegerLessThanNode extends IntegerLowerThanNode {
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    56
    public static final NodeClass<IntegerLessThanNode> TYPE = NodeClass.create(IntegerLessThanNode.class);
47798
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
    57
    private static final LessThanOp OP = new LessThanOp();
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    58
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    59
    public IntegerLessThanNode(ValueNode x, ValueNode y) {
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
    60
        super(TYPE, x, y, OP);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    61
        assert !x.getStackKind().isNumericFloat() && x.getStackKind() != JavaKind.Object;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    62
        assert !y.getStackKind().isNumericFloat() && y.getStackKind() != JavaKind.Object;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    63
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    64
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
    65
    public static LogicNode create(ValueNode x, ValueNode y, NodeView view) {
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
    66
        return OP.create(x, y, view);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    67
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    68
46393
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
    69
    public static LogicNode create(ConstantReflectionProvider constantReflection, MetaAccessProvider metaAccess, OptionValues options, Integer smallestCompareWidth,
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
    70
                    ValueNode x, ValueNode y, NodeView view) {
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
    71
        LogicNode value = OP.canonical(constantReflection, metaAccess, options, smallestCompareWidth, OP.getCondition(), false, x, y, view);
46393
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
    72
        if (value != null) {
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
    73
            return value;
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
    74
        }
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
    75
        return create(x, y, view);
46393
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
    76
    }
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
    77
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    78
    @Override
46393
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
    79
    public Node canonical(CanonicalizerTool tool, ValueNode forX, ValueNode forY) {
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
    80
        NodeView view = NodeView.from(tool);
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
    81
        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
    82
        if (value != null) {
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
    83
            return 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
        return this;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    86
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    87
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    88
    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
    89
        long r = x - y;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    90
        // 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
    91
        // 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
    92
        return (((x ^ y) & (x ^ r)) < 0) || r <= minValue;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    93
    }
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
    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
    96
        long r = x - y;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    97
        // 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
    98
        // 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
    99
        return (((x ^ y) & (x ^ r)) < 0) || r > maxValue;
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
46393
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   102
    public static class LessThanOp extends LowerOp {
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   103
        @Override
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   104
        protected CompareNode duplicateModified(ValueNode newX, ValueNode newY, boolean unorderedIsTrue, NodeView view) {
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   105
            if (newX.stamp(view) instanceof FloatStamp && newY.stamp(view) instanceof FloatStamp) {
46393
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   106
                return new FloatLessThanNode(newX, newY, unorderedIsTrue); // TODO: Is the last arg
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   107
                                                                           // supposed to be true?
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   108
            } else if (newX.stamp(view) instanceof IntegerStamp && newY.stamp(view) instanceof IntegerStamp) {
46393
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   109
                return new IntegerLessThanNode(newX, newY);
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   110
            }
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   111
            throw GraalError.shouldNotReachHere();
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   112
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   113
46393
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   114
        @Override
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   115
        protected LogicNode optimizeNormalizeCompare(ConstantReflectionProvider constantReflection, MetaAccessProvider metaAccess, OptionValues options, Integer smallestCompareWidth,
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   116
                        Constant constant, NormalizeCompareNode normalizeNode, boolean mirrored, NodeView view) {
46393
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   117
            PrimitiveConstant primitive = (PrimitiveConstant) constant;
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   118
            /* @formatter:off
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   119
             * a NC b < c  (not mirrored)
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   120
             * cases for c:
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   121
             *  0         -> a < b
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   122
             *  [MIN, -1] -> false
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   123
             *  1         -> a <= b
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   124
             *  [2, MAX]  -> true
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   125
             * unordered-is-less means unordered-is-true.
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   126
             *
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   127
             * c < a NC b  (mirrored)
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   128
             * cases for c:
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   129
             *  0         -> a > b
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   130
             *  [1, MAX]  -> false
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   131
             *  -1        -> a >= b
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   132
             *  [MIN, -2] -> true
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   133
             * unordered-is-less means unordered-is-false.
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   134
             *
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   135
             *  We can handle mirroring by swapping a & b and negating the constant.
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   136
             *  @formatter:on
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   137
             */
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   138
            ValueNode a = mirrored ? normalizeNode.getY() : normalizeNode.getX();
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   139
            ValueNode b = mirrored ? normalizeNode.getX() : normalizeNode.getY();
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   140
            long cst = mirrored ? -primitive.asLong() : primitive.asLong();
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   141
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   142
            if (cst == 0) {
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   143
                if (normalizeNode.getX().getStackKind() == JavaKind.Double || normalizeNode.getX().getStackKind() == JavaKind.Float) {
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   144
                    return FloatLessThanNode.create(constantReflection, metaAccess, options, smallestCompareWidth, a, b, mirrored ^ normalizeNode.isUnorderedLess, view);
46393
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   145
                } else {
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   146
                    return IntegerLessThanNode.create(constantReflection, metaAccess, options, smallestCompareWidth, a, b, view);
46393
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   147
                }
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   148
            } else if (cst == 1) {
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   149
                // a <= b <=> !(a > b)
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   150
                LogicNode compare;
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   151
                if (normalizeNode.getX().getStackKind() == JavaKind.Double || normalizeNode.getX().getStackKind() == JavaKind.Float) {
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   152
                    // since we negate, we have to reverse the unordered result
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   153
                    compare = FloatLessThanNode.create(constantReflection, metaAccess, options, smallestCompareWidth, b, a, mirrored == normalizeNode.isUnorderedLess, view);
46393
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   154
                } else {
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   155
                    compare = IntegerLessThanNode.create(constantReflection, metaAccess, options, smallestCompareWidth, b, a, view);
46393
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   156
                }
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   157
                return LogicNegationNode.create(compare);
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   158
            } else if (cst <= -1) {
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   159
                return LogicConstantNode.contradiction();
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   160
            } else {
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   161
                assert cst >= 2;
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   162
                return LogicConstantNode.tautology();
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   163
            }
d497d892ab11 8178864: Update Graal
iveresov
parents: 46371
diff changeset
   164
        }
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   165
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   166
        @Override
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   167
        protected LogicNode findSynonym(ValueNode forX, ValueNode forY, NodeView view) {
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   168
            LogicNode result = super.findSynonym(forX, forY, view);
46371
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
   169
            if (result != null) {
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
   170
                return result;
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
   171
            }
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   172
            if (forX.stamp(view) instanceof IntegerStamp && forY.stamp(view) instanceof IntegerStamp) {
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   173
                if (IntegerStamp.sameSign((IntegerStamp) forX.stamp(view), (IntegerStamp) forY.stamp(view))) {
46371
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
   174
                    return new IntegerBelowNode(forX, forY);
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
   175
                }
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
   176
            }
52578
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   177
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   178
            // 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
   179
            // from an add or sub) into the constant on the right side.
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   180
            if (forY.isConstant()) {
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   181
                if (forX instanceof SubNode) {
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   182
                    SubNode sub = (SubNode) forX;
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   183
                    ValueNode xx = null;
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   184
                    ValueNode yy = null;
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   185
                    boolean negate = false;
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   186
                    if (forY.asConstant().isDefaultForKind()) {
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   187
                        // (x - y) < 0 when x - y is known not to underflow <=> x < y
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   188
                        xx = sub.getX();
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   189
                        yy = sub.getY();
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   190
                    } else if (forY.isJavaConstant() && forY.asJavaConstant().asLong() == 1) {
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   191
                        // (x - y) < 1 when x - y is known not to underflow <=> !(y < x)
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   192
                        xx = sub.getY();
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   193
                        yy = sub.getX();
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   194
                        negate = true;
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   195
                    }
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   196
                    if (xx != null) {
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   197
                        assert yy != null;
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   198
                        IntegerStamp xStamp = (IntegerStamp) sub.getX().stamp(view);
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   199
                        IntegerStamp yStamp = (IntegerStamp) sub.getY().stamp(view);
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   200
                        long minValue = CodeUtil.minValue(xStamp.getBits());
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   201
                        long maxValue = CodeUtil.maxValue(xStamp.getBits());
46371
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
   202
52578
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   203
                        if (!subtractMayUnderflow(xStamp.lowerBound(), yStamp.upperBound(), minValue) && !subtractMayOverflow(xStamp.upperBound(), yStamp.lowerBound(), maxValue)) {
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   204
                            LogicNode logic = new IntegerLessThanNode(xx, yy);
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   205
                            if (negate) {
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   206
                                logic = LogicNegationNode.create(logic);
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   207
                            }
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   208
                            return logic;
46371
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
   209
                        }
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
   210
                    }
52578
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   211
                } else if (forX instanceof AddNode) {
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   212
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   213
                    // (x + xConstant) < forY => x < (forY - xConstant)
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   214
                    AddNode addNode = (AddNode) forX;
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   215
                    if (addNode.getY().isJavaConstant()) {
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   216
                        IntegerStamp xStamp = (IntegerStamp) addNode.getX().stamp(view);
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   217
                        if (!IntegerStamp.addCanOverflow(xStamp, (IntegerStamp) addNode.getY().stamp(view))) {
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   218
                            long minValue = CodeUtil.minValue(xStamp.getBits());
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   219
                            long maxValue = CodeUtil.maxValue(xStamp.getBits());
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   220
                            long yConstant = forY.asJavaConstant().asLong();
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   221
                            long xConstant = addNode.getY().asJavaConstant().asLong();
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   222
                            if (!subtractMayUnderflow(yConstant, xConstant, minValue) && !subtractMayOverflow(yConstant, xConstant, maxValue)) {
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   223
                                long newConstant = yConstant - xConstant;
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   224
                                return IntegerLessThanNode.create(addNode.getX(), ConstantNode.forIntegerStamp(xStamp, newConstant), view);
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   225
                            }
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   226
                        }
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   227
                    }
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   228
46371
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
   229
                }
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
   230
            }
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
   231
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   232
            if (forX.stamp(view) instanceof IntegerStamp) {
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   233
                assert forY.stamp(view) instanceof IntegerStamp;
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   234
                int bits = ((IntegerStamp) forX.stamp(view)).getBits();
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   235
                assert ((IntegerStamp) forY.stamp(view)).getBits() == bits;
52578
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   236
                LogicNode logic = canonicalizeRangeFlip(forX, forY, bits, true, view);
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   237
                if (logic != null) {
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   238
                    return logic;
46371
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
   239
                }
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
   240
            }
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
   241
            return null;
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
   242
        }
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
   243
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
   244
        @Override
48861
47f19ff9903c 8194819: Update Graal
iveresov
parents: 48190
diff changeset
   245
        protected CanonicalCondition getCondition() {
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   246
            return LT;
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   247
        }
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   248
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   249
        @Override
46371
0337d0617e7b 8178088: Update Graal
iveresov
parents: 46344
diff changeset
   250
        protected IntegerLowerThanNode createNode(ValueNode x, ValueNode y) {
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   251
            return new IntegerLessThanNode(x, y);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   252
        }
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   253
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   254
        @Override
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   255
        protected long upperBound(IntegerStamp stamp) {
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   256
            return stamp.upperBound();
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   257
        }
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   258
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   259
        @Override
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   260
        protected long lowerBound(IntegerStamp stamp) {
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   261
            return stamp.lowerBound();
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   262
        }
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   263
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   264
        @Override
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   265
        protected int compare(long a, long b) {
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   266
            return Long.compare(a, b);
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   267
        }
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   268
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   269
        @Override
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   270
        protected long min(long a, long b) {
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   271
            return Math.min(a, b);
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   272
        }
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   273
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   274
        @Override
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   275
        protected long max(long a, long b) {
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   276
            return Math.max(a, b);
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   277
        }
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   278
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   279
        @Override
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   280
        protected long cast(long a, int bits) {
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   281
            return CodeUtil.signExtend(a, bits);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   282
        }
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   283
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   284
        @Override
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   285
        protected long minValue(int bits) {
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   286
            return NumUtil.minValue(bits);
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   287
        }
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   288
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   289
        @Override
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   290
        protected long maxValue(int bits) {
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   291
            return NumUtil.maxValue(bits);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   292
        }
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   293
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   294
        @Override
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   295
        protected IntegerStamp forInteger(int bits, long min, long max) {
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   296
            return StampFactory.forInteger(bits, cast(min, bits), cast(max, bits));
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   297
        }
43972
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
}