[PATCH] D49710: [GlobalMerge] Allow merging globals with arbitrary alignment.

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 24 18:16:49 PDT 2018


efriedma updated this revision to Diff 157179.
efriedma added a comment.

Use DataLayout::getPreferredAlignment so we end up with the same alignment the AsmPrinter would have computed.  Not sure if the AsmPrinter is actually behaving the way we want it to, but we should consider that separately.


Repository:
  rL LLVM

https://reviews.llvm.org/D49710

Files:
  lib/CodeGen/GlobalMerge.cpp
  test/Transforms/GlobalMerge/alignment-2.ll
  test/Transforms/GlobalMerge/alignment.ll
  test/Transforms/GlobalMerge/basic.ll
  test/Transforms/GlobalMerge/debug-info.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D49710.157179.patch
Type: text/x-patch
Size: 9937 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180725/6d5beedf/attachment.bin>


More information about the llvm-commits mailing list