[PATCH] D79907: [AMDGPU] Peephole adjacent equivalent S_SET_GPR_IDX_ON

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/SIPreEmitPeephole.cpp:152
+  MachineOperand *Idx = TII->getNamedOperand(MI, AMDGPU::OpName::src0);
+  Register IdxReg = Idx->isReg() ? Idx->getReg() : AMDGPU::NoRegister;
+  SmallVector<MachineInstr *, 4> ToRemove;
----------------
Register() instead of NoRegister. When does the NoRegister case happen?


================
Comment at: llvm/lib/Target/AMDGPU/SIPreEmitPeephole.cpp:157-159
+  for (MachineBasicBlock::reverse_iterator
+       I = std::next(MI.getReverseIterator()),
+       E = MBB.rend(); I != E; ++I) {
----------------
Should have a search threshold


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

https://reviews.llvm.org/D79907