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

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 2 17:55:45 PST 2018

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


More information about the llvm-commits mailing list