[TypeChecker/CodeCompletion] Re-introduce expression sanitization before solving

A call to `SanitizeExpr` has been incorrectly removed from
`typeCheckForCodeCompletion` by refactoring to use `ASTNode`.

It is still required because fallback calls could have partially
type-checked AST.

Resolves: #59315
Resolves: rdar://94619388
(cherry picked from commit 3159591)

Author: xedin

lib/Sema/TypeCheckCodeCompletion.cpp

@@ -575,6 +575,11 @@ bool TypeChecker::typeCheckForCodeCompletion(
   if (!node)
     return false;
 
+  if (auto *expr = getAsExpr(node)) {
+    node = expr->walk(SanitizeExpr(Context,
+                                   /*shouldReusePrecheckedType=*/false));
+  }
+
   CompletionContextFinder contextAnalyzer(node, DC);
 
   // If there was no completion expr (e.g. if the code completion location was

validation-test/IDE/stress_tester_issues_fixed/rdar94619388.swift

@@ -0,0 +1,8 @@
+// RUN: %swift-ide-test -code-completion -source-filename %s -code-completion-token COMPLETE
+
+func foo(closure: (String) -> Void) -> String? {
+  return nil
+}
+
+func test() {
+  if let key = foo(closure: { str in str.suffix(2) == #^COMPLETE^#