[Sema] Swap init accessor check in MemberwiseInitPropertiesRequest

Currently `init` accessors are always memberwise initialized, but I'm
planning on changing that. As such we need to make sure we record
the `init` accessor regardless of whether it is included in the memberwise
init.

Author: hamishknight

lib/Sema/TypeCheckStorage.cpp

@@ -386,10 +386,6 @@ MemberwiseInitPropertiesRequest::evaluate(Evaluator &evaluator,
   SmallPtrSet<VarDecl *, 4> subsumedViaInitAccessor;
 
   for (auto *var : initableVars) {
-    // We only care about properties that are memberwise initialized.
-    if (!var->isMemberwiseInitialized(/*preferDeclaredProperties=*/true))
-      continue;
-
     // If this property has an init accessor, it subsumes all of the stored properties
     // that the accessor initializes. Mark those stored properties as being subsumed; we'll
     // get back to them later.
@@ -399,6 +395,10 @@ MemberwiseInitPropertiesRequest::evaluate(Evaluator &evaluator,
       }
     }
 
+    // We only care about properties that are memberwise initialized.
+    if (!var->isMemberwiseInitialized(/*preferDeclaredProperties=*/true))
+      continue;
+
     // Add this property.
     results.push_back(var);
   }