[PATCH] D59797: [COFF] Reorder fields in Chunk and SectionChunk to reduce their size

[Reid Kleckner via Phabricator via llvm-commits]

rnk created this revision.
rnk added reviewers: ruiu, mstorsjo, aganea.
Herald added a subscriber: jdoerfert.
Herald added a project: LLVM.

This reduces the total size of all LLD heap allocations by 5.22% when
linking Chromium's media_blink.dll without /DEBUG.  This also adds
static_asserts to reduce the likelihood that significant memory usage
size regressions creep in over time.  The new size of SectionChunk is
168 bytes, and I think there is still room for improvement.

I added more comments for some fields while I'm here.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D59797

Files:
  lld/COFF/Chunks.cpp
  lld/COFF/Chunks.h
  llvm/include/llvm/BinaryFormat/COFF.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D59797.192192.patch
Type: text/x-patch
Size: 5256 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190325/a5463db4/attachment.bin>