[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