Skip to content
This repository has been archived by the owner on Dec 22, 2021. It is now read-only.

Fix SIMD load splat/extend names in binary and text section of spec #405

Merged

Conversation

ngzhian
Copy link
Member

@ngzhian ngzhian commented Dec 17, 2020

Follow up to #297 for renaming the instructions in binary and text
sections in the spec.

Follow up to WebAssembly#297 for renaming the instructions in binary and text
sections in the spec.
@ngzhian
Copy link
Member Author

ngzhian commented Jan 13, 2021

@rossberg ptal, thanks!

@Maratyszcza
Copy link
Contributor

IMO we should similarly rename bitmask, i8x16.shuffle, and i8x16.swizzle instructions

@ngzhian ngzhian added the spectext changes to spec text label Jan 14, 2021
@ngzhian
Copy link
Member Author

ngzhian commented Jan 14, 2021

IMO we should similarly rename bitmask, i8x16.shuffle, and i8x16.swizzle instructions

Are you saying i8x16.{shuffle,swizzle} should be v8x16? If so, #316 for context on why we chose we went with i8x16. If you object to that decision feel free to reopen that issue.

@ngzhian ngzhian merged commit 10d3153 into WebAssembly:master Jan 14, 2021
@ngzhian ngzhian deleted the fix-simd-load-store-instruction-names branch January 14, 2021 03:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
spectext changes to spec text
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants