[PATCH] D67794: [MachineCopyPropagation] Extend MCP to do trivial copy backward propagation

Quentin Colombet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 2 16:48:48 PST 2019


qcolombet added inline comments.


================
Comment at: llvm/lib/CodeGen/MachineCopyPropagation.cpp:212
+    return AvailCopy;
+  }
+
----------------
It seems to me that we could just reuse directly findAvailCopy and thus eliminate findCopyDefViaUnit.
The only difference would be the that we use a different iterator for the. forward and the backward case.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D67794/new/

https://reviews.llvm.org/D67794





More information about the llvm-commits mailing list