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 missing path to Kokkos Perlmutter Makefile #3874

Merged
merged 1 commit into from
Aug 2, 2023
Merged

Conversation

stanmoore1
Copy link
Contributor

@stanmoore1 stanmoore1 commented Aug 2, 2023

Summary

Add missing path to Kokkos Perlmutter Makefile

Related Issue(s)

None

Author(s)

Stan Moore (SNL), reported by Rahul Gayatri (NERSC)

Licensing

By submitting this pull request, I agree, that my contribution will be included in LAMMPS and redistributed under either the GNU General Public License version 2 (GPL v2) or the GNU Lesser General Public License version 2.1 (LGPL v2.1).

Backward Compatibility

Yes

@stanmoore1
Copy link
Contributor Author

@rgayatri23 can you confirm this fixes the LAMMPS build on Perlmutter?

@rgayatri23
Copy link

rgayatri23 commented Aug 2, 2023

The ${CRAY_CUDATOOLKIT_POST_LINK_OPTS} will add the path to cufft on Perlmutter when the cudatoolkit module is loaded.
I confirm that the change works on Perlmutter.

@stanmoore1
Copy link
Contributor Author

@akohlmey can this make it into the stable? It is pretty trivial.

@akohlmey
Copy link
Member

akohlmey commented Aug 2, 2023

Sure.

@akohlmey akohlmey merged commit 91fdf69 into develop Aug 2, 2023
7 checks passed
@akohlmey akohlmey deleted the perlmutter branch August 2, 2023 23:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants