Stream: git-wasmtime

Topic: wasmtime / PR #3600 aarch64: Migrate `uextend`/`sextend` ...


view this post on Zulip Wasmtime GitHub notifications bot (Dec 14 2021 at 15:03):

alexcrichton opened PR #3600 from isle-5-uextend-sextend to main:

This commit migrates the sign/zero extension instructions from
lower_inst.rs to ISLE. There's actually a fair amount going on in this
migration since a few other pieces needed touching up along the way as
well:

Overall I think that the translation to ISLE was a net benefit for these
instructions. It's relatively obvious what all the cases are now unlike
before where it took a few reads of the code and some boolean switches
to figure out which path was taken for each flavor of input. I think
there's still possible improvements here where, for example, the
put_in_reg_{s,z}ext64 helper doesn't use this logic so technically
those helpers could also pattern match the "well atomic loads and vector
extractions automatically do this for us" but that's a possible future
improvement for later (and shouldn't be too too hard with some ISLE
refactoring).

<!--

Please ensure that the following steps are all taken care of before submitting
the PR.

Please ensure all communication adheres to the code of conduct.
-->

view this post on Zulip Wasmtime GitHub notifications bot (Dec 14 2021 at 15:03):

alexcrichton requested fitzgen for a review on PR #3600.

view this post on Zulip Wasmtime GitHub notifications bot (Dec 14 2021 at 21:51):

fitzgen submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Dec 14 2021 at 21:51):

fitzgen merged PR #3600.


Last updated: Dec 23 2024 at 12:05 UTC