[llvm-commits] CVS: llvm/lib/Target/SparcV8/InstSelectSimple.cpp

[Brian Gaeke]

Changes in directory llvm/lib/Target/SparcV8:

InstSelectSimple.cpp updated: 1.4 -> 1.5

---
Log message:

Support return values of basic integer types.
Emit RETL instruction to return instead of funny JMPL.
Fix indentation.


---
Diffs of the changes:  (+37 -24)

Index: llvm/lib/Target/SparcV8/InstSelectSimple.cpp
diff -u llvm/lib/Target/SparcV8/InstSelectSimple.cpp:1.4 llvm/lib/Target/SparcV8/InstSelectSimple.cpp:1.5
--- llvm/lib/Target/SparcV8/InstSelectSimple.cpp:1.4	Wed Mar  3 22:37:45 2004
+++ llvm/lib/Target/SparcV8/InstSelectSimple.cpp	Fri Mar  5 23:32:28 2004
@@ -227,12 +227,25 @@
 
 
 void V8ISel::visitReturnInst(ReturnInst &I) {
-  if (I.getNumOperands() == 0) {
-    // Just emit a 'jmpl' instruction to return.
-    BuildMI(BB, V8::JMPLi, 2, V8::G0).addZImm(8).addReg(V8::I7);
-    return;
+  if (I.getNumOperands () == 1) {
+    unsigned RetValReg = getReg (I.getOperand (0));
+    switch (getClass (I.getOperand (0)->getType ())) {
+      case cByte:
+      case cShort:
+      case cInt:
+        // Schlep it over into i0 (where it will become o0 after restore).
+        BuildMI (BB, V8::ORrr, 2, V8::I0).addReg(V8::G0).addReg(RetValReg);
+        break;
+      default:
+        visitInstruction (I);
+        return;
+    }
+  } else if (I.getNumOperands () != 1) {
+    visitInstruction (I);
   }
-  visitInstruction(I);
+  // Just emit a 'retl' instruction to return.
+  BuildMI(BB, V8::RETL, 0);
+  return;
 }
 
 void V8ISel::visitBinaryOperator (BinaryOperator &I) {
@@ -249,36 +262,36 @@
       BuildMI (BB, V8::SUBrr, 2, ResultReg).addReg (Op0Reg).addReg (Op1Reg);
       break;
     default:
-	  visitInstruction (I);
+      visitInstruction (I);
       return;
   }
 
   switch (getClass (I.getType ())) {
     case cByte: 
-	  if (I.getType ()->isSigned ()) { // add byte
-		BuildMI (BB, V8::ANDri, 2, DestReg).addReg (ResultReg).addZImm (0xff);
-	  } else { // add ubyte
-		unsigned TmpReg = makeAnotherReg (I.getType ());
-		BuildMI (BB, V8::SLLri, 2, TmpReg).addReg (ResultReg).addZImm (24);
-		BuildMI (BB, V8::SRAri, 2, DestReg).addReg (TmpReg).addZImm (24);
-	  }
+      if (I.getType ()->isSigned ()) { // add byte
+        BuildMI (BB, V8::ANDri, 2, DestReg).addReg (ResultReg).addZImm (0xff);
+      } else { // add ubyte
+        unsigned TmpReg = makeAnotherReg (I.getType ());
+        BuildMI (BB, V8::SLLri, 2, TmpReg).addReg (ResultReg).addZImm (24);
+        BuildMI (BB, V8::SRAri, 2, DestReg).addReg (TmpReg).addZImm (24);
+      }
       break;
     case cShort:
-	  if (I.getType ()->isSigned ()) { // add short
-		unsigned TmpReg = makeAnotherReg (I.getType ());
-		BuildMI (BB, V8::SLLri, 2, TmpReg).addReg (ResultReg).addZImm (16);
-		BuildMI (BB, V8::SRAri, 2, DestReg).addReg (TmpReg).addZImm (16);
-	  } else { // add ushort
-		unsigned TmpReg = makeAnotherReg (I.getType ());
-		BuildMI (BB, V8::SLLri, 2, TmpReg).addReg (ResultReg).addZImm (24);
-		BuildMI (BB, V8::SRLri, 2, DestReg).addReg (TmpReg).addZImm (24);
-	  }
+      if (I.getType ()->isSigned ()) { // add short
+        unsigned TmpReg = makeAnotherReg (I.getType ());
+        BuildMI (BB, V8::SLLri, 2, TmpReg).addReg (ResultReg).addZImm (16);
+        BuildMI (BB, V8::SRAri, 2, DestReg).addReg (TmpReg).addZImm (16);
+      } else { // add ushort
+        unsigned TmpReg = makeAnotherReg (I.getType ());
+        BuildMI (BB, V8::SLLri, 2, TmpReg).addReg (ResultReg).addZImm (24);
+        BuildMI (BB, V8::SRLri, 2, DestReg).addReg (TmpReg).addZImm (24);
+      }
       break;
     case cInt:
-	  BuildMI (BB, V8::ORrr, 2, DestReg).addReg (V8::G0).addReg (ResultReg);
+      BuildMI (BB, V8::ORrr, 2, DestReg).addReg (V8::G0).addReg (ResultReg);
       break;
     default:
-	  visitInstruction (I);
+      visitInstruction (I);
       return;
   }
 }