Skip to content

vNxM.load_splat instructions#2350

Merged
tlively merged 1 commit intoWebAssembly:masterfrom
tlively:simd-load-splat
Sep 24, 2019
Merged

vNxM.load_splat instructions#2350
tlively merged 1 commit intoWebAssembly:masterfrom
tlively:simd-load-splat

Conversation

@tlively
Copy link
Copy Markdown
Member

@tlively tlively commented Sep 23, 2019

Introduces a new instruction class, SIMDLoad. Implements encoding,
decoding, parsing, printing, and interpretation of the load and splat
instructions, including in the C and JS APIs. v128.load remains in
the Load instruction class for now because the interpreter code
expects a Load to be able to load any memory value type.

Introduces a new instruction class, `SIMDLoad`. Implements encoding,
decoding, parsing, printing, and interpretation of the load and splat
instructions, including in the C and JS APIs. `v128.load` remains in
the `Load` instruction class for now because the interpreter code
expects a `Load` to be able to load any memory value type.
@tlively tlively requested review from aheejin and kripken September 23, 2019 17:29
Comment thread src/wasm-interpreter.h
load.align = curr->align;
load.isAtomic = false;
load.ptr = curr->ptr;
Literal (Literal::*splat)() const = nullptr;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fancy... but I guess it's nice here :)

Comment thread Contributing.md
- [ ] JS API support added in build-js.sh and src/js/binaryen.js-post.js
- [ ] C API tested in test/example/c-api-kitchen-sink.c
- [ ] JS API tested in test/binaryen.js/kitchen-sink.js
- [ ] Tests added in test/spec
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice docs!

@tlively tlively merged commit 835581f into WebAssembly:master Sep 24, 2019
@tlively tlively deleted the simd-load-splat branch April 24, 2020 23:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants