[PATCH] D44898: [ELF] Fix incorrect deduplication of MergeSyntheticSection's

Andrew Ng via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 27 06:07:40 PDT 2018


andrewng added a comment.

In https://reviews.llvm.org/D44898#1048789, @ruiu wrote:

> > 2.8%. Somehow clang-fsds got 2.39% faster.
>
> That speedup is interesting. I don't have a good theory to explain it though.
>
> Even if this patch is somewhat performance-neutral, we still should avoid extra memory allocation if we can. Let's try ignoring synthetic sections in ICF. That's I think better.


Alternative fix here: https://reviews.llvm.org/D44923.


https://reviews.llvm.org/D44898





More information about the llvm-commits mailing list