[clang-tools-extra] [clang-tidy] support query based custom check (PR #131804)

[via cfe-commits]

github-actions[bot] wrote:

<!--LLVM CODE FORMAT COMMENT: {clang-format}-->


:warning: C/C++ code formatter, clang-format found issues in your code. 
:warning:

<details>
<summary>
You can test this locally with the following command:
</summary>

``````````bash
git-clang-format --diff origin/main HEAD --extensions cpp,h -- 
clang-tools-extra/clang-tidy/custom/CustomTidyModule.cpp 
clang-tools-extra/clang-tidy/custom/QueryCheck.cpp 
clang-tools-extra/clang-tidy/custom/QueryCheck.h 
clang-tools-extra/test/clang-tidy/checkers/custom/query-incorrect-query.cpp 
clang-tools-extra/test/clang-tidy/checkers/custom/query-partially-active-check.cpp
 clang-tools-extra/test/clang-tidy/checkers/custom/query.cpp 
clang-tools-extra/test/clang-tidy/infrastructure/custom-query-check-not-enable.cpp
 clang-tools-extra/test/clang-tidy/infrastructure/custom-query-check.cpp 
clang-tools-extra/clang-tidy/ClangTidy.cpp 
clang-tools-extra/clang-tidy/ClangTidy.h 
clang-tools-extra/clang-tidy/ClangTidyDiagnosticConsumer.cpp 
clang-tools-extra/clang-tidy/ClangTidyDiagnosticConsumer.h 
clang-tools-extra/clang-tidy/ClangTidyForceLinker.h 
clang-tools-extra/clang-tidy/ClangTidyModule.h 
clang-tools-extra/clang-tidy/ClangTidyOptions.cpp 
clang-tools-extra/clang-tidy/ClangTidyOptions.h 
clang-tools-extra/clang-tidy/tool/ClangTidyMain.cpp 
clang-tools-extra/unittests/clang-tidy/ClangTidyTest.h
``````````

:warning:
The reproduction instructions above might return results for more than one PR
in a stack if you are using a stacked PR workflow. You can limit the results by
changing `origin/main` to the base branch/commit you want to compare against.
:warning:

</details>

<details>
<summary>
View the diff from clang-format here.
</summary>

``````````diff
diff --git a/clang-tools-extra/clang-tidy/ClangTidy.cpp 
b/clang-tools-extra/clang-tidy/ClangTidy.cpp
index cee5c1596..2451d4eb9 100644
--- a/clang-tools-extra/clang-tidy/ClangTidy.cpp
+++ b/clang-tools-extra/clang-tidy/ClangTidy.cpp
@@ -515,8 +515,7 @@ std::vector<std::string> getCheckNames(const 
ClangTidyOptions &Options,
   clang::tidy::ClangTidyContext Context(
       std::make_unique<DefaultOptionsProvider>(ClangTidyGlobalOptions(),
                                                Options),
-      AllowEnablingAnalyzerAlphaCheckers, false,
-      ExperimentalCustomChecks);
+      AllowEnablingAnalyzerAlphaCheckers, false, ExperimentalCustomChecks);
   ClangTidyASTConsumerFactory Factory(Context);
   return Factory.getCheckNames();
 }
@@ -542,8 +541,7 @@ getCheckOptions(const ClangTidyOptions &Options,
   clang::tidy::ClangTidyContext Context(
       std::make_unique<DefaultOptionsProvider>(ClangTidyGlobalOptions(),
                                                Options),
-      AllowEnablingAnalyzerAlphaCheckers, false,
-      ExperimentalCustomChecks);
+      AllowEnablingAnalyzerAlphaCheckers, false, ExperimentalCustomChecks);
   ClangTidyDiagnosticConsumer DiagConsumer(Context);
   auto DiagOpts = std::make_unique<DiagnosticOptions>();
   DiagnosticsEngine DE(llvm::makeIntrusiveRefCnt<DiagnosticIDs>(), *DiagOpts,
@@ -687,8 +685,7 @@ ChecksAndOptions getAllChecksAndOptions(bool 
AllowEnablingAnalyzerAlphaCheckers,
   Opts.Checks = "*";
   clang::tidy::ClangTidyContext Context(
       std::make_unique<DefaultOptionsProvider>(ClangTidyGlobalOptions(), Opts),
-      AllowEnablingAnalyzerAlphaCheckers, false,
-      ExperimentalCustomChecks);
+      AllowEnablingAnalyzerAlphaCheckers, false, ExperimentalCustomChecks);
   ClangTidyCheckFactories Factories;
 #if CLANG_TIDY_ENABLE_QUERY_BASED_CUSTOM_CHECKS
   if (ExperimentalCustomChecks)
diff --git a/clang-tools-extra/clang-tidy/ClangTidyDiagnosticConsumer.h 
b/clang-tools-extra/clang-tidy/ClangTidyDiagnosticConsumer.h
index cbded96e8..fd346669b 100644
--- a/clang-tools-extra/clang-tidy/ClangTidyDiagnosticConsumer.h
+++ b/clang-tools-extra/clang-tidy/ClangTidyDiagnosticConsumer.h
@@ -216,9 +216,7 @@ public:
 
   // whether experimental custom checks can be enabled.
   // enabled with `--experimental-custom-checks`
-  bool canExperimentalCustomChecks() const {
-    return ExperimentalCustomChecks;
-  }
+  bool canExperimentalCustomChecks() const { return ExperimentalCustomChecks; }
 
   void setSelfContainedDiags(bool Value) { SelfContainedDiags = Value; }
 
diff --git a/clang-tools-extra/clang-tidy/tool/ClangTidyMain.cpp 
b/clang-tools-extra/clang-tidy/tool/ClangTidyMain.cpp
index adc5d97b2..cd059a86c 100644
--- a/clang-tools-extra/clang-tidy/tool/ClangTidyMain.cpp
+++ b/clang-tools-extra/clang-tidy/tool/ClangTidyMain.cpp
@@ -354,7 +354,7 @@ see 
https://clang.llvm.org/extra/clang-tidy/QueryBasedCustomChecks.html.
 Note this function is still under development
 and the API is subject to change.
 )"),
-                                   cl::init(false), 
cl::cat(ClangTidyCategory));
+                             cl::init(false), cl::cat(ClangTidyCategory));
 
 namespace clang::tidy {
 

``````````

</details>


https://github.com/llvm/llvm-project/pull/131804
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits