-
Notifications
You must be signed in to change notification settings - Fork 973
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
SSE41 kernels are broken #1959
Comments
With DNNL_ENABLE_PRIMITIVE_CPU_ISA=SSE41 and ONEDNN_ENABLE_GEMM_KERNELS_ISA=AVX2:
With DNNL_ENABLE_PRIMITIVE_CPU_ISA=AVX2 and ONEDNN_ENABLE_GEMM_KERNELS_ISA=SSE41:
|
@nwnk Thank you for reporting the issue. I can reproduce most of the test failures locally.
I will report this issue internally to have better coordination of all the test cases for the ISAs. |
@nwnk, thank you for the report. Looks like we have an issue with tests and/or dispatcher when The library is functional on platforms with Intel SSE 4.1 instruction set support. This can be validated either by running tests on system with Intel SSE4.1, running tests with environment variable Setting the bug aside, could you please elaborate on what you are using |
Experimentation, mostly. Trying to get a feel for how much larger each ISA makes the resulting binary, and validating that the library works as documented on the minimal feature set that it claims to support. |
Thanks for the details. We're looking into options to address the test issue. |
Issue with |
Issue with |
I'm trying to build oneDNN CPU-only (OMP backed) and with only the minimal SSE41 kernels, on Fedora 40. The result does not pass ctest:
Specifically this is with both
DNNL_ENABLE_PRIMITIVE_CPU_ISA
andONEDNN_ENABLE_GEMM_KERNELS_ISA
set to SSE41. If I set them both to AVX2 all tests pass. I have not tested AVX512 or AMX yet. I will try setting just one to AVX2 (both ways) and see if that helps narrow it down.Relevant environment and version info:
The text was updated successfully, but these errors were encountered: