[PATCH] [lld][LayoutPass] Add comment on _followOn{Nexts,Roots}.

Rui Ueyama ruiu at google.com
Wed May 22 21:59:35 PDT 2013


Hi shankarke,

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

Files:
  include/lld/Passes/LayoutPass.h

Index: include/lld/Passes/LayoutPass.h
===================================================================
--- include/lld/Passes/LayoutPass.h
+++ include/lld/Passes/LayoutPass.h
@@ -67,8 +67,19 @@
 
   typedef llvm::DenseMap<const DefinedAtom *, const DefinedAtom *> AtomToAtomT;
   typedef llvm::DenseMap<const DefinedAtom *, uint64_t> AtomToOrdinalT;
+
+  // A map to be used to sort atoms. It represents the order of atoms in the
+  // result; if Atom X is mapped to atom Y in this map, X will be located
+  // immediately before Y in the output file. Y might be mapped to another
+  // atom, constructing a follow-on chain. An atom cannot be mapped to more
+  // than one atom unless all but one atom are of size zero.
   AtomToAtomT _followOnNexts;
+
+  // A map to be used to sort atoms. It's a map from an atom to its follow-on
+  // root. If an atom is not in _followOnNexts, the atom is not in this map,
+  // and vice versa.
   AtomToAtomT _followOnRoots;
+
   AtomToOrdinalT _ordinalOverrideMap;
   CompareAtoms _compareAtoms;
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D847.1.patch
Type: text/x-patch
Size: 1041 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130522/275b1803/attachment.bin>


More information about the llvm-commits mailing list