[Serialization] Fix isUserAccessible serialization on functions only

Author: xymus

lib/Serialization/Deserialization.cpp

@@ -3194,14 +3194,16 @@ class DeclDeserializer {
     fn->setForcedStaticDispatch(hasForcedStaticDispatch);
     ctx.evaluator.cacheOutput(NeedsNewVTableEntryRequest{fn},
                               std::move(needsNewVTableEntry));
-    fn->setUserAccessible(isUserAccessible);
 
     if (opaqueReturnTypeID) {
       ctx.evaluator.cacheOutput(
           OpaqueResultTypeRequest{fn},
           cast<OpaqueTypeDecl>(MF.getDecl(opaqueReturnTypeID)));
     }
 
+    if (!isAccessor)
+      fn->setUserAccessible(isUserAccessible);
+
     return fn;
   }
 

lib/Serialization/ModuleFormat.h

@@ -55,7 +55,7 @@ const uint16_t SWIFTMODULE_VERSION_MAJOR = 0;
 /// describe what change you made. The content of this comment isn't important;
 /// it just ensures a conflict if two people change the module format.
 /// Don't worry about adhering to the 80-column limit for this line.
-const uint16_t SWIFTMODULE_VERSION_MINOR = 573; // @async on SILFunctionType
+const uint16_t SWIFTMODULE_VERSION_MINOR = 574; // reapply isUserAccessible
 
 /// A standard hash seed used for all string hashes in a serialized module.
 ///