Skip to content

[lldb] Add PlatformWebInspectorWasm (#188751)#12666

Merged
JDevlieghere merged 1 commit intostable/21.xfrom
cherrypick-webinspector-platform
Mar 27, 2026
Merged

[lldb] Add PlatformWebInspectorWasm (#188751)#12666
JDevlieghere merged 1 commit intostable/21.xfrom
cherrypick-webinspector-platform

Conversation

@JDevlieghere
Copy link
Copy Markdown

Add a new PlatformWebInspectorWasm, which is a Wasm platform that automatically connects to the WebInspector platform server.

The existing "wasm" platform handles WebAssembly generally and allows you to configure a runtime to launch under. The "webinspector-wasm" platform does the inverse, and only supports attaching to an already running WebAssembly instance in Safari. The workflow here is always platform process list followed by platform process attach. This explains why you can only force create this platform and it's never automatically selected when loading a Wasm target.

(cherry picked from commit 7119610)

Add a new PlatformWebInspectorWasm, which is a Wasm platform that
automatically connects to the WebInspector platform server.

The existing "wasm" platform handles WebAssembly generally and allows
you to configure a runtime to launch under. The "webinspector-wasm"
platform does the inverse, and only supports attaching to an already
running WebAssembly instance in Safari. The workflow here is always
`platform process list` followed by `platform process attach`. This
explains why you can only force create this platform and it's never
automatically selected when loading a Wasm target.

(cherry picked from commit 7119610)
@JDevlieghere JDevlieghere requested a review from a team as a code owner March 27, 2026 15:25
@JDevlieghere
Copy link
Copy Markdown
Author

@swift-ci test

@JDevlieghere JDevlieghere merged commit c50bd74 into stable/21.x Mar 27, 2026
3 checks passed
@JDevlieghere JDevlieghere deleted the cherrypick-webinspector-platform branch March 27, 2026 21:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants