jameysharp opened PR #5729 from backport-5723
to release-6.0.0
:
This backports #5723 which fixes incorrect optimizations found while auditing
algebraic.isle
for misuse of theiconst
constructor. No bugs have been reported implicating these optimizations, but this will be the first Cranelift release with egraph optimization enabled by default, so most people wouldn't have run into these bugs sooner.Since Wasmtime doesn't use the I128 type, it is not affected by the
icmp
fix. I'm not sure whether thebxor
issue was reachable from WebAssembly.
jameysharp requested fitzgen for a review on PR #5729.
cfallin submitted PR review.
jameysharp merged PR #5729.
Last updated: Nov 22 2024 at 16:03 UTC