[PATCH] D45519: [ELF] - Change the way of sorting local symbols.
Rafael Avila de Espindola via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Apr 11 17:20:53 PDT 2018
espindola added a comment.
In https://reviews.llvm.org/D45519#1065031, @ruiu wrote:
> George, can you try to not use hash table like this in the future? Both this patch and your previous patch use a hash table to store a lot of symbols, which is a violation of the rule I set to achieve a good performance. If it is unavoidable, please explain in the commit message so that we can discuss and examine if it is really unavoidable. Just using a hash table without an explanation isn't good.
Note that this hash in on InputFiles, not symbols.
https://reviews.llvm.org/D45519
More information about the llvm-commits
mailing list