Skip to content

Commit 3e004d1

Browse files
[Github] Enable warnings as errors on flang sphinx build (llvm#72723)
Now that the number of warnings in the flang sphinx build has come down significantly, we can turn on warnings as errors in the sphinx build, which is the default configuration in CMake.
1 parent 6a8ab12 commit 3e004d1

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

.github/workflows/docs.yml

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -164,8 +164,6 @@ jobs:
164164
TZ=UTC ninja -C polly-build docs-polly-html docs-polly-man
165165
- name: Build Flang docs
166166
if: steps.docs-changed-subprojects.outputs.flang_any_changed == 'true'
167-
# TODO(boomanaiden154): Remove the SPHINX_WARNINGS_AS_ERRORS from the
168-
# CMake invocation once the warnings in the flang docs build are fixed.
169167
run: |
170-
cmake -B flang-build -GNinja -DCMAKE_BUILD_TYPE=Release -DLLVM_ENABLE_PROJECTS="clang;mlir;flang" -DLLVM_ENABLE_SPHINX=ON -DSPHINX_WARNINGS_AS_ERRORS=OFF ./llvm
171-
TZ=UTC ninja -C flang-build docs-flang-html docs-flang-man
168+
cmake -B flang-build -GNinja -DCMAKE_BUILD_TYPE=Release -DLLVM_ENABLE_PROJECTS="clang;mlir;flang" -DLLVM_ENABLE_SPHINX=ON ./llvm
169+
TZ=UTC ninja -C flang-build docs-flang-html

0 commit comments

Comments
 (0)