# HG changeset patch # User Roland Schatz # Date 1432216382 -7200 # Node ID a0ae5a2ac5cf6e9bd533344391e784158bd5cdfb # Parent 24dd9af9ac26fba1f6db549281a7a04526158817 Support detection of long induction variables. diff -r 24dd9af9ac26 -r a0ae5a2ac5cf graal/com.oracle.graal.loop/src/com/oracle/graal/loop/DerivedConvertedInductionVariable.java --- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/graal/com.oracle.graal.loop/src/com/oracle/graal/loop/DerivedConvertedInductionVariable.java Thu May 21 15:53:02 2015 +0200 @@ -0,0 +1,108 @@ +/* + * Copyright (c) 2015, 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 com.oracle.graal.loop; + +import com.oracle.graal.compiler.common.type.*; +import com.oracle.graal.nodes.*; +import com.oracle.graal.nodes.calc.*; + +public class DerivedConvertedInductionVariable extends DerivedInductionVariable { + + private final Stamp stamp; + private final ValueNode value; + + public DerivedConvertedInductionVariable(LoopEx loop, InductionVariable base, Stamp stamp, ValueNode value) { + super(loop, base); + this.stamp = stamp; + this.value = value; + } + + @Override + public ValueNode valueNode() { + return value; + } + + @Override + public Direction direction() { + return base.direction(); + } + + @Override + public ValueNode initNode() { + return IntegerConvertNode.convert(base.initNode(), stamp, graph()); + } + + @Override + public ValueNode strideNode() { + return IntegerConvertNode.convert(base.strideNode(), stamp, graph()); + } + + @Override + public boolean isConstantInit() { + return base.isConstantInit(); + } + + @Override + public boolean isConstantStride() { + return base.isConstantStride(); + } + + @Override + public long constantInit() { + return base.constantInit(); + } + + @Override + public long constantStride() { + return base.constantStride(); + } + + @Override + public ValueNode extremumNode(boolean assumePositiveTripCount, Stamp s) { + return base.extremumNode(assumePositiveTripCount, s); + } + + @Override + public ValueNode exitValueNode() { + return IntegerConvertNode.convert(base.exitValueNode(), stamp, graph()); + } + + @Override + public boolean isConstantExtremum() { + return base.isConstantExtremum(); + } + + @Override + public long constantExtremum() { + return base.constantExtremum(); + } + + @Override + public void deleteUnusedNodes() { + } + + @Override + public String toString() { + return String.format("DerivedConvertedInductionVariable base (%s) %s %s", base, value.getNodeClass().shortName(), stamp); + } +} diff -r 24dd9af9ac26 -r a0ae5a2ac5cf graal/com.oracle.graal.loop/src/com/oracle/graal/loop/DerivedScaledInductionVariable.java --- a/graal/com.oracle.graal.loop/src/com/oracle/graal/loop/DerivedScaledInductionVariable.java Thu May 21 15:00:11 2015 +0200 +++ b/graal/com.oracle.graal.loop/src/com/oracle/graal/loop/DerivedScaledInductionVariable.java Thu May 21 15:53:02 2015 +0200 @@ -42,7 +42,7 @@ public DerivedScaledInductionVariable(LoopEx loop, InductionVariable base, NegateNode value) { super(loop, base); - this.scale = ConstantNode.forInt(-1, value.graph()); + this.scale = ConstantNode.forIntegerStamp(value.stamp(), -1, value.graph()); this.value = value; } diff -r 24dd9af9ac26 -r a0ae5a2ac5cf graal/com.oracle.graal.loop/src/com/oracle/graal/loop/LoopEx.java --- a/graal/com.oracle.graal.loop/src/com/oracle/graal/loop/LoopEx.java Thu May 21 15:00:11 2015 +0200 +++ b/graal/com.oracle.graal.loop/src/com/oracle/graal/loop/LoopEx.java Thu May 21 15:53:02 2015 +0200 @@ -1,5 +1,5 @@ /* - * Copyright (c) 2012, 2012, Oracle and/or its affiliates. All rights reserved. + * Copyright (c) 2012, 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 @@ -333,6 +333,16 @@ iv = new DerivedScaledInductionVariable(loop, baseIv, (NegateNode) op); } else if ((scale = mul(loop, op, baseIvNode)) != null) { iv = new DerivedScaledInductionVariable(loop, baseIv, scale, op); + } else { + boolean isValidConvert = op instanceof PiNode || op instanceof SignExtendNode; + if (!isValidConvert && op instanceof ZeroExtendNode) { + IntegerStamp inputStamp = (IntegerStamp) ((ZeroExtendNode) op).getValue().stamp(); + isValidConvert = inputStamp.isPositive(); + } + + if (isValidConvert) { + iv = new DerivedConvertedInductionVariable(loop, baseIv, op.stamp(), op); + } } if (iv != null) { @@ -368,7 +378,7 @@ if (op instanceof LeftShiftNode) { LeftShiftNode shift = (LeftShiftNode) op; if (shift.getX() == base && shift.getY().isConstant()) { - return ConstantNode.forInt(1 << shift.getY().asJavaConstant().asInt(), base.graph()); + return ConstantNode.forIntegerStamp(base.stamp(), 1 << shift.getY().asJavaConstant().asInt(), base.graph()); } } return null;