[PATCH] D16203: [AArch64] Add pass to remove redundant copy after RA

[Jun Bum Lim via llvm-commits]

junbuml added inline comments.

================
Comment at: lib/Target/AArch64/AArch64RedundantCopyElimination.cpp:8
@@ +7,3 @@
+//
+// This pass removes unnecessary zero copies in BBs that are targets of
+// cbz/cbnz instructions. For instance, the copy instruction in the code below
----------------
mcrosier wrote:
> Were we planning on adding support for handling arbitrary const copies, per Matthias' suggestion?
In this patch, I want to handle zero first which applied in most spec2000/2006 benchmakrs. And then I will remove below FIXME as I handle the arbitrary mov #imm cases as a separate patch.  I believe this could lead review process easier.


http://reviews.llvm.org/D16203