Merge pull request #41764 from ktoso/wip-flag-module

[Distributed] rename _Distributed to module

Author: swift-ci

include/swift/AST/DiagnosticsSema.def

@@ -4534,7 +4534,7 @@ ERROR(distributed_actor_isolated_non_self_reference,none,
       "non-isolated context",
       (DescriptiveDeclKind, DeclName))
 ERROR(distributed_actor_needs_explicit_distributed_import,none,
-      "'_Distributed' module not imported, required for 'distributed actor'",
+      "'Distributed' module not imported, required for 'distributed actor'",
       ())
 ERROR(actor_isolated_inout_state,none,
       "actor-isolated %0 %1 cannot be passed 'inout' to"

include/swift/AST/KnownIdentifiers.def

@@ -60,7 +60,7 @@ IDENTIFIER(CoreFoundation)
 IDENTIFIER(count)
 IDENTIFIER(CVarArg)
 IDENTIFIER(Darwin)
-IDENTIFIER_(Distributed)
+IDENTIFIER(Distributed)
 IDENTIFIER(dealloc)
 IDENTIFIER(debugDescription)
 IDENTIFIER(Decodable)

include/swift/AST/TypeCheckRequests.h

@@ -2738,7 +2738,7 @@ class HasCircularRawValueRequest
   bool isCached() const { return true; }
 };
 
-/// Checks if the _Distributed module is available.
+/// Checks if the Distributed module is available.
 class DistributedModuleIsAvailableRequest
     : public SimpleRequest<DistributedModuleIsAvailableRequest, bool(Decl *),
                            RequestFlags::Cached> {

include/swift/Option/FrontendOptions.td

@@ -408,7 +408,7 @@ def disable_implicit_concurrency_module_import : Flag<["-"],
 
 def disable_implicit_distributed_module_import : Flag<["-"],
   "disable-implicit-distributed-module-import">,
-  HelpText<"Disable the implicit import of the _Distributed module.">;
+  HelpText<"Disable the implicit import of the Distributed module.">;
 
 def disable_arc_opts : Flag<["-"], "disable-arc-opts">,
   HelpText<"Don't run SIL ARC optimization passes.">;

include/swift/Strings.h

@@ -25,7 +25,7 @@ constexpr static const StringLiteral SWIFT_ONONE_SUPPORT = "SwiftOnoneSupport";
 /// The name of the Concurrency module, which supports that extension.
 constexpr static const StringLiteral SWIFT_CONCURRENCY_NAME = "_Concurrency";
 /// The name of the Distributed module, which supports that extension.
-constexpr static const StringLiteral SWIFT_DISTRIBUTED_NAME = "_Distributed";
+constexpr static const StringLiteral SWIFT_DISTRIBUTED_NAME = "Distributed";
 /// The name of the StringProcessing module, which supports that extension.
 constexpr static const StringLiteral SWIFT_STRING_PROCESSING_NAME = "_StringProcessing";
 /// The name of the SwiftShims module, which contains private stdlib decls.

lib/AST/Decl.cpp

@@ -5569,7 +5569,7 @@ void ProtocolDecl::computeKnownProtocolKind() const {
       !module->getName().is("Foundation") &&
       !module->getName().is("_Differentiation") &&
       !module->getName().is("_Concurrency") &&
-      !module->getName().is("_Distributed")) {
+      !module->getName().is("Distributed")) {
     const_cast<ProtocolDecl *>(this)->Bits.ProtocolDecl.KnownProtocol = 1;
     return;
   }

lib/SILGen/SILGenDistributed.cpp

@@ -115,7 +115,7 @@ static void emitDistributedIfRemoteBranch(SILGenFunction &SGF, SILLocation Loc,
 
   FuncDecl *isRemoteFn = ctx.getIsRemoteDistributedActor();
   assert(isRemoteFn && "Could not find 'is remote' function, is the "
-                       "'_Distributed' module available?");
+                       "'Distributed' module available?");
 
   ManagedValue selfAnyObject = B.createInitExistentialRef(
       Loc, SGF.getLoweredType(ctx.getAnyObjectType()), CanType(selfTy),

lib/Sema/TypeCheckDistributed.cpp

@@ -47,7 +47,7 @@ DistributedModuleIsAvailableRequest::evaluate(Evaluator &evaluator,
   if (C.getLoadedModule(C.Id_Distributed))
     return true;
 
-  // seems we're missing the _Distributed module, ask to import it explicitly
+  // seems we're missing the Distributed module, ask to import it explicitly
   decl->diagnose(diag::distributed_actor_needs_explicit_distributed_import);
   return false;
 }
@@ -639,7 +639,7 @@ void TypeChecker::checkDistributedActor(SourceFile *SF, NominalTypeDecl *nominal
   if (!nominal)
     return;
 
-  // ==== Ensure the _Distributed module is available,
+  // ==== Ensure the Distributed module is available,
   // without it there's no reason to check the decl in more detail anyway.
   if (!swift::ensureDistributedModuleLoaded(nominal))
     return;

lib/Sema/TypeCheckDistributed.h

@@ -34,7 +34,7 @@ class NominalTypeDecl;
 /********************* Distributed Actor Type Checking ************************/
 /******************************************************************************/
 
-// Diagnose an error if the _Distributed module is not loaded.
+// Diagnose an error if the Distributed module is not loaded.
 bool ensureDistributedModuleLoaded(Decl *decl);
 
 /// Check for illegal property declarations (e.g. re-declaring transport or id)

stdlib/cmake/modules/AddSwiftStdlib.cmake

@@ -1750,7 +1750,7 @@ function(add_swift_target_library name)
   # Turn off implicit import of _Concurrency when building libraries
   list(APPEND SWIFTLIB_SWIFT_COMPILE_FLAGS "-Xfrontend;-disable-implicit-concurrency-module-import")
 
-  # Turn off implicit import of _Distributed when building libraries
+  # Turn off implicit import of Distributed when building libraries
   if(SWIFT_ENABLE_EXPERIMENTAL_DISTRIBUTED)
     list(APPEND SWIFTLIB_SWIFT_COMPILE_FLAGS
                       "-Xfrontend;-disable-implicit-distributed-module-import")