Skip to content
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

[DWARF] Translate this in a more accessible way #9855

Open
SingleAccretion opened this issue Dec 18, 2024 · 0 comments
Open

[DWARF] Translate this in a more accessible way #9855

SingleAccretion opened this issue Dec 18, 2024 · 0 comments
Labels
wasmtime:debugging Issues related to debugging of JIT'ed code

Comments

@SingleAccretion
Copy link
Contributor

Currently, for instance methods, this pointers are translated like all other pointers - wrapped in the synthetic types. However, this breaks LLDB's ability to do anything with them as it expects this to be a proper pointer.

(There are still ways to inspect the underlying data, of course, but they're more clunky than just p Member.)

The proposed fix, consistent with the existing system, would be to rename this and effectively transform all instance methods into static ones.

The user experience would then be p __this->Member instead of p Member.

A more ambitious alternative is to translate it into a proper pointer. Unlike for a number of other cases, we can do that (since we have the frame context available), but it's a bit more complicated (need to intelligently intersect ranges of __vmctx with those of this) and more importantly, it wouldn't be consistent with how the rest of the system operates.

@alexcrichton alexcrichton added the wasmtime:debugging Issues related to debugging of JIT'ed code label Dec 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wasmtime:debugging Issues related to debugging of JIT'ed code
Projects
None yet
Development

No branches or pull requests

2 participants