[PATCH] D115488: [amdgpu] Increase alignment of all LDS variables
Matt Arsenault via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Dec 9 18:23:49 PST 2021
arsenm added a comment.
In D115488#3184510 <https://reviews.llvm.org/D115488#3184510>, @JonChesterfield wrote:
> This will increase the LDS used by some cases (specifically, padding around variables that aren't moved into the module.lds block). I assume that's expected to be a net win. Internal CI running at 611966.
If this is supposed to be the guaranteed to work in all cases pass, isn't that an issue? Also it's probably not a net win if the increase pushes over an occupancy window
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D115488/new/
https://reviews.llvm.org/D115488
More information about the llvm-commits
mailing list