[Explicit Module Builds] Always rebuild the target module in explicit -compile-module-from-interface builds.

We have long switched to delegate the checking of whether a module is up-to-date to the build system (SwiftDriver) and stopped serializing file dependencies for interface-built binary modules.

Resolves rdar://162881032

Author: artemcm

include/swift/Basic/LangOptions.h

@@ -270,9 +270,6 @@ namespace swift {
     /// Emit a remark when indexing a system module.
     bool EnableIndexingSystemModuleRemarks = false;
 
-    /// Emit a remark on early exit in explicit interface build
-    bool EnableSkipExplicitInterfaceModuleBuildRemarks = false;
-
     ///
     /// Support for alternate usage modes
     ///

lib/Frontend/CompilerInvocation.cpp

@@ -1469,8 +1469,6 @@ static bool ParseLangArgs(LangOptions &Opts, ArgList &Args,
   Opts.EnableMacroLoadingRemarks = Args.hasArg(OPT_remark_macro_loading);
   Opts.EnableIndexingSystemModuleRemarks = Args.hasArg(OPT_remark_indexing_system_module);
 
-  Opts.EnableSkipExplicitInterfaceModuleBuildRemarks = Args.hasArg(OPT_remark_skip_explicit_interface_build);
-
   if (Args.hasArg(OPT_experimental_skip_non_exportable_decls)) {
     // Only allow -experimental-skip-non-exportable-decls if either library
     // evolution is enabled (in which case the module's ABI is independent of

lib/Frontend/ModuleInterfaceLoader.cpp

@@ -1609,27 +1609,6 @@ bool ModuleInterfaceLoader::buildExplicitSwiftModuleFromSwiftInterface(
     StringRef outputPath, bool ShouldSerializeDeps,
     ArrayRef<std::string> CompiledCandidates,
     DependencyTracker *tracker) {
-
-  if (!Instance.getInvocation().getIRGenOptions().AlwaysCompile) {
-    // First, check if the expected output already exists and possibly
-    // up-to-date w.r.t. all of the dependencies it was built with. If so, early
-    // exit.
-    UpToDateModuleCheker checker(
-        Instance.getASTContext());
-    ModuleRebuildInfo rebuildInfo;
-    SmallVector<FileDependency, 3> allDeps;
-    std::unique_ptr<llvm::MemoryBuffer> moduleBuffer;
-    if (checker.swiftModuleIsUpToDate(outputPath, rebuildInfo, allDeps,
-                                      moduleBuffer)) {
-      if (Instance.getASTContext()
-              .LangOpts.EnableSkipExplicitInterfaceModuleBuildRemarks) {
-        Instance.getDiags().diagnose(
-            SourceLoc(), diag::explicit_interface_build_skipped, outputPath);
-      }
-      return false;
-    }
-  }
-
   // Read out the compiler version.
   llvm::BumpPtrAllocator alloc;
   llvm::StringSaver ArgSaver(alloc);

test/ScanDependencies/explicit-interface-build.swift

@@ -0,0 +1,25 @@
+// RUN: %empty-directory(%t)
+// RUN: %empty-directory(%t/module-cache)
+// RUN: %empty-directory(%t/Foo.swiftmodule)
+// RUN: split-file %s %t
+
+// Build module from interface
+// RUN: %target-swift-frontend -compile-module-from-interface -o %t/Foo.swiftmodule/Foo.swiftmodule -module-name Foo %t/Foo.swiftmodule/Foo.swiftinterface -explicit-interface-module-build
+
+// Add to the textual interface
+// RUN: echo "struct GoodBye {}" >> %t/Foo.swiftmodule/Foo.swiftinterface
+
+// Build module from interface
+// RUN: %target-swift-frontend -compile-module-from-interface -o %t/Foo.swiftmodule/Foo.swiftmodule -module-name Foo %t/Foo.swiftmodule/Foo.swiftinterface -explicit-interface-module-build
+
+// Get the list of strings and ensure the binary module reflects the new addition
+// RUN: %llvm-strings %t/Foo.swiftmodule/Foo.swiftmodule > %t/module_strings.txt
+
+// RUN: cat %t/module_strings.txt | %FileCheck %s
+// CHECK-DAG: Hello
+// CHECK-DAG: GoodBye
+
+//--- Foo.swiftmodule/Foo.swiftinterface
+// swift-interface-format-version: 1.0
+// swift-module-flags: -module-name Foo
+struct Hello {}