[PATCH] D133003: [ELF] Parallelize relocation scanning

Andrew Ng via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 9 09:17:17 PDT 2022


andrewng added a comment.

Performance on Windows looks good! Every test case I've tried has shown an improvement.



================
Comment at: lld/ELF/SyntheticSections.cpp:1603-1606
+  for (auto &v : relocsVec)
+    newSize += v.size();
+  relocs.reserve(newSize);
+  for (auto &v : relocsVec)
----------------
Perhaps `const auto &v`? Same for `RelrBaseSection::mergeRels()`.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D133003/new/

https://reviews.llvm.org/D133003



More information about the llvm-commits mailing list