[PATCH] D36940: Replace std::vector to std::array in SymbolSerializer

Zachary Turner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Aug 20 15:57:36 PDT 2017


zturner added a comment.

The results of this and the other 2 patches look promising.  Some initial benchmarks show:

MSVC linker - 71.59s
lld without these 3 patches - 47.98s
lld with these 3 patches - 29.91s

Definitely a solid improvement.  I'll run some more timings on Monday when I'm back in the office (as well as looking more closely at the 3 patches).

Thanks!


https://reviews.llvm.org/D36940





More information about the llvm-commits mailing list