Update AVX512 implementation to use AVX512F intrinsics

kokkos / simd-math

Library for length agnostic SIMD intrinsic support and the corresponding math operations

Other

20 stars 10 forks source link

Update AVX512 implementation to use AVX512F intrinsics #19

Closed jvo1012 closed 4 years ago

ibaned commented 4 years ago

Hi @jvo1012 this looks fine, can you just explain briefly the rationale for the changes? Does this make the code more portable?

jvo1012 commented 4 years ago

@ibaned, yes this is for portability. Only the core extension AVX512F (foundation) are required by all AVX512 implementations. We ran into some issues on KNL systems where certain intrinsics were not available, resulting in illegal instruction error. This change fixes those errors.