[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