[PATCH] D59797: [COFF] Reduce the size of Chunk and SectionChunk, NFC

2019-04-03 Thread Reid Kleckner via Phabricator via cfe-commits
rnk added a comment. In D59797#1452440 , @MaskRay wrote: > > this reduces the sum of heap allocations ... These numbers exclude memory > > mapped files > > May I ask how you counted the memory usage minus memory mapped files? I'm using the Visual

[PATCH] D59797: [COFF] Reduce the size of Chunk and SectionChunk, NFC

2019-04-02 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay added a comment. > this reduces the sum of heap allocations ... These numbers exclude memory > mapped files May I ask how you counted the memory usage minus memory mapped files? Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59797/new/

[PATCH] D59797: [COFF] Reduce the size of Chunk and SectionChunk, NFC

2019-04-02 Thread Reid Kleckner via Phabricator via cfe-commits
This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit rL357535: [COFF] Reduce the size of Chunk and SectionChunk, NFC (authored by rnk, committed by ). Changed prior to commit:

[PATCH] D59797: [COFF] Reduce the size of Chunk and SectionChunk, NFC

2019-04-02 Thread Reid Kleckner via Phabricator via cfe-commits
rnk updated this revision to Diff 193371. rnk added a comment. - last update Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59797/new/ https://reviews.llvm.org/D59797 Files: lld/COFF/Chunks.cpp lld/COFF/Chunks.h lld/COFF/ICF.cpp