afonso360 opened PR #5991 from implement-simd-arith
to main
:
:wave: Hey,
The interpreter can already deal with some opcodes in a polymorphic way, but this wasn't enabled for most binary and unary opcodes. This PR changes that. We get a bunch of opcodes implemented in a relatively easy way.
Please review this commit by commit, The first commits are just moving ops that weren't implemented from the main
simd-arithmetic.clif
into their own separate files. So that we could later enable that file for testing in the interpreter.Will add fuzzer support in a separate PR.
afonso360 requested jameysharp for a review on PR #5991.
jameysharp submitted PR review.
jameysharp submitted PR review.
jameysharp created PR review comment:
Pre-existing: should this comment have said "Same as
binary
"?
afonso360 submitted PR review.
afonso360 created PR review comment:
Yeah, I think so, I'll update this when doing the refactoring above ^
afonso360 updated PR #5991 from implement-simd-arith
to main
.
afonso360 updated PR #5991 from implement-simd-arith
to main
.
afonso360 updated PR #5991 from implement-simd-arith
to main
.
jameysharp submitted PR review.
alexcrichton merged PR #5991.
Last updated: Nov 22 2024 at 17:03 UTC