[PATCH] D44045: [AMDGPU] Adjusted alignment-check for local address space;

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: lib/Target/AMDGPU/AMDGPUISelDAGToDAG.cpp:661-663
+  // LDS space is allocated to a work-group or wavefront in contiguous blocks of
+  // 64 Dwords on 64-Dword alignment; checking offset being multiple of 8 is
+  // sufficient to prove that the address is 8 byte aligned.
----------------
Also if you're referring to the allocation granularity for the entire program, that doesn't reflect the individual symbols allocated. We certainly don't allocate the individual globals with at least 8 byte alignment


https://reviews.llvm.org/D44045