[PATCH] D21859: [DSE] Fix bug in partial overwrite tracking

Hal Finkel via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 29 15:39:30 PDT 2016


hfinkel accepted this revision.
hfinkel added a comment.
This revision is now accepted and ready to land.

LGTM.


================
Comment at: lib/Transforms/Scalar/DeadStoreElimination.cpp:402
@@ +401,3 @@
+      //
+      while (ILI != IM.end() && (ILI->second<= LaterIntEnd)) {
+        assert(ILI->second > LaterIntStart && "Unexpected interval");
----------------
There's a missing space in between second and <=. The inner parenthesis are not necessary.


http://reviews.llvm.org/D21859





More information about the llvm-commits mailing list