[clang] [Clang] include attribute scope in diagnostics (PR #144619)

[Aaron Ballman via cfe-commits]

================
@@ -506,7 +506,15 @@ void clang::FormatASTNodeDiagnosticArgument(
     case DiagnosticsEngine::ak_attr: {
       const Attr *At = reinterpret_cast<Attr *>(Val);
       assert(At && "Received null Attr object!");
-      OS << '\'' << At->getSpelling() << '\'';
+
+      OS << '\'';
+      if (At->hasScope()) {
----------------
AaronBallman wrote:

Do we still need `ak_attr` at all? `Attr` inherits from `AttributeCommonInfo`, so I think we end up handling `AttributeCommonInfo`, `ParsedAttr`, and `Attr` all the same way now, right?

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