[PATCH] D100117: [AMDGPU] SIFoldOperands: eagerly delete dead copies

[Jay Foad via Phabricator via llvm-commits]

foad added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/SIFoldOperands.cpp:1789
+      // delete it now.
+      if (MRI->use_empty(MI.getOperand(0).getReg()))
+        MI.eraseFromParent();
----------------
arsenm wrote:
> Slightly worried about the case where the only remaining copy user is a debug instruction
Good point. What can you do in that case?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100117