Skip to content

Comments

OptimizeInstructions: Improve areConsecutiveInputsEqualAndFoldable#8355

Merged
kripken merged 2 commits intoWebAssembly:mainfrom
kripken:oi.fold.no.remove
Feb 23, 2026
Merged

OptimizeInstructions: Improve areConsecutiveInputsEqualAndFoldable#8355
kripken merged 2 commits intoWebAssembly:mainfrom
kripken:oi.fold.no.remove

Conversation

@kripken
Copy link
Member

@kripken kripken commented Feb 20, 2026

To check if we can fold A, B into A, we can just check if B has
effects we can't remove. The old code used a utility that checks if
A is also removable, but that is suboptimal, and that entire
utility is not needed.

@kripken kripken requested a review from tlively February 20, 2026 18:56
@kripken kripken mentioned this pull request Feb 20, 2026
Copy link
Member

@tlively tlively left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM if the fuzzer is happy.

@kripken kripken merged commit 9656b8e into WebAssembly:main Feb 23, 2026
17 checks passed
@kripken kripken deleted the oi.fold.no.remove branch February 23, 2026 21:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants