changeset 13185:06d9596a909e

refactored all PTX reporting to use PTXTestBase.printReport()
author Doug Simon <doug.simon@oracle.com>
date Wed, 27 Nov 2013 20:14:35 +0100
parents 6531170a229f
children 32f606699ce5
files graal/com.oracle.graal.compiler.ptx.test/src/com/oracle/graal/compiler/ptx/test/ArrayPTXTest.java graal/com.oracle.graal.compiler.ptx.test/src/com/oracle/graal/compiler/ptx/test/BasicPTXTest.java graal/com.oracle.graal.compiler.ptx.test/src/com/oracle/graal/compiler/ptx/test/ControlPTXTest.java graal/com.oracle.graal.compiler.ptx.test/src/com/oracle/graal/compiler/ptx/test/FloatPTXTest.java graal/com.oracle.graal.compiler.ptx.test/src/com/oracle/graal/compiler/ptx/test/IntegerPTXTest.java graal/com.oracle.graal.compiler.ptx.test/src/com/oracle/graal/compiler/ptx/test/LogicPTXTest.java graal/com.oracle.graal.compiler.ptx.test/src/com/oracle/graal/compiler/ptx/test/PTXTestBase.java
diffstat 7 files changed, 17 insertions(+), 32 deletions(-) [+]
line wrap: on
line diff
--- a/graal/com.oracle.graal.compiler.ptx.test/src/com/oracle/graal/compiler/ptx/test/ArrayPTXTest.java	Wed Nov 27 20:13:58 2013 +0100
+++ b/graal/com.oracle.graal.compiler.ptx.test/src/com/oracle/graal/compiler/ptx/test/ArrayPTXTest.java	Wed Nov 27 20:14:35 2013 +0100
@@ -24,12 +24,12 @@
 
 import static com.oracle.graal.lir.ptx.ThreadDimension.*;
 
-import com.oracle.graal.lir.ptx.ParallelOver;
-import com.oracle.graal.lir.ptx.Warp;
+import java.lang.reflect.*;
+import java.util.*;
 
-import java.lang.reflect.Method;
-import java.util.Arrays;
-import org.junit.Test;
+import org.junit.*;
+
+import com.oracle.graal.lir.ptx.*;
 
 public class ArrayPTXTest extends PTXTestBase {
 
@@ -71,9 +71,7 @@
         for (Method m : ArrayPTXTest.class.getMethods()) {
             String name = m.getName();
             if (m.getAnnotation(Test.class) == null && name.startsWith("test")) {
-                // CheckStyle: stop system..print check
-                System.out.println(name + ": \n" + new String(test.compile(name).getTargetCode()));
-                // CheckStyle: resume system..print check
+                printReport(name + ": \n" + new String(test.compile(name).getTargetCode()));
             }
         }
     }
--- a/graal/com.oracle.graal.compiler.ptx.test/src/com/oracle/graal/compiler/ptx/test/BasicPTXTest.java	Wed Nov 27 20:13:58 2013 +0100
+++ b/graal/com.oracle.graal.compiler.ptx.test/src/com/oracle/graal/compiler/ptx/test/BasicPTXTest.java	Wed Nov 27 20:14:35 2013 +0100
@@ -22,9 +22,9 @@
  */
 package com.oracle.graal.compiler.ptx.test;
 
-import java.lang.reflect.Method;
+import java.lang.reflect.*;
 
-import org.junit.Test;
+import org.junit.*;
 
 /**
  * Test class for small Java methods compiled to PTX kernels.
@@ -51,9 +51,7 @@
         for (Method m : methods) {
             String name = m.getName();
             if (m.getAnnotation(Test.class) == null && name.startsWith("test")) {
-                // CheckStyle: stop system..print check
-                System.out.println(name + ": \n" + new String(test.compile(name).getTargetCode()));
-                // CheckStyle: resume system..print check
+                printReport(name + ": \n" + new String(test.compile(name).getTargetCode()));
             }
         }
     }
--- a/graal/com.oracle.graal.compiler.ptx.test/src/com/oracle/graal/compiler/ptx/test/ControlPTXTest.java	Wed Nov 27 20:13:58 2013 +0100
+++ b/graal/com.oracle.graal.compiler.ptx.test/src/com/oracle/graal/compiler/ptx/test/ControlPTXTest.java	Wed Nov 27 20:14:35 2013 +0100
@@ -22,9 +22,9 @@
  */
 package com.oracle.graal.compiler.ptx.test;
 
