Skip to content

Fix some comparison-related clif optimization rules#13099

Merged
fitzgen merged 2 commits intobytecodealliance:mainfrom
alexcrichton:fix-another-clif-rule
Apr 14, 2026
Merged

Fix some comparison-related clif optimization rules#13099
fitzgen merged 2 commits intobytecodealliance:mainfrom
alexcrichton:fix-another-clif-rule

Conversation

@alexcrichton
Copy link
Copy Markdown
Member

The fix in #13063 failed to take into account some extra rules in bitops.isle which now works for vector types and didn't correctly use the cmp_true helper to create a "true" value for the destination type.

The fix in bytecodealliance#13063 failed to take into account some extra rules in
`bitops.isle` which now works for vector types and didn't correctly use
the `cmp_true` helper to create a "true" value for the destination type.
@alexcrichton alexcrichton requested a review from a team as a code owner April 14, 2026 20:44
@alexcrichton alexcrichton requested review from fitzgen and removed request for a team April 14, 2026 20:44
@github-actions github-actions Bot added cranelift Issues related to the Cranelift code generator isle Related to the ISLE domain-specific language labels Apr 14, 2026
@fitzgen fitzgen added this pull request to the merge queue Apr 14, 2026
Merged via the queue into bytecodealliance:main with commit 22e81e9 Apr 14, 2026
79 checks passed
@alexcrichton alexcrichton deleted the fix-another-clif-rule branch April 15, 2026 00:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cranelift Issues related to the Cranelift code generator isle Related to the ISLE domain-specific language

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants