llvm-cov: Use uint32_t for loop variables to be more consistent.
authorBob Wilson <bob.wilson@apple.com>
Tue, 22 Oct 2013 20:02:36 +0000 (20:02 +0000)
committerBob Wilson <bob.wilson@apple.com>
Tue, 22 Oct 2013 20:02:36 +0000 (20:02 +0000)
The loop bounds here are uint32_t variables, so it makes sense for the
loop variables to have the same type.

Patch by Yuchen Wu!

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

lib/IR/GCOV.cpp

index 84a7d63d6124023905a05e2cdb59c0b7fec3d77e..e16c7e6c76a73e9552134ddb1fc8eaf5ce560bfe 100644 (file)
@@ -117,7 +117,7 @@ bool GCOVFunction::read(GCOVBuffer &Buff, GCOV::GCOVFormat Format) {
   (void)BlockTagFound;
   assert(BlockTagFound && "Block Tag not found!");
   uint32_t BlockCount = Buff.readInt();
-  for (int i = 0, e = BlockCount; i != e; ++i) {
+  for (uint32_t i = 0, e = BlockCount; i != e; ++i) {
     Buff.readInt(); // Block flags;
     Blocks.push_back(new GCOVBlock(i));
   }
@@ -127,7 +127,7 @@ bool GCOVFunction::read(GCOVBuffer &Buff, GCOV::GCOVFormat Format) {
     uint32_t EdgeCount = (Buff.readInt() - 1) / 2;
     uint32_t BlockNo = Buff.readInt();
     assert(BlockNo < BlockCount && "Unexpected Block number!");
-    for (int i = 0, e = EdgeCount; i != e; ++i) {
+    for (uint32_t i = 0, e = EdgeCount; i != e; ++i) {
       Blocks[BlockNo]->addEdge(Buff.readInt());
       Buff.readInt(); // Edge flag
     }