-import org.junit.*;
+import java.lang.reflect.*;
 
-import java.lang.reflect.Method;
+import org.junit.*;
 
 public class ControlPTXTest extends PTXTestBase {
 
@@ -214,9 +214,7 @@
         for (Method m : ControlPTXTest.class.getMethods()) {
             String name = m.getName();
             if (m.getAnnotation(Test.class) == null && name.startsWith("test")) {
-                // CheckStyle: stop system..print check
-                System.out.println(name + ": \n" + new String(test.compile(name).getTargetCode()));
-                // CheckStyle: resume system..print check
+                printReport(name + ": \n" + new String(test.compile(name).getTargetCode()));
             }
         }
     }
--- a/graal/com.oracle.graal.compiler.ptx.test/src/com/oracle/graal/compiler/ptx/test/FloatPTXTest.java	Wed Nov 27 20:13:58 2013 +0100
+++ b/graal/com.oracle.graal.compiler.ptx.test/src/com/oracle/graal/compiler/ptx/test/FloatPTXTest.java	Wed Nov 27 20:14:35 2013 +0100
@@ -366,9 +366,7 @@
         for (Method m : FloatPTXTest.class.getMethods()) {
             String name = m.getName();
             if (m.getAnnotation(Test.class) == null && name.startsWith("test") && name.startsWith("testRem") == false) {
-                // CheckStyle: stop system..print check
-                System.out.println(name + ": \n" + new String(test.compile(name).getTargetCode()));
-                // CheckStyle: resume system..print check
+                printReport(name + ": \n" + new String(test.compile(name).getTargetCode()));
             }
         }
     }
--- a/graal/com.oracle.graal.compiler.ptx.test/src/com/oracle/graal/compiler/ptx/test/IntegerPTXTest.java	Wed Nov 27 20:13:58 2013 +0100
+++ b/graal/com.oracle.graal.compiler.ptx.test/src/com/oracle/graal/compiler/ptx/test/IntegerPTXTest.java	Wed Nov 27 20:14:35 2013 +0100
@@ -352,9 +352,7 @@
         for (Method m : IntegerPTXTest.class.getMethods()) {
             String name = m.getName();
             if (m.getAnnotation(Test.class) == null && name.startsWith("test")) {
-                // CheckStyle: stop system..print check
-                System.out.println(name + ": \n" + new String(test.compile(name).getTargetCode()));
-                // CheckStyle: resume system..print check
+                printReport(name + ": \n" + new String(test.compile(name).getTargetCode()));
             }
         }
     }
--- a/graal/com.oracle.graal.compiler.ptx.test/src/com/oracle/graal/compiler/ptx/test/LogicPTXTest.java	Wed Nov 27 20:13:58 2013 +0100
+++ b/graal/com.oracle.graal.compiler.ptx.test/src/com/oracle/graal/compiler/ptx/test/LogicPTXTest.java	Wed Nov 27 20:14:35 2013 +0100
@@ -128,9 +128,7 @@
         for (Method m : LogicPTXTest.class.getMethods()) {
             String name = m.getName();
             if (m.getAnnotation(Test.class) == null && name.startsWith("test")) {
-                // CheckStyle: stop system..print check
-                System.out.println(name + ": \n" + new String(test.compile(name).getTargetCode()));
-                // CheckStyle: resume system..print check
+                printReport(name + ": \n" + new String(test.compile(name).getTargetCode()));
             }
         }
     }
--- a/graal/com.oracle.graal.compiler.ptx.test/src/com/oracle/graal/compiler/ptx/test/PTXTestBase.java	Wed Nov 27 20:13:58 2013 +0100
+++ b/graal/com.oracle.graal.compiler.ptx.test/src/com/oracle/graal/compiler/ptx/test/PTXTestBase.java	Wed Nov 27 20:14:35 2013 +0100
@@ -48,11 +48,8 @@
 
     private StructuredGraph sg;
 
-    public void printReport(String message) {
-        // CheckStyle: stop system..print check
-        System.out.println(message);
-        // CheckStyle: resume system..print check
-
+    public static void printReport(String message) {
+        TTY.println(message);
     }
 
     public PTXTestBase() {