[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