Merge pull request #38168 from xedin/drop-unused-flag

[BuilderTransform] Remove unused `skipPrecheck` from `PreCheckResultB…

Author: xedin

lib/Sema/BuilderTransform.cpp

@@ -1906,13 +1906,6 @@ class PreCheckResultBuilderApplication : public ASTWalker {
 
 ResultBuilderBodyPreCheck PreCheckResultBuilderRequest::evaluate(
     Evaluator &evaluator, PreCheckResultBuilderDescriptor owner) const {
-  // We don't want to do the precheck if it will already have happened in
-  // the enclosing expression.
-  bool skipPrecheck = false;
-  if (auto closure = dyn_cast_or_null<ClosureExpr>(
-          owner.Fn.getAbstractClosureExpr()))
-    skipPrecheck = shouldTypeCheckInEnclosingExpression(closure);
-
   return PreCheckResultBuilderApplication(
              owner.Fn, /*skipPrecheck=*/false,
              /*suppressDiagnostics=*/owner.SuppressDiagnostics)