diff options
| author | Guillaume Chatelet <gchatelet@google.com> | 2020-06-29 11:56:06 +0000 |
|---|---|---|
| committer | Guillaume Chatelet <gchatelet@google.com> | 2020-06-29 11:56:06 +0000 |
| commit | 52911428ef50556edbddd9e32220c22e8755a002 (patch) | |
| tree | 67b62085f7b3d48fef3b844602ccc39615c04171 /llvm/lib/Target/AMDGPU/AMDGPUMachineFunction.cpp | |
| parent | 368a5e3a666ff38432be538370c46ff800fface1 (diff) | |
[Alignment][NFC] Migrate AMDGPU backend to Align
This patch is part of a series to introduce an Alignment type.
See this thread for context: http://lists.llvm.org/pipermail/llvm-dev/2019-July/133851.html
See this patch for the introduction of the type: https://reviews.llvm.org/D64790
Differential Revision: https://reviews.llvm.org/D82743
Diffstat (limited to 'llvm/lib/Target/AMDGPU/AMDGPUMachineFunction.cpp')
| -rw-r--r-- | llvm/lib/Target/AMDGPU/AMDGPUMachineFunction.cpp | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/llvm/lib/Target/AMDGPU/AMDGPUMachineFunction.cpp b/llvm/lib/Target/AMDGPU/AMDGPUMachineFunction.cpp index 4c82d056ef65..64acd6efe028 100644 --- a/llvm/lib/Target/AMDGPU/AMDGPUMachineFunction.cpp +++ b/llvm/lib/Target/AMDGPU/AMDGPUMachineFunction.cpp @@ -43,14 +43,13 @@ unsigned AMDGPUMachineFunction::allocateLDSGlobal(const DataLayout &DL, if (!Entry.second) return Entry.first->second; - unsigned Align = GV.getAlignment(); - if (Align == 0) - Align = DL.getABITypeAlignment(GV.getValueType()); + Align Alignment = + DL.getValueOrABITypeAlignment(GV.getAlign(), GV.getValueType()); /// TODO: We should sort these to minimize wasted space due to alignment /// padding. Currently the padding is decided by the first encountered use /// during lowering. - unsigned Offset = LDSSize = alignTo(LDSSize, Align); + unsigned Offset = LDSSize = alignTo(LDSSize, Alignment); Entry.first->second = Offset; LDSSize += DL.getTypeAllocSize(GV.getValueType()); |