Skip to content

Conversation

Daniel-B-Smith
Copy link
Contributor

Builds off of #862 to add the scatter intrinsics.

@rust-highfive
Copy link

r? @gnzlbg

(rust_highfive has picked a reviewer for you, use r? to override)

@Daniel-B-Smith Daniel-B-Smith marked this pull request as ready for review June 7, 2020 18:32
@Amanieu
Copy link
Member

Amanieu commented Jun 9, 2020

You are missing these scatter intrinsics:

  • _mm512_i32scatter_ps
  • _mm512_mask_i32scatter_ps
  • _mm512_i32scatter_epi32
  • _mm512_mask_i32scatter_epi32

@Daniel-B-Smith
Copy link
Contributor Author

Added the remaining intrinsics

@Amanieu
Copy link
Member

Amanieu commented Jun 14, 2020

Does this PR also contain the other two? If so then just close the other two and we'll use this one. It also needs a rebase due to merge conflicts.

@Daniel-B-Smith
Copy link
Contributor Author

I closed the other two PRs since this contains them. I'm not sure what needs to be rebased as this contains the latest master already.

@Amanieu Amanieu changed the title Add AVX 512f scatter intrinsics Add AVX 512f gather, scatter and compare intrinsics Jun 16, 2020
@Amanieu Amanieu merged commit b17efd8 into rust-lang:master Jun 16, 2020
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.

4 participants