Merge pull request #25516 from akyrtzi/5.1-print-some-as-keyword

Author: swift-ci

lib/AST/ASTPrinter.cpp

@@ -4216,7 +4216,7 @@ class TypePrinter : public TypeVisitor<TypePrinter> {
   void visitOpaqueTypeArchetypeType(OpaqueTypeArchetypeType *T) {
     switch (Options.OpaqueReturnTypePrinting) {
     case PrintOptions::OpaqueReturnTypePrintingMode::WithOpaqueKeyword:
-      Printer << "some ";
+      Printer.printKeyword("some", Options, /*Suffix=*/" ");
       LLVM_FALLTHROUGH;
     case PrintOptions::OpaqueReturnTypePrintingMode::WithoutOpaqueKeyword: {
       visit(T->getExistentialType());

lib/AST/TypeRepr.cpp

@@ -555,7 +555,7 @@ void ProtocolTypeRepr::printImpl(ASTPrinter &Printer,
 
 void OpaqueReturnTypeRepr::printImpl(ASTPrinter &Printer,
                                      const PrintOptions &Opts) const {
-  Printer << "some ";
+  Printer.printKeyword("some", Opts, /*Suffix=*/" ");
   printTypeRepr(Constraint, Printer, Opts);
 }
 

test/SourceKit/CursorInfo/cursor_some_type.swift

@@ -0,0 +1,12 @@
+public class Base {}
+
+public protocol Proto {}
+
+public func foo() -> some Base & Proto {
+  class Derived: Base, Proto {}
+  return Derived()
+}
+
+// RUN: %sourcekitd-test -req=cursor -pos=5:13 %s -- %s | %FileCheck %s
+// CHECK: <Declaration>public func foo() -&gt; some <Type usr=[[Base_USR:.*]]>Base</Type> &amp; <Type usr=[[Proto_USR:.*]]>Proto</Type></Declaration>
+// CHECK: <decl.function.free><syntaxtype.keyword>public</syntaxtype.keyword> <syntaxtype.keyword>func</syntaxtype.keyword> <decl.name>foo</decl.name>() -&gt; <decl.function.returntype><syntaxtype.keyword>some</syntaxtype.keyword> <ref.class usr=[[Base_USR]]>Base</ref.class> &amp; <ref.protocol usr=[[Proto_USR]]>Proto</ref.protocol></decl.function.returntype></decl.function.free>