[PATCH] D102334: [AMDGPU] Sort LDS globals based on thier size and alignment.

[Stanislav Mekhanoshin via Phabricator via llvm-commits]

rampitec added a comment.

In D102334#2754646 <https://reviews.llvm.org/D102334#2754646>, @JonChesterfield wrote:

> This seems to put all variables into one struct. I was suggesting a struct type and instance per kernel, populated by the subset of variables that can be accessed by kernel and functions called by it. Strictly speaking this does do that, but the subset can usually be smaller than the set of all variables in the module.

Right, I think we need a minimal (or as small as we can if not minimal) set per kernel.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D102334/new/

https://reviews.llvm.org/D102334