Skip to content

Commit c875bb8

Browse files
authored
[flang] Minor revision of the getting started guide. (#149632)
- C++17 is now the default, no need to specify explicitly. https://github.com/llvm/llvm-project/blob/b3c9ed151f18fbbfe027cf93dd7957f36bcbaccf/flang/CMakeLists.txt#L13 - The current recommended way to set up OpenMP is as a runtime, not as a project. https://github.com/llvm/llvm-project/blob/b3c9ed151f18fbbfe027cf93dd7957f36bcbaccf/llvm/CMakeLists.txt#L209-L214
1 parent ef49ed4 commit c875bb8

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

flang/docs/GettingStarted.md

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -74,15 +74,14 @@ cmake \
7474
-G Ninja \
7575
-DCMAKE_BUILD_TYPE=Release \
7676
-DCMAKE_INSTALL_PREFIX=$INSTALLDIR \
77-
-DCMAKE_CXX_STANDARD=17 \
7877
-DCMAKE_EXPORT_COMPILE_COMMANDS=ON \
7978
-DCMAKE_CXX_LINK_FLAGS="-Wl,-rpath,$LD_LIBRARY_PATH" \
8079
-DFLANG_ENABLE_WERROR=ON \
8180
-DLLVM_ENABLE_ASSERTIONS=ON \
8281
-DLLVM_TARGETS_TO_BUILD=host \
8382
-DLLVM_LIT_ARGS=-v \
84-
-DLLVM_ENABLE_PROJECTS="clang;mlir;flang;openmp" \
85-
-DLLVM_ENABLE_RUNTIMES="compiler-rt;flang-rt" \
83+
-DLLVM_ENABLE_PROJECTS="clang;mlir;flang" \
84+
-DLLVM_ENABLE_RUNTIMES="compiler-rt;flang-rt;openmp" \
8685
../llvm-project/llvm
8786

8887
ninja
@@ -141,7 +140,6 @@ cd build
141140
cmake \
142141
-G Ninja \
143142
-DCMAKE_BUILD_TYPE=Release \
144-
-DCMAKE_CXX_STANDARD=17 \
145143
-DCMAKE_CXX_LINK_FLAGS="-Wl,-rpath,$LD_LIBRARY_PATH" \
146144
-DCMAKE_EXPORT_COMPILE_COMMANDS=ON \
147145
-DFLANG_ENABLE_WERROR=ON \

0 commit comments

Comments
 (0)