Review for new VEX encodings? · cranelift

Stream: cranelift

Topic: Review for new VEX encodings?

Andrew Brown (Feb 16 2023 at 00:26):

@Alex Crichton, thanks for pushing #5795. I was planning to take a look at it but let me know if anyone else wants to do the review: @Chris Fallin, @Trevor Elliott, @Jamey Sharp?

x64: Add rudimentary support for some AVX instructions by alexcrichton · Pull Request #5795 · bytecodealliance/wasmtime

I was poking around Spidermonkey's wasm backend and saw that the various assembler functions used are all v*-prefixed which look like they're intended for use with AVX instructions. I looked at Cra...

Chris Fallin (Feb 16 2023 at 00:26):

Please feel free to review! I'll skim it too if I have time at some point

Trevor Elliott (Feb 16 2023 at 00:27):

happy to look through later today!

Alex Crichton (Feb 16 2023 at 00:35):

oh no rush of course, I just needed a break from components for a bit

Chris Fallin (Feb 16 2023 at 00:41):

(ordinary human: takes a break by watching a good series on Netflix; Alex: takes a break by casually implementing 3-register forms using VEX in an afternoon)

Chris Fallin (Feb 16 2023 at 00:41):

(thanks)

Last updated: Nov 22 2024 at 17:03 UTC