[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