Merge pull request #68589 from xedin/remove-isinresultbuildercontext

[Sema] NFC: Remove obsolete `isInResultBuilderContext` method

Author: xedin

include/swift/Sema/ConstraintSystem.h

@@ -5497,10 +5497,6 @@ class ConstraintSystem {
   /// part of the constraint system.
   void forEachExpr(Expr *expr, llvm::function_ref<Expr *(Expr *)> callback);
 
-  /// Determine whether one of the parent closures the given one is nested
-  /// in (if any) has a result builder applied to its body.
-  bool isInResultBuilderContext(ClosureExpr *closure) const;
-
   /// Determine whether referencing the given member on the
   /// given existential base type is supported. This is the case only if the
   /// type of the member, spelled in the context of \p baseTy, does not contain

lib/Sema/CSSyntacticElement.cpp

@@ -1539,25 +1539,6 @@ void ConstraintSystem::generateConstraints(ArrayRef<ExprPattern *> exprPatterns,
                     referencedTypeVars);
 }
 
-bool ConstraintSystem::isInResultBuilderContext(ClosureExpr *closure) const {
-  if (!closure->hasSingleExpressionBody()) {
-    auto *DC = closure->getParent();
-    do {
-      // Result builder is applied to a function/getter body.
-      if (auto *AFD = dyn_cast<AbstractFunctionDecl>(DC)) {
-        if (resultBuilderTransformed.count(AFD))
-          return true;
-      }
-
-      if (auto *parentClosure = dyn_cast<ClosureExpr>(DC)) {
-        if (resultBuilderTransformed.count(parentClosure))
-          return true;
-      }
-    } while ((DC = DC->getParent()));
-  }
-  return false;
-}
-
 bool isConditionOfStmt(ConstraintLocatorBuilder locator) {
   if (!locator.endsWith<LocatorPathElt::Condition>())
     return false;