-
Notifications
You must be signed in to change notification settings - Fork 111
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
DW_OP_WASM_location is unsupported #544
Comments
Yes, it will be nice to have Wasm DWARF support. At least under the feature. We are using crude hack in the wasmtime project. |
Are the semantics of this operation similar to |
@philipc I guess depends what you mean by semantics... The attached link likely explains it better than I could. |
Does it complete the expression or piece (as in |
AFAIK it's the former, yes. |
So having implemented this in #546, I see that llvm is emitting a |
Note that this is being fixed in https://reviews.llvm.org/D94347/new/, which also adds support for indirect values (0x02). So |
When trying to
dwarfdump
any Wasm file produced with latest Clang or Rust,DW_AT_frame_base
looks like this:Looks like Gimli needs to be taught
DW_OP_WASM_location
as described at https://yurydelendik.github.io/webassembly-dwarf/#DWARF-expressions-and-location-descriptions. /cc @yurydelendikThe text was updated successfully, but these errors were encountered: