[PATCH] D67101: Target hooks for custom COPY insertion.

[Stanislav Mekhanoshin via Phabricator via llvm-commits]

rampitec added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/SIInstrInfo.cpp:6417
+    return BuildMI(MBB, InsPt, InsPt->getDebugLoc(),
+                   get(AMDGPU::S_MOV_B64_term), Dst)
+        .addReg(Src, 0, SrcSubReg)
----------------
alex-t wrote:
> alex-t wrote:
> > rampitec wrote:
> > > So you think it is always scalar and always 64 bit?
> > Since only case is SI_CF yes.
> Oh... yes. in WAVE32 it will be S_MOV_B32_term
What ensures that is just SI_CF? Why it cannot be some another PHI?


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

https://reviews.llvm.org/D67101