8000 GCC minimum version for cxx 20 by lucbv · Pull Request #8130 · kokkos/kokkos · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

GCC minimum version for cxx 20 #8130

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

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

lucbv
Copy link
Contributor
@lucbv lucbv commented May 30, 2025

Proposing to raise the minimum version of gcc to avoid running into the issue of LWG-3446: https://cplusplus.github.io/LWG/issue3446 This issue is causing failures in Trilinos but not in Kokkos yet. Note that the gcc versions currently affected by LWG-3446 are:

  • gcc < 10.4.0
  • gcc < 11.1.0
  • clang < 14

Please let me know if the proposed change of minimum gcc raises issues with any important customer and/or platform.

Proposing to raise the minimum version of gcc to avoid
running into the issue of LWG-3446: https://cplusplus.github.io/LWG/issue3446
This issue is causing failures in Trilinos but not in Kokkos yet.
Note that the gcc versions currently affected by LWG-3446 are:
 - gcc < 10.4.0
 - gcc < 11.1.0
 - clang < 14

Please let me know if the proposed change of minimum gcc raises
issues with any important customer and/or platform.

Signed-off-by: Luc Berger-Vergiat <lberge@sandia.gov>
@lucbv lucbv requested review from dalg24, masterleinad and crtrott May 30, 2025 16:29
@lucbv lucbv self-assigned this May 30, 2025
@lucbv lucbv added CMake C++20 Related to the transition to C++ 20 labels May 30, 2025
@masterleinad
Copy link
Contributor

Is it gcc < 10.40 or gcc < 11.1.0? I guess you meant a different compiler for the second version number?

@lucbv
Copy link
Contributor Author
lucbv commented May 30, 2025

No, gcc keeps patching multiple major versions at the same time. Based on the timeline of their releases I suspect they fixed the issue in gcc/12.0.0 and then back-ported it gcc/10.4.0 and gcc/11.1.0 which means that gcc/11.0.0 has the bug while 10.4.0 and 10.5.0 do not...

@masterleinad
Copy link
Contributor

No, gcc keeps patching multiple major versions at the same time. Based on the timeline of their releases I suspect they fixed the issue in gcc/12.0.0 and then back-ported it gcc/10.4.0 and gcc/11.1.0 which means that gcc/11.0.0 has the bug while 10.4.0 and 10.5.0 do not...

Should we then try going to gcc-12? We should also consider what the latest distros ship. Ubuntu 24.04 seems to ship gcc-13.2.

@JBludau
Copy link
Contributor
JBludau commented May 30, 2025

I think 11.1 is reasonable and avoids logic on our side. As long as no one needs gcc 10 ...

@@ -173,7 +173,7 @@ else()
set(KOKKOS_CLANG_CPU_MINIMUM 14.0.0)
set(KOKKOS_CLANG_CUDA_MINIMUM 14.0.0)
set(KOKKOS_CLANG_OPENMPTARGET_MINIMUM 15.0.0)
set(KOKKOS_GCC_MINIMUM 10.1.0)
set(KOKKOS_GCC_MINIMUM 10.4.0)
Copy link
Contributor
B7E1

Choose a reason for hiding this comment

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

Suggested change
set(KOKKOS_GCC_MINIMUM 10.4.0)
set(KOKKOS_GCC_MINIMUM 11.1.0)

@crtrott
Copy link
Member
crtrott commented May 30, 2025

I looked at release dates. The GCC 10.4 is fairly new June 2022. 11.1 is April 2021, 10.1 is May 2020, 12.1 is May 22. I feel slightly uncomfortable raising the date all the way to 22. Let me confer with folks on our side regarding minimum version. Note that Kokkos itself would be fine with the older GCC.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C++20 Related to the transition to C++ 20 CMake
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
0