diff options
| author | Roman Lebedev <lebedev.ri@gmail.com> | 2018-05-08 13:14:21 +0000 |
|---|---|---|
| committer | Roman Lebedev <lebedev.ri@gmail.com> | 2018-05-08 13:14:21 +0000 |
| commit | 0ae5fe435fe0cc658ef0bca8769b022d963b4498 (patch) | |
| tree | 96de9f05058ef97ab647749102946f9c75383ac8 /clang-tidy/ClangTidy.h | |
| parent | 77e60e54f9c127ca9023d00dca047e7638ad4744 (diff) | |
[clang-tidy] Profile is a per-AST (per-TU) data.
Summary:
As discussed in D45931, currently, profiling output of clang-tidy is somewhat not great.
It outputs one profile at the end of the execution, and that profile contains the data
from the last TU that was processed. So if the tool run on multiple TU's, the data is
not accumulated, it is simply discarded.
It would be nice to improve this.
This differential is the first step - make this profiling info per-TU,
and output it after the tool has finished processing each TU.
In particular, when `ClangTidyASTConsumer` destructor runs.
Next step will be to add a CSV (JSON?) printer to store said profiles under user-specified directory prefix.
Reviewers: alexfh, sbenza
Reviewed By: alexfh
Subscribers: Eugene.Zelenko, mgorny, xazax.hun, mgrang, klimek, cfe-commits
Tags: #clang-tools-extra
Differential Revision: https://reviews.llvm.org/D46504
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@331763 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'clang-tidy/ClangTidy.h')
| -rw-r--r-- | clang-tidy/ClangTidy.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/clang-tidy/ClangTidy.h b/clang-tidy/ClangTidy.h index 7c9ef760..c68f6ed3 100644 --- a/clang-tidy/ClangTidy.h +++ b/clang-tidy/ClangTidy.h @@ -228,7 +228,7 @@ void runClangTidy(clang::tidy::ClangTidyContext &Context, const tooling::CompilationDatabase &Compilations, ArrayRef<std::string> InputFiles, llvm::IntrusiveRefCntPtr<vfs::FileSystem> BaseFS, - ProfileData *Profile = nullptr); + bool EnableCheckProfile = false); // FIXME: This interface will need to be significantly extended to be useful. // FIXME: Implement confidence levels for displaying/fixing errors. |