[clang-tools-extra] [clang-tidy] support ak_attr_info in diagnostic forwarding (PR #147503)

Mikael Holmén via cfe-commits cfe-commits at lists.llvm.org
Tue Jul 8 04:03:29 PDT 2025


================
@@ -544,6 +544,9 @@ void ClangTidyDiagnosticConsumer::forwardDiagnostic(const Diagnostic &Info) {
     case clang::DiagnosticsEngine::ak_attr:
       Builder << reinterpret_cast<Attr *>(Info.getRawArg(Index));
       break;
+    case clang::DiagnosticsEngine::ak_attr_info:
----------------
mikaelholmen wrote:

I don't know if anyone cares but the case value in this switch are handled in the same order as the values are defined in the ArgumentKind enum, so if we should continue with that I guess this new case should be placed last, after ak_expr (which then would need an explicit "break;".

https://github.com/llvm/llvm-project/pull/147503


More information about the cfe-commits mailing list