[COFF] Move ghash timers under the "add objects" timer
authorReid Kleckner <rnk@google.com>
Sat, 31 Oct 2020 18:08:58 +0000 (11:08 -0700)
committerReid Kleckner <rnk@google.com>
Sat, 31 Oct 2020 18:08:59 +0000 (11:08 -0700)
I had envisioned the ghash step as a big up front step, but as currently
written, the timers are nested, and we are notionally adding types from
objects, so we might as well arrange the timers this way.

lld/COFF/PDB.cpp

index 846d7a11fbfa95bd87f3f360b3cca862d1c3f5aa..5e65eca5b9fe4d128c6771e0d5aff61b78a827ac 100644 (file)
@@ -66,9 +66,9 @@ using llvm::object::coff_section;
 static ExitOnError exitOnErr;
 
 static Timer totalPdbLinkTimer("PDB Emission (Cumulative)", Timer::root());
-Timer lld::coff::loadGHashTimer("Global Type Hashing", totalPdbLinkTimer);
-Timer lld::coff::mergeGHashTimer("GHash Type Merging", totalPdbLinkTimer);
 static Timer addObjectsTimer("Add Objects", totalPdbLinkTimer);
+Timer lld::coff::loadGHashTimer("Global Type Hashing", addObjectsTimer);
+Timer lld::coff::mergeGHashTimer("GHash Type Merging", addObjectsTimer);
 static Timer typeMergingTimer("Type Merging", addObjectsTimer);
 static Timer symbolMergingTimer("Symbol Merging", addObjectsTimer);
 static Timer publicsLayoutTimer("Publics Stream Layout", totalPdbLinkTimer);