[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