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

[Jay Foad via Phabricator via llvm-commits]

foad added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/GCNPreRAOptimizations.cpp:104
+      if (!IsAGPRDst)
+        break;
+
----------------
piotr wrote:
> 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?
@vangthao ping! This is causing codegen differences on gfx900 which has no AGPRs. Can you please comment on whether that is expected, and if so why? Thanks!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108830