[PATCH] D81070: [AMDGPU] Fixup use of StackPtrOffsetReg when not initialized

Scott Linder via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 4 14:42:43 PDT 2020


scott.linder added a comment.

> The other option is to treat any frame that can possibly have an offset that won't fit in the immediate field as needing an SP

I think this would simplify things a lot, and doesn't seem like it should have a huge effect on performance? If you are already indexing into your stack that far does it really matter that you have one fewer SGPR free?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81070



More information about the llvm-commits mailing list