[PATCH] D46080: Pack symbols a bit more

Rafael Avila de Espindola via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 25 13:56:52 PDT 2018


espindola created this revision.
espindola added reviewers: ruiu, grimar.
Herald added subscribers: arichardson, emaste.

Before this patch:

Symbol 56
Defined 80
Undefined 56
SharedSymbol 88
LazyArchive 72
LazyObject 56

With this patch

Symbol 48
Defined 72
Undefined 48
SharedSymbol 80
LazyArchive 64
LazyObject 48

The result is that peak allocation when linking chromium (according to heaptrack) goes from 578 to 568 MB.


https://reviews.llvm.org/D46080

Files:
  ELF/Symbols.cpp
  ELF/Symbols.h
  include/lld/Common/Strings.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D46080.144000.patch
Type: text/x-patch
Size: 4736 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180425/25d9bd6d/attachment.bin>


More information about the llvm-commits mailing list