[PATCH] [lld] Move AtomLayout from ELF to Core so that it can be used by non-ELF writers.

Rui Ueyama ruiu at google.com
Fri Jun 14 13:26:05 PDT 2013


  Michael,

  Currently AtomLayout has an object of Atom. Do you think we should change the type to DefinedAtom? My understanding is that atoms that will be layout are only defined ones. Undefined or external atoms does not have contents and does not occupy bytes in segments. So it feel an AtomLayout should only have a pointer to DefinedAtom. That change would eliminate some dyn_cast from our code.

http://llvm-reviews.chandlerc.com/D977



More information about the llvm-commits mailing list