alexcrichton opened PR #6354 from alexcrichton:x64-sse2-extractlane
to bytecodealliance:main
:
This affects some specializations of extract-to-memory which involved adding guards, and for the general cases involved adding specialized lowerings that don't use the
pextr*
instructions.Additionally this also removes the optimization where an extend-to-larger-width operation was removed after an
extractlane
since now the lowerings forextractlane
may conditionally auto-zero-extend and may not. To avoid any subtle future bugs here the optimization is removed, and if needed in the future it's hoped by that point it's either easier to add back in or it's more clear why the maintenance burden should be required.<!--
Please make sure you include the following information:
If this work has been discussed elsewhere, please include a link to that
conversation. If it was discussed in an issue, just mention "issue #...".Explain why this change is needed. If the details are in an issue already,
this can be brief.Our development process is documented in the Wasmtime book:
https://docs.wasmtime.dev/contributing-development-process.htmlPlease ensure all communication follows the code of conduct:
https://github.com/bytecodealliance/wasmtime/blob/main/CODE_OF_CONDUCT.md
-->
alexcrichton requested wasmtime-compiler-reviewers for a review on PR #6354.
alexcrichton requested abrown for a review on PR #6354.
abrown submitted PR review:
LGTM!
abrown submitted PR review:
LGTM!
abrown created PR review comment:
Maybe we don't need to document this everywhere but whenever I see this kind of thing I have to remind myself that the type system never lets us see the other bits. It's up to you if it is helpful to remind future readers of that.
alexcrichton updated PR #6354.
alexcrichton has enabled auto merge for PR #6354.
alexcrichton merged PR #6354.
Last updated: Dec 23 2024 at 13:07 UTC