Fix bug in copying long constants to register pairs. We were getting

the top and bottom halves backwards...how embarrassing.
Support 'cast long to long' and other similar no-op casts to long.
Support 'ret long'.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@14683 91177308-0d34-0410-b5e6-96231b3b80d8

diff --git a/lib/Target/Sparc/InstSelectSimple.cpp b/lib/Target/Sparc/InstSelectSimple.cpp
index 982ac865325709c9b133c2e0d192b35943d4185d..43f2646d1e879809229c52bce7598ccdac18a0b3 100644 (file)
--- a/lib/Target/Sparc/InstSelectSimple.cpp
+++ b/lib/Target/Sparc/InstSelectSimple.cpp
@@ -232,8 +232,8 @@ void V8ISel::copyConstantToRegister(MachineBasicBlock *MBB,
       // Copy the value into the register pair.
       // R = top(more-significant) half, R+1 = bottom(less-significant) half
       uint64_t Val = cast<ConstantInt>(C)->getRawValue();
-      unsigned topHalf = Val & 0xffffffffU;
-      unsigned bottomHalf = Val >> 32;
+      unsigned bottomHalf = Val & 0xffffffffU;
+      unsigned topHalf = Val >> 32;
       unsigned HH = topHalf >> 10;
       unsigned HM = topHalf & 0x03ff;
       unsigned LM = bottomHalf >> 10;
@@ -565,6 +565,17 @@ void V8ISel::emitCastOperation(MachineBasicBlock *BB,
         break;
       }
       }
