[PATCH] D64393: [AMDGPU] Fix DPP combiner check for exec modification

[Jay Foad via Phabricator via llvm-commits]

foad added a comment.

In D64393#1578035 <https://reviews.llvm.org/D64393#1578035>, @vpykhtin wrote:

> Jay, I'm ok with your patch, just a note that the prt set can be avoided if we count the number of use instructions and use the count to limit the scan.


I did think of that but I wasn't confident that I could implement it correctly. Does MRI.use_nodbg_instructions() guarantee not to repeat instructions, even if one instruction has multiple uses of the register?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D64393