alexcrichton edited PR #14056.
alexcrichton edited PR #14056:
A local
fibbenchmark 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.
alexcrichton commented on PR #14056:
Sorry, hit submit too soon with too many commits, updated PR/description now
github-actions[bot] added the label cranelift on PR #14056.
github-actions[bot] added the label cranelift:area:x64 on PR #14056.
:thumbs_up: cfallin submitted PR review.
cfallin added PR #14056 x64: Remove an add128 lower rule that turns out just hurts to the merge queue.
github-merge-queue[bot] removed PR #14056 x64: Remove an add128 lower rule that turns out just hurts from the merge queue.
alexcrichton added PR #14056 x64: Remove an add128 lower rule that turns out just hurts to the merge queue.
github-merge-queue[bot] removed PR #14056 x64: Remove an add128 lower rule that turns out just hurts from the merge queue.
alexcrichton added PR #14056 x64: Remove an add128 lower rule that turns out just hurts to the merge queue.
:check: alexcrichton merged PR #14056.
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