[PATCH] D16897: AMDGPU: Account for LDS alignment

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 4 13:16:17 PST 2016


arsenm created this revision.
arsenm added a reviewer: tstellarAMD.
arsenm added a subscriber: llvm-commits.
Herald added a subscriber: arsenm.

The current situation isn't great, because the amount of padding
requires is determined by the inverse order of the first encountered
use. We should eventually somehow sort these to minimize wasted space.
    
Another problem is the alignment of kernel arguments isn't
respected. The group_segment_alignment is always emitted as
the default 16, and typed arguments with higher alignments
or an explicitly set alignment are also ignored.

http://reviews.llvm.org/D16897

Files:
  lib/Target/AMDGPU/AMDGPUISelLowering.cpp
  test/CodeGen/AMDGPU/lds-alignment.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D16897.46953.patch
Type: text/x-patch
Size: 17078 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160204/55c692b0/attachment.bin>


More information about the llvm-commits mailing list