[PATCH] D17889: [MachineCopyPropagation] Expose more dead copies across instructions with regmasks

Matthias Braun via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 23 22:42:30 PDT 2016


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

LGTM


================
Comment at: lib/CodeGen/MachineCopyPropagation.cpp:289-290
@@ +288,4 @@
+      for (SmallSetVector<MachineInstr *, 8>::iterator DI =
+               MaybeDeadCopies.begin();
+           DI != MaybeDeadCopies.end();) {
+        MachineInstr *MaybeDead = *DI;
----------------
can these two lines get merged to a single one? (Ignore this if they are actually need more than 80 columns when combined).


http://reviews.llvm.org/D17889





More information about the llvm-commits mailing list