Mercurial > hg > truffle

changeset 16942:ae0f235469db

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
[SPARC] Fix findbugs/checkstyle warnings
author Stefan Anzinger <stefan.anzinger@gmail.com>
date Mon, 25 Aug 2014 14:42:42 -0700
parents 4e3b63e7a9f6
children f8e60d11c0ec
files graal/com.oracle.graal.asm.sparc/src/com/oracle/graal/asm/sparc/SPARCMacroAssembler.java graal/com.oracle.graal.compiler.sparc.test/src/com/oracle/graal/compiler/sparc/test/SPARCAllocatorTest.java graal/com.oracle.graal.compiler.sparc/src/com/oracle/graal/compiler/sparc/SPARCLIRGenerator.java graal/com.oracle.graal.jtt/src/com/oracle/graal/jtt/micro/BigMixedParams04.java graal/com.oracle.graal.lir.sparc/src/com/oracle/graal/lir/sparc/SPARCControlFlow.java
diffstat 5 files changed, 4 insertions(+), 6 deletions(-) [+]
line wrap: on
line diff
--- a/graal/com.oracle.graal.asm.sparc/src/com/oracle/graal/asm/sparc/SPARCMacroAssembler.java	Mon Aug 25 11:32:04 2014 -0700
+++ b/graal/com.oracle.graal.asm.sparc/src/com/oracle/graal/asm/sparc/SPARCMacroAssembler.java	Mon Aug 25 14:42:42 2014 -0700
@@ -27,7 +27,6 @@
 
 import com.oracle.graal.api.code.*;
 import com.oracle.graal.asm.*;
-import com.oracle.graal.sparc.*;
 
 public class SPARCMacroAssembler extends SPARCAssembler {
 
--- a/graal/com.oracle.graal.compiler.sparc.test/src/com/oracle/graal/compiler/sparc/test/SPARCAllocatorTest.java	Mon Aug 25 11:32:04 2014 -0700
+++ b/graal/com.oracle.graal.compiler.sparc.test/src/com/oracle/graal/compiler/sparc/test/SPARCAllocatorTest.java	Mon Aug 25 14:42:42 2014 -0700
@@ -55,7 +55,7 @@
 
     @Test
     public void test3() {
-        test("test3snippet", 3, 0, 0);
+        test("test3snippet", 3, 1, 0);
     }
 
     public static long test3snippet(long x) {
--- a/graal/com.oracle.graal.compiler.sparc/src/com/oracle/graal/compiler/sparc/SPARCLIRGenerator.java	Mon Aug 25 11:32:04 2014 -0700
+++ b/graal/com.oracle.graal.compiler.sparc/src/com/oracle/graal/compiler/sparc/SPARCLIRGenerator.java	Mon Aug 25 14:42:42 2014 -0700
@@ -708,6 +708,7 @@
                 op = IUDIV;
                 actualA = emitZeroExtend(actualA, 32, 64);
                 actualB = emitZeroExtend(actualB, 32, 64);
+                break;
             case Long:
                 op = LUDIV;
                 break;
--- a/graal/com.oracle.graal.jtt/src/com/oracle/graal/jtt/micro/BigMixedParams04.java	Mon Aug 25 11:32:04 2014 -0700
+++ b/graal/com.oracle.graal.jtt/src/com/oracle/graal/jtt/micro/BigMixedParams04.java	Mon Aug 25 14:42:42 2014 -0700
@@ -27,7 +27,6 @@
 
 import org.junit.*;
 
-import com.oracle.graal.compiler.common.*;
 import com.oracle.graal.jtt.*;
 
 /**
--- a/graal/com.oracle.graal.lir.sparc/src/com/oracle/graal/lir/sparc/SPARCControlFlow.java	Mon Aug 25 11:32:04 2014 -0700
+++ b/graal/com.oracle.graal.lir.sparc/src/com/oracle/graal/lir/sparc/SPARCControlFlow.java	Mon Aug 25 14:42:42 2014 -0700
@@ -31,7 +31,6 @@
 import com.oracle.graal.api.meta.*;
 import com.oracle.graal.asm.*;
 import com.oracle.graal.asm.sparc.*;
-import com.oracle.graal.asm.sparc.SPARCAssembler.*;
 import com.oracle.graal.asm.sparc.SPARCMacroAssembler.*;
 import com.oracle.graal.compiler.common.*;
 import com.oracle.graal.compiler.common.calc.*;
@@ -112,7 +111,7 @@
                 if (actualCondition != null) {
                     emitCompare(masm, actualTarget, actualCondition, cc);
                 } else if (actualConditionFlag != null) {
-                    emitCompare(masm, actualTarget, actualConditionFlag, cc);
+                    emitCompare(masm, actualTarget, actualConditionFlag);
                 } else {
                     GraalInternalError.shouldNotReachHere();
                 }
@@ -155,7 +154,7 @@
         new Nop().emit(masm);
     }
 
-    private static void emitCompare(SPARCMacroAssembler masm, Label target, ConditionFlag actualCondition, CC cc) {
+    private static void emitCompare(SPARCMacroAssembler masm, Label target, ConditionFlag actualCondition) {
         new Fmt00b(false, actualCondition, Op2s.Br, target).emit(masm);
     }