Author: Victor Chernyakin Date: 2026-03-01T11:50:53-08:00 New Revision: cf1e76835febe1e6f1bcd5c3ebf4178b4c92dbe7
URL: https://github.com/llvm/llvm-project/commit/cf1e76835febe1e6f1bcd5c3ebf4178b4c92dbe7 DIFF: https://github.com/llvm/llvm-project/commit/cf1e76835febe1e6f1bcd5c3ebf4178b4c92dbe7.diff LOG: [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. Added: Modified: clang-tools-extra/clang-tidy/bugprone/ThrowingStaticInitializationCheck.h clang-tools-extra/clang-tidy/llvm/UseNewMLIROpBuilderCheck.h Removed: ################################################################################ diff --git a/clang-tools-extra/clang-tidy/bugprone/ThrowingStaticInitializationCheck.h b/clang-tools-extra/clang-tidy/bugprone/ThrowingStaticInitializationCheck.h index 4ec554fc78fc6..a25d7fe889e16 100644 --- a/clang-tools-extra/clang-tidy/bugprone/ThrowingStaticInitializationCheck.h +++ b/clang-tools-extra/clang-tidy/bugprone/ThrowingStaticInitializationCheck.h @@ -23,7 +23,7 @@ class ThrowingStaticInitializationCheck : public ClangTidyCheck { 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 0842699823a65..28c7477a18f7f 100644 --- a/clang-tools-extra/clang-tidy/llvm/UseNewMLIROpBuilderCheck.h +++ b/clang-tools-extra/clang-tidy/llvm/UseNewMLIROpBuilderCheck.h @@ -20,7 +20,7 @@ class UseNewMlirOpBuilderCheck : public utils::TransformerClangTidyCheck { UseNewMlirOpBuilderCheck(StringRef Name, ClangTidyContext *Context); bool isLanguageVersionSupported(const LangOptions &LangOpts) const override { - return getLangOpts().CPlusPlus; + return LangOpts.CPlusPlus; } }; _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
