Skip to content

Add support for more source map fields#7473

Merged
dschuff merged 2 commits intomainfrom
sourcemap-sources
Apr 9, 2025
Merged

Add support for more source map fields#7473
dschuff merged 2 commits intomainfrom
sourcemap-sources

Conversation

@dschuff
Copy link
Copy Markdown
Member

@dschuff dschuff commented Apr 9, 2025

Read the "sourcesContent", "file", and "sourceRoot" fields from incoming
source maps, attach them to the wasm IR module, and write them back to
the output source map. These fields are unchanged by Binaryen's
updates to the mappings, so they do not need to be decoded or interpreted.
Fixes #6805

Read the "sourcesContent", "file", and "sourceRoot" fields from incoming
source maps, attach them to the wasm IR module, and write them back to
the output source map. These fields are unchanged by Binaryen's
updates to the mappings, so they do not need to be decoded or interpreted.
Fixes #6805
@dschuff dschuff requested a review from kripken April 9, 2025 17:02
@dschuff dschuff enabled auto-merge (squash) April 9, 2025 17:55
@dschuff dschuff merged commit 2172402 into main Apr 9, 2025
14 checks passed
@dschuff dschuff deleted the sourcemap-sources branch April 9, 2025 18:25
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.

wasm-emscripten-finalize doesn't preserve sourcesContent

2 participants