[llvm-commits] CVS: llvm/lib/Transforms/ExprTypeConvert.cpp TransformInternals.cpp

[Chris Lattner]

Changes in directory llvm/lib/Transforms:

ExprTypeConvert.cpp updated: 1.99 -> 1.100
TransformInternals.cpp updated: 1.46 -> 1.47
---
Log message:

Fix VS warnings


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

Index: llvm/lib/Transforms/ExprTypeConvert.cpp
diff -u llvm/lib/Transforms/ExprTypeConvert.cpp:1.99 llvm/lib/Transforms/ExprTypeConvert.cpp:1.100
--- llvm/lib/Transforms/ExprTypeConvert.cpp:1.99	Tue Sep 28 09:42:44 2004
+++ llvm/lib/Transforms/ExprTypeConvert.cpp	Sat Jan  8 13:48:40 2005
@@ -53,9 +53,9 @@
   ExprType Expr = ClassifyExpr(MI->getArraySize());
 
   // Get information about the base datatype being allocated, before & after
-  int ReqTypeSize = TD.getTypeSize(Ty);
+  uint64_t ReqTypeSize = TD.getTypeSize(Ty);
   if (ReqTypeSize == 0) return false;
-  unsigned OldTypeSize = TD.getTypeSize(MI->getType()->getElementType());
+  uint64_t OldTypeSize = TD.getTypeSize(MI->getType()->getElementType());
 
   // Must have a scale or offset to analyze it...
   if (!Expr.Offset && !Expr.Scale && OldTypeSize == 1) return false;
@@ -66,8 +66,8 @@
 
   // The old type might not be of unit size, take old size into consideration
   // here...
-  int64_t Offset = OffsetVal * OldTypeSize;
-  int64_t Scale  = ScaleVal  * OldTypeSize;
+  uint64_t Offset = OffsetVal * OldTypeSize;
+  uint64_t Scale  = ScaleVal  * OldTypeSize;
   
   // In order to be successful, both the scale and the offset must be a multiple
   // of the requested data type's size.
@@ -92,8 +92,8 @@
   const PointerType *AllocTy = cast<PointerType>(Ty);
   const Type *ElType = AllocTy->getElementType();
 
-  unsigned DataSize = TD.getTypeSize(ElType);
-  unsigned OldTypeSize = TD.getTypeSize(MI->getType()->getElementType());
+  uint64_t DataSize = TD.getTypeSize(ElType);
+  uint64_t OldTypeSize = TD.getTypeSize(MI->getType()->getElementType());
 
   // Get the offset and scale coefficients that we are allocating...
   int64_t OffsetVal = (Expr.Offset ? getConstantValue(Expr.Offset) : 0);
@@ -101,8 +101,8 @@
 
   // The old type might not be of unit size, take old size into consideration
   // here...
-  unsigned Offset = (uint64_t)OffsetVal * OldTypeSize / DataSize;
-  unsigned Scale  = (uint64_t)ScaleVal  * OldTypeSize / DataSize;
+  unsigned Offset = OffsetVal * OldTypeSize / DataSize;
+  unsigned Scale  = ScaleVal  * OldTypeSize / DataSize;
 
   // Locate the malloc instruction, because we may be inserting instructions
   It = MI;
@@ -784,7 +784,7 @@
     //
     if (I->getNumOperands() == 2) {
       const Type *OldElTy = cast<PointerType>(I->getType())->getElementType();
-      unsigned DataSize = TD.getTypeSize(OldElTy);
+      uint64_t DataSize = TD.getTypeSize(OldElTy);
       Value *Index = I->getOperand(1);
       Instruction *TempScale = 0;
 
@@ -1103,7 +1103,7 @@
     //
     BasicBlock::iterator It = I;
     const Type *OldElTy = cast<PointerType>(I->getType())->getElementType();
-    unsigned DataSize = TD.getTypeSize(OldElTy);
+    uint64_t DataSize = TD.getTypeSize(OldElTy);
     Value *Index = I->getOperand(1);
 
     if (DataSize != 1) {


Index: llvm/lib/Transforms/TransformInternals.cpp
diff -u llvm/lib/Transforms/TransformInternals.cpp:1.46 llvm/lib/Transforms/TransformInternals.cpp:1.47
--- llvm/lib/Transforms/TransformInternals.cpp:1.46	Thu Jul 29 07:17:33 2004
+++ llvm/lib/Transforms/TransformInternals.cpp	Sat Jan  8 13:48:40 2005
@@ -73,7 +73,7 @@
            "Offset not in composite!");
 
     NextType = ATy->getElementType();
-    unsigned ChildSize = TD.getTypeSize(NextType);
+    unsigned ChildSize = (unsigned)TD.getTypeSize(NextType);
     if (ConstantSInt::isValueValidForType(Type::IntTy, Offset/ChildSize))
       Indices.push_back(ConstantSInt::get(Type::IntTy, Offset/ChildSize));
     else
@@ -84,10 +84,10 @@
     return Ty;    // Return the leaf type
   }
 
-  unsigned SubOffs = Offset - ThisOffset;
+  unsigned SubOffs = unsigned(Offset - ThisOffset);
   const Type *LeafTy = getStructOffsetType(NextType, SubOffs,
                                            Indices, TD, StopEarly);
-  Offset = ThisOffset + SubOffs;
+  Offset = unsigned(ThisOffset + SubOffs);
   return LeafTy;
 }