afonso360 opened PR #6683 from afonso360:cprop-ireduce
to bytecodealliance:main
:
:wave: Hey,
I was playing around with cranelift over the weekend, and found that we currently don't have a cprop rule for
ireduce
, in my case this was preventing a bunch of the other optimizations from running.I've also added some tests for the other extends which were missing.
afonso360 requested elliottt for a review on PR #6683.
afonso360 requested wasmtime-compiler-reviewers for a review on PR #6683.
elliottt submitted PR review:
Looks good to me!
elliottt submitted PR review:
Looks good to me!
elliottt created PR review comment:
Is it worth adding the
fits_in_64
check here anyway just to future-proof this a bit?
afonso360 updated PR #6683.
afonso360 has enabled auto merge for PR #6683.
afonso360 merged PR #6683.
Last updated: Dec 23 2024 at 12:05 UTC