[PATCH] D82788: AMDGPU: Fix alignment requirements for 96bit and 128bit local loads and stores

Nicolai Hähnle via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 8 08:13:50 PDT 2020


nhaehnle added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUSubtarget.h:700
+  bool hasUnalignedDSAccess() const {
+    return UnalignedDSAccess;
+  }
----------------
arsenm wrote:
> I believe this is actually the same control as UnalignedBufferAccess, so a new feature isn't needed (but this needs double checking)
I believe LDS only become fully-featured with gfx9.


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

https://reviews.llvm.org/D82788





More information about the llvm-commits mailing list