You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We recently updated Axom's C++ language requirement to c++20, which has some potential implications on the CMake requirements on various supported architectures. Let's reinvestigate the minimum required CMake versions for our supported CPU and GPU architectures.
Among other concerns, this potential impacts:
our stated version in the cmake_minimum_required call, which impacts our available CMake policies and their defaults
This might also be affected by a BLT PR to improve CUDA support: llnl/blt#769
See also, the discussion on a recent docs update PR: #1957 (comment)
In particular, @white238 wrote:
CMake 3.18 + CUDA + c++20 actually is a failure due to CMake not knowing that CUDA supports C++20 that early in CMake. It was set to this because many moons ago we noticed that it was changing cmake policies based on the cmake_minimum_required and this caused failures. We should reevaluate this at some point.
We recently updated Axom's C++ language requirement to
c++20, which has some potential implications on the CMake requirements on various supported architectures. Let's reinvestigate the minimum required CMake versions for our supported CPU and GPU architectures.Among other concerns, this potential impacts:
cmake_minimum_requiredcall, which impacts our available CMake policies and their defaultsaxom/src/CMakeLists.txt
Lines 10 to 20 in 97f1e49
axom/src/CMakeLists.txt
Lines 134 to 137 in 97f1e49
axom/src/docs/sphinx/quickstart_guide/config_build.rst
Lines 26 to 33 in 97f1e49
(note that we are updating this section for C++20 in Some touchups to reflect that C++20 is now required for Axom #1957)
This might also be affected by a BLT PR to improve CUDA support: llnl/blt#769
See also, the discussion on a recent docs update PR: #1957 (comment)
In particular, @white238 wrote: