SwiftModuleBuildDescription: Fix diagnostic file paths for WMO builds

WMO builds have a single frontend invocation and produce a single
diagnostic file named after the module.

Author: AnthonyLatsis

Sources/Build/BuildDescription/SwiftModuleBuildDescription.swift

@@ -1126,10 +1126,22 @@ extension SwiftModuleBuildDescription {
 
 extension SwiftModuleBuildDescription {
     package var diagnosticFiles: [AbsolutePath] {
-        self.sources.compactMap { self.diagnosticFile(sourceFile: $0) }
+        // WMO builds have a single frontend invocation and produce a single
+        // diagnostic file named after the module.
+        if self.useWholeModuleOptimization {
+            return [
+                self.diagnosticFile(name: self.target.name)
+            ]
+        }
+
+        return self.sources.map(self.diagnosticFile(sourceFile:))
+    }
+
+    private func diagnosticFile(name: String) -> AbsolutePath {
+        self.tempsPath.appending(component: "\(name).dia")
     }
 
     private func diagnosticFile(sourceFile: AbsolutePath) -> AbsolutePath {
-        self.tempsPath.appending(component: "\(sourceFile.basenameWithoutExt).dia")
+        self.diagnosticFile(name: sourceFile.basenameWithoutExt)
     }
 }

Tests/CommandsTests/PackageCommandTests.swift

@@ -2168,7 +2168,7 @@ class PackageCommandTestCase: CommandsBuildProviderTestCase {
         }
     }
 
-    func testMigrateCommand() async throws {
+    func _testMigrateCommand(configuration: BuildConfiguration) async throws {
         try XCTSkipIf(
             !UserToolchain.default.supportesSupportedFeatures,
             "skipping because test environment compiler doesn't support `-print-supported-features`"
@@ -2196,7 +2196,7 @@ class PackageCommandTestCase: CommandsBuildProviderTestCase {
                 }
 
                 let (stdout, _) = try await self.execute(
-                    ["migrate", "--to-feature", featureName],
+                    ["migrate", "-c", configuration.rawValue, "--to-feature", featureName],
                     packagePath: fixturePath
                 )
 
@@ -2220,6 +2220,14 @@ class PackageCommandTestCase: CommandsBuildProviderTestCase {
         try await doMigration(featureName: "InferIsolatedConformances", expectedSummary: "Applied 3 fix-its in 2 files")
     }
 
+    func testMigrateCommandDebug() async throws {
+        try await _testMigrateCommand(configuration: .debug)
+    }
+
+    func testMigrateCommandRelease() async throws {
+        try await _testMigrateCommand(configuration: .release)
+    }
+
     func testMigrateCommandWithBuildToolPlugins() async throws {
         try XCTSkipIf(
             !UserToolchain.default.supportesSupportedFeatures,
@@ -4432,7 +4440,19 @@ class PackageCommandSwiftBuildTests: PackageCommandTestCase {
         try await super.testNoParameters()
     }
 
-    override func testMigrateCommand() async throws {
+    override func testMigrateCommandDebug() async throws {
+        try XCTSkipOnWindows(
+            because: """
+            Possibly https://github.com/swiftlang/swift-package-manager/issues/8602:
+            error: Could not choose a single platform for target 'AllIncludingTests' from the supported platforms 'android qnx webassembly'. Specialization parameters imposed by workspace: platform 'nil' sdkVariant 'nil' supportedPlatforms: 'nil' toolchain: 'nil'
+            """,
+            skipPlatformCi: true,
+        )
+
+        try await super.testMigrateCommandDebug()
+    }
+
+    override func testMigrateCommandRelease() async throws {
         try XCTSkipOnWindows(
             because: """
             Possibly https://github.com/swiftlang/swift-package-manager/issues/8602:
@@ -4441,7 +4461,7 @@ class PackageCommandSwiftBuildTests: PackageCommandTestCase {
             skipPlatformCi: true,
         )
 
-        try await super.testMigrateCommand()
+        try await super.testMigrateCommandRelease()
     }
 
     override func testMigrateCommandUpdateManifest2Targets() async throws {