simde icon indicating copy to clipboard operation
simde copied to clipboard

Is `simde_mm512_load_ps` missing?

Open lkskstlr opened this issue 2 years ago • 1 comments

Hi,

thank you very much for this cool library.

I noticed that simde_mm512_load_ps is missing while simde_mm512_loadu_ps is implemented. However, both seem to exist for Intel intrinsics (see screenshot). Maybe I oversaw one or maybe they do the same anyways?

Thanks Lukas

Screenshot from 2021-10-25 13-28-29

lkskstlr avatar Oct 25 '21 11:10 lkskstlr

Hello @lkskstlr You are correct, https://www.intel.com/content/www/us/en/docs/intrinsics-guide/index.html#text=_mm512_load_ps&ig_expand=4246,4245&techs=AVX_512 is not yet implemented in SIMDe

You can see the status for the AVX-512F intrinsics implementations at https://github.com/simd-everywhere/implementation-status/blob/main/x86.md#avx512f ; currently 1121 of 3154 (35.54%) are implemented

*_load_* functions are some of the simpler ones to implement. I suggest reading https://github.com/simd-everywhere/simde/wiki/Implementing-a-New-Function and looking at the implementations of some of the other _load_ps functions if you'd like to add it yourself. We are happy to assist you!

mr-c avatar Oct 25 '21 16:10 mr-c

This was fixed in https://github.com/simd-everywhere/simde/commit/d5880491cc07564a01f840b580d59701a51bb082

mr-c avatar Jan 18 '23 17:01 mr-c