[llvm] r369308 - [Support][Time profiler] Make FE codegen blocks to be inside frontend blocks
Anton Afanasyev via llvm-commits
llvm-commits at lists.llvm.org
Mon Aug 19 15:58:26 PDT 2019
Author: anton-afanasyev
Date: Mon Aug 19 15:58:26 2019
New Revision: 369308
URL: http://llvm.org/viewvc/llvm-project?rev=369308&view=rev
Log:
[Support][Time profiler] Make FE codegen blocks to be inside frontend blocks
Summary:
Add `Frontend` time trace entry to `HandleTranslationUnit()` function.
Add test to check all codegen blocks are inside frontend blocks.
Also, change `--time-trace-granularity` option a bit to make sure very small
time blocks are outputed to json-file when using `--time-trace-granularity=0`.
This fixes http://llvm.org/pr41969
Reviewers: russell.gallop, lebedev.ri, thakis
Reviewed By: russell.gallop
Subscribers: vsapsai, aras-p, lebedev.ri, hiraditya, cfe-commits, llvm-commits
Tags: #clang, #llvm
Differential Revision: https://reviews.llvm.org/D63325
Modified:
llvm/trunk/lib/Support/TimeProfiler.cpp
Modified: llvm/trunk/lib/Support/TimeProfiler.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Support/TimeProfiler.cpp?rev=369308&r1=369307&r2=369308&view=diff
==============================================================================
--- llvm/trunk/lib/Support/TimeProfiler.cpp (original)
+++ llvm/trunk/lib/Support/TimeProfiler.cpp Mon Aug 19 15:58:26 2019
@@ -58,8 +58,8 @@ struct TimeTraceProfiler {
auto &E = Stack.back();
E.Duration = steady_clock::now() - E.Start;
- // Only include sections longer than TimeTraceGranularity msec.
- if (duration_cast<microseconds>(E.Duration).count() > TimeTraceGranularity)
+ // Only include sections longer or equal to TimeTraceGranularity msec.
+ if (duration_cast<microseconds>(E.Duration).count() >= TimeTraceGranularity)
Entries.emplace_back(E);
// Track total time taken by each "name", but only the topmost levels of
More information about the llvm-commits
mailing list