From cf1e76835febe1e6f1bcd5c3ebf4178b4c92dbe7 Mon Sep 17 00:00:00 2001 From: Victor Chernyakin Date: Sun, 1 Mar 2026 12:50:53 -0700 Subject: [PATCH] [clang-tidy][NFC] Don't call `getLangOpts` in `isLanguageVersionSupported` (#184029) This is just a little inconsistency I noticed. Basically all checks inspect the `LangOpts` parameter, but these two ignore the parameter and call `getLangOpts` instead. --- .../clang-tidy/bugprone/ThrowingStaticInitializationCheck.h | 2 +- clang-tools-extra/clang-tidy/llvm/UseNewMLIROpBuilderCheck.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/clang-tools-extra/clang-tidy/bugprone/ThrowingStaticInitializationCheck.h b/clang-tools-extra/clang-tidy/bugprone/ThrowingStaticInitializationCheck.h index 4ec554fc78fc..a25d7fe889e1 100644 --- a/clang-tools-extra/clang-tidy/bugprone/ThrowingStaticInitializationCheck.h +++ b/clang-tools-extra/clang-tidy/bugprone/ThrowingStaticInitializationCheck.h @@ -23,7 +23,7 @@ public: ThrowingStaticInitializationCheck(StringRef Name, ClangTidyContext *Context) : ClangTidyCheck(Name, Context) {} bool isLanguageVersionSupported(const LangOptions &LangOpts) const override { - return getLangOpts().CPlusPlus && getLangOpts().CXXExceptions; + return LangOpts.CPlusPlus && LangOpts.CXXExceptions; } void registerMatchers(ast_matchers::MatchFinder *Finder) override; void check(const ast_matchers::MatchFinder::MatchResult &Result) override; diff --git a/clang-tools-extra/clang-tidy/llvm/UseNewMLIROpBuilderCheck.h b/clang-tools-extra/clang-tidy/llvm/UseNewMLIROpBuilderCheck.h index 0842699823a6..28c7477a18f7 100644 --- a/clang-tools-extra/clang-tidy/llvm/UseNewMLIROpBuilderCheck.h +++ b/clang-tools-extra/clang-tidy/llvm/UseNewMLIROpBuilderCheck.h @@ -20,7 +20,7 @@ public: UseNewMlirOpBuilderCheck(StringRef Name, ClangTidyContext *Context); bool isLanguageVersionSupported(const LangOptions &LangOpts) const override { - return getLangOpts().CPlusPlus; + return LangOpts.CPlusPlus; } };