Skip to content

Conversation

c8ef
Copy link
Contributor

@c8ef c8ef commented Jul 19, 2025

  • C++17 is now the default, no need to specify explicitly.

set(CMAKE_CXX_STANDARD 17)

  • The current recommended way to set up OpenMP is as a runtime, not as a project.

if ("openmp" IN_LIST LLVM_ENABLE_PROJECTS)
message(WARNING "Using LLVM_ENABLE_PROJECTS=openmp is deprecated now, and will "
"become a fatal error in the LLVM 21 release. Please use "
"-DLLVM_ENABLE_RUNTIMES=openmp or see the instructions at "
"https://openmp.llvm.org/ for building the runtimes.")
endif()

@llvmbot llvmbot added the flang Flang issues not falling into any other category label Jul 19, 2025
@c8ef c8ef requested review from Meinersbur, jhuber6 and mjklemm July 19, 2025 02:20
@mjklemm
Copy link
Contributor

mjklemm commented Jul 19, 2025

LGTM, thanks for catching this

@c8ef c8ef merged commit c875bb8 into llvm:main Jul 19, 2025
12 checks passed
@c8ef c8ef deleted the flang-start branch July 19, 2025 05:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

flang Flang issues not falling into any other category

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants