commit c29ea8fb903f9f7e3d8bdf8cb57b87eb3713826b
author Ken Dyck <kd@kendyck.com> Fri Mar 11 23:42:54 2011 +0000
committer Ken Dyck <kd@kendyck.com> Fri Mar 11 23:42:54 2011 +0000
tree 0b0236ec8cb3c35ce342deffaa4ded998894a521
parent d078bd2e02bd4e6661fe06645d98280a2a5a3e8c

Change parameter to AppendPadding from bytes to CharUnits. No change in
functionality intended.


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

lib/CodeGen/CGExprConstant.cpp

diff --git a/lib/CodeGen/CGExprConstant.cpp b/lib/CodeGen/CGExprConstant.cpp
index 99554753..01af80c 100644
--- a/lib/CodeGen/CGExprConstant.cpp
+++ b/lib/CodeGen/CGExprConstant.cpp
@@ -56,7 +56,7 @@
   void AppendBitField(const FieldDecl *Field, uint64_t FieldOffset,
                       llvm::ConstantInt *InitExpr);
 
-  void AppendPadding(uint64_t NumBytes);
+  void AppendPadding(CharUnits PadSize);
 
   void AppendTailPadding(CharUnits RecordSize);
 
@@ -99,7 +99,8 @@
 
   if (AlignedNextFieldOffsetInBytes < FieldOffsetInBytes) {
     // We need to append padding.
-    AppendPadding(FieldOffsetInBytes - NextFieldOffsetInBytes);
+    AppendPadding(
+        CharUnits::fromQuantity(FieldOffsetInBytes - NextFieldOffsetInBytes));
 
     assert(NextFieldOffsetInBytes == FieldOffsetInBytes &&
            "Did not add enough padding!");
@@ -129,7 +130,7 @@
       llvm::RoundUpToAlignment(FieldOffset -
                                NextFieldOffsetInBytes * 8, 8) / 8;
 
-    AppendPadding(NumBytes);
+    AppendPadding(CharUnits::fromQuantity(NumBytes));
   }
 
   uint64_t FieldSize =
@@ -212,8 +213,8 @@
         Elements.pop_back();
         
         // Add the padding back in two chunks.
-        AppendPadding(AT->getNumElements()-1);
-        AppendPadding(1);
+        AppendPadding(CharUnits::fromQuantity(AT->getNumElements()-1));
+        AppendPadding(CharUnits::One());
         assert(isa<llvm::UndefValue>(Elements.back()) &&
                Elements.back()->getType()->isIntegerTy(8) &&
                "Padding addition didn't work right");
@@ -265,13 +266,13 @@
   NextFieldOffsetInBytes++;
 }
 
-void ConstStructBuilder::AppendPadding(uint64_t NumBytes) {
-  if (!NumBytes)
+void ConstStructBuilder::AppendPadding(CharUnits PadSize) {
+  if (PadSize.isZero())
     return;
 
   const llvm::Type *Ty = llvm::Type::getInt8Ty(CGM.getLLVMContext());
-  if (NumBytes > 1)
-    Ty = llvm::ArrayType::get(Ty, NumBytes);
+  if (PadSize > CharUnits::One())
+    Ty = llvm::ArrayType::get(Ty, PadSize.getQuantity());
 
   llvm::Constant *C = llvm::UndefValue::get(Ty);
   Elements.push_back(C);
@@ -284,8 +285,7 @@
   assert(NextFieldOffsetInBytes <= RecordSize.getQuantity() && 
          "Size mismatch!");
 
-  unsigned NumPadBytes = RecordSize.getQuantity() - NextFieldOffsetInBytes;
-  AppendPadding(NumPadBytes);
+  AppendPadding(RecordSize - CharUnits::fromQuantity(NextFieldOffsetInBytes));
 }
 
 void ConstStructBuilder::ConvertStructToPacked() {