commit f7cedec1f2c6a8b054e54f98922d5d295d086cff
author Vikram S. Adve <vadve@cs.uiuc.edu> Sun Mar 31 00:13:12 2002 +0000
committer Vikram S. Adve <vadve@cs.uiuc.edu> Sun Mar 31 00:13:12 2002 +0000
tree dede3552d0c9f970418e9b531ccd9b8c7b13884d
parent 072d3a02241a27473ee245b5aac783501d78161e

Fix CreateUIntSetInstruction() to handle sign extensions correctly.


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

lib/Target/SparcV9/SparcV9InstrInfo.cpp

diff --git a/lib/Target/SparcV9/SparcV9InstrInfo.cpp b/lib/Target/SparcV9/SparcV9InstrInfo.cpp
index 1317dc3..a44dc7f 100644
--- a/lib/Target/SparcV9/SparcV9InstrInfo.cpp
+++ b/lib/Target/SparcV9/SparcV9InstrInfo.cpp
@@ -45,8 +45,8 @@
   else
     {
       minstr = new MachineInstr(SETSW);
-      minstr->SetMachineOperandConst(0, MachineOperand::MO_SignExtendedImmed, C);
-      minstr->SetMachineOperandVal(1, MachineOperand::MO_VirtualRegister, dest);
+      minstr->SetMachineOperandConst(0,MachineOperand::MO_SignExtendedImmed,C);
+      minstr->SetMachineOperandVal(1, MachineOperand::MO_VirtualRegister,dest);
     }
   
   return minstr;
@@ -59,20 +59,50 @@
   MachineInstr* minstr;
   if (C > (unsigned int) ~0)
     { // C does not fit in 32 bits
+      assert(dest->getType() == Type::ULongTy && "Sign extension problems");
       TmpInstruction *tmpReg = new TmpInstruction(Type::IntTy);
       tempVec.push_back(tmpReg);
       
       minstr = new MachineInstr(SETX);
-      minstr->SetMachineOperandConst(0, MachineOperand::MO_SignExtendedImmed, C);
-      minstr->SetMachineOperandVal(1, MachineOperand::MO_VirtualRegister, tmpReg,
-                                   /*isdef*/ true);
+      minstr->SetMachineOperandConst(0,MachineOperand::MO_SignExtendedImmed,C);
+      minstr->SetMachineOperandVal(1, MachineOperand::MO_VirtualRegister,
+                                   tmpReg, /*isdef*/ true);
       minstr->SetMachineOperandVal(2, MachineOperand::MO_VirtualRegister,dest);
     }
-  else
+  else if (dest->getType() == Type::ULongTy)
     {
       minstr = new MachineInstr(SETUW);
       minstr->SetMachineOperandConst(0, MachineOperand::MO_UnextendedImmed, C);
-      minstr->SetMachineOperandVal(1, MachineOperand::MO_VirtualRegister, dest);
+      minstr->SetMachineOperandVal(1, MachineOperand::MO_VirtualRegister,dest);
+    }
+  else
+    { // cast to signed type of the right length and use signed op (SETSW)
+      // to get correct sign extension
+      // 
+      minstr = new MachineInstr(SETSW);
+      minstr->SetMachineOperandVal(1, MachineOperand::MO_VirtualRegister,dest);
+      
+      switch (dest->getType()->getPrimitiveID())
+        {
+        case Type::UIntTyID:
+          minstr->SetMachineOperandConst(0,
+                                         MachineOperand::MO_SignExtendedImmed,
+                                         (int) C);
+          break;
+        case Type::UShortTyID:
+          minstr->SetMachineOperandConst(0,
+                                         MachineOperand::MO_SignExtendedImmed,
+                                         (short) C);
+          break;
+        case Type::UByteTyID:
+          minstr->SetMachineOperandConst(0,
+                                         MachineOperand::MO_SignExtendedImmed,
+                                         (char) C);
+          break;
+        default:
+          assert(0 && "Unexpected unsigned type");
+          break;
+        }
     }
   
   return minstr;