Skip to content

Commit

Permalink
Revert rL362792 : [Support][Test] Time profiler: add regression test
Browse files Browse the repository at this point in the history
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?

llvm-svn: 362816
  • Loading branch information
RKSimon committed Jun 7, 2019
1 parent bb8de35 commit 96c421d
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 28 deletions.
23 changes: 0 additions & 23 deletions clang/test/Driver/check-time-trace.cpp

This file was deleted.

5 changes: 0 additions & 5 deletions clang/tools/driver/cc1_main.cpp
Expand Up @@ -241,11 +241,6 @@ 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
Expand Down

0 comments on commit 96c421d

Please sign in to comment.