From 9468c9fcfbc497a58a22e36b9c1f4ed63297d208 Mon Sep 17 00:00:00 2001 From: Finagolfin Date: Sun, 5 Nov 2023 20:31:29 +0530 Subject: [PATCH] Don't add an autolink-extract job unless actually linking ELF/Wasm objects, matching the original C++ driver --- Sources/SwiftDriver/Jobs/AutolinkExtractJob.swift | 3 ++- Tests/SwiftDriverTests/SwiftDriverTests.swift | 14 ++++---------- 2 files changed, 6 insertions(+), 11 deletions(-) diff --git a/Sources/SwiftDriver/Jobs/AutolinkExtractJob.swift b/Sources/SwiftDriver/Jobs/AutolinkExtractJob.swift index aa47fd95f..9b6eda562 100644 --- a/Sources/SwiftDriver/Jobs/AutolinkExtractJob.swift +++ b/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? { diff --git a/Tests/SwiftDriverTests/SwiftDriverTests.swift b/Tests/SwiftDriverTests/SwiftDriverTests.swift index d23f94a02..13eae549b 100644 --- a/Tests/SwiftDriverTests/SwiftDriverTests.swift +++ b/Tests/SwiftDriverTests/SwiftDriverTests.swift @@ -3347,17 +3347,11 @@ 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"))) } @@ -3365,7 +3359,7 @@ final class SwiftDriverTests: XCTestCase { // 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"))) }