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

[oneAPI_Support] Upgrade to v2024.2.1 #9552

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

amontoison
Copy link
Contributor

Related issue JuliaGPU/oneAPI.jl#465

@amontoison amontoison added the oneAPI 1️⃣ Builders related to the oneAPI toolkit. label Sep 29, 2024
@maleadt
Copy link
Contributor

maleadt commented Sep 30, 2024

Seems like this runs into the same issue I was running into.

You should be able to remove the OpenCL hack, oneapi-src/oneMKL#472 (comment). That probably necessitates a major version bump, to be sure we don't accidentally push this update to users.

Also, Conda installed far fewer dependencies when I tried this locally (no intel-openmp, intel-sycl-rt, onemkl-sycl-vm, or tbb), so I think we can remove a couple. See https://github.com/JuliaPackaging/Yggdrasil/pull/9554/files.

@amontoison
Copy link
Contributor Author

The missing symbols seem to be an issue in the Intel libraries that we can't solve.

@maleadt
Copy link
Contributor

maleadt commented Sep 30, 2024

Then why does the local build succeed? There must be a difference between how we're invoking cc here, and how dpcpp (or whatever the Clang wrapper is called now) links the library.

@amontoison
Copy link
Contributor Author

I don't know, Tim. Maybe the shared libraries have the same issue as the local build; it's just that the local build doesn't include the audit process to check the libraries.

@maleadt
Copy link
Contributor

maleadt commented Sep 30, 2024

The audit seems to complain it can't even dlopen the library though, which has to work for oneAPI.jl tests to succeed.

I'll take a closer look next week if you don't beat me to it :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
oneAPI 1️⃣ Builders related to the oneAPI toolkit.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants