[PATCH] D79776: [AMDGPU] Allow use of StackPtrOffsetReg when building spills

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 14 09:11:17 PDT 2020


arsenm added inline comments.


================
Comment at: llvm/test/CodeGen/AMDGPU/spill-scavenge-offset.ll:65
 
+attributes #0 = { "amdgpu-num-sgpr"="30" }
 attributes #1 = { nounwind readnone }
----------------
critson wrote:
> arsenm wrote:
> > Can you use amdgpu-max-waves-per-eu? I'm trying to move away from the direct register limit attributes
> Unfortunately amdgpu-waves-per-eu does not allow us to constrain the SGPRs low enough to trigger the bug without making the test much bigger.
Don't you just need a big block of sgpr use asm to cover the difference?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79776





More information about the llvm-commits mailing list