[Sema] Check cs.isForCodeCompletion() instead of checking for presence of CompletionCallback

This shouldn’t really change anything but is just a little cleaner.

Author: ahoppen

lib/Sema/CSRanking.cpp

@@ -1008,7 +1008,7 @@ SolutionCompareResult ConstraintSystem::compareSolutions(
     // problems with restating requirements in protocols.
     identical = false;
 
-    if (cs.Context.CompletionCallback) {
+    if (cs.isForCodeCompletion()) {
       // Don't rank based on overload choices of function calls that contain the
       // code completion token.
       ASTNode anchor = simplifyLocatorToAnchor(overload.locator);

lib/Sema/CSStep.cpp

@@ -945,7 +945,7 @@ StepResult ConjunctionStep::resume(bool prevFailed) {
 
       if (Solutions.size() == 1) {
         auto score = Solutions.front().getFixedScore();
-        if (score.Data[SK_Fix] > 0 && !CS.getASTContext().CompletionCallback)
+        if (score.Data[SK_Fix] > 0 && !CS.isForCodeCompletion())
           Producer.markExhausted();
       }
     } else if (Solutions.size() != 1) {