Skip to content

Commit

Permalink
[Github] Enable warnings as errors on flang sphinx build (llvm#72723)
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
boomanaiden154 authored Feb 15, 2024
1 parent 6a8ab12 commit 3e004d1
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions .github/workflows/docs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -164,8 +164,6 @@ jobs:
TZ=UTC ninja -C polly-build docs-polly-html docs-polly-man
- name: Build Flang docs
if: steps.docs-changed-subprojects.outputs.flang_any_changed == 'true'
# TODO(boomanaiden154): Remove the SPHINX_WARNINGS_AS_ERRORS from the
# CMake invocation once the warnings in the flang docs build are fixed.
run: |
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
TZ=UTC ninja -C flang-build docs-flang-html docs-flang-man
cmake -B flang-build -GNinja -DCMAKE_BUILD_TYPE=Release -DLLVM_ENABLE_PROJECTS="clang;mlir;flang" -DLLVM_ENABLE_SPHINX=ON ./llvm
TZ=UTC ninja -C flang-build docs-flang-html

0 comments on commit 3e004d1

Please sign in to comment.