+    } else if (newTyClass == cLong) {
+      if (oldTyClass == cLong) {
+        // Just copy it
+        BuildMI (*BB, IP, V8::ORrr, 2, DestReg).addReg (V8::G0).addReg (SrcReg);
+        BuildMI (*BB, IP, V8::ORrr, 2, DestReg+1).addReg (V8::G0)
+          .addReg (SrcReg+1);
+      } else {
+        std::cerr << "Cast still unsupported: SrcTy = "
+                  << *SrcTy << ", DestTy = " << *DestTy << "\n";
+        abort ();
+      }
     } else {
       std::cerr << "Cast still unsupported: SrcTy = "
                 << *SrcTy << ", DestTy = " << *DestTy << "\n";
@@ -705,6 +716,10 @@ void V8ISel::visitReturnInst(ReturnInst &I) {
       case cFloat:
         BuildMI (BB, V8::FMOVS, 2, V8::F0).addReg(RetValReg);
         break;
+      case cLong:
+        BuildMI (BB, V8::ORrr, 2, V8::I0).addReg(V8::G0).addReg(RetValReg);
+        BuildMI (BB, V8::ORrr, 2, V8::I1).addReg(V8::G0).addReg(RetValReg+1);
+        break;
       default:
         std::cerr << "Return instruction of this type not handled: " << I;
         abort ();

diff --git a/lib/Target/Sparc/SparcV8ISelSimple.cpp b/lib/Target/Sparc/SparcV8ISelSimple.cpp
index 982ac865325709c9b133c2e0d192b35943d4185d..43f2646d1e879809229c52bce7598ccdac18a0b3 100644 (file)
--- a/lib/Target/Sparc/SparcV8ISelSimple.cpp
+++ b/lib/Target/Sparc/SparcV8ISelSimple.cpp
@@ -232,8 +232,8 @@ void V8ISel::copyConstantToRegister(MachineBasicBlock *MBB,
       // Copy the value into the register pair.
       // R = top(more-significant) half, R+1 = bottom(less-significant) half
       uint64_t Val = cast<ConstantInt>(C)->getRawValue();
-      unsigned topHalf = Val & 0xffffffffU;
-      unsigned bottomHalf = Val >> 32;
+      unsigned bottomHalf = Val & 0xffffffffU;
+      unsigned topHalf = Val >> 32;
       unsigned HH = topHalf >> 10;
       unsigned HM = topHalf & 0x03ff;
       unsigned LM = bottomHalf >> 10;
@@ -565,6 +565,17 @@ void V8ISel::emitCastOperation(MachineBasicBlock *BB,
         break;
       }
       }
+    } else if (newTyClass == cLong) {
+      if (oldTyClass == cLong) {
+        // Just copy it
+        BuildMI (*BB, IP, V8::ORrr, 2, DestReg).addReg (V8::G0).addReg (SrcReg);
+        BuildMI (*BB, IP, V8::ORrr, 2, DestReg+1).addReg (V8::G0)
+          .addReg (SrcReg+1);
+      } else {
+        std::cerr << "Cast still unsupported: SrcTy = "
+                  << *SrcTy << ", DestTy = " << *DestTy << "\n";
+        abort ();
+      }
     } else {
       std::cerr << "Cast still unsupported: SrcTy = "
                 << *SrcTy << ", DestTy = " << *DestTy << "\n";
@@ -705,6 +716,10 @@ void V8ISel::visitReturnInst(ReturnInst &I) {
       case cFloat:
         BuildMI (BB, V8::FMOVS, 2, V8::F0).addReg(RetValReg);
         break;
+      case cLong:
+        BuildMI (BB, V8::ORrr, 2, V8::I0).addReg(V8::G0).addReg(RetValReg);
+        BuildMI (BB, V8::ORrr, 2, V8::I1).addReg(V8::G0).addReg(RetValReg+1);
+        break;
       default:
         std::cerr << "Return instruction of this type not handled: " << I;
         abort ();

diff --git a/lib/Target/SparcV8/InstSelectSimple.cpp b/lib/Target/SparcV8/InstSelectSimple.cpp
index 982ac865325709c9b133c2e0d192b35943d4185d..43f2646d1e879809229c52bce7598ccdac18a0b3 100644 (file)
--- a/lib/Target/SparcV8/InstSelectSimple.cpp
+++ b/lib/Target/SparcV8/InstSelectSimple.cpp
@@ -232,8 +232,8 @@ void V8ISel::copyConstantToRegister(MachineBasicBlock *MBB,
       // Copy the value into the register pair.
       // R = top(more-significant) half, R+1 = bottom(less-significant) half
       uint64_t Val = cast<ConstantInt>(C)->getRawValue();
-      unsigned topHalf = Val & 0xffffffffU;
-      unsigned bottomHalf = Val >> 32;
+      unsigned bottomHalf = Val & 0xffffffffU;
+      unsigned topHalf = Val >> 32;
       unsigned HH = topHalf >> 10;
       unsigned HM = topHalf & 0x03ff;
       unsigned LM = bottomHalf >> 10;
@@ -565,6 +565,17 @@ void V8ISel::emitCastOperation(MachineBasicBlock *BB,
         break;
       }
       }
+    } else if (newTyClass == cLong) {
+      if (oldTyClass == cLong) {
+        // Just copy it
+        BuildMI (*BB, IP, V8::ORrr, 2, DestReg).addReg (V8::G0).addReg (SrcReg);
+        BuildMI (*BB, IP, V8::ORrr, 2, DestReg+1).addReg (V8::G0)
+          .addReg (SrcReg+1);
+      } else {
+        std::cerr << "Cast still unsupported: SrcTy = "
+                  << *SrcTy << ", DestTy = " << *DestTy << "\n";
+        abort ();
+      }
     } else {
       std::cerr << "Cast still unsupported: SrcTy = "
                 << *SrcTy << ", DestTy = " << *DestTy << "\n";
@@ -705,6 +716,10 @@ void V8ISel::visitReturnInst(ReturnInst &I) {
       case cFloat:
         BuildMI (BB, V8::FMOVS, 2, V8::F0).addReg(RetValReg);
         break;
+      case cLong:
+        BuildMI (BB, V8::ORrr, 2, V8::I0).addReg(V8::G0).addReg(RetValReg);
+        BuildMI (BB, V8::ORrr, 2, V8::I1).addReg(V8::G0).addReg(RetValReg+1);
+        break;
       default:
         std::cerr << "Return instruction of this type not handled: " << I;
         abort ();

diff --git a/lib/Target/SparcV8/SparcV8ISelSimple.cpp b/lib/Target/SparcV8/SparcV8ISelSimple.cpp
index 982ac865325709c9b133c2e0d192b35943d4185d..43f2646d1e879809229c52bce7598ccdac18a0b3 100644 (file)
--- a/lib/Target/SparcV8/SparcV8ISelSimple.cpp
+++ b/lib/Target/SparcV8/SparcV8ISelSimple.cpp
@@ -232,8 +232,8 @@ void V8ISel::copyConstantToRegister(MachineBasicBlock *MBB,
       // Copy the value into the register pair.
       // R = top(more-significant) half, R+1 = bottom(less-significant) half
       uint64_t Val = cast<ConstantInt>(C)->getRawValue();
-      unsigned topHalf = Val & 0xffffffffU;
-      unsigned bottomHalf = Val >> 32;
+      unsigned bottomHalf = Val & 0xffffffffU;
+      unsigned topHalf = Val >> 32;
       unsigned HH = topHalf >> 10;
       unsigned HM = topHalf & 0x03ff;
       unsigned LM = bottomHalf >> 10;
@@ -565,6 +565,17 @@ void V8ISel::emitCastOperation(MachineBasicBlock *BB,
         break;
       }
       }
+    } else if (newTyClass == cLong) {
+      if (oldTyClass == cLong) {
+        // Just copy it
+        BuildMI (*BB, IP, V8::ORrr, 2, DestReg).addReg (V8::G0).addReg (SrcReg);
+        BuildMI (*BB, IP, V8::ORrr, 2, DestReg+1).addReg (V8::G0)
+          .addReg (SrcReg+1);
+      } else {
+        std::cerr << "Cast still unsupported: SrcTy = "
+                  << *SrcTy << ", DestTy = " << *DestTy << "\n";
+        abort ();
+      }
     } else {
       std::cerr << "Cast still unsupported: SrcTy = "
                 << *SrcTy << ", DestTy = " << *DestTy << "\n";
@@ -705,6 +716,10 @@ void V8ISel::visitReturnInst(ReturnInst &I) {
       case cFloat:
         BuildMI (BB, V8::FMOVS, 2, V8::F0).addReg(RetValReg);
         break;
+      case cLong:
+        BuildMI (BB, V8::ORrr, 2, V8::I0).addReg(V8::G0).addReg(RetValReg);
+        BuildMI (BB, V8::ORrr, 2, V8::I1).addReg(V8::G0).addReg(RetValReg+1);
+        break;
       default:
         std::cerr << "Return instruction of this type not handled: " << I;
         abort ();