[PATCH] D100594: [AMDGPU] Strengthen the logic behind collection of LDS globals for lowering.

Mahesha S via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 15 12:24:27 PDT 2021


hsmhsm created this revision.
hsmhsm added reviewers: JonChesterfield, arsenm.
Herald added subscribers: kerbowa, hiraditya, t-tye, tpr, dstuttard, yaxunl, nhaehnle, jvesely, kzhuravl.
hsmhsm requested review of this revision.
Herald added subscribers: llvm-commits, wdng.
Herald added a project: LLVM.

Correct and strengthen the logic behind collection of LDS globals for lowering.
If LDS global is used within non-kernel functions, then, it certainly requires
lowering. Else, if it is used as a global initializer, then, lowering requires
if it is used as an initilizer of normal globals (*NON* llvm.used and
llvm.compiler.used globals). All other uses of LDS global can be ignored.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D100594

Files:
  llvm/lib/Target/AMDGPU/Utils/AMDGPULDSUtils.cpp
  llvm/lib/Target/AMDGPU/Utils/AMDGPULDSUtils.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D100594.337867.patch
Type: text/x-patch
Size: 9001 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210415/c9578a69/attachment.bin>


More information about the llvm-commits mailing list