Skip to content

Commit 5fcf712

Browse files
vbvictorlocalspook
andauthored
[clang-tidy][NFC] Add misc-include-cleaner to clang-tidy codebase (#171903)
Enabling `misc-include-cleaner` with a couple of `IWYU pragma: keep` lines seems beneficial to me: In my observation, we've made far more comments in reviews like "this header seems unused" than we've looked at includes of "top-level" files like `ClangTidy.cpp`, `ClangTidyForceLinker.h`, `ClangTidyMain.cpp` and so on. So I think it's a net benefit to keep check files clean from unused headers (which we review a lot) but make rarely-touched areas a little "polluted" with `IWYU pragma`'s --------- Co-authored-by: Victor Chernyakin <chernyakin.victor.j@outlook.com>
1 parent 57fcbd4 commit 5fcf712

File tree

7 files changed

+9
-10
lines changed

7 files changed

+9
-10
lines changed

clang-tools-extra/clang-tidy/.clang-tidy

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ Checks: >
1515
cppcoreguidelines-virtual-class-destructor,
1616
google-readability-casting,
1717
misc-const-correctness,
18+
misc-include-cleaner,
1819
modernize-*,
1920
-modernize-avoid-c-arrays,
2021
-modernize-pass-by-value,
@@ -42,3 +43,5 @@ Checks: >
4243
CheckOptions:
4344
- key: performance-move-const-arg.CheckTriviallyCopyableMove
4445
value: false
46+
- key: misc-include-cleaner.MissingIncludes
47+
value: false

clang-tools-extra/clang-tidy/ClangTidy.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@
3333
#include "clang/Lex/PreprocessorOptions.h"
3434
#include "clang/Rewrite/Frontend/FixItRewriter.h"
3535
#include "clang/Tooling/Core/Diagnostic.h"
36-
#include "clang/Tooling/DiagnosticsYaml.h"
36+
#include "clang/Tooling/DiagnosticsYaml.h" // IWYU pragma: keep
3737
#include "clang/Tooling/Refactoring.h"
3838
#include "clang/Tooling/Tooling.h"
3939
#include "llvm/Support/Process.h"

clang-tools-extra/clang-tidy/ClangTidyForceLinker.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@
1010
#define LLVM_CLANG_TOOLS_EXTRA_CLANG_TIDY_CLANGTIDYFORCELINKER_H
1111

1212
#include "clang-tidy-config.h"
13-
#include "llvm/Support/Compiler.h"
1413

1514
namespace clang::tidy {
1615

clang-tools-extra/clang-tidy/GlobList.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ static llvm::StringRef extractNextGlob(StringRef &GlobList) {
3131
}
3232

3333
static llvm::Regex createRegexFromGlob(StringRef &Glob) {
34-
SmallString<128> RegexText("^");
34+
llvm::SmallString<128> RegexText("^");
3535
const StringRef MetaChars("()^$|*+?.[]\\{}");
3636
for (const char C : Glob) {
3737
if (C == '*')

clang-tools-extra/clang-tidy/modernize/UseStdNumbersCheck.cpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,6 @@
2727
#include "llvm/ADT/SmallVector.h"
2828
#include "llvm/ADT/StringRef.h"
2929
#include "llvm/Support/FormatVariadic.h"
30-
#include "llvm/Support/MathExtras.h"
3130
#include <array>
3231
#include <cmath>
3332
#include <cstdint>

clang-tools-extra/clang-tidy/readability/IdentifierNamingCheck.cpp

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,6 @@
1616
#include "llvm/ADT/ArrayRef.h"
1717
#include "llvm/ADT/StringRef.h"
1818
#include "llvm/Support/Debug.h"
19-
#include "llvm/Support/FormatVariadic.h"
2019
#include "llvm/Support/Path.h"
2120
#include "llvm/Support/Regex.h"
2221
#include "llvm/Support/YAMLParser.h"
@@ -1363,9 +1362,8 @@ IdentifierNamingCheck::getFailureInfo(
13631362
if (StringRef(Fixup) == Name) {
13641363
if (!IgnoreFailedSplit) {
13651364
LLVM_DEBUG(Location.print(llvm::dbgs(), SM);
1366-
llvm::dbgs()
1367-
<< llvm::formatv(": unable to split words for {0} '{1}'\n",
1368-
KindName, Name));
1365+
llvm::dbgs() << ": unable to split words for " << KindName
1366+
<< " '" << Name << "'\n");
13691367
}
13701368
return std::nullopt;
13711369
}

clang-tools-extra/clang-tidy/tool/ClangTidyMain.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,14 +16,14 @@
1616

1717
#include "ClangTidyMain.h"
1818
#include "../ClangTidy.h"
19-
#include "../ClangTidyForceLinker.h"
19+
#include "../ClangTidyForceLinker.h" // IWYU pragma: keep
2020
#include "../GlobList.h"
2121
#include "../utils/OptionsUtils.h"
2222
#include "clang/Tooling/CommonOptionsParser.h"
2323
#include "llvm/ADT/StringSet.h"
2424
#include "llvm/Support/CommandLine.h"
2525
#include "llvm/Support/InitLLVM.h"
26-
#include "llvm/Support/PluginLoader.h"
26+
#include "llvm/Support/PluginLoader.h" // IWYU pragma: keep
2727
#include "llvm/Support/Process.h"
2828
#include "llvm/Support/Signals.h"
2929
#include "llvm/Support/TargetSelect.h"

0 commit comments

Comments
 (0)