Use an iterator instead of calling .size() on the worklist every time, which is wasteful.
authorBill Wendling <isanbard@gmail.com>
Thu, 15 Mar 2012 11:19:41 +0000 (11:19 +0000)
committerBill Wendling <isanbard@gmail.com>
Thu, 15 Mar 2012 11:19:41 +0000 (11:19 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@152794 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Transforms/InstCombine/InstCombineWorklist.h

index 32009c39ec2554a1cf235a103b913fad63dddcf2..99a02fc0df3f688c4476f5a47a4fbcc83ac23f94 100644 (file)
@@ -55,9 +55,9 @@ public:
     Worklist.reserve(NumEntries+16);
     WorklistMap.resize(NumEntries);
     DEBUG(errs() << "IC: ADDING: " << NumEntries << " instrs to worklist\n");
-    for (; NumEntries; --NumEntries) {
+    for (unsigned Idx = 0; NumEntries; --NumEntries) {
       Instruction *I = List[NumEntries-1];
-      WorklistMap.insert(std::make_pair(I, Worklist.size()));
+      WorklistMap.insert(std::make_pair(I, Idx++));
       Worklist.push_back(I);
     }
   }