This repository was archived by the owner on Dec 22, 2021. It is now read-only.
Update ImplementationStatus to reflect status quo #223
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Updates based on latest LLVM & latest v8 (earlier v8 versions aren't useful because of renumbering anyway): - v8 implements load_splat and load with zero/sign extension - v8 implements i64x2_mul - v8 implements swizzle and andnot - LLVM supports integer abs instructions and intrinsics - Removing note about shuffle opcode - irrelevant post-renumbering Notable remaining omissions: - v8 doesn't implement v128.const - LLVM doesn't implement (or at least doesn't expose an intrinsic for) i64x2_mul - A lot of instructions are still part of unimplemented subset in LLVM
Member
|
V8 side LGTM, thank you! |
Member
|
@lars-t-hansen It'd be great to get the spidermonkey status too! |
Contributor
In due time :-) We've landed in baseline (x64 only) and preffed-off, but once we land in the optimizing compiler I'll post a PR for implementation status. Hoping this will happen in May. |
tlively
added a commit
to llvm/llvm-project
that referenced
this pull request
May 15, 2020
Summary: Move instructions that have recently been implemented in V8 from the `unimplemented-simd128` target feature to the `simd128` target feature. The updated instructions match the update at WebAssembly/simd#223. Reviewers: aheejin Subscribers: dschuff, sbc100, jgravelle-google, hiraditya, sunfish, cfe-commits, llvm-commits Tags: #clang, #llvm Differential Revision: https://reviews.llvm.org/D79973
hanswinderix
pushed a commit
to hanswinderix/llvm-project
that referenced
this pull request
May 27, 2020
Summary: Move instructions that have recently been implemented in V8 from the `unimplemented-simd128` target feature to the `simd128` target feature. The updated instructions match the update at WebAssembly/simd#223. Reviewers: aheejin Subscribers: dschuff, sbc100, jgravelle-google, hiraditya, sunfish, cfe-commits, llvm-commits Tags: #clang, #llvm Differential Revision: https://reviews.llvm.org/D79973
arichardson
pushed a commit
to arichardson/llvm-project
that referenced
this pull request
Jul 2, 2020
Summary: Move instructions that have recently been implemented in V8 from the `unimplemented-simd128` target feature to the `simd128` target feature. The updated instructions match the update at WebAssembly/simd#223. Reviewers: aheejin Subscribers: dschuff, sbc100, jgravelle-google, hiraditya, sunfish, cfe-commits, llvm-commits Tags: #clang, #llvm Differential Revision: https://reviews.llvm.org/D79973
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
5 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Updates based on latest LLVM & latest v8 (earlier v8 versions aren't
useful because of renumbering anyway):
Notable remaining omissions:
i64x2_mul