@vrabaud Please take a look on this patch.

I believe you also need to fix intrin_avx2.hpp
Just check for cvtps_ph and cvtph_ps

This patch solves my initial problem, thx !
(but I still believe intrin_avx2.hpp needs to be fixed)

intrin_avx2.hpp

BTW, there is no intrin_avx2.hpp file (intrin_avx.hpp is used for AVX2 code due to legacy reasons)

Ah, my bad, I meant: intrin_avx512.hpp

Hmm, actually wait before merging, I still have an error when compiling for haswell with clang.
I get the error: error: use of undeclared identifier '_mm_cvtps_ph'; did you mean '_mm_cvtss_sd' in cvdef.h

compiling for haswell with clang.

Do you compile OpenCV itself or use OpenCV?

@vrabaud Could you please describe observed problem here ? (press "Share" after modification and dump here provided link)

(you can dump included headers in your code through adding of -H option)

@vrabaud Updated, please take a look.

Hi, sorry for the delay. You did fix the initial issue.
Now, the other issue I mentioned (with haswell) is actually a different problem (independent from compiler flags): if you only include types_c.h in your code, you have a problem because it includes cvdef.h at https://github.com/opencv/opencv/blob/master/modules/core/include/opencv2/core/types_c.h#L84
which does SIMD stuff while no SIMD header was included. Hence the error:
error: missing '#include <immintrin.h>'; '_mm_cvtph_ps' must be declared before it is used
(I had only looked at the beginning of the haswell error, not the end)

This case should be handled in cv_cpu_dispatch.h now.

Thx a bunch, I confirm my two issues are fixed ! Ready to merge !

👍