[PATCH] D47633: [GlobalMerge] Set the alignment on merged global structs

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 5 07:15:32 PDT 2018


dmgreen updated this revision to Diff 149974.
dmgreen added a comment.

Added a comment. I could change it to calculate the MaxAlignment from the alignment on the globals, if you think that's better in the long run.


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.149974.patch
Type: text/x-patch
Size: 7715 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180605/7eba04c4/attachment.bin>


More information about the llvm-commits mailing list