[PATCH] D29344: [AArch64] Extend redundant copy elimination pass to handle non-zero stores.

Jun Bum Lim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 2 10:18:04 PST 2017


junbuml added inline comments.


================
Comment at: lib/Target/AArch64/AArch64RedundantCopyElimination.cpp:314-315
+
+            // Don't remove a move immediate that implicitly defines the upper
+            // bits when only the lower 32 bits are known.
+            MCPhysReg CmpReg = KnownReg.Reg;
----------------
If this is needed, it will be good to add comment why this is needed only for mov, not in Copy. 


https://reviews.llvm.org/D29344





More information about the llvm-commits mailing list