[PATCH] D108830: [AMDGPU] Propagate defining src reg for AGPR to AGPR Copys

Piotr Sobczak via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 30 08:03:43 PDT 2021


piotr added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/GCNPreRAOptimizations.cpp:104
+      if (!IsAGPRDst)
+        break;
+
----------------
I'm seeing issues with this commit on targets without agpr (e.g. gfx900).
I think there is a change in behaviour here - previously on COPY there was an early return. Now, there is only a switch break, so the loop continues. Is this intended?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108830



More information about the llvm-commits mailing list