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

Venado cmake fixes #49

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

Conversation

dmageeLANL
Copy link
Contributor

CMAKE

  • Update lanl-setup.cmake for new machines.
  • OpenMP
    • Using cce on venado, the OpenMP flag doesn't get set when openmp is found even though it finds OpenMP_CXX_FLAGS (which is really what's needed).
    • Removed OpenMP version check, cce's openmp (libcraymp.so) doesn't return a version on venado and it's unlikely to run into an OpenMP version over 13 years old.
    • Use OpenMP_CXX_FLAGS directly in branson_deps because on venado, using cce, cmake doesn't return anything in OpenMP::OpenMP_CXX.
  • CUDA
    • Add SMS option for the compute capability of cuda samples. This is picked up from the environment. The name SMS matches the variable used by cuda-samples. Default to 80 (Ampere).
    • Add the host compiler (CXX) to the CUDA_FLAGS and set -allow-unsupported-compiler in case it's too new.
  • Cray Compiler
    • The CXX_COMPILER_ID for cce is 'Clang', change the conditional to catch that.

Source

  • Only output GPU status updates if ENABLE_VERBOSE_GPU_TRANSPORT is set.
  • Tell if CUDA is enabled in header summary.
  • Reuse total_sigma_s in transport kernel.

…penMP_CXX_FLAGS to cmake deps directly. Take compute capability (variable SMS) as environment variable, set default to 80 (Ampere). Use CMAKE_CXX_COMPILER as host compiler for nvcc and add -allow-unsupported-compiler flag so it works with everything. Move printouts to end of CMakeLists.txt to print CUDA compiler config. Update lanl-setup config for new machines.
…GPU status messages with ENABLE_VERBOSE_GPU_TRANSPORT option. Add message to header if CUDA is enabled in binary.
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.

1 participant