-
Notifications
You must be signed in to change notification settings - Fork 407
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
Add SKL and ICL as architectures #5013
Conversation
cmake/kokkos_arch.cmake
Outdated
DECLARE_AND_CHECK_HOST_ARCH(ICX "Intel Ice Lake Server CPUs (AVX512)") | ||
DECLARE_AND_CHECK_HOST_ARCH(SKL "Intel Sky Lake Xeon E-class Client CPUs") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove the Xeon on the Skylake and you can add a Xeon on the ICX i.e. also I think its one word. I.e. like this:
Intel Icelake Client CPUs (AVX512)
Intel Icelake Xeon Server CPUs (AVX512)
Intel Skylake Client CPUs
Intel Skylake Xeon Server CPUs (AVX512)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's Ice Lake
and Skylake
.
DECLARE_AND_CHECK_HOST_ARCH(SKX "Intel Sky Lake Xeon E-class HPC CPUs (AVX512)") | ||
DECLARE_AND_CHECK_HOST_ARCH(ICL "Intel Ice Lake Client CPUs (AVX512)") | ||
DECLARE_AND_CHECK_HOST_ARCH(ICX "Intel Ice Lake Xeon Server CPUs (AVX512)") | ||
DECLARE_AND_CHECK_HOST_ARCH(SKL "Intel Skylake Client CPUs") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this one not missing the (AVX512)
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, not all of them support AVX512.
In response to #4929 (comment).
I also removed linking with
rtm
since transactional memory is not used anymore anyway.