[PATCH] D47633: [GlobalMerge] Set the alignment on merged global structs
Dave Green via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jun 1 07:00:38 PDT 2018
dmgreen created this revision.
dmgreen added reviewers: efriedma, qcolombet, haicheng, john.brawn.
Herald added a reviewer: javed.absar.
If no alignment is set, the abi/preferred alignment of structs will be
used which may be higher than required. This can lead to extra padding
and in the end an increase in data size.
https://reviews.llvm.org/D47633
Files:
lib/CodeGen/GlobalMerge.cpp
test/CodeGen/AArch64/global-merge-1.ll
test/CodeGen/AArch64/global-merge-2.ll
test/CodeGen/AArch64/global-merge-3.ll
test/CodeGen/AArch64/global-merge-4.ll
test/CodeGen/AArch64/global-merge-group-by-use.ll
test/CodeGen/AArch64/global-merge-ignore-single-use-minsize.ll
test/CodeGen/AArch64/global-merge-ignore-single-use.ll
test/CodeGen/ARM/global-merge-1.ll
test/CodeGen/ARM/global-merge-alignment.ll
test/Transforms/GlobalMerge/debug-info.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D47633.149444.patch
Type: text/x-patch
Size: 7463 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180601/e2d20eb9/attachment.bin>
More information about the llvm-commits
mailing list