[PATCH] D19912: [SimplifyCFG] isSafeToSpeculateStore now ignores debug info

[Chad Rosier via llvm-commits]

mcrosier requested changes to this revision.
mcrosier added a reviewer: mcrosier.
This revision now requires changes to proceed.

================
Comment at: lib/Transforms/Utils/SimplifyCFG.cpp:1451
@@ -1450,3 +1450,3 @@
   // Look for a store to the same pointer in BrBB.
   unsigned MaxNumInstToLookAt = 10;
   for (BasicBlock::reverse_iterator RI = BrBB->rbegin(),
----------------
Actually, in the original code we were doing a pre-decrement prior to the check.  Now the decrement happens after the check, so I believe we're now doing one extra iteration of the loop.


http://reviews.llvm.org/D19912