Merge pull request #1479 from finagolfin/autolink

Don't add an autolink-extract job unless actually linking ELF/Wasm objects, matching the original C++ driver

Author: bnbarham

Sources/SwiftDriver/Jobs/AutolinkExtractJob.swift

@@ -18,7 +18,8 @@ import struct TSCBasic.RelativePath
 // FIXME: Also handle Cygwin and MinGW
 extension Driver {
   /*@_spi(Testing)*/ public var isAutolinkExtractJobNeeded: Bool {
-    [.elf, .wasm].contains(targetTriple.objectFormat) && lto == nil
+    [.elf, .wasm].contains(targetTriple.objectFormat) && lto == nil &&
+    linkerOutputType != nil
   }
 
   mutating func autolinkExtractJob(inputs: [TypedVirtualPath]) throws -> Job? {

Tests/SwiftDriverTests/SwiftDriverTests.swift

@@ -3347,25 +3347,19 @@ final class SwiftDriverTests: XCTestCase {
       }
     }
 
-    #if os(Linux) || os(Android)
-    let autoLinkExtractJob = 1
-    #else
-    let autoLinkExtractJob = 0
-    #endif
-
     do {
       // non library-evolution builds require a single job, because cross-module-optimization is enabled by default.
       var driver = try Driver(args: ["swiftc", "foo.swift", "bar.swift", "-module-name", "Test", "-emit-module-path", rebase("Test.swiftmodule", at: root), "-c", "-o", rebase("test.o", at: root), "-wmo", "-O" ])
       let plannedJobs = try driver.planBuild()
-      XCTAssertEqual(plannedJobs.count, 1 + autoLinkExtractJob)
+      XCTAssertEqual(plannedJobs.count, 1)
       XCTAssertTrue(plannedJobs[0].commandLine.contains(.flag("-enable-default-cmo")))
     }
 
     do {
       // library-evolution builds can emit the module in a separate job.
       var driver = try Driver(args: ["swiftc", "foo.swift", "bar.swift", "-module-name", "Test", "-emit-module-path", rebase("Test.swiftmodule", at: root), "-c", "-o", rebase("test.o", at: root), "-wmo", "-O", "-enable-library-evolution" ])
       let plannedJobs = try driver.planBuild()
-      XCTAssertEqual(plannedJobs.count, 2 + autoLinkExtractJob)
+      XCTAssertEqual(plannedJobs.count, 2)
       XCTAssertFalse(plannedJobs[0].commandLine.contains(.flag("-enable-default-cmo")))
       XCTAssertFalse(plannedJobs[1].commandLine.contains(.flag("-enable-default-cmo")))
     }
@@ -3374,7 +3368,7 @@ final class SwiftDriverTests: XCTestCase {
       // When disabling cross-module-optimization, the module can be emitted in a separate job.
       var driver = try Driver(args: ["swiftc", "foo.swift", "bar.swift", "-module-name", "Test", "-emit-module-path", rebase("Test.swiftmodule", at: root), "-c", "-o", rebase("test.o", at: root), "-wmo", "-O", "-disable-cmo" ])
       let plannedJobs = try driver.planBuild()
-      XCTAssertEqual(plannedJobs.count, 2 + autoLinkExtractJob)
+      XCTAssertEqual(plannedJobs.count, 2)
       XCTAssertFalse(plannedJobs[0].commandLine.contains(.flag("-enable-default-cmo")))
       XCTAssertFalse(plannedJobs[1].commandLine.contains(.flag("-enable-default-cmo")))
     }
@@ -3383,7 +3377,7 @@ final class SwiftDriverTests: XCTestCase {
       // non optimized builds can emit the module in a separate job.
       var driver = try Driver(args: ["swiftc", "foo.swift", "bar.swift", "-module-name", "Test", "-emit-module-path", rebase("Test.swiftmodule", at: root), "-c", "-o", rebase("test.o", at: root), "-wmo" ])
       let plannedJobs = try driver.planBuild()
-      XCTAssertEqual(plannedJobs.count, 2 + autoLinkExtractJob)
+      XCTAssertEqual(plannedJobs.count, 2)
       XCTAssertFalse(plannedJobs[0].commandLine.contains(.flag("-enable-default-cmo")))
       XCTAssertFalse(plannedJobs[1].commandLine.contains(.flag("-enable-default-cmo")))
     }