[PATCH] Add DefinedAtom::sectionSize().

Rui Ueyama ruiu at google.com
Fri Jun 6 01:06:50 PDT 2014


Hi Bigcheese, shankarke, atanasyan,

Merge::mergeByLargestSection is defined in terms of section size,
but there was no way to get the section size for an atom. Previously
we followed layout-before/layout-after chains to collect all atoms
for a section, but that's not the right way to do it because there's
no guarantee that layout-before/layout-after exist. Even if they
exist, it's not guaranteed that they represent a section.

So this patch adds sectionSize() member function to DefinedAtom.

http://reviews.llvm.org/D4042

Files:
  include/lld/Core/DefinedAtom.h
  lib/Core/SymbolTable.cpp
  lib/ReaderWriter/PECOFF/Atoms.h
  lib/ReaderWriter/PECOFF/ReaderCOFF.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D4042.10168.patch
Type: text/x-patch
Size: 6191 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140606/4a1cbd84/attachment.bin>


More information about the llvm-commits mailing list