review: break the attribute name into its own fragment again

Author: QuietMisdreavus

lib/SymbolGraphGen/DeclarationFragmentPrinter.cpp

@@ -136,8 +136,13 @@ void DeclarationFragmentPrinter::printStructurePre(PrintStructureKind Kind,
 void DeclarationFragmentPrinter::printTypeRef(Type T, const TypeDecl *RefTo,
     Identifier Name,
     PrintNameContext NameContext) {
-  if (Kind != FragmentKind::Attribute)
+  if (Kind != FragmentKind::Attribute) {
     openFragment(FragmentKind::TypeIdentifier);
+  } else {
+    // create a separate fragment so that only the attribute name is linkable
+    closeFragment();
+    openFragment(FragmentKind::Attribute);
+  }
   printText(Name.str());
   USR.clear();
 

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

@@ -32,7 +32,11 @@ public func wrapped(@SomeWrapper arg: Int) {}
 // CHECK-NEXT:  },
 // CHECK-NEXT:  {
 // CHECK-NEXT:    "kind": "attribute",
-// CHECK-NEXT:    "spelling": "@SomeWrapper",
+// CHECK-NEXT:    "spelling": "@"
+// CHECK-NEXT:  },
+// CHECK-NEXT:  {
+// CHECK-NEXT:    "kind": "attribute",
+// CHECK-NEXT:    "spelling": "SomeWrapper",
 // CHECK-NEXT:    "preciseIdentifier": "s:10CustomAttr11SomeWrapperV"
 // CHECK-NEXT:  },
 // CHECK-NEXT:  {