Skip to content

Clarify _mm256_shuffle_epi8 documentation #990

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jan 30, 2021
Merged

Clarify _mm256_shuffle_epi8 documentation #990

merged 1 commit into from
Jan 30, 2021

Conversation

jethrogb
Copy link
Contributor

The old documentation seemed to suggest that the low 4 bits of bytes in b would index into the 32-element array a. That's not quite right and you'd need 5 bits to index 32 elements.

@jethrogb
Clarify _mm256_shuffle_epi8 documentation
fc3d42e 
The old documentation seemed to suggest that the low 4 bits of bytes in `b` would index into the 32-element array `a`. That's not quite right and you'd need 5 bits to index 32 elements.
@rust-highfive
Copy link

r? @Amanieu

(rust-highfive has picked a reviewer for you, use r? to override)

@jethrogb
Copy link
Contributor Author

CI failure seems unrelated:

---- core_arch::x86::abm::assert__lzcnt_u32_lzcnt stdout ----
exit code: 1
LLVM ERROR: out of memory
fatal error: /Applications/Xcode_12.3.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/otool: fatal error in /Applications/Xcode_12.3.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/otool-classic

@Amanieu Amanieu merged commit da69c54 into rust-lang:master Jan 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@Amanieu Amanieu

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jethrogb @rust-highfive @Amanieu