[PATCH] D87704: [AMDGPU] Reduce stack pointer alignment
Matt Arsenault via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Nov 18 17:11:15 PST 2022
arsenm resigned from this revision.
arsenm added a comment.
Herald added subscribers: kosarev, foad, arsenm.
Herald added a project: All.
I think this should be abandoned. Requiring 16-byte stack alignment doesn't require 16-byte alignment for a VGPR spill. The spill alignment requirement is only 4. This mostly matters for when stack realignment is necessary. The stack waste comes only in call frame contexts
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D87704/new/
https://reviews.llvm.org/D87704
More information about the llvm-commits
mailing list