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
Update HierarchicalMNS for Intel OneAPI compilers #3653
Conversation
# required for use of ClangGCC toolchain | ||
'Clang,GCC': ('Clang-GCC', '%(Clang)s-%(GCC)s'), | ||
# required for use of gcccuda toolchain, and for CUDA installed with GCC toolchain | ||
'CUDA,GCC': ('GCC-CUDA', '%(GCC)s-%(CUDA)s'), | ||
# required for use of iccifortcuda toolchain | ||
'CUDA,icc,ifort': ('intel-CUDA', '%(icc)s-%(CUDA)s'), | ||
'CUDA,iccifort': ('intel-CUDA', '%(iccifort)s-%(CUDA)s'), | ||
'CUDA,intel-compilers': ('intel-CUDA', '%(intel-compilers)s-%(CUDA)s'), # OneAPI compilers |
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.
I'm going to remove this since we are about to unify iccifort
and iccifortcuda
(so I have no toolchain to test this on, and we don't plan on having one).
I tested this locally:
|
@boegel I think we are hitting API limits on GitHub or something. I've rerun these tests twice now and which ones fail is a moving target but seems to be related to GitHub:
|
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.
lgtm
No description provided.