Daniel Jasper | d07c840 | 2013-07-29 08:19:24 +0000 | [diff] [blame] | 1 | //===--- tools/extra/clang-tidy/ClangTidyMain.cpp - Clang tidy tool -------===// |
| 2 | // |
| 3 | // The LLVM Compiler Infrastructure |
| 4 | // |
| 5 | // This file is distributed under the University of Illinois Open Source |
| 6 | // License. See LICENSE.TXT for details. |
| 7 | // |
| 8 | //===----------------------------------------------------------------------===// |
| 9 | /// |
| 10 | /// \file This file implements a clang-tidy tool. |
| 11 | /// |
| 12 | /// This tool uses the Clang Tooling infrastructure, see |
| 13 | /// http://clang.llvm.org/docs/HowToSetupToolingForLLVM.html |
| 14 | /// for details on setting it up with LLVM source tree. |
| 15 | /// |
| 16 | //===----------------------------------------------------------------------===// |
| 17 | |
| 18 | #include "../ClangTidy.h" |
Manuel Klimek | 814f9bd | 2013-11-14 15:49:44 +0000 | [diff] [blame] | 19 | #include "clang/Tooling/CommonOptionsParser.h" |
Alexander Kornienko | e995154 | 2014-09-24 18:36:03 +0000 | [diff] [blame] | 20 | #include "llvm/Support/Process.h" |
Daniel Jasper | d07c840 | 2013-07-29 08:19:24 +0000 | [diff] [blame] | 21 | |
| 22 | using namespace clang::ast_matchers; |
| 23 | using namespace clang::driver; |
| 24 | using namespace clang::tooling; |
| 25 | using namespace llvm; |
| 26 | |
Alexander Kornienko | 99c9d6a | 2014-02-05 13:43:27 +0000 | [diff] [blame] | 27 | static cl::OptionCategory ClangTidyCategory("clang-tidy options"); |
Daniel Jasper | d07c840 | 2013-07-29 08:19:24 +0000 | [diff] [blame] | 28 | |
Manuel Klimek | 814f9bd | 2013-11-14 15:49:44 +0000 | [diff] [blame] | 29 | static cl::extrahelp CommonHelp(CommonOptionsParser::HelpMessage); |
Alexander Kornienko | d53d268 | 2014-09-04 14:23:36 +0000 | [diff] [blame] | 30 | static cl::extrahelp ClangTidyHelp( |
| 31 | "Configuration files:\n" |
| 32 | " clang-tidy attempts to read configuration for each source file from a\n" |
| 33 | " .clang-tidy file located in the closest parent directory of the source\n" |
| 34 | " file. If any configuration options have a corresponding command-line\n" |
| 35 | " option, command-line option takes precedence. The effective\n" |
Alexander Kornienko | 5c42331 | 2014-12-03 14:03:03 +0000 | [diff] [blame] | 36 | " configuration can be inspected using -dump-config:\n" |
| 37 | "\n" |
| 38 | " $ clang-tidy -dump-config - --\n" |
| 39 | " ---\n" |
| 40 | " Checks: '-*,some-check'\n" |
| 41 | " HeaderFilterRegex: ''\n" |
| 42 | " AnalyzeTemporaryDtors: false\n" |
| 43 | " User: user\n" |
| 44 | " CheckOptions: \n" |
| 45 | " - key: some-check.SomeOption\n" |
| 46 | " value: 'some value'\n" |
| 47 | " ...\n" |
| 48 | "\n\n"); |
Daniel Jasper | d07c840 | 2013-07-29 08:19:24 +0000 | [diff] [blame] | 49 | |
Alexander Kornienko | 50ab156 | 2014-10-29 18:25:09 +0000 | [diff] [blame] | 50 | const char DefaultChecks[] = // Enable these checks: |
| 51 | "clang-diagnostic-*," // * compiler diagnostics |
| 52 | "clang-analyzer-*," // * Static Analyzer checks |
| 53 | "-clang-analyzer-alpha*"; // * but not alpha checks: many false positives |
Alexander Kornienko | dad4acb | 2014-05-22 16:07:11 +0000 | [diff] [blame] | 54 | |
Alexander Kornienko | 23fe959 | 2014-05-15 14:27:36 +0000 | [diff] [blame] | 55 | static cl::opt<std::string> |
Alexander Kornienko | 3ab3467 | 2014-05-16 13:07:18 +0000 | [diff] [blame] | 56 | Checks("checks", cl::desc("Comma-separated list of globs with optional '-'\n" |
| 57 | "prefix. Globs are processed in order of appearance\n" |
| 58 | "in the list. Globs without '-' prefix add checks\n" |
| 59 | "with matching names to the set, globs with the '-'\n" |
| 60 | "prefix remove checks with matching names from the\n" |
Alexander Kornienko | d53d268 | 2014-09-04 14:23:36 +0000 | [diff] [blame] | 61 | "set of enabled checks.\n" |
| 62 | "This option's value is appended to the value read\n" |
| 63 | "from a .clang-tidy file, if any."), |
Alexander Kornienko | 23fe959 | 2014-05-15 14:27:36 +0000 | [diff] [blame] | 64 | cl::init(""), cl::cat(ClangTidyCategory)); |
Alexander Kornienko | dad4acb | 2014-05-22 16:07:11 +0000 | [diff] [blame] | 65 | |
Alexander Kornienko | 3ab3467 | 2014-05-16 13:07:18 +0000 | [diff] [blame] | 66 | static cl::opt<std::string> |
| 67 | HeaderFilter("header-filter", |
| 68 | cl::desc("Regular expression matching the names of the\n" |
Alexander Kornienko | dad4acb | 2014-05-22 16:07:11 +0000 | [diff] [blame] | 69 | "headers to output diagnostics from. Diagnostics\n" |
| 70 | "from the main file of each translation unit are\n" |
| 71 | "always displayed.\n" |
Alexander Kornienko | d53d268 | 2014-09-04 14:23:36 +0000 | [diff] [blame] | 72 | "Can be used together with -line-filter.\n" |
| 73 | "This option overrides the value read from a\n" |
| 74 | ".clang-tidy file."), |
Alexander Kornienko | 3ab3467 | 2014-05-16 13:07:18 +0000 | [diff] [blame] | 75 | cl::init(""), cl::cat(ClangTidyCategory)); |
Alexander Kornienko | dad4acb | 2014-05-22 16:07:11 +0000 | [diff] [blame] | 76 | |
Alexander Kornienko | 37f7abe | 2014-10-28 22:16:13 +0000 | [diff] [blame] | 77 | static cl::opt<bool> |
| 78 | SystemHeaders("system-headers", |
Alexander Kornienko | 5eac3c6 | 2014-11-03 14:06:31 +0000 | [diff] [blame] | 79 | cl::desc("Display the errors from system headers."), |
Alexander Kornienko | 37f7abe | 2014-10-28 22:16:13 +0000 | [diff] [blame] | 80 | cl::init(false), cl::cat(ClangTidyCategory)); |
Alexander Kornienko | dad4acb | 2014-05-22 16:07:11 +0000 | [diff] [blame] | 81 | static cl::opt<std::string> |
| 82 | LineFilter("line-filter", |
| 83 | cl::desc("List of files with line ranges to filter the\n" |
| 84 | "warnings. Can be used together with\n" |
| 85 | "-header-filter. The format of the list is a JSON\n" |
| 86 | "array of objects:\n" |
| 87 | " [\n" |
| 88 | " {\"name\":\"file1.cpp\",\"lines\":[[1,3],[5,7]]},\n" |
| 89 | " {\"name\":\"file2.h\"}\n" |
| 90 | " ]"), |
| 91 | cl::init(""), cl::cat(ClangTidyCategory)); |
| 92 | |
Alexander Kornienko | 5eac3c6 | 2014-11-03 14:06:31 +0000 | [diff] [blame] | 93 | static cl::opt<bool> |
| 94 | Fix("fix", cl::desc("Apply suggested fixes. Without -fix-errors\n" |
| 95 | "clang-tidy will bail out if any compilation\n" |
| 96 | "errors were found."), |
| 97 | cl::init(false), cl::cat(ClangTidyCategory)); |
| 98 | |
| 99 | static cl::opt<bool> |
| 100 | FixErrors("fix-errors", |
| 101 | cl::desc("Apply suggested fixes even if compilation errors\n" |
| 102 | "were found. If compiler errors have attached\n" |
| 103 | "fix-its, clang-tidy will apply them as well."), |
| 104 | cl::init(false), cl::cat(ClangTidyCategory)); |
Daniel Jasper | d07c840 | 2013-07-29 08:19:24 +0000 | [diff] [blame] | 105 | |
Alexander Kornienko | 3ab3467 | 2014-05-16 13:07:18 +0000 | [diff] [blame] | 106 | static cl::opt<bool> |
| 107 | ListChecks("list-checks", |
| 108 | cl::desc("List all enabled checks and exit. Use with\n" |
| 109 | "-checks='*' to list all available checks."), |
| 110 | cl::init(false), cl::cat(ClangTidyCategory)); |
Daniel Jasper | d07c840 | 2013-07-29 08:19:24 +0000 | [diff] [blame] | 111 | |
Alexander Kornienko | eff4e92 | 2014-09-26 11:42:29 +0000 | [diff] [blame] | 112 | static cl::opt<std::string> Config( |
| 113 | "config", |
| 114 | cl::desc("Specifies a configuration in YAML/JSON format:\n" |
Alexander Kornienko | 5c42331 | 2014-12-03 14:03:03 +0000 | [diff] [blame] | 115 | " -config=\"{Checks: '*', CheckOptions: [{key: x, value: y}]}\"\n" |
Alexander Kornienko | eff4e92 | 2014-09-26 11:42:29 +0000 | [diff] [blame] | 116 | "When the value is empty, clang-tidy will attempt to find\n" |
Alexander Kornienko | 7165e89 | 2014-09-26 11:48:53 +0000 | [diff] [blame] | 117 | "a file named .clang-tidy for each source file in its parent\n" |
Alexander Kornienko | eff4e92 | 2014-09-26 11:42:29 +0000 | [diff] [blame] | 118 | "directories."), |
| 119 | cl::init(""), cl::cat(ClangTidyCategory)); |
| 120 | |
Alexander Kornienko | 5c42331 | 2014-12-03 14:03:03 +0000 | [diff] [blame] | 121 | static cl::opt<bool> DumpConfig( |
| 122 | "dump-config", |
| 123 | cl::desc("Dumps configuration in the YAML format to stdout. This option\n" |
| 124 | "should be used along with a file name (and '--' if the file is\n" |
| 125 | "outside of a project with configured compilation database). The\n" |
| 126 | "configuration used for this file will be printed."), |
| 127 | cl::init(false), cl::cat(ClangTidyCategory)); |
Alexander Kornienko | d53d268 | 2014-09-04 14:23:36 +0000 | [diff] [blame] | 128 | |
Samuel Benzaquen | aedd994 | 2014-10-23 17:23:20 +0000 | [diff] [blame] | 129 | static cl::opt<bool> EnableCheckProfile( |
| 130 | "enable-check-profile", |
| 131 | cl::desc("Enable per-check timing profiles, and print a report to stderr."), |
| 132 | cl::init(false), cl::cat(ClangTidyCategory)); |
| 133 | |
Alexander Kornienko | d53d268 | 2014-09-04 14:23:36 +0000 | [diff] [blame] | 134 | static cl::opt<bool> AnalyzeTemporaryDtors( |
| 135 | "analyze-temporary-dtors", |
| 136 | cl::desc("Enable temporary destructor-aware analysis in\n" |
| 137 | "clang-analyzer- checks.\n" |
| 138 | "This option overrides the value read from a\n" |
| 139 | ".clang-tidy file."), |
| 140 | cl::init(false), cl::cat(ClangTidyCategory)); |
Alex McCarthy | fec08c7 | 2014-04-30 14:09:24 +0000 | [diff] [blame] | 141 | |
Benjamin Kramer | fb98b74 | 2014-09-04 10:31:23 +0000 | [diff] [blame] | 142 | static cl::opt<std::string> ExportFixes( |
| 143 | "export-fixes", |
| 144 | cl::desc("YAML file to store suggested fixes in. The\n" |
| 145 | "stored fixes can be applied to the input source\n" |
| 146 | "code with clang-apply-replacements."), |
| 147 | cl::value_desc("filename"), cl::cat(ClangTidyCategory)); |
| 148 | |
Alexander Kornienko | c28c32d | 2014-09-10 11:43:09 +0000 | [diff] [blame] | 149 | namespace clang { |
| 150 | namespace tidy { |
| 151 | |
| 152 | static void printStats(const ClangTidyStats &Stats) { |
Alexander Kornienko | dad4acb | 2014-05-22 16:07:11 +0000 | [diff] [blame] | 153 | if (Stats.errorsIgnored()) { |
| 154 | llvm::errs() << "Suppressed " << Stats.errorsIgnored() << " warnings ("; |
Alexander Kornienko | 5d17454 | 2014-05-07 09:06:53 +0000 | [diff] [blame] | 155 | StringRef Separator = ""; |
| 156 | if (Stats.ErrorsIgnoredNonUserCode) { |
| 157 | llvm::errs() << Stats.ErrorsIgnoredNonUserCode << " in non-user code"; |
| 158 | Separator = ", "; |
| 159 | } |
Alexander Kornienko | dad4acb | 2014-05-22 16:07:11 +0000 | [diff] [blame] | 160 | if (Stats.ErrorsIgnoredLineFilter) { |
| 161 | llvm::errs() << Separator << Stats.ErrorsIgnoredLineFilter |
| 162 | << " due to line filter"; |
| 163 | Separator = ", "; |
| 164 | } |
Alexander Kornienko | 5d17454 | 2014-05-07 09:06:53 +0000 | [diff] [blame] | 165 | if (Stats.ErrorsIgnoredNOLINT) { |
| 166 | llvm::errs() << Separator << Stats.ErrorsIgnoredNOLINT << " NOLINT"; |
| 167 | Separator = ", "; |
| 168 | } |
| 169 | if (Stats.ErrorsIgnoredCheckFilter) |
| 170 | llvm::errs() << Separator << Stats.ErrorsIgnoredCheckFilter |
| 171 | << " with check filters"; |
| 172 | llvm::errs() << ").\n"; |
| 173 | if (Stats.ErrorsIgnoredNonUserCode) |
| 174 | llvm::errs() << "Use -header-filter='.*' to display errors from all " |
| 175 | "non-system headers.\n"; |
| 176 | } |
| 177 | } |
| 178 | |
Samuel Benzaquen | aedd994 | 2014-10-23 17:23:20 +0000 | [diff] [blame] | 179 | static void printProfileData(const ProfileData &Profile, |
| 180 | llvm::raw_ostream &OS) { |
| 181 | // Time is first to allow for sorting by it. |
| 182 | std::vector<std::pair<llvm::TimeRecord, StringRef>> Timers; |
| 183 | TimeRecord Total; |
| 184 | |
| 185 | for (const auto& P : Profile.Records) { |
| 186 | Timers.emplace_back(P.getValue(), P.getKey()); |
| 187 | Total += P.getValue(); |
| 188 | } |
| 189 | |
| 190 | std::sort(Timers.begin(), Timers.end()); |
| 191 | |
| 192 | std::string Line = "===" + std::string(73, '-') + "===\n"; |
| 193 | OS << Line; |
| 194 | |
| 195 | if (Total.getUserTime()) |
| 196 | OS << " ---User Time---"; |
| 197 | if (Total.getSystemTime()) |
| 198 | OS << " --System Time--"; |
| 199 | if (Total.getProcessTime()) |
| 200 | OS << " --User+System--"; |
| 201 | OS << " ---Wall Time---"; |
| 202 | if (Total.getMemUsed()) |
| 203 | OS << " ---Mem---"; |
| 204 | OS << " --- Name ---\n"; |
| 205 | |
| 206 | // Loop through all of the timing data, printing it out. |
| 207 | for (auto I = Timers.rbegin(), E = Timers.rend(); I != E; ++I) { |
| 208 | I->first.print(Total, OS); |
| 209 | OS << I->second << '\n'; |
| 210 | } |
| 211 | |
| 212 | Total.print(Total, OS); |
| 213 | OS << "Total\n"; |
| 214 | OS << Line << "\n"; |
| 215 | OS.flush(); |
| 216 | } |
| 217 | |
Benjamin Kramer | e710371 | 2015-03-23 12:49:15 +0000 | [diff] [blame^] | 218 | static std::unique_ptr<ClangTidyOptionsProvider> createOptionsProvider() { |
Alexander Kornienko | c28c32d | 2014-09-10 11:43:09 +0000 | [diff] [blame] | 219 | ClangTidyGlobalOptions GlobalOptions; |
| 220 | if (std::error_code Err = parseLineFilter(LineFilter, GlobalOptions)) { |
Alexander Kornienko | dad4acb | 2014-05-22 16:07:11 +0000 | [diff] [blame] | 221 | llvm::errs() << "Invalid LineFilter: " << Err.message() << "\n\nUsage:\n"; |
| 222 | llvm::cl::PrintHelpMessage(/*Hidden=*/false, /*Categorized=*/true); |
Alexander Kornienko | eff4e92 | 2014-09-26 11:42:29 +0000 | [diff] [blame] | 223 | return nullptr; |
Alexander Kornienko | dad4acb | 2014-05-22 16:07:11 +0000 | [diff] [blame] | 224 | } |
Alexander Kornienko | 33a9bcc | 2014-04-29 15:20:10 +0000 | [diff] [blame] | 225 | |
Alexander Kornienko | e995154 | 2014-09-24 18:36:03 +0000 | [diff] [blame] | 226 | ClangTidyOptions DefaultOptions; |
| 227 | DefaultOptions.Checks = DefaultChecks; |
| 228 | DefaultOptions.HeaderFilterRegex = HeaderFilter; |
Alexander Kornienko | 37f7abe | 2014-10-28 22:16:13 +0000 | [diff] [blame] | 229 | DefaultOptions.SystemHeaders = SystemHeaders; |
Alexander Kornienko | e995154 | 2014-09-24 18:36:03 +0000 | [diff] [blame] | 230 | DefaultOptions.AnalyzeTemporaryDtors = AnalyzeTemporaryDtors; |
| 231 | DefaultOptions.User = llvm::sys::Process::GetEnv("USER"); |
| 232 | // USERNAME is used on Windows. |
| 233 | if (!DefaultOptions.User) |
| 234 | DefaultOptions.User = llvm::sys::Process::GetEnv("USERNAME"); |
Alexander Kornienko | a469522 | 2014-06-05 13:31:45 +0000 | [diff] [blame] | 235 | |
Alexander Kornienko | c28c32d | 2014-09-10 11:43:09 +0000 | [diff] [blame] | 236 | ClangTidyOptions OverrideOptions; |
Alexander Kornienko | d53d268 | 2014-09-04 14:23:36 +0000 | [diff] [blame] | 237 | if (Checks.getNumOccurrences() > 0) |
| 238 | OverrideOptions.Checks = Checks; |
| 239 | if (HeaderFilter.getNumOccurrences() > 0) |
| 240 | OverrideOptions.HeaderFilterRegex = HeaderFilter; |
Alexander Kornienko | 37f7abe | 2014-10-28 22:16:13 +0000 | [diff] [blame] | 241 | if (SystemHeaders.getNumOccurrences() > 0) |
| 242 | OverrideOptions.SystemHeaders = SystemHeaders; |
Alexander Kornienko | d53d268 | 2014-09-04 14:23:36 +0000 | [diff] [blame] | 243 | if (AnalyzeTemporaryDtors.getNumOccurrences() > 0) |
| 244 | OverrideOptions.AnalyzeTemporaryDtors = AnalyzeTemporaryDtors; |
| 245 | |
Alexander Kornienko | eff4e92 | 2014-09-26 11:42:29 +0000 | [diff] [blame] | 246 | if (!Config.empty()) { |
| 247 | if (llvm::ErrorOr<ClangTidyOptions> ParsedConfig = |
| 248 | parseConfiguration(Config)) { |
| 249 | return llvm::make_unique<DefaultOptionsProvider>( |
| 250 | GlobalOptions, ClangTidyOptions::getDefaults() |
| 251 | .mergeWith(DefaultOptions) |
| 252 | .mergeWith(*ParsedConfig) |
| 253 | .mergeWith(OverrideOptions)); |
| 254 | } else { |
| 255 | llvm::errs() << "Error: invalid configuration specified.\n" |
| 256 | << ParsedConfig.getError().message() << "\n"; |
| 257 | return nullptr; |
| 258 | } |
| 259 | } |
| 260 | return llvm::make_unique<FileOptionsProvider>(GlobalOptions, DefaultOptions, |
| 261 | OverrideOptions); |
| 262 | } |
| 263 | |
Benjamin Kramer | e710371 | 2015-03-23 12:49:15 +0000 | [diff] [blame^] | 264 | static int clangTidyMain(int argc, const char **argv) { |
Alexander Kornienko | eff4e92 | 2014-09-26 11:42:29 +0000 | [diff] [blame] | 265 | CommonOptionsParser OptionsParser(argc, argv, ClangTidyCategory); |
| 266 | |
| 267 | auto OptionsProvider = createOptionsProvider(); |
| 268 | if (!OptionsProvider) |
| 269 | return 1; |
Alexander Kornienko | d53d268 | 2014-09-04 14:23:36 +0000 | [diff] [blame] | 270 | |
| 271 | std::string FileName = OptionsParser.getSourcePathList().front(); |
Alexander Kornienko | c28c32d | 2014-09-10 11:43:09 +0000 | [diff] [blame] | 272 | ClangTidyOptions EffectiveOptions = OptionsProvider->getOptions(FileName); |
| 273 | std::vector<std::string> EnabledChecks = getCheckNames(EffectiveOptions); |
Alexander Kornienko | fbf9258 | 2014-06-02 20:32:06 +0000 | [diff] [blame] | 274 | |
Alexander Kornienko | fb9e92b | 2013-12-19 19:57:05 +0000 | [diff] [blame] | 275 | // FIXME: Allow using --list-checks without positional arguments. |
| 276 | if (ListChecks) { |
Alexander Kornienko | fb9e92b | 2013-12-19 19:57:05 +0000 | [diff] [blame] | 277 | llvm::outs() << "Enabled checks:"; |
Alexander Kornienko | fbf9258 | 2014-06-02 20:32:06 +0000 | [diff] [blame] | 278 | for (auto CheckName : EnabledChecks) |
Alexander Kornienko | 16ac6ce | 2014-03-05 13:14:32 +0000 | [diff] [blame] | 279 | llvm::outs() << "\n " << CheckName; |
Alexander Kornienko | fb9e92b | 2013-12-19 19:57:05 +0000 | [diff] [blame] | 280 | llvm::outs() << "\n\n"; |
| 281 | return 0; |
| 282 | } |
| 283 | |
Alexander Kornienko | d53d268 | 2014-09-04 14:23:36 +0000 | [diff] [blame] | 284 | if (DumpConfig) { |
Alexander Kornienko | 6e0cbc8 | 2014-09-12 08:53:36 +0000 | [diff] [blame] | 285 | EffectiveOptions.CheckOptions = getCheckOptions(EffectiveOptions); |
Alexander Kornienko | c28c32d | 2014-09-10 11:43:09 +0000 | [diff] [blame] | 286 | llvm::outs() << configurationAsText(ClangTidyOptions::getDefaults() |
| 287 | .mergeWith(EffectiveOptions)) |
Alexander Kornienko | d53d268 | 2014-09-04 14:23:36 +0000 | [diff] [blame] | 288 | << "\n"; |
| 289 | return 0; |
| 290 | } |
| 291 | |
Alexander Kornienko | fbf9258 | 2014-06-02 20:32:06 +0000 | [diff] [blame] | 292 | if (EnabledChecks.empty()) { |
| 293 | llvm::errs() << "Error: no checks enabled.\n"; |
| 294 | llvm::cl::PrintHelpMessage(/*Hidden=*/false, /*Categorized=*/true); |
| 295 | return 1; |
| 296 | } |
| 297 | |
Samuel Benzaquen | aedd994 | 2014-10-23 17:23:20 +0000 | [diff] [blame] | 298 | ProfileData Profile; |
| 299 | |
Alexander Kornienko | c28c32d | 2014-09-10 11:43:09 +0000 | [diff] [blame] | 300 | std::vector<ClangTidyError> Errors; |
| 301 | ClangTidyStats Stats = |
| 302 | runClangTidy(std::move(OptionsProvider), OptionsParser.getCompilations(), |
Samuel Benzaquen | aedd994 | 2014-10-23 17:23:20 +0000 | [diff] [blame] | 303 | OptionsParser.getSourcePathList(), &Errors, |
| 304 | EnableCheckProfile ? &Profile : nullptr); |
Alexander Kornienko | 5eac3c6 | 2014-11-03 14:06:31 +0000 | [diff] [blame] | 305 | bool FoundErrors = |
| 306 | std::find_if(Errors.begin(), Errors.end(), [](const ClangTidyError &E) { |
| 307 | return E.DiagLevel == ClangTidyError::Error; |
| 308 | }) != Errors.end(); |
| 309 | |
| 310 | const bool DisableFixes = Fix && FoundErrors && !FixErrors; |
| 311 | |
| 312 | // -fix-errors implies -fix. |
| 313 | handleErrors(Errors, (FixErrors || Fix) && !DisableFixes); |
Daniel Jasper | d07c840 | 2013-07-29 08:19:24 +0000 | [diff] [blame] | 314 | |
Alexander Kornienko | 4153da2 | 2014-09-04 15:19:49 +0000 | [diff] [blame] | 315 | if (!ExportFixes.empty() && !Errors.empty()) { |
Benjamin Kramer | fb98b74 | 2014-09-04 10:31:23 +0000 | [diff] [blame] | 316 | std::error_code EC; |
| 317 | llvm::raw_fd_ostream OS(ExportFixes, EC, llvm::sys::fs::F_None); |
| 318 | if (EC) { |
| 319 | llvm::errs() << "Error opening output file: " << EC.message() << '\n'; |
| 320 | return 1; |
| 321 | } |
Alexander Kornienko | c28c32d | 2014-09-10 11:43:09 +0000 | [diff] [blame] | 322 | exportReplacements(Errors, OS); |
Benjamin Kramer | fb98b74 | 2014-09-04 10:31:23 +0000 | [diff] [blame] | 323 | } |
| 324 | |
Alexander Kornienko | 5d17454 | 2014-05-07 09:06:53 +0000 | [diff] [blame] | 325 | printStats(Stats); |
Alexander Kornienko | 5eac3c6 | 2014-11-03 14:06:31 +0000 | [diff] [blame] | 326 | if (DisableFixes) |
Alexander Kornienko | b0a9b70 | 2014-12-09 15:02:17 +0000 | [diff] [blame] | 327 | llvm::errs() |
| 328 | << "Found compiler errors, but -fix-errors was not specified.\n" |
| 329 | "Fixes have NOT been applied.\n\n"; |
Alexander Kornienko | 5eac3c6 | 2014-11-03 14:06:31 +0000 | [diff] [blame] | 330 | |
Samuel Benzaquen | aedd994 | 2014-10-23 17:23:20 +0000 | [diff] [blame] | 331 | if (EnableCheckProfile) |
| 332 | printProfileData(Profile, llvm::errs()); |
| 333 | |
Daniel Jasper | d07c840 | 2013-07-29 08:19:24 +0000 | [diff] [blame] | 334 | return 0; |
| 335 | } |
Daniel Jasper | 89bbab0 | 2013-08-04 15:56:30 +0000 | [diff] [blame] | 336 | |
Daniel Jasper | 89bbab0 | 2013-08-04 15:56:30 +0000 | [diff] [blame] | 337 | // This anchor is used to force the linker to link the LLVMModule. |
| 338 | extern volatile int LLVMModuleAnchorSource; |
| 339 | static int LLVMModuleAnchorDestination = LLVMModuleAnchorSource; |
| 340 | |
| 341 | // This anchor is used to force the linker to link the GoogleModule. |
| 342 | extern volatile int GoogleModuleAnchorSource; |
| 343 | static int GoogleModuleAnchorDestination = GoogleModuleAnchorSource; |
| 344 | |
Alexander Kornienko | 16ac6ce | 2014-03-05 13:14:32 +0000 | [diff] [blame] | 345 | // This anchor is used to force the linker to link the MiscModule. |
| 346 | extern volatile int MiscModuleAnchorSource; |
| 347 | static int MiscModuleAnchorDestination = MiscModuleAnchorSource; |
| 348 | |
Alexander Kornienko | 2192a8e | 2014-10-26 01:41:14 +0000 | [diff] [blame] | 349 | // This anchor is used to force the linker to link the ReadabilityModule. |
| 350 | extern volatile int ReadabilityModuleAnchorSource; |
| 351 | static int ReadabilityModuleAnchorDestination = ReadabilityModuleAnchorSource; |
| 352 | |
Daniel Jasper | 89bbab0 | 2013-08-04 15:56:30 +0000 | [diff] [blame] | 353 | } // namespace tidy |
| 354 | } // namespace clang |
Alexander Kornienko | c28c32d | 2014-09-10 11:43:09 +0000 | [diff] [blame] | 355 | |
| 356 | int main(int argc, const char **argv) { |
| 357 | return clang::tidy::clangTidyMain(argc, argv); |
| 358 | } |