From 2b56a904417394416a61fd2df44ffe40cb49b829 Mon Sep 17 00:00:00 2001 From: Simon Pilgrim Date: Fri, 7 Jun 2019 17:37:04 +0000 Subject: [PATCH] Revert rL362792 : [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 ........ Breaks buildbots - @anton-afanasyev please can you take a look? git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@362816 91177308-0d34-0410-b5e6-96231b3b80d8 --- test/Driver/check-time-trace.cpp | 23 ----------------------- tools/driver/cc1_main.cpp | 5 ----- 2 files changed, 28 deletions(-) delete mode 100644 test/Driver/check-time-trace.cpp diff --git a/test/Driver/check-time-trace.cpp b/test/Driver/check-time-trace.cpp deleted file mode 100644 index 9e02537a0b..0000000000 --- a/test/Driver/check-time-trace.cpp +++ /dev/null @@ -1,23 +0,0 @@ -// 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 - -int main() { - std::cout << "Foo" << std::endl; - return 0; -} diff --git a/tools/driver/cc1_main.cpp b/tools/driver/cc1_main.cpp index caf73195fe..ae60fefdcf 100644 --- a/tools/driver/cc1_main.cpp +++ b/tools/driver/cc1_main.cpp @@ -241,11 +241,6 @@ int cc1_main(ArrayRef 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 -- 2.40.0