Skip to content

Commit

Permalink
Note that clang accepts _mm512_set1_epi64 on 32-bit.
Browse files Browse the repository at this point in the history
Per #676 (comment) , LLVM is able to generate code for this intrinsic on `x86` targets.
  • Loading branch information
hdevalence authored and gnzlbg committed Feb 11, 2019
1 parent b1d365d commit b035a7e
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion crates/stdsimd-verify/tests/x86-intel.rs
Original file line number Diff line number Diff line change
Expand Up @@ -369,7 +369,7 @@ fn matches(rust: &Function, intel: &Intrinsic) -> Result<(), String> {
// Apparently all of clang/msvc/gcc accept these intrinsics on
// 32-bit, so let's do the same
"_mm_set_epi64x" | "_mm_set1_epi64x" | "_mm256_set_epi64x" | "_mm256_setr_epi64x"
| "_mm256_set1_epi64x" => true,
| "_mm256_set1_epi64x" | "_mm512_set1_epi64" => true,

// These return a 64-bit argument but they're assembled from other
// 32-bit registers, so these work on 32-bit just fine. See #308 for
Expand Down

0 comments on commit b035a7e

Please sign in to comment.