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

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 25 13:31:44 PDT 2019


ruiu added inline comments.


================
Comment at: lld/COFF/Chunks.cpp:47
 
+namespace {
+// This class exists just for the purpose of calculating the expected size of
----------------
This might be useful but at the same time it looks a bit overly cautious? Perhaps the symbol size is more important but we don't have something like this for them, for example.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D59797/new/

https://reviews.llvm.org/D59797





More information about the llvm-commits mailing list