[PATCH] D103261: [AMDGPU] Increase alignment of LDS globals if necessary before LDS lowering.

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 3 14:44:26 PDT 2021


arsenm added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/Utils/AMDGPULDSUtils.h:47
+// that we can use aligned LDS instructions to access them.
+void increaseAlignment(ArrayRef<GlobalVariable *> LDSGlobals,
+                       const DataLayout &DL);
----------------
This should handle one GV at a time. I don't see a benefit to pushing the loop into the function


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103261



More information about the llvm-commits mailing list