myunbin opened PR #13787 from myunbin:remove-redundants to bytecodealliance:main:
This PR removes a redundant simplification rule that is already covered by constant canonicalization in
cprop.isle.The rule
(2^n) * x --> x << nis redundant because it can be derived by first canonicalizing the constant operand:(2^n) * x --> x * (2^n) --> x << nSo only
x * (2^n) --> (x << n)rule is needed.
myunbin requested alexcrichton for a review on PR #13787.
myunbin requested wasmtime-compiler-reviewers for a review on PR #13787.
myunbin requested wasmtime-core-reviewers for a review on PR #13787.
github-actions[bot] added the label cranelift on PR #13787.
github-actions[bot] added the label isle on PR #13787.
github-actions[bot] commented on PR #13787:
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:
- cfallin: isle
- fitzgen: isle
To subscribe or unsubscribe from this label, edit the <code>.github/subscribe-to-label.json</code> configuration file.
Learn more.
</details>
:thumbs_up: alexcrichton submitted PR review.
alexcrichton added PR #13787 [Cranelift] remove redundant simplifictaion rule to the merge queue.
github-merge-queue[bot] removed PR #13787 [Cranelift] remove redundant simplifictaion rule from the merge queue.
alexcrichton added PR #13787 [Cranelift] remove redundant simplifictaion rule to the merge queue.
github-merge-queue[bot] removed PR #13787 [Cranelift] remove redundant simplifictaion rule from the merge queue.
alexcrichton added PR #13787 [Cranelift] remove redundant simplifictaion rule to the merge queue.
github-merge-queue[bot] removed PR #13787 [Cranelift] remove redundant simplifictaion rule from the merge queue.
alexcrichton commented on PR #13787:
Sorry this PR seems to be getting extremely unlucky...
alexcrichton added PR #13787 [Cranelift] remove redundant simplifictaion rule to the merge queue.
:check: alexcrichton merged PR #13787.
alexcrichton removed PR #13787 [Cranelift] remove redundant simplifictaion rule from the merge queue.
Last updated: Jul 29 2026 at 05:03 UTC