RequirementMachine: Move a bit of code in computeCompletion() for clarity

Author: slavapestov

lib/AST/RequirementMachine/RequirementMachine.cpp

@@ -485,6 +485,10 @@ RequirementMachine::computeCompletion(RewriteSystem::ValidityPolicy policy) {
 
       unsigned rulesAdded = (System.getRules().size() - ruleCount);
 
+      // If buildPropertyMap() didn't add any new rules, we are done.
+      if (rulesAdded == 0)
+        break;
+
       if (Stats) {
         Stats->getFrontendCounters()
           .NumRequirementMachineUnifiedConcreteTerms += rulesAdded;
@@ -507,10 +511,6 @@ RequirementMachine::computeCompletion(RewriteSystem::ValidityPolicy policy) {
         return std::make_pair(CompletionResult::MaxRuleCount,
                               System.getRules().size() - 1);
       }
-
-      // If buildPropertyMap() didn't add any new rules, we are done.
-      if (rulesAdded == 0)
-        break;
     }
   }