[Explicit Module Builds] Avoid adding cross-target input dependencies to intermediate build artifact jobs.

For a given target, in the Explicit Module Build world, swift-driver produces jobs to build the main module and jobs to build all module dependencies.
cross-target dependencies should be added as explicit inputs to a target's main module build job, but not intermediate build dependency jobs.

Always adding cross-target dependencies as inputs can lead to cycles in the build manifest when multiple targets share common intermediate module dependencies.

Author: artemcm

Sources/Build/ManifestBuilder.swift

@@ -206,6 +206,7 @@ extension LLBuildManifestBuilder {
             if buildParameters.useExplicitModuleBuild {
               commandLine.append("-experimental-explicit-module-build")
             }
+
             var driver = try Driver(args: commandLine, fileSystem: target.fs)
             let jobs = try driver.planBuild()
             let resolver = try ArgsResolver(fileSystem: target.fs)
@@ -218,22 +219,35 @@ extension LLBuildManifestBuilder {
                 let jobInputs = job.inputs.map { $0.resolveToNode() }
                 let jobOutputs = job.outputs.map { $0.resolveToNode() }
 
+                // Add target dependencies as inputs to the main module build command.
+                //
+                // Jobs for a target's intermediate build artifacts, such as PCMs or
+                // modules built from a .swiftinterface, do not have a
+                // dependency on cross-target build products. If multiple targets share
+                // common intermediate dependency modules, such dependencies can lead
+                // to cycles in the resulting manifest.
+                var manifestNodeInputs = jobInputs
+                if !job.inputs.contains(where: { $0.type == .swiftInterface }) &&
+                    !job.outputs.contains(where: { $0.type == .pcm }) {
+                  manifestNodeInputs = (inputs + jobInputs).uniqued()
+                }
+
                 let moduleName = target.target.c99name
                 let description = job.description
                 if job.kind.isSwiftFrontend {
                     manifest.addSwiftFrontendCmd(
                         name: jobOutputs.first!.name,
                         moduleName: moduleName,
                         description: description,
-                        inputs: (inputs + jobInputs).uniqued(),
+                        inputs: manifestNodeInputs,
                         outputs: jobOutputs,
                         args: arguments
                     )
                 } else {
                     manifest.addShellCmd(
                         name: jobOutputs.first!.name,
                         description: description,
-                        inputs: (inputs + jobInputs).uniqued(),
+                        inputs: manifestNodeInputs,
                         outputs: jobOutputs,
                         args: arguments
                     )