Skip to content

Conversation

@asantagata
Copy link
Contributor

Ported to Clang 20 as described in #191

@tpatki
Copy link
Member

tpatki commented Oct 28, 2025

@asantagata

  • We need to change the clang version in CI as well, so your tests pass here. That change is here.
  • You also need to change the CMakeLists to ensure that Clang 20.0 is the pinned version instead of Clang18. That change is here:
    # Fail if using Clang < 18.0
    if("${CMAKE_CXX_COMPILER_ID}" STREQUAL "Clang")
    if (CMAKE_CXX_COMPILER_VERSION VERSION_LESS 18.0)
    message(FATAL_ERROR "Clang++ version must be equal to 18.0!")
    endif()
    else()
    message(FATAL_ERROR "Unsupported CXX compiler: please use Clang == 18.0")
    endif()

Could you fix these in this PR as well?

@tpatki tpatki merged commit d58868e into flux-framework:main Oct 29, 2025
9 checks passed
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.

2 participants