[llvm-commits] CVS: llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp InstructionCombining.cpp

[Chris Lattner]

Changes in directory llvm/lib/Transforms/Scalar:

DeadStoreElimination.cpp updated: 1.9 -> 1.10
InstructionCombining.cpp updated: 1.296 -> 1.297
---
Log message:

Fix uint64_t -> unsigned VS warnings.


---
Diffs of the changes:  (+13 -12)

Index: llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp
diff -u llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp:1.9 llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp:1.10
--- llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp:1.9	Tue Dec 28 22:36:02 2004
+++ llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp	Sat Jan  8 13:42:22 2005
@@ -73,7 +73,7 @@
         unsigned Size = ~0U;
         if (!AI->isArrayAllocation() &&
             AI->getType()->getElementType()->isSized())
-          Size = TD.getTypeSize(AI->getType()->getElementType());
+          Size = (unsigned)TD.getTypeSize(AI->getType()->getElementType());
         KillLocs.add(AI, Size);
       }
   }
@@ -106,7 +106,7 @@
     // the stored location is already in the tracker, then this is a dead
     // store.  We can just delete it here, but while we're at it, we also
     // delete any trivially dead expression chains.
-    unsigned ValSize = TD.getTypeSize(I->getOperand(0)->getType());
+    unsigned ValSize = (unsigned)TD.getTypeSize(I->getOperand(0)->getType());
     Value *Ptr = I->getOperand(1);
 
     if (AliasSet *AS = KillLocs.getAliasSetForPointerIfExists(Ptr, ValSize))


Index: llvm/lib/Transforms/Scalar/InstructionCombining.cpp
diff -u llvm/lib/Transforms/Scalar/InstructionCombining.cpp:1.296 llvm/lib/Transforms/Scalar/InstructionCombining.cpp:1.297
--- llvm/lib/Transforms/Scalar/InstructionCombining.cpp:1.296	Sat Jan  1 10:22:27 2005
+++ llvm/lib/Transforms/Scalar/InstructionCombining.cpp	Sat Jan  8 13:42:22 2005
@@ -2274,7 +2274,7 @@
 
             if (LHSI->hasOneUse()) {
               // Otherwise strength reduce the shift into an and.
-              unsigned ShAmtVal = ShAmt->getValue();
+              unsigned ShAmtVal = (unsigned)ShAmt->getValue();
               unsigned TypeBits = CI->getType()->getPrimitiveSize()*8;
               uint64_t Val = (1ULL << (TypeBits-ShAmtVal))-1;
 
@@ -2317,7 +2317,7 @@
             }
               
             if (LHSI->hasOneUse() || CI->isNullValue()) {
-              unsigned ShAmtVal = ShAmt->getValue();
+              unsigned ShAmtVal = (unsigned)ShAmt->getValue();
 
               // Otherwise strength reduce the shift into an and.
               uint64_t Val = ~0ULL;          // All ones.
@@ -2931,8 +2931,8 @@
     if (ShiftInst *Op0SI = dyn_cast<ShiftInst>(Op0))
       if (ConstantUInt *ShiftAmt1C =
                                  dyn_cast<ConstantUInt>(Op0SI->getOperand(1))) {
-        unsigned ShiftAmt1 = ShiftAmt1C->getValue();
-        unsigned ShiftAmt2 = CUI->getValue();
+        unsigned ShiftAmt1 = (unsigned)ShiftAmt1C->getValue();
+        unsigned ShiftAmt2 = (unsigned)CUI->getValue();
         
         // Check for (A << c1) << c2   and   (A >> c1) >> c2
         if (I.getOpcode() == Op0SI->getOpcode()) {
@@ -3169,8 +3169,8 @@
         const Type *AllocElTy = AI->getAllocatedType();
         const Type *CastElTy = PTy->getElementType();
         if (AllocElTy->isSized() && CastElTy->isSized()) {
-          unsigned AllocElTySize = TD->getTypeSize(AllocElTy);
-          unsigned CastElTySize = TD->getTypeSize(CastElTy);
+          uint64_t AllocElTySize = TD->getTypeSize(AllocElTy);
+          uint64_t CastElTySize = TD->getTypeSize(CastElTy);
 
           // If the allocation is for an even multiple of the cast type size
           if (CastElTySize && (AllocElTySize % CastElTySize == 0)) {
@@ -4190,12 +4190,13 @@
       ConstantUInt *CU = cast<ConstantUInt>(I.getOperand());
       assert(CU->getValue() < STy->getNumElements() &&
              "Struct index out of range!");
+      unsigned El = (unsigned)CU->getValue();
       if (ConstantStruct *CS = dyn_cast<ConstantStruct>(C)) {
-        C = CS->getOperand(CU->getValue());
+        C = CS->getOperand(El);
       } else if (isa<ConstantAggregateZero>(C)) {
-	C = Constant::getNullValue(STy->getElementType(CU->getValue()));
+	C = Constant::getNullValue(STy->getElementType(El));
       } else if (isa<UndefValue>(C)) {
-	C = UndefValue::get(STy->getElementType(CU->getValue()));
+	C = UndefValue::get(STy->getElementType(El));
       } else {
         return 0;
       }
@@ -4203,7 +4204,7 @@
       const ArrayType *ATy = cast<ArrayType>(*I);
       if ((uint64_t)CI->getRawValue() >= ATy->getNumElements()) return 0;
       if (ConstantArray *CA = dyn_cast<ConstantArray>(C))
-        C = CA->getOperand(CI->getRawValue());
+        C = CA->getOperand((unsigned)CI->getRawValue());
       else if (isa<ConstantAggregateZero>(C))
         C = Constant::getNullValue(ATy->getElementType());
       else if (isa<UndefValue>(C))