Skip to content

Conversation

@ax3l
Copy link
Member

@ax3l ax3l commented Nov 10, 2025

Summary

We already use features of FindCUDAToolkit for nvtx3 that are CMake 3.25+. Modernize and remove pre CMake 3.20 CUDA logic.

Additional background

CMake 3.25 was released in November 2022 and is widely available and very easy to install as a user, if missing.

Also a bit of prep for #4572

As in WarpX, we disable SENSEI CI because the package has gone unsupported and deprecated for a few years now.

Checklist

The proposed changes:

  • fix a bug or incorrect behavior in AMReX
  • add new capabilities to AMReX
  • changes answers in the test suite to more than roundoff level
  • are likely to significantly affect the results of downstream AMReX users
  • include documentation in the code and/or rst files, if appropriate

We already use features of `FindCUDAToolkit` for nvtx3 that
are CMake 3.25+. Modernize and remove pre CMake 3.20 CUDA
logic.
@ax3l ax3l added the GPU label Nov 10, 2025
@ax3l
Copy link
Member Author

ax3l commented Nov 10, 2025

@WeiqunZhang can you please remove SENSEI Adaptor [test] from the required tests in CI?

@WeiqunZhang
Copy link
Member

Should we also update this?

"CMAKE_CUDA_COMPILER_VERSION VERSION_GREATER_EQUAL 10.2.89;CMAKE_VERSION VERSION_LESS 3.17" OFF)

Also don't think we need to worry about CUDA 10 anymore.

target_link_libraries(amrex_${D}d PUBLIC CUDA::cusparse)
endif ()

if(CMAKE_CUDA_COMPILER_VERSION VERSION_LESS 11.2)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://amrex-codes.github.io/amrex/docs_html/GPU_Chapter.html

Do we need to update the docs (not that I expect someone to find cuda 11.0 or 11.1)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated the docs 👍

@ax3l
Copy link
Member Author

ax3l commented Nov 11, 2025

Yes, couple of options we need to refresh now.

E.g., add support for LTO on AMReX_CUDA_LTO.
Supported by CMake 3.25+, too: https://cmake.org/cmake/help/latest/module/CheckIPOSupported.html

@jmsexton03 jmsexton03 mentioned this pull request Nov 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants