[PATCH] D98659: [MachineCopyPropagation] Do more backward copy propagations
yshui via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Mar 18 04:07:04 PDT 2021
yshui added inline comments.
================
Comment at: llvm/lib/CodeGen/MachineCopyPropagation.cpp:88
+/// Check that \p MI does not have implicit uses that overlap with it's \p Use
+/// operand (the register being replaced), since these can sometimes be
----------------
lkail wrote:
> Any reason of moving these two functions?
I realized these two functions don't have to be part of CopyTracker. If this is not fine I can put them back.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D98659/new/
https://reviews.llvm.org/D98659
More information about the llvm-commits
mailing list