[PATCH] D127848: [CodeGen] Fix the bug of machine sink

[ChenZheng via Phabricator via llvm-commits]

shchenz added inline comments.


================
Comment at: llvm/lib/CodeGen/MachineSink.cpp:802
     // Ignore non-register operands.
     if (!MO.isReg())
       continue;
----------------
LuoYuanke wrote:
> shchenz wrote:
> > Maybe better to exclude the undef case at the very beginning?
> So do we also ingore undef value for physical register?
Do we have undef physical register? I don't see such usage and sounds weird to me, it is a physical register but it is undef....


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127848