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

Support for ARM SVE and better performance for NEON #281

Merged
merged 13 commits into from
Jun 17, 2024

Conversation

fwessels
Copy link
Contributor

No description provided.

@klauspost
Copy link
Owner

Wow, getting SVE support is sluggish: https://go-review.googlesource.com/c/go/+/365034 🐌

How is the output generated? It is pretty much non-maintainable without the generator(s) but I realize the lack of SVE support probably means there are some external programs needed.

@fwessels
Copy link
Contributor Author

fwessels commented Jun 6, 2024

Wow, getting SVE support is sluggish: https://go-review.googlesource.com/c/go/+/365034 🐌

Yes, have seen that too -- SVE is a quite extensive and rich instruction set and with predicate support etc. it is far from trivial (and a huge effort) to define an alternative syntax that covers essentially everything. So not surprised that it's been going on for a while now.

Copy link
Owner

@klauspost klauspost left a comment

Choose a reason for hiding this comment

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

Just one minor thing. Otherwise LGTM.

_gen/gen.go Outdated Show resolved Hide resolved
@klauspost klauspost merged commit 7202dfc into klauspost:master Jun 17, 2024
13 checks passed
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