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

Configuration on Cori with nvcc #2413

Merged
merged 9 commits into from
Sep 1, 2021

Conversation

bendudson
Copy link
Contributor

Updates to enable configuration and compilation on Cori.
Selectively merging some scripts from Holger's branch
Deleting a number of unneeded / unused files.

Testing which configurations work on Cori
Not used anywhere, duplicates functionality in
include/bout/build_config.hxx
Set of header files included in the source list, but not used anywhere
Two step process: First set the environment, then configure BOUT++.
If all sources are marked as CUDA files, then nvcc attempts
to compile everything, including header files.

One solution is to split the sources into header and cxx files.
This uses list FILTER to do that automatically.
Previously hard-wired, now can be changed if needed
Copy and swap compiles fine with nvcc, with BOUT_HOST_DEVICE
@github-actions
Copy link
Contributor

github-actions bot commented Sep 1, 2021

clang-tidy review says "All clean, LGTM! 👍"

@bendudson
Copy link
Contributor Author

Library compiling on Cori (with nvcc) but doesn't link. Error is

make[2]: *** No rule to make target '/tmp/build/80754af9/libnetcdf_1545349563001/_build_env/x86_64-conda_cos6-linux-gnu/sysroot/usr/lib/librt.so', 
needed by 'examples/laplacexy/simple/CMakeFiles/laplacexy-simple.dir/cmake_device_link.o'

Suspect NetCDF configuration issue.

Used for debugging, but not necessary in outputs
nvcc seems to need a constructor, rather than
inferring a constructor which gcc does
@bendudson bendudson merged commit 1ea779f into next-hypre-outerloop-cuda-merged Sep 1, 2021
@bendudson bendudson deleted the merging-hj branch September 1, 2021 20:35
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

1 participant