# HG changeset patch # User Thomas Wuerthinger # Date 1359980467 -3600 # Node ID 10a72efd23bc49c993b90241606af700b85a2acd # Parent 31540ca73e81e913a8ccff83bfd48ca2fc93aa4b Clean up more TODO comments. diff -r 31540ca73e81 -r 10a72efd23bc graal/com.oracle.graal.jtt/src/com/oracle/graal/jtt/bytecode/BC_l2i.java --- a/graal/com.oracle.graal.jtt/src/com/oracle/graal/jtt/bytecode/BC_l2i.java Fri Feb 01 19:53:52 2013 +0100 +++ b/graal/com.oracle.graal.jtt/src/com/oracle/graal/jtt/bytecode/BC_l2i.java Mon Feb 04 13:21:07 2013 +0100 @@ -25,9 +25,6 @@ import com.oracle.graal.jtt.*; import org.junit.*; -/* - * TODO: test roundoff behavior - */ public class BC_l2i extends JTTTest { public static int test(long a) { diff -r 31540ca73e81 -r 10a72efd23bc graal/com.oracle.graal.lir/src/com/oracle/graal/lir/StandardOp.java --- a/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/StandardOp.java Fri Feb 01 19:53:52 2013 +0100 +++ b/graal/com.oracle.graal.lir/src/com/oracle/graal/lir/StandardOp.java Mon Feb 04 13:21:07 2013 +0100 @@ -78,8 +78,6 @@ * LIR operation that is an unconditional jump to {@link #destination()}. When the LIR is * constructed, the last operation of every block must implement this interface. After register * allocation, unnecessary jumps can be deleted. - * - * TODO (cwimmer) Currently, a block can also end with an XIR operation. */ public static class JumpOp extends LIRInstruction { diff -r 31540ca73e81 -r 10a72efd23bc graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/calc/ShiftNode.java --- a/graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/calc/ShiftNode.java Fri Feb 01 19:53:52 2013 +0100 +++ b/graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/calc/ShiftNode.java Mon Feb 04 13:21:07 2013 +0100 @@ -38,7 +38,5 @@ */ public ShiftNode(Kind kind, ValueNode x, ValueNode s) { super(kind, x, s); - // TODO (cwimmer) Why check for null here - what is a shift with no left operand? - assert x == null || x.kind() == kind; } }