Stream: git-wasmtime

Topic: wasmtime / PR #11522 [Cranelift] `X * (1 << Y) --> X << Y`


view this post on Zulip Wasmtime GitHub notifications bot (Aug 25 2025 at 01:54):

bongjunj opened PR #11522 from bongjunj:mul-shl1-to-shl to bytecodealliance:main:

<!--
Please make sure you include the following information:

Our development process is documented in the Wasmtime book:
https://docs.wasmtime.dev/contributing-development-process.html

Please ensure all communication follows the code of conduct:
https://github.com/bytecodealliance/wasmtime/blob/main/CODE_OF_CONDUCT.md
-->

This adds an optimization X * (1 << Y) --> X << Y.
The same proof with Crocus is used to verify the correctness. (See https://github.com/bytecodealliance/wasmtime/pull/11359)

view this post on Zulip Wasmtime GitHub notifications bot (Aug 25 2025 at 01:54):

bongjunj requested cfallin for a review on PR #11522.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 25 2025 at 01:54):

bongjunj requested wasmtime-compiler-reviewers for a review on PR #11522.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 25 2025 at 01:54):

bongjunj updated PR #11522.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 25 2025 at 03:10):

github-actions[bot] commented on PR #11522:

Subscribe to Label Action

cc @cfallin, @fitzgen

<details>
This issue or pull request has been labeled: "cranelift", "isle"

Thus the following users have been cc'd because of the following labels:

To subscribe or unsubscribe from this label, edit the <code>.github/subscribe-to-label.json</code> configuration file.

Learn more.
</details>

view this post on Zulip Wasmtime GitHub notifications bot (Aug 25 2025 at 15:07):

cfallin submitted PR review:

Thanks!

view this post on Zulip Wasmtime GitHub notifications bot (Aug 25 2025 at 18:22):

fitzgen submitted PR review:

And a runtest for this one as well, please. Thank you, it is very appreciated!

view this post on Zulip Wasmtime GitHub notifications bot (Aug 25 2025 at 18:24):

fitzgen commented on PR #11522:

(CI failure looks like spurious apt-related stuff, probably won't happen next time it is enqueued for merging)

view this post on Zulip Wasmtime GitHub notifications bot (Aug 27 2025 at 00:43):

bongjunj updated PR #11522.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 27 2025 at 00:44):

bongjunj commented on PR #11522:

Thank you for the review.
I've added the runtests for the new opt.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 27 2025 at 15:22):

fitzgen submitted PR review:

Thanks!

view this post on Zulip Wasmtime GitHub notifications bot (Aug 27 2025 at 16:02):

fitzgen merged PR #11522.


Last updated: Dec 06 2025 at 06:05 UTC