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

Jun Bum Lim via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 24 08:37:42 PDT 2016


junbuml added a comment.

Thanks Matthias for the review. 
This will be committed after committing http://reviews.llvm.org/D18281.


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


http://reviews.llvm.org/D17889





More information about the llvm-commits mailing list