changeset 7667:10a72efd23bc

Clean up more TODO comments.
author Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
date Mon, 04 Feb 2013 13:21:07 +0100
parents 31540ca73e81
children d19837d236e5
files graal/com.oracle.graal.jtt/src/com/oracle/graal/jtt/bytecode/BC_l2i.java graal/com.oracle.graal.lir/src/com/oracle/graal/lir/StandardOp.java graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/calc/ShiftNode.java
diffstat 3 files changed, 0 insertions(+), 7 deletions(-) [+]
line wrap: on
line diff
--- 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) {
--- 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 {
 
--- 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;
     }
 }