[PATCH] D76042: [PowerPC] Add the Uses of implicit register for the BCLRn instruction

[Qing Shan Zhang via Phabricator via llvm-commits]

steven.zhang resigned from this revision.
steven.zhang added inline comments.


================
Comment at: llvm/lib/Target/PowerPC/PPCEarlyReturn.cpp:95
-                  .addReg(J->getOperand(1).getReg())
-                  .copyImplicitOps(*I);
               MachineBasicBlock::iterator K = J--;
----------------
I still have concern on expand the BuildMI here. If it is the problem of copyImplicitOps, we need to add some new API or extend it to support the merge. And I don't think your new solution fix this issue as where is the implicit operands of BCCLR ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76042