Merge pull request #2027 from swiftlang/maxd/wasm-stack-size

Wasm: fix stack size flag (no double dash, add `-z`)

Author: rjmansfield

Sources/SwiftDriver/Jobs/WebAssemblyToolchain+LinkerSupport.swift

@@ -170,7 +170,9 @@ extension WebAssemblyToolchain {
       // workflows like Swift Testing can run within this limited stack space.
       let SWIFT_WASM_DEFAULT_STACK_SIZE = 1024 * 128
       commandLine.appendFlag(.Xlinker)
-      commandLine.appendFlag("--stack-size=\(SWIFT_WASM_DEFAULT_STACK_SIZE)")
+      commandLine.appendFlag("-z")
+      commandLine.appendFlag(.Xlinker)
+      commandLine.appendFlag("stack-size=\(SWIFT_WASM_DEFAULT_STACK_SIZE)")
 
       // Delegate to Clang for sanitizers. It will figure out the correct linker
       // options.

Tests/SwiftDriverTests/SwiftDriverTests.swift

@@ -2766,7 +2766,7 @@ final class SwiftDriverTests: XCTestCase {
         XCTAssertTrue(cmd.contains(.responseFilePath(.absolute(path.appending(components: "wasi", "static-executable-args.lnk")))))
         XCTAssertTrue(cmd.contains(subsequence: [.flag("-Xlinker"), .flag("--global-base=4096")]))
         XCTAssertTrue(cmd.contains(subsequence: [.flag("-Xlinker"), .flag("--table-base=4096")]))
-        XCTAssertTrue(cmd.contains(subsequence: [.flag("-Xlinker"), .flag("--stack-size=\(128 * 1024)")]))
+        XCTAssertTrue(cmd.contains(subsequence: [.flag("-Xlinker"), .flag("-z"), .flag("-Xlinker"), .flag("stack-size=\(128 * 1024)")]))
         XCTAssertTrue(cmd.contains(.flag("-O3")))
         XCTAssertEqual(linkJob.outputs[0].file, try toPath("Test"))