================ @@ -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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits