[PATCH] D98419: [lld-macho] minimal TimeTrace support

Nico Weber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 11 06:27:47 PST 2021


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

Nice!



================
Comment at: lld/MachO/Driver.cpp:948
+  {
+    llvm::TimeTraceScope timeScope("Link", StringRef("ExecuteLinker"));
+
----------------
ELF has just `"Link"` here, no 2nd arg. Do we need the 2nd arg?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98419



More information about the llvm-commits mailing list