[PATCH] D94648: [amdgpu] Implement lower function LDS pass

Mahesha S via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 20 05:43:43 PDT 2021


hsmhsm added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPULowerModuleLDSPass.cpp:303
+        CurrentOffset += DL.getTypeAllocSize(FGV->getValueType());
+      }
+    }
----------------
Will the logic within above *for* loop ensures that first *non-padding* (real) member of the struct will be accessed at the same address as that of struct instance?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94648



More information about the llvm-commits mailing list