Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update cranelift, impl low hanging fruit SIMD opcodes #2307

Merged
merged 10 commits into from
May 14, 2021

Conversation

MarkMcCaskey
Copy link
Contributor

Bumps to clif 0.73 and implements the new SIMD opcodes that are 0 effort to do. (I thought i8x16popcnt would be zero effort too, but popcnt doesn't work on vectors in cranelift). Also updates ignores.txt due to the newly passing spectests

@MarkMcCaskey MarkMcCaskey force-pushed the feature/update-clif-small-simd-improvement branch from 86d90a0 to 9e04e26 Compare May 12, 2021 16:16
@syrusakbary
Copy link
Member

bors r+

bors bot added a commit that referenced this pull request May 12, 2021
2307: Update cranelift, impl low hanging fruit SIMD opcodes r=syrusakbary a=MarkMcCaskey

Bumps to clif 0.73 and implements the new SIMD opcodes that are 0 effort to do. (I thought i8x16popcnt would be zero effort too, but popcnt doesn't work on vectors in cranelift). Also updates `ignores.txt` due to the newly passing spectests



Co-authored-by: Mark McCaskey <mark@wasmer.io>
Co-authored-by: Syrus <me@syrusakbary.com>
@bors
Copy link
Contributor

bors bot commented May 12, 2021

Build failed:

@MarkMcCaskey
Copy link
Contributor Author

MarkMcCaskey commented May 13, 2021

Looks like we're getting some verifier errors with your changes, probably need to rewrite / fix those or disable them for now

@MarkMcCaskey
Copy link
Contributor Author

bors r+

bors bot added a commit that referenced this pull request May 13, 2021
2307: Update cranelift, impl low hanging fruit SIMD opcodes r=MarkMcCaskey a=MarkMcCaskey

Bumps to clif 0.73 and implements the new SIMD opcodes that are 0 effort to do. (I thought i8x16popcnt would be zero effort too, but popcnt doesn't work on vectors in cranelift). Also updates `ignores.txt` due to the newly passing spectests



Co-authored-by: Mark McCaskey <mark@wasmer.io>
Co-authored-by: Syrus <me@syrusakbary.com>
Co-authored-by: Mark McCaskey <5770194+MarkMcCaskey@users.noreply.github.com>
@bors
Copy link
Contributor

bors bot commented May 13, 2021

Build failed:

@MarkMcCaskey
Copy link
Contributor Author

bors try

bors bot added a commit that referenced this pull request May 13, 2021
@bors
Copy link
Contributor

bors bot commented May 13, 2021

try

Build failed:

@MarkMcCaskey
Copy link
Contributor Author

bors try

bors bot added a commit that referenced this pull request May 13, 2021
@bors
Copy link
Contributor

bors bot commented May 13, 2021

try

Build failed:

@MarkMcCaskey
Copy link
Contributor Author

bors try

bors bot added a commit that referenced this pull request May 14, 2021
@MarkMcCaskey
Copy link
Contributor Author

bors r+

@bors
Copy link
Contributor

bors bot commented May 14, 2021

try

Build failed:

bors bot added a commit that referenced this pull request May 14, 2021
2307: Update cranelift, impl low hanging fruit SIMD opcodes r=MarkMcCaskey a=MarkMcCaskey

Bumps to clif 0.73 and implements the new SIMD opcodes that are 0 effort to do. (I thought i8x16popcnt would be zero effort too, but popcnt doesn't work on vectors in cranelift). Also updates `ignores.txt` due to the newly passing spectests



Co-authored-by: Mark McCaskey <mark@wasmer.io>
Co-authored-by: Syrus <me@syrusakbary.com>
Co-authored-by: Mark McCaskey <5770194+MarkMcCaskey@users.noreply.github.com>
@bors
Copy link
Contributor

bors bot commented May 14, 2021

Canceled.

@syrusakbary
Copy link
Member

bors r-

@syrusakbary
Copy link
Member

bors r+

@bors
Copy link
Contributor

bors bot commented May 14, 2021

@bors bors bot merged commit 28edc78 into master May 14, 2021
@bors bors bot deleted the feature/update-clif-small-simd-improvement branch May 14, 2021 22:41
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.

None yet

2 participants