=?utf-8?q?Donát?= Nagy <donat.n...@ericsson.com>, =?utf-8?q?Donát?= Nagy <donat.n...@ericsson.com>, =?utf-8?q?Donát?= Nagy <donat.n...@ericsson.com>, =?utf-8?q?Donát?= Nagy <donat.n...@ericsson.com>, =?utf-8?q?Donát?= Nagy <donat.n...@ericsson.com>, =?utf-8?q?Donát?= Nagy <donat.n...@ericsson.com>,Balazs Benics <benicsbal...@gmail.com>,Balazs Benics <benicsbal...@gmail.com>, =?utf-8?q?Donát?= Nagy <donat.n...@ericsson.com>, =?utf-8?q?Donát?= Nagy <donat.n...@ericsson.com> Message-ID: In-Reply-To: <llvm.org/llvm/llvm-project/pull/139...@github.com>
================ @@ -115,9 +115,22 @@ class CheckerRegistry { public: /// Adds a checker to the registry. Use this non-templated overload when your /// checker requires custom initialization. - void addChecker(RegisterCheckerFn Fn, ShouldRegisterFunction sfn, + void addChecker(RegisterCheckerFn Fn, ShouldRegisterFunction Sfn, + StringRef FullName, StringRef DebugName, StringRef Desc, + StringRef DocsUri, bool IsHidden); + + /// Adds a checker to the registry. This overload doesn't take a `DebugName` + /// (which usually looks like `DivZeroChecker`), so it uses the user-facing + /// `FullName` (which usually looks like ``core.DivideZero`) as a debug name. + /// THIS IS DEPRECATED and is only provided to preserve compatibility with + /// legacy plugins. + /// TODO: Eventually remove this from the codebase. ---------------- steakhal wrote: We can break compatibility. In fact, a clang plugin is only compatible with the same major version of clang already. There is no such thing in Clang as API or ABI compatibility across major releases. I'd recommend dropping this compatibility overload. https://github.com/llvm/llvm-project/pull/139256 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits