[PATCH] D21613: [DSE] Avoid iterator invalidation bugs.

Chad Rosier via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 6 07:53:29 PDT 2016


mcrosier marked 4 inline comments as done.

================
Comment at: lib/Transforms/Scalar/DeadStoreElimination.cpp:822
@@ -814,3 +821,3 @@
     if (CallInst *F = isFreeCall(Inst, TLI)) {
       MadeChange |= handleFree(F, AA, MD, DT, TLI);
       continue;
----------------
eli.friedman wrote:
> BBI doesn't point at the free instruction here.
Ah, right.  Should be fixed now.

================
Comment at: lib/Transforms/Scalar/DeadStoreElimination.cpp:922
@@ -923,3 +921,3 @@
           // Delete the store and now-dead instructions that feed it.
-          deleteDeadInstruction(DepWrite, *MD, *TLI);
+          deleteDeadInstruction(DepWrite, &BBI, *MD, *TLI);
           ++NumFastStores;
----------------
Fixed.


http://reviews.llvm.org/D21613





More information about the llvm-commits mailing list