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

Make kokkos_has_string() function in Makefile.kokkos case insensitive #3091

Merged
merged 1 commit into from
Jun 11, 2020

Conversation

akohlmey
Copy link
Contributor

This simplifies the configuration and documentation when compiling Kokkos via GNU make in LAMMPS while maintaining backward compatibility.
For compiling LAMMPS with non-CMake procedure one can do:
make kokkos_cuda_mpi KOKKOS_ARCH=pascal61
or:
make kokkos_cuda_mpi KOKKOS_ARCH=Pascal61
or:
make kokkos_cuda_mpi KOKKOS_ARCH=PASCAL61
and all will have the same outcome.

This has been tested with LAMMPS on Fedora 31 Linux x86_64 only.

technical note: i looked up a way doing this without using shell commands to improve speed and portability.

@dalg24-jenkins
Copy link
Collaborator

Can one of the admins verify this patch?

@akohlmey
Copy link
Contributor Author

@stanmoore1 submitted to upstream.

@stanmoore1
Copy link
Contributor

I confirm this works on Power9+V100, with KOKKOS_ARCH=VOLTA70 instead of KOKKOS_ARCH=Volta70

@akohlmey
Copy link
Contributor Author

Also tested successfully with LAMMPS on Ubuntu 18.04LTS x86_64

@dalg24 dalg24 requested a review from jeffmiles63 June 10, 2020 21:24
@codecov-commenter
Copy link

codecov-commenter commented Jun 10, 2020

Codecov Report

Merging #3091 into develop will decrease coverage by 0.0%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##           develop   #3091     +/-   ##
=========================================
- Coverage     85.7%   85.6%   -0.1%     
=========================================
  Files          122     122             
  Lines        10392   10391      -1     
=========================================
- Hits          8906    8905      -1     
  Misses        1486    1486             
Flag Coverage Δ
#clang 76.1% <ø> (+<0.1%) ⬆️
#gcc 86.4% <ø> (-0.1%) ⬇️
Impacted Files Coverage Δ
core/src/impl/Kokkos_HostThreadTeam.cpp 96.0% <0.0%> (-0.8%) ⬇️
core/src/OpenMP/Kokkos_OpenMP_Task.hpp 98.7% <0.0%> (-0.1%) ⬇️
core/src/impl/Kokkos_TaskQueueCommon.hpp 86.7% <0.0%> (+0.8%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 78f4004...bdf7d5d. Read the comment docs.

@crtrott
Copy link
Member

crtrott commented Jun 11, 2020

OK to test

@crtrott crtrott merged commit f48b709 into kokkos:develop Jun 11, 2020
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.

None yet

5 participants