[PATCH] D41915: [lldCOFF] Print detailed timing information with /VERBOSE

Zachary Turner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 11 10:34:24 PST 2018


zturner updated this revision to Diff 129475.
zturner added a comment.

- Renamed `LinkTimer` to `Timer`, as suggested by Rui.
- Moved the LTO setup into the proper place.
- Changed from `system_clock` to `high_resolution_clock`.

I didn't change ms to nanoseconds because I find nanoseconds hard to read.  It prints more than 10 digits of numbers, and it takes some mental strain to translate that back into something I can understand.  If anyone feels strongly that ms is not a fine enough granularity though, let me know.  Making the milliseconds a floating point might be another alternative.


https://reviews.llvm.org/D41915

Files:
  lld/COFF/Config.h
  lld/COFF/Driver.cpp
  lld/COFF/ICF.cpp
  lld/COFF/MarkLive.cpp
  lld/COFF/Options.td
  lld/COFF/PDB.cpp
  lld/COFF/SymbolTable.cpp
  lld/COFF/Timing.h
  lld/COFF/Writer.cpp
  lld/COFF/Writer.h
  lld/Common/CMakeLists.txt
  lld/Common/Timer.cpp
  lld/include/lld/Common/Timer.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D41915.129475.patch
Type: text/x-patch
Size: 19440 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180111/45e5b31a/attachment.bin>


More information about the llvm-commits mailing list