[PATCH] D88703: [LLD][COFF] Add more type record information to /summary

Reid Kleckner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 1 14:52:32 PDT 2020


rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.

lgtm, this is useful info



================
Comment at: lld/COFF/PDB.cpp:977
+  if (config->showSummary) {
+    for_each(TpiSource::instances, [&](TpiSource *source) {
+      nbTypeRecords += source->nbTypeRecords;
----------------
A little voice in my head says, "addition is associative, parallelize it!" But I know the performance of this operation is not important. :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88703



More information about the llvm-commits mailing list