Stream: git-wasmtime

Topic: wasmtime / issue #10733 `--emit-clif` files do not includ...

view this post on Zulip Wasmtime GitHub notifications bot (May 05 2025 at 21:35):

MonoidMusician opened issue #10733:

The IR included in explore includes function names with each section of IR, but this info doesn't end up in the similar files produced by --emit-clif which makes it hard to see what each function is (without cross-referencing with the explore output or something).

view this post on Zulip Wasmtime GitHub notifications bot (May 05 2025 at 21:49):

jameysharp commented on issue #10733:

As the person who asked you to open this issue, I want to add a couple details for whoever has time to poke at it:

wasmtime explore gets the function name here: https://github.com/bytecodealliance/wasmtime/blob/ede670dd937d73122ae36104fcea6a345c0890c1/crates/explorer/src/lib.rs#L222-L231

The implementation of --emit-clif is here: https://github.com/bytecodealliance/wasmtime/blob/a4700cb7eb86484ba4d5246cdb85bf07311a51a0/crates/cranelift/src/compiler.rs#L1000-L1004

In addition to explore, tests/disas.rs also uses emit-clif internally and IIRC also wants function names, so maybe this can all be unified a bit.

view this post on Zulip Wasmtime GitHub notifications bot (May 08 2025 at 16:28):

alexcrichton added the wasmtime:debugging label to Issue #10733.

view this post on Zulip Wasmtime GitHub notifications bot (May 08 2025 at 16:28):

alexcrichton added the good first issue label to Issue #10733.

view this post on Zulip Wasmtime GitHub notifications bot (Jun 06 2025 at 20:46):

PhantomInTheWire commented on issue #10733:

hey @alexcrichton! This looks like a good first issue! I think I have a solution for this and plan to submit a PR shortly. LMK if you'd like me to consider anything specific.

view this post on Zulip Wasmtime GitHub notifications bot (Jun 06 2025 at 20:48):

PhantomInTheWire edited a comment on issue #10733:

hey @alexcrichton! This looks like a good first issue! I think I have a solution for this I'd like to work on this. Could you please assign it to me?

view this post on Zulip Wasmtime GitHub notifications bot (Jun 06 2025 at 20:48):

PhantomInTheWire edited a comment on issue #10733:

hey @alexcrichton! This looks like a good first issue! I think I have a solution for this. Could you please assign it to me?

view this post on Zulip Wasmtime GitHub notifications bot (Jun 06 2025 at 20:54):

alexcrichton commented on issue #10733:

Sure thing, and one thing to be aware of perhaps is that https://github.com/bytecodealliance/wasmtime/pull/10947 just landed which changed things slightly in this area, but I believe this is still applicable with respect to wasmtime explore.

view this post on Zulip Wasmtime GitHub notifications bot (Jun 06 2025 at 20:54):

alexcrichton assigned PhantomInTheWire to issue #10733.

view this post on Zulip Wasmtime GitHub notifications bot (Jun 06 2025 at 21:05):

PhantomInTheWire commented on issue #10733:

Awesome, thanks for assigning it! I'll dig into this

view this post on Zulip Wasmtime GitHub notifications bot (Jun 16 2025 at 15:38):

fitzgen closed issue #10733:

The IR included in explore includes function names with each section of IR, but this info doesn't end up in the similar files produced by --emit-clif which makes it hard to see what each function is (without cross-referencing with the explore output or something).

Last updated: Dec 06 2025 at 06:05 UTC