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

Mahesha S via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 12 18:05:16 PDT 2021


hsmhsm added a comment.

In D102334#2754554 <https://reviews.llvm.org/D102334#2754554>, @rampitec wrote:

> As far as I understand all kernels in the module will use all the LDS globals irrespective if they actually need it? We may run out of LDS.

Hmm, I did not realize it. This patch is totally bad then.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102334



More information about the llvm-commits mailing list