[PATCH] D122286: [AMDGPU] VGPR need to be reserved for AGPR copy *only* on subtarget GFX908

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 23 02:27:06 PDT 2022


foad added a comment.

Can you simplify the patch? I think all it is doing is (a) adding a new test and (b) simplifying the "if" on line 609 because the condition is always true. All the other changes make it harder to see what the patch is really doing.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122286



More information about the llvm-commits mailing list