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

[Thorsten via Phabricator via llvm-commits]

tschuett added inline comments.


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


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98419