BridgeJS: Remove forced capture workaround in async thunk body closures once swiftlang/swift#89715 lands

Offen

#761 erstellt am 10.6.2026 0 Kommentare 0 Reaktionen 1 zugewiesene Person Auf GitHub ansehen

@krodak arbeitet bereits daran.

Seit 10.6.2026.

Bewertung

Dieses Issue wurde noch nicht bewertet.

Beschreibung

Tracks removal of the workaround introduced in #760.

Context

A zero-parameter async throws(JSException) export generates a captureless _bjs_makePromise body closure, which lowers via thin_to_thick_function and miscompiles on Wasm (swiftlang/swift#89320): the thrown error is corrupted across the async unwind, trapping in Promise_reject or rejecting with a garbage value. #760 works around this at the codegen level by forcing a capture in the emitted body closure (the capture must also be read in the body, since an unread capture list entry is dropped by capture analysis and the closure stays thin):

let __bjs_capture = 0
return _bjs_makePromise(resolve: Promise_resolve_SS, reject: Promise_reject) { [__bjs_capture] () async throws(JSException) -> String in
    _ = __bjs_capture
    return try await ping()
}

What to do once the compiler fix ships

The proper fix is swiftlang/swift#89715 (IRGen: fix async typed throws miscompiles on Wasm). Once it is available in the oldest Swift toolchain JavaScriptKit supports:

Remove asyncThrowsBodyForcesCapture and the forced-capture emission from ExportedThunkBuilder in Plugins/BridgeJS/Sources/BridgeJSCore/ExportSwift.swift.
Regenerate snapshots and the AoT-committed bindings.
Keep the zeroArgAsyncThrows end-to-end regression test as the guard that the reject path still works without the workaround.

Vorherrschende Sprache: Swift
Sterne: 986
Forks: 76
Ø Merge: 21 Std. 11 Min.
Gemergte PRs (30 T.): 4

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus swiftwasm/JavaScriptKit

BridgeJS: Add diagnostics for unsupported `from: .global` / `from: .module(...)` parameters for imported symbols Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 55/100

swiftwasm/JavaScriptKit#793 · Erstellt am 23.7.2026
BridgeJS: Same-named `@JS` types across modules collide silently in generated glue Offen

swiftwasm/JavaScriptKit#785 · 1 zugewiesene Person · Erstellt am 06.7.2026
[SwiftBuild] Preserve duplicate archive members when SwiftBuild expands static archives for linking Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 48/100

swiftwasm/JavaScriptKit#781 · 3 Kommentare · Erstellt am 23.6.2026
☂️ `--build-system swiftbuild` compatibility issues Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 25/100

swiftwasm/JavaScriptKit#773 · Erstellt am 23.6.2026
BridgeJS: Remove JSException storage boxing once swiftlang/swift#89715 lands Offen

swiftwasm/JavaScriptKit#767 · 1 zugewiesene Person · Erstellt am 11.6.2026

Alle Issues in swiftwasm/JavaScriptKit