[PATCH] D82551: [AMDGPU] Don't combine DPP if DPP register is used more than once per instruction

Valery Pykhtin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 26 02:41:05 PDT 2020


vpykhtin marked an inline comment as done.
vpykhtin added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/GCNDPPCombine.cpp:516
+        Src0->getSubReg() == Src1->getSubReg()) {
+      assert(Use == Src0 || Use == Src1);
+      LLVM_DEBUG(
----------------
foad wrote:
> This doesn't seem right. Use could be Src2 or some other operand, couldn't it?
I cound't find example for this, but ok, lets check this too.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82551





More information about the llvm-commits mailing list