Skip to content
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 icpx 2024.0 with OpenMP to the CI #2209

Draft
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

psychocoderHPC
Copy link
Member

fix #2205

CI_FILTER: ^linux_icpx

CI_FILTER: ^linux_icpx
@psychocoderHPC
Copy link
Member Author

The bug will be fixed with #2213 and I keep this PR to add icpx 204.0 to the CI. We still need to think about how we can enable OpenMP backends. OneDPL is using by default the TBB backend. It is currently not clear how we should test the OpenMP backend in our CI.
A possible solution would be to define a second icpx compiler in our job generator where we ohne install OpenMP dependencies for icpx, see the changes in this PR.

@bernhardmgruber bernhardmgruber changed the title fix icpx OpenMP atomics Add icpx 2024.0 with OpenMP to the CI Jan 10, 2024
@bernhardmgruber
Copy link
Member

I renamed the PR so I am less confused when looking at the list of open PRs.

@psychocoderHPC psychocoderHPC added this to the 1.2.0 milestone Jan 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Compilation failure with icpx 2024.0.1 and OpenMP backend
2 participants