[PATCH] D13815: [ELF2] Add support for Gnu Hash section

Igor Kudrin via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 19 07:38:22 PDT 2015


ikudrin added a comment.

> What about:

> 

> - In the current loop: Out<ELFT>::DynSymTab->addSymbol(...)

> - Sort the DynSymTab if needed: if (Out<ELFT>::GnuHashTab) GnuHashTable<ELFT>::sortDynamicSymbols()

> - Walk the DynSymTab, calling setDynamicSymbolTableIndex and adding the symbol to one or both hash tables.


I've almost done a new variant which is close to your suggestion, I hope. Let's look at it.


http://reviews.llvm.org/D13815





More information about the llvm-commits mailing list