[PATCH] D90686: [lld][ELF] Add additional time trace categories

[George Rimar via Phabricator via llvm-commits]

grimar added inline comments.


================
Comment at: lld/ELF/Writer.cpp:2182
+  {
+    llvm::TimeTraceScope timeScope("Finalize synthetic sections");
+    // finalizeAddressDependentContent may have added local symbols to the
----------------
Does it make sense to name this and the scope above as:

"Finalize synthetic sections (1/2)"
"Finalize synthetic sections (2/2)"

?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90686