Skip to content

[flang] Minor revision of the getting started guide.#149632

Merged
c8ef merged 1 commit intollvm:mainfrom
c8ef:flang-start
Jul 19, 2025
Merged

[flang] Minor revision of the getting started guide.#149632
c8ef merged 1 commit intollvm:mainfrom
c8ef:flang-start

Conversation

@c8ef
Copy link
Copy Markdown
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
Copy Markdown
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