[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