https://github.com/localspook created
https://github.com/llvm/llvm-project/pull/187558
```txt
---User Time--- --System Time-- --User+System--
---Wall Time--- --- Name ---
Status quo: 0.4743 (100.0%) 0.3802 (100.0%) 0.8546 (100.0%) 0.8567
(100.0%) bugprone-suspicious-semicolon
With this change: 0.0103 (100.0%) 0.0027 (100.0%) 0.0130 (100.0%) 0.0133
(100.0%) bugprone-suspicious-semicolon
```
Continuing the trend of registering one `anyOf` matcher being slower than
registering each of its matchers separately (see #178829 for a previous
example).
(This PR also changes the traversal mode, but I only saw a small speedup from
that. Most of it came from registering the matchers separately.)
This check wasn't super expensive to begin with, but the speedup is still
pretty nice.
>From 32940133b136a87c841680881f9cc4737acf2340 Mon Sep 17 00:00:00 2001
From: Victor Chernyakin <[email protected]>
Date: Thu, 19 Mar 2026 18:56:44 +0000
Subject: [PATCH] [clang-tidy] Speed up `bugprone-suspicious-semicolon`
---
.../bugprone/SuspiciousSemicolonCheck.cpp | 16 +++++++++-------
.../bugprone/SuspiciousSemicolonCheck.h | 3 +++
2 files changed, 12 insertions(+), 7 deletions(-)
diff --git a/clang-tools-extra/clang-tidy/bugprone/SuspiciousSemicolonCheck.cpp
b/clang-tools-extra/clang-tidy/bugprone/SuspiciousSemicolonCheck.cpp
index 94041c3260384..ee2a4638f0a5d 100644
--- a/clang-tools-extra/clang-tidy/bugprone/SuspiciousSemicolonCheck.cpp
+++ b/clang-tools-extra/clang-tidy/bugprone/SuspiciousSemicolonCheck.cpp
@@ -16,14 +16,16 @@ using namespace clang::ast_matchers;
namespace clang::tidy::bugprone {
void SuspiciousSemicolonCheck::registerMatchers(MatchFinder *Finder) {
+ Finder->addMatcher(ifStmt(hasThen(nullStmt().bind("semi")),
+ unless(hasElse(stmt())), unless(isConstexpr()))
+ .bind("stmt"),
+ this);
+ Finder->addMatcher(forStmt(hasBody(nullStmt().bind("semi"))).bind("stmt"),
+ this);
Finder->addMatcher(
- stmt(anyOf(ifStmt(hasThen(nullStmt().bind("semi")),
- unless(hasElse(stmt())), unless(isConstexpr())),
- forStmt(hasBody(nullStmt().bind("semi"))),
- cxxForRangeStmt(hasBody(nullStmt().bind("semi"))),
- whileStmt(hasBody(nullStmt().bind("semi")))))
- .bind("stmt"),
- this);
+ cxxForRangeStmt(hasBody(nullStmt().bind("semi"))).bind("stmt"), this);
+ Finder->addMatcher(whileStmt(hasBody(nullStmt().bind("semi"))).bind("stmt"),
+ this);
}
void SuspiciousSemicolonCheck::check(const MatchFinder::MatchResult &Result) {
diff --git a/clang-tools-extra/clang-tidy/bugprone/SuspiciousSemicolonCheck.h
b/clang-tools-extra/clang-tidy/bugprone/SuspiciousSemicolonCheck.h
index 10bdf328df0ff..e789d5bfa4cc1 100644
--- a/clang-tools-extra/clang-tidy/bugprone/SuspiciousSemicolonCheck.h
+++ b/clang-tools-extra/clang-tidy/bugprone/SuspiciousSemicolonCheck.h
@@ -24,6 +24,9 @@ class SuspiciousSemicolonCheck : public ClangTidyCheck {
: ClangTidyCheck(Name, Context) {}
void registerMatchers(ast_matchers::MatchFinder *Finder) override;
void check(const ast_matchers::MatchFinder::MatchResult &Result) override;
+ std::optional<TraversalKind> getCheckTraversalKind() const override {
+ return TK_IgnoreUnlessSpelledInSource;
+ }
};
} // namespace clang::tidy::bugprone
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
