[ASTPrinter] Skip SPI extensions too

Author: xymus

lib/AST/ASTPrinter.cpp

@@ -145,8 +145,11 @@ PrintOptions PrintOptions::printSwiftInterfaceFile(bool preferTypeRepr,
       if (D->getAttrs().hasAttribute<ImplementationOnlyAttr>())
         return false;
 
-      // Skip anything that isn't 'public' or '@usableFromInline',
-      // or SPI if desired.
+      // Skip SPI decls if `PrintSPIs`.
+      if (!options.PrintSPIs && D->isSPI())
+        return false;
+
+      // Skip anything that isn't 'public' or '@usableFromInline'.
       if (auto *VD = dyn_cast<ValueDecl>(D)) {
         if (!isPublicOrUsableFromInline(VD)) {
           // We do want to print private stored properties, without their
@@ -155,10 +158,6 @@ PrintOptions PrintOptions::printSwiftInterfaceFile(bool preferTypeRepr,
             if (contributesToParentTypeStorage(ASD))
               return true;
 
-          // Always print SPI decls if `PrintSPIs`.
-          if (options.PrintSPIs && VD->isSPI())
-            return true;
-
           return false;
         }
       }