[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