[PATCH] D17409: Add support for merging strings with alignment larger than one char

Rafael Ávila de Espíndola via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 18 13:19:02 PST 2016


rafael created this revision.
rafael added a reviewer: ruiu.
rafael added a subscriber: llvm-commits.

This reduces the .rodata of scyladb from 4501932 to 4334639 bytes (1.038 times smaller).

I don't think it is critical to support tail merging, just exact duplicates, but given the code organization it was actually a bit easier to support both.


http://reviews.llvm.org/D17409

Files:
  ELF/InputFiles.cpp
  ELF/OutputSections.cpp
  ELF/OutputSections.h
  ELF/Writer.cpp
  test/ELF/merge-string-align.s
  test/ELF/tail-merge-string-align.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D17409.48391.patch
Type: text/x-patch
Size: 7885 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160218/ef41f77f/attachment.bin>


More information about the llvm-commits mailing list