Merge pull request #26318 from nkcsgexi/52982457

AST: avoid unnecessarily calling getStartLoc() to avoid exponential growth of the stack trace

Author: nkcsgexi

include/swift/AST/Expr.h

@@ -2422,8 +2422,9 @@ class UnresolvedDotExpr : public Expr {
   }
 
   SourceLoc getStartLoc() const {
-    if (SubExpr->getStartLoc().isValid())
-      return SubExpr->getStartLoc();
+    auto SubLoc = SubExpr->getStartLoc();
+    if (SubLoc.isValid())
+      return SubLoc;
     else if (DotLoc.isValid())
       return DotLoc;
     else

test/IDE/coloring_52982457.swift

@@ -0,0 +1,116 @@
+// To ensure walking the long chained call expression doesn't take long time to finish
+// RUN: %target-swift-ide-test -syntax-coloring -source-filename %s | %FileCheck %s
+// RUN: %target-swift-ide-test -syntax-coloring -typecheck -source-filename %s | %FileCheck %s
+
+func migrate() -> EitherIO<Error, Prelude.Unit> {
+// CHECK: <kw>func</kw> migrate() -> <type>EitherIO</type><<type>Error</type>, <type>Prelude</type>.<type>Unit</type>> {
+  return self.execute(
+    """
+  """
+    )
+    .flatMap(const(execute(
+      """
+    """
+    )))
+    .flatMap(const(execute(
+      """
+    """
+    )))
+    .flatMap(const(execute(
+      """
+    """
+    )))
+    .flatMap(const(execute(
+      """
+    """
+    )))
+    .flatMap(const(execute(
+      """
+    """
+    )))
+    .flatMap(const(execute(
+      """
+    """
+    )))
+    .flatMap(const(execute(
+      """
+    """
+    )))
+    .flatMap(const(execute(
+      """
+    """
+    )))
+    .flatMap(const(execute(
+      """
+    """
+    )))
+    .flatMap(const(execute(
+      """
+    """
+    )))
+    .flatMap(const(execute(
+      """
+    """
+    )))
+    .flatMap(const(execute(
+      """
+    """
+    )))
+    .flatMap(const(execute(
+      """
+    """
+    )))
+    .flatMap(const(execute(
+      """
+    """
+    )))
+    .flatMap(const(execute(
+      """
+    """
+    )))
+    .flatMap(const(execute(
+      """
+    """
+    )))
+    .flatMap(const(execute(
+      """
+    """
+    )))
+    .flatMap(const(execute(
+      """
+    """
+    )))
+    .flatMap(const(execute(
+      """
+    """
+    )))
+    .flatMap(const(execute(
+      """
+    """
+    )))
+    .flatMap(const(execute(
+      """
+    """
+    )))
+    .flatMap(const(execute(
+      """
+    """
+    )))
+    .flatMap(const(execute(
+      """
+    """
+    )))
+    .flatMap(const(execute(
+      """
+    """
+    )))
+    .flatMap(const(execute(
+      """
+    """
+    )))
+    .flatMap(const(execute(
+      """
+    """
+    )))
+    .map(const(unit))
+}