Repository navigation
worker-build: emit Wasm source maps for emscripten builds - #1110
JesusFileto wants to merge 3 commits into
Conversation
Adds a `source-map` profile option for `--emscripten` builds. emcc links with `-gsource-map` and builds the map from the DWARF line tables of a `debug = "line-tables-only"` profile. The map is written to `build/index_bg.wasm.map` with workspace-relative sources, and the module's `sourceMappingURL` points to it. The Wasm binary helpers move from producers.rs to wasm_util.rs so both can use them.
|
Nit: It seems like if you build once with source maps on, then build again with them off, the old source maps stay in the build folder. Unless the upload step later has some way of knowing that the build its uploading doesn't contain source maps, I think it would upload the stale source maps which wouldn't be good. |
logan-gatlin
left a comment
logan-gatlin
left a comment
There was a problem hiding this comment.
Looks good, marked do-not-merge while we wait on emscripten to update so we can bump the pin
A build with `source-map` turned off left the previous build's index_bg.wasm.map in build/, beside a module it no longer describes. Remove any index_bg.wasm.* file the current build didn't write.
Callers no longer need to know the header's size, and custom_section returns a Range.
Good catch, in latest commit now all wasm source maps are deleted if its in the build folder before |
Adds a
source-mapprofile option for--emscriptenbuilds. Withdebug = "line-tables-only", emcc links with-gsource-map, which builds a source map from the DWARF line tables before Binaryen optimizes and keeps the debug info out of the module, so it stays fully optimized. worker-build writes the map tobuild/index_bg.wasm.mapand points the module'ssourceMappingURLat it.emcc names sources relative to the directory it wrote the map in, deep in cargo's
target/, so once the map moves tobuild/they point nowhere and still carry the build machine's home directory into uploaded maps. worker-build renames them as Cargo's still unstabletrim-pathswould: workspace sources relative to the workspace root, registry and git dependencies without$CARGO_HOME, and other paths absolute. This keeps names stable across machines and readable in stack traces, which is what they are used for, rather than paths relative to the map.Like debug builds, source maps send DWARF through wasm-bindgen, so they need wasm-bindgen 0.2.129, the first release whose output Binaryen accepts. The Wasm binary helpers move from producers.rs to wasm_util.rs, since the source map code needs them too.
This can only be merged once worker-build's Emscripten pin moves to 6.0.14 or later, the first release with emscripten-core/emscripten#28033. Without it, code at DWARF line 0, such as an
unreachablethat LLVM shares between several branches, maps to the line before it instead of to no location.