Closed Bug 1682466 Opened 5 years ago Closed 5 years ago

Implement the SIMD "extended multiplication" instruction

Categories

(Core :: JavaScript: WebAssembly, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED

People

(Reporter: lth, Assigned: lth)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

See https://github.com/WebAssembly/simd/pull/376. This has been accepted into the spec, and V8, LLVM, emscripten, binaryen all support it now. Checklist:

[ x ] Test cases
[ x ] Wast
[ x ] Baseline (x86, x64, arm64)
[ x ] Ion (x86, x64)

Ion arm64 will land separately, with all the other Ion arm64 SIMD support (bug 1687629).

Opcodes as implemented in V8 and the tools: https://chromium-review.googlesource.com/c/v8/v8/+/2469156/4/src/wasm/wasm-opcodes.h

Blocks: 1664131

I'll take care of the non-cranelift bits.

Assignee: nobody → lhansen
Status: NEW → ASSIGNED
Depends on: 1682500

I would like to have i64x.widen_i32x4_{s,u} for the test cases for the extending multiplication, so take a dependency.

Depends on: 1685998

This version contains new opcodes for ExtMul.

Attachment #9193202 - Attachment description: Bug 1682466 - SIMD extended multiply, baseline + ion → Bug 1682466 - SIMD extending multiply, baseline + ion + tests.
Keywords: leave-open
Attachment #9193202 - Attachment description: Bug 1682466 - SIMD extending multiply, baseline + ion + tests. → Bug 1682466 - SIMD extending multiply, baseline + ion + tests. r?jseward
Blocks: 1687629
Blocks: 1686004
Depends on: 1687949
No longer depends on: 1685998
Blocks: 1685999
Pushed by lhansen@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/c25ac5c7acc0 SIMD extending multiply, baseline + ion + tests. r=jseward
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Keywords: leave-open
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: