Stream: git-wasmtime

Topic: wasmtime / PR #14056 x64: Remove an add128 lower rule tha...


view this post on Zulip Wasmtime GitHub notifications bot (Jul 31 2026 at 19:31):

alexcrichton edited PR #14056.

view this post on Zulip Wasmtime GitHub notifications bot (Jul 31 2026 at 19:31):

alexcrichton edited PR #14056:

A local fib benchmark shows that without this lowering rule wasm is 8%
slower than native, whereas with this lowering rule it's 24% slower than
native. The "FIXME" comment turns out can be resolved by just deleting
the entire rule.

view this post on Zulip Wasmtime GitHub notifications bot (Jul 31 2026 at 19:32):

alexcrichton commented on PR #14056:

Sorry, hit submit too soon with too many commits, updated PR/description now

view this post on Zulip Wasmtime GitHub notifications bot (Jul 31 2026 at 21:03):

github-actions[bot] added the label cranelift on PR #14056.

view this post on Zulip Wasmtime GitHub notifications bot (Jul 31 2026 at 21:03):

github-actions[bot] added the label cranelift:area:x64 on PR #14056.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 05 2026 at 18:54):

:thumbs_up: cfallin submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 05 2026 at 18:54):

cfallin added PR #14056 x64: Remove an add128 lower rule that turns out just hurts to the merge queue.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 05 2026 at 19:19):

github-merge-queue[bot] removed PR #14056 x64: Remove an add128 lower rule that turns out just hurts from the merge queue.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 06 2026 at 03:51):

alexcrichton added PR #14056 x64: Remove an add128 lower rule that turns out just hurts to the merge queue.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 06 2026 at 04:39):

github-merge-queue[bot] removed PR #14056 x64: Remove an add128 lower rule that turns out just hurts from the merge queue.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 06 2026 at 14:04):

alexcrichton added PR #14056 x64: Remove an add128 lower rule that turns out just hurts to the merge queue.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 06 2026 at 14:31):

:check: alexcrichton merged PR #14056.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 06 2026 at 14:31):

alexcrichton removed PR #14056 x64: Remove an add128 lower rule that turns out just hurts from the merge queue.


Last updated: Aug 30 2026 at 10:08 UTC