[PATCH] D104023: [AMDGPU] Make sure that the size of module (or kernel) lds struct is multiple of its alignment.

Mahesha S via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 10 05:26:06 PDT 2021


hsmhsm created this revision.
hsmhsm added reviewers: rampitec, foad, arsenm, JonChesterfield.
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.

When we create module (or per kernel) lds struct, we fix its alignment as the largest of struct
members. Add padding at the end of the struct so that the size of struct is multiple of its alignment.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D104023

Files:
  llvm/lib/Target/AMDGPU/AMDGPULowerModuleLDSPass.cpp
  llvm/test/CodeGen/AMDGPU/GlobalISel/hip.extern.shared.array.ll
  llvm/test/CodeGen/AMDGPU/hip.extern.shared.array.ll
  llvm/test/CodeGen/AMDGPU/lds-alignment.ll
  llvm/test/CodeGen/AMDGPU/lower-kernel-and-module-lds.ll
  llvm/test/CodeGen/AMDGPU/lower-kernel-lds-global-uses.ll
  llvm/test/CodeGen/AMDGPU/lower-kernel-lds.ll
  llvm/test/CodeGen/AMDGPU/lower-module-lds-global-alias.ll
  llvm/test/CodeGen/AMDGPU/lower-module-lds-global-uses.ll
  llvm/test/CodeGen/AMDGPU/lower-module-lds-indirect.ll
  llvm/test/CodeGen/AMDGPU/lower-module-lds-used-list.ll
  llvm/test/CodeGen/AMDGPU/lower-module-lds.ll
  llvm/test/CodeGen/AMDGPU/update-lds-alignment.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D104023.351137.patch
Type: text/x-patch
Size: 21456 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210610/c043f2a7/attachment.bin>


More information about the llvm-commits mailing list