--- /dev/null 2017-01-22 10:16:57.869617664 -0800 +++ new/src/jdk.internal.vm.compiler/share/classes/org.graalvm.compiler.nodes/src/org/graalvm/compiler/nodes/calc/AddNode.java 2017-02-15 17:06:55.538372377 -0800 @@ -0,0 +1,132 @@ +/* + * Copyright (c) 2011, 2015, Oracle and/or its affiliates. All rights reserved. + * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. + * + * This code is free software; you can redistribute it and/or modify it + * under the terms of the GNU General Public License version 2 only, as + * published by the Free Software Foundation. + * + * This code is distributed in the hope that it will be useful, but WITHOUT + * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or + * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License + * version 2 for more details (a copy is included in the LICENSE file that + * accompanied this code). + * + * You should have received a copy of the GNU General Public License version + * 2 along with this work; if not, write to the Free Software Foundation, + * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. + * + * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA + * or visit www.oracle.com if you need additional information or have any + * questions. + */ +package org.graalvm.compiler.nodes.calc; + +import org.graalvm.compiler.core.common.type.ArithmeticOpTable; +import org.graalvm.compiler.core.common.type.ArithmeticOpTable.BinaryOp; +import org.graalvm.compiler.core.common.type.ArithmeticOpTable.BinaryOp.Add; +import org.graalvm.compiler.core.common.type.Stamp; +import org.graalvm.compiler.graph.NodeClass; +import org.graalvm.compiler.graph.spi.Canonicalizable.BinaryCommutative; +import org.graalvm.compiler.graph.spi.CanonicalizerTool; +import org.graalvm.compiler.lir.gen.ArithmeticLIRGeneratorTool; +import org.graalvm.compiler.nodeinfo.NodeInfo; +import org.graalvm.compiler.nodes.ConstantNode; +import org.graalvm.compiler.nodes.ValueNode; +import org.graalvm.compiler.nodes.spi.NodeLIRBuilderTool; + +import jdk.vm.ci.meta.Constant; +import jdk.vm.ci.meta.Value; + +@NodeInfo(shortName = "+") +public class AddNode extends BinaryArithmeticNode implements NarrowableArithmeticNode, BinaryCommutative { + + public static final NodeClass TYPE = NodeClass.create(AddNode.class); + + public AddNode(ValueNode x, ValueNode y) { + this(TYPE, x, y); + } + + protected AddNode(NodeClass c, ValueNode x, ValueNode y) { + super(c, ArithmeticOpTable::getAdd, x, y); + } + + public static ValueNode create(ValueNode x, ValueNode y) { + BinaryOp op = ArithmeticOpTable.forStamp(x.stamp()).getAdd(); + Stamp stamp = op.foldStamp(x.stamp(), y.stamp()); + ConstantNode tryConstantFold = tryConstantFold(op, x, y, stamp); + if (tryConstantFold != null) { + return tryConstantFold; + } else { + return new AddNode(x, y).maybeCommuteInputs(); + } + } + + @Override + public ValueNode canonical(CanonicalizerTool tool, ValueNode forX, ValueNode forY) { + ValueNode ret = super.canonical(tool, forX, forY); + if (ret != this) { + return ret; + } + + if (forX.isConstant() && !forY.isConstant()) { + // we try to swap and canonicalize + ValueNode improvement = canonical(tool, forY, forX); + if (improvement != this) { + return improvement; + } + // if this fails we only swap + return new AddNode(forY, forX); + } + BinaryOp op = getOp(forX, forY); + boolean associative = op.isAssociative(); + if (associative) { + if (forX instanceof SubNode) { + SubNode sub = (SubNode) forX; + if (sub.getY() == forY) { + // (a - b) + b + return sub.getX(); + } + } + if (forY instanceof SubNode) { + SubNode sub = (SubNode) forY; + if (sub.getY() == forX) { + // b + (a - b) + return sub.getX(); + } + } + } + if (forY.isConstant()) { + Constant c = forY.asConstant(); + if (op.isNeutral(c)) { + return forX; + } + if (associative) { + // canonicalize expressions like "(a + 1) + 2" + BinaryNode reassociated = reassociate(this, ValueNode.isConstantPredicate(), forX, forY); + if (reassociated != this) { + return reassociated; + } + } + } + if (forX instanceof NegateNode) { + return BinaryArithmeticNode.sub(forY, ((NegateNode) forX).getValue()); + } else if (forY instanceof NegateNode) { + return BinaryArithmeticNode.sub(forX, ((NegateNode) forY).getValue()); + } + return this; + } + + @Override + public void generate(NodeLIRBuilderTool nodeValueMap, ArithmeticLIRGeneratorTool gen) { + Value op1 = nodeValueMap.operand(getX()); + assert op1 != null : getX() + ", this=" + this; + Value op2 = nodeValueMap.operand(getY()); + if (shouldSwapInputs(nodeValueMap)) { + Value tmp = op1; + op1 = op2; + op2 = tmp; + } + nodeValueMap.setResult(this, gen.emitAdd(op1, op2, false)); + } +}