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

[SHTns] up to v3.7 and add CUDA support #9813

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

Conversation

fgerick
Copy link
Contributor

@fgerick fgerick commented Nov 19, 2024

No description provided.

@fgerick
Copy link
Contributor Author

fgerick commented Nov 19, 2024

Does someone know what has to be done to keep the cuda builds from failing? When I run this locally it works fine and it seems it can build the library just fine, but then the dlopen checks fail. When using dont_dlopen=true the builds end up in an infinite loop.

@fgerick
Copy link
Contributor Author

fgerick commented Nov 27, 2024

Okay, after quite some painful trial and error I believe this is finally working. I've removed the micro-architecture expansion, that I think is not necessary with the removal of the -mtune=skylake flag that was still around in the build.

I hope this is correct how the CUDA and non-CUDA versions are build now and that the x86_64-linux-gnu-cuda+none is the correct way to do it. I do not manage to build everything without this flag for the CPU-only version.

@fgerick
Copy link
Contributor Author

fgerick commented Nov 29, 2024

Who should I get in touch with to approve this? Thank you.

S/SHTns/build_tarballs.jl Outdated Show resolved Hide resolved
S/SHTns/build_tarballs.jl Outdated Show resolved Hide resolved
fgerick and others added 2 commits November 29, 2024 14:06
Co-authored-by: Mosè Giordano <[email protected]>
Co-authored-by: Mosè Giordano <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants