Merge pull request #36847 from apple/revert-36767-implicit-concurrency-import

Revert "Enable import of the _Concurrency module by default."

Author: DougGregor

lib/Frontend/Frontend.cpp

@@ -754,7 +754,8 @@ CompilerInstance::openModuleDoc(const InputFile &input) {
 }
 
 bool CompilerInvocation::shouldImportSwiftConcurrency() const {
-  return !getLangOptions().DisableImplicitConcurrencyModuleImport &&
+  return getLangOptions().EnableExperimentalConcurrency
+      && !getLangOptions().DisableImplicitConcurrencyModuleImport &&
       getFrontendOptions().InputMode !=
         FrontendOptions::ParseInputMode::SwiftModuleInterface;
 }

test/Concurrency/async_main_no_concurrency.swift

@@ -1,4 +1,4 @@
-// RUN: %target-typecheck-verify-swift -parse-as-library -disable-implicit-concurrency-module-import %s
+// RUN: %target-typecheck-verify-swift -parse-as-library %s
 // RUN: %target-typecheck-verify-swift -parse-as-library -enable-experimental-concurrency -parse-stdlib %s
 
 @main struct Main {

test/Driver/Inputs/android.sdk/usr/lib/swift/android/x86_64/_Concurrency.swiftinterface

@@ -1,5 +1,4 @@
 import SwiftOnoneSupport
-import _Concurrency
 
 func %%% (lhs: Int, rhs: Int) -> Int
 

test/IDE/Inputs/foo_swift_module.printed.comments.txt

@@ -1,5 +1,5 @@
 // RUN: %empty-directory(%t)
-// RUN: %target-swift-ide-test -disable-implicit-concurrency-module-import -batch-code-completion -source-filename %s -filecheck %raw-FileCheck -completion-output-dir %t
+// RUN: %target-swift-ide-test -batch-code-completion -source-filename %s -filecheck %raw-FileCheck -completion-output-dir %t
 
 // NO_STDLIB_PRIVATE: Begin completions
 // NO_STDLIB_PRIVATE-NOT: Decl{{[^:]*}}/IsSystem: _

test/IDE/complete_from_stdlib.swift

@@ -1,5 +1,5 @@
 // RUN: %empty-directory(%t)
-// RUN: %target-swift-ide-test -disable-implicit-concurrency-module-import -batch-code-completion -source-filename %s -filecheck %raw-FileCheck -completion-output-dir %t
+// RUN: %target-swift-ide-test -batch-code-completion -source-filename %s -filecheck %raw-FileCheck -completion-output-dir %t
 // RUN: sed -n -e '1,/NO_ERRORS_UP_TO_HERE$/ p' %s > %t_no_errors.swift
 // RUN: %target-swift-frontend -typecheck -verify -disable-objc-attr-requires-foundation-module -enable-objc-interop %t_no_errors.swift
 

test/IDE/complete_override.swift

@@ -1,11 +1,11 @@
-// RUN: %swiftc_driver -emit-ir %s -o - -Xfrontend -disable-implicit-concurrency-module-import | %FileCheck %s -check-prefix ELF
+// RUN: %swiftc_driver -emit-ir %s -o - | %FileCheck %s -check-prefix ELF
 
 // Check that the swift auto link section is available in the object file.
-// RUN: %swiftc_driver -c %s -o %t -Xfrontend -disable-implicit-concurrency-module-import
+// RUN: %swiftc_driver -c %s -o %t
 // RUN: llvm-readelf %t -S | %FileCheck %s -check-prefix SECTION
 
 // Checks that the swift auto link section is removed from the final binary.
-// RUN: %swiftc_driver  -emit-executable %s -o %t -Xfrontend -disable-implicit-concurrency-module-import
+// RUN: %swiftc_driver  -emit-executable %s -o %t
 // RUN: llvm-readelf %t -S | %FileCheck %s -check-prefix NOSECTION
 
 // REQUIRES: OS=linux-gnu

test/IRGen/ELF-remove-autolink-section.swift

@@ -30,7 +30,6 @@ from__ABAdditionsCAdditions()
 // MAIN-NEXT: is-debug: 1
 // MAIN-NEXT: DEPEND START
 // MAIN-NEXT: Unit | system | Swift | {{.*}}{{/|\\}}Swift.swiftmodule
-// MAIN-NEXT: Unit | system | _Concurrency | {{.*}}{{/|\\}}_Concurrency.swiftmodule
 // MAIN-NEXT: Unit | system | B | {{.*}}{{/|\\}}B.swiftmodule{{/|\\}}{{.*}}
 // MAIN-NEXT: Unit | system | C | {{.*}}{{/|\\}}C.swiftmodule{{/|\\}}{{.*}}
 // MAIN-NEXT: Unit | system | A | {{.*}}{{/|\\}}__ABAdditionsCAdditions.swiftmodule{{/|\\}}{{.*}}

test/Index/Store/cross-import-overlay.swift

@@ -16,7 +16,6 @@
 
 // CHECK: DEPEND START
 // CHECK: Unit | system | {{.*}}{{/|\\}}Swift.swiftmodule
-// CHECK: Unit | system | _Concurrency
-// CHECK: DEPEND END (2)
+// CHECK: DEPEND END (1)
 
 // OPT: is-debug: 1

test/Index/Store/unit-from-compile.swift

@@ -8,9 +8,8 @@
 // CHECK: 00-output_for_index
 // CHECK: DEPEND START
 // CHECK: Unit | system | Swift | [[MODULE:.*[/\\]Swift[.]swiftmodule([/\\].+[.]swiftmodule)?]] | [[SWIFT:.+[.]swiftmodule-[A-Z0-9]*]]
-// CHECK: Unit | system | _Concurrency | {{.*}}
 // CHECK: Record | user | {{.*}}{{/|\\}}unit-one-file-multi-file-invocation.swift |
-// CHECK: DEPEND END (3)
+// CHECK: DEPEND END (2)
 
 // CHECK: [[SWIFT]]
 // CHECK: DEPEND START