From e104086ec98028b419b9b634497cbb52e814e4aa Mon Sep 17 00:00:00 2001 From: Anton Afanasyev Date: Mon, 19 Aug 2019 22:58:26 +0000 Subject: [PATCH] [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 git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@369308 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Support/TimeProfiler.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/Support/TimeProfiler.cpp b/lib/Support/TimeProfiler.cpp index b2f340bf9a0..fae8c15b6e3 100644 --- a/lib/Support/TimeProfiler.cpp +++ b/lib/Support/TimeProfiler.cpp @@ -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(E.Duration).count() > TimeTraceGranularity) + // Only include sections longer or equal to TimeTraceGranularity msec. + if (duration_cast(E.Duration).count() >= TimeTraceGranularity) Entries.emplace_back(E); // Track total time taken by each "name", but only the topmost levels of -- 2.49.0