Merge pull request swiftlang#37574 from rintaro/sourcekit-open-noop-rdar77693214

[SourceKit] Avoid unnecessary parsing in 'open' request

Author: rintaro

test/SourceKit/Sema/edit_nowait.swift

@@ -1,26 +1,36 @@
 // RUN: %empty-directory(%t)
-// RUN: echo "" > %t/t.swift
+// RUN: echo "func f() { f() }" > %t/t.swift
 
 // RUN: %sourcekitd-test \
-// RUN:   -req=open %t/t.swift -- %t/t.swift == \
-// RUN:   -req=edit -offset=0 -replace="func foo() { warn("") }" -length=0 -req-opts=enablesyntaxmap=0,enablesubstructure=0,enablediagnostics=0 %t/t.swift -print-raw-response == \
+// RUN:   -req=open -req-opts=enablesyntaxmap=0,enablesubstructure=0,enablediagnostics=0 %t/t.swift -print-raw-response -- %t/t.swift == \
+// RUN:   -req=edit -offset=0 -replace="func foo() { warn("") }" -length=16 -req-opts=enablesyntaxmap=0,enablesubstructure=0,enablediagnostics=0 %t/t.swift -print-raw-response == \
 // RUN:   -req=edit -offset=13 -replace="print" -length=5 -req-opts=enablesyntaxmap=0,enablesubstructure=0,enablediagnostics=0 %t/t.swift -print-raw-response \
 // RUN: | %FileCheck --check-prefix=EDIT_NOWAIT %s
 
 // EDIT_NOWAIT:      {
 // EDIT_NOWAIT-NEXT: }
 // EDIT_NOWAIT-NEXT: {
 // EDIT_NOWAIT-NEXT: }
+// EDIT_NOWAIT-NEXT: {
+// EDIT_NOWAIT-NEXT: }
 
 // RUN: %sourcekitd-test \
-// RUN: %sourcekitd-test \
-// RUN:   -req=open %t/t.swift -- %t/t.swift == \
-// RUN:   -req=edit -offset=0 -replace="func foo() { warn("") }" -length=0 -req-opts=enablesyntaxmap=0,enablesubstructure=0,enablediagnostics=0 %t/t.swift == \
+// RUN:   -req=open -req-opts=enablesyntaxmap=0,enablesubstructure=0,enablediagnostics=0 %t/t.swift -- %t/t.swift == \
+// RUN:   -req=print-annotations %t/t.swift  == \
+// RUN:   -req=edit -offset=0 -replace="func foo() { warn("") }" -length=16 -req-opts=enablesyntaxmap=0,enablesubstructure=0,enablediagnostics=0 %t/t.swift == \
 // RUN:   -req=edit -offset=13 -replace="print" -length=4 -req-opts=enablesyntaxmap=0,enablesubstructure=0,enablediagnostics=0 %t/t.swift == \
-// RUN:   -req=print-annotations %s \
+// RUN:   -req=print-annotations %t/t.swift \
 // RUN: | %FileCheck --check-prefix=ANNOTATION %s
 
-// ANNOTATION:      [
+// ANNOTATION-LABEL: [
+// ANNOTATION-NEXT:   {
+// ANNOTATION-NEXT:     key.kind: source.lang.swift.ref.function.free,
+// ANNOTATION-NEXT:     key.offset: 11,
+// ANNOTATION-NEXT:     key.length: 1
+// ANNOTATION-NEXT:   }
+// ANNOTATION-NEXT: ]
+
+// ANNOTATION-LABEL: [
 // ANNOTATION-NEXT:   {
 // ANNOTATION-NEXT:     key.kind: source.lang.swift.ref.function.free,
 // ANNOTATION-NEXT:     key.offset: 13,

tools/SourceKit/lib/SwiftLang/SwiftEditor.cpp

@@ -2358,6 +2358,13 @@ void SwiftLangSupport::editorOpen(
     EditorDoc->updateSemaInfo();
   }
 
+  if (!Consumer.documentStructureEnabled() &&
+      !Consumer.syntaxMapEnabled() &&
+      !Consumer.diagnosticsEnabled() &&
+      !Consumer.syntaxTreeEnabled()) {
+    return;
+  }
+
   EditorDoc->readSyntaxInfo(Consumer, /*ReportDiags=*/true);
 
   if (Consumer.syntaxTreeEnabled()) {