[AST] ASTPrinter: Make sure that attributes are printed as attributes and specifiers as keywords

Author: xedin

lib/AST/ASTPrinter.cpp

@@ -3890,7 +3890,7 @@ static void printParameterFlags(ASTPrinter &printer,
 
   if (flags.isSending()) {
     if (!options.SuppressSendingArgsAndResults) {
-      printer.printAttrName("sending ");
+      printer.printKeyword("sending", options, " ");
     } else if (flags.getOwnershipSpecifier() ==
                ParamSpecifier::ImplicitlyCopyableConsuming) {
       // Ok. We are suppressing sending. If our ownership specifier was
@@ -3911,14 +3911,14 @@ static void printParameterFlags(ASTPrinter &printer,
   // `inout` implies `@escaping`
   if (flags.getOwnershipSpecifier() != ParamSpecifier::InOut) {
     if (!options.excludeAttrKind(TypeAttrKind::Escaping) && escaping)
-      printer.printKeyword("@escaping", options, " ");
+      printer.printAttrName("@escaping ");
   }
 
   if (flags.isCompileTimeLiteral())
     printer.printKeyword("_const", options, " ");
 
   if (flags.isConstValue())
-    printer.printKeyword("@const", options, " ");
+    printer.printAttrName("@const ");
 }
 
 void PrintAST::visitVarDecl(VarDecl *decl) {

test/SymbolGraph/Symbols/Mixins/DeclarationFragments/Full/Function.swift

@@ -41,12 +41,12 @@ public func foo<S>(f: @escaping () -> (), ext int: Int = 2, s: S) where S: Seque
 // FOO-NEXT:          "spelling": ": "
 // FOO-NEXT:        },
 // FOO-NEXT:        {
-// FOO-NEXT:          "kind": "keyword",
-// FOO-NEXT:          "spelling": "@escaping"
+// FOO-NEXT:          "kind": "attribute",
+// FOO-NEXT:          "spelling": "@escaping "
 // FOO-NEXT:        },
 // FOO-NEXT:        {
 // FOO-NEXT:          "kind": "text",
-// FOO-NEXT:          "spelling": " () -> (), "
+// FOO-NEXT:          "spelling": "() -> (), "
 // FOO-NEXT:        },
 // FOO-NEXT:        {
 // FOO-NEXT:          "kind": "externalParam",