diff options
author | Anton Afanasyev <anton.a.afanasyev@gmail.com> | 2019-06-07 12:59:32 +0000 |
---|---|---|
committer | Anton Afanasyev <anton.a.afanasyev@gmail.com> | 2019-06-07 12:59:32 +0000 |
commit | e00c217376e21d109d001ead542c257397c9f82a (patch) | |
tree | 1c8350df550eef0e2ab1a7cd98388c0f2a43e07e | |
parent | 74050c35b8b25f673ebbbe747444a49345972a95 (diff) | |
download | clang-e00c217376e21d109d001ead542c257397c9f82a.tar.gz |
[Support][Test] Time profiler: add regression test
Summary:
Add output to `llvm::errs()` when `-ftime-trace` option is enabled,
add regression test checking this option works as expected.
Reviewers: thakis, aganea
Subscribers: cfe-commits, llvm-commits
Tags: #clang, #llvm
Differential Revision: https://reviews.llvm.org/D61914
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@362792 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r-- | test/Driver/check-time-trace.cpp | 23 | ||||
-rw-r--r-- | tools/driver/cc1_main.cpp | 5 |
2 files changed, 28 insertions, 0 deletions
diff --git a/test/Driver/check-time-trace.cpp b/test/Driver/check-time-trace.cpp new file mode 100644 index 0000000000..9e02537a0b --- /dev/null +++ b/test/Driver/check-time-trace.cpp @@ -0,0 +1,23 @@ +// RUN: %clangxx -ftime-trace %s 2>&1 | grep "Time trace json-file dumped to" \ +// RUN: | awk '{print $NF}' | xargs cat \ +// RUN: | %python -c 'import json, sys; json.dump(json.loads(sys.stdin.read()), sys.stdout, sort_keys=True, indent=2)' \ +// RUN: | FileCheck %s + +// CHECK: "traceEvents": [ +// CHECK: "args": +// CHECK: "detail": +// CHECK: "dur": +// CHECK: "name": "Source" +// CHECK-NEXT: "ph": +// CHECK-NEXT: "pid": +// CHECK-NEXT: "tid": +// CHECK-NEXT: "ts": +// CHECK: "name": "clang" +// CHECK: "name": "process_name" + +#include <iostream> + +int main() { + std::cout << "Foo" << std::endl; + return 0; +} diff --git a/tools/driver/cc1_main.cpp b/tools/driver/cc1_main.cpp index ae60fefdcf..caf73195fe 100644 --- a/tools/driver/cc1_main.cpp +++ b/tools/driver/cc1_main.cpp @@ -241,6 +241,11 @@ int cc1_main(ArrayRef<const char *> Argv, const char *Argv0, void *MainAddr) { llvm::timeTraceProfilerWrite(*profilerOutput); llvm::timeTraceProfilerCleanup(); + + llvm::errs() << "Time trace json-file dumped to " << Path.str() << "\n"; + llvm::errs() + << "Use chrome://tracing or Speedscope App " + "(https://www.speedscope.app) for flamegraph visualization\n"; } // Our error handler depends on the Diagnostics object, which we're |