[PATCH] D75098: Add TCOPY, a terminator form of the COPY instr
Bill Wendling via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri May 8 16:41:06 PDT 2020
void marked an inline comment as done.
void added inline comments.
================
Comment at: llvm/lib/CodeGen/MachineSink.cpp:857-860
+ // The copy no longer needs to be a terminator, so convert it to a normal
+ // COPY.
+ if (MI.getOpcode() == TargetOpcode::TCOPY)
+ MI.setDesc(TII->get(TargetOpcode::COPY));
----------------
arsenm wrote:
> I don't see this captured in a test?
This will happen now that we correctly mark `TCOPY` as sinkable. I'll see if I can craft an MIR test to explicitly do this.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D75098/new/
https://reviews.llvm.org/D75098
More information about the llvm-commits
mailing list