Merge pull request #74063 from ahoppen/validate-system-dependencies

[SourceKit] Don’t set `DisableModulesValidateSystemDependencies` to `true`

Author: ahoppen

lib/IDETool/CompilerInvocation.cpp

@@ -256,8 +256,6 @@ bool ide::initCompilerInvocation(
                                      std::to_string(sessionTimestamp - 1));
     ImporterOpts.ExtraArgs.push_back(
         "-fmodules-validate-once-per-build-session");
-
-    SearchPathOpts.DisableModulesValidateSystemDependencies = true;
   }
 
   // Disable expensive SIL options to reduce time spent in SILGen.

test/SourceKit/test-change-modulemap-of-system-module.swift

@@ -0,0 +1,25 @@
+// RUN: %empty-directory(%t)
+// RUN: split-file %s %t
+// RUN: %sourcekitd-test -req=diags %t/test.swift -- %t/test.swift -Xcc -fmodule-map-file=%t/module.modulemap -module-cache-path %t/module-cache | %FileCheck %s
+// Sleep for 1 second so that touching the modulemap modifies its timestamp on the second level.
+// RUN: sleep 1
+// RUN: touch %t/module.modulemap
+// RUN: %sourcekitd-test -req=diags %t/test.swift -- %t/test.swift -Xcc -fmodule-map-file=%t/module.modulemap -module-cache-path %t/module-cache | %FileCheck %s
+
+// CHECK: 'guard' body must not fall through
+
+//--- test.swift
+
+import Lib
+
+func test(value: Bool) {
+  guard value else {
+
+  }
+}
+
+//--- module.modulemap
+
+module Lib [system] {
+  export *
+}