Skip to content

Conversation

@boomanaiden154
Copy link
Contributor

This patch enables building the polly docs in CI to catch warnings/build regressions in the documentation.

@llvmbot
Copy link
Member

llvmbot commented Oct 28, 2023

@llvm/pr-subscribers-github-workflow

Author: Aiden Grossman (boomanaiden154)

Changes

This patch enables building the polly docs in CI to catch warnings/build regressions in the documentation.


Full diff: https://github.com/llvm/llvm-project/pull/70531.diff

1 Files Affected:

  • (modified) .github/workflows/docs.yml (+9)
diff --git a/.github/workflows/docs.yml b/.github/workflows/docs.yml index e6af2f41167e0a1..ebe3c9e462b6886 100644 --- a/.github/workflows/docs.yml +++ b/.github/workflows/docs.yml @@ -21,6 +21,7 @@ on: - 'libcxx/docs/**' - 'libc/docs/**' - 'lld/docs/**' + - 'polly/docs/**' pull_request: paths: - 'llvm/docs/**' @@ -31,6 +32,7 @@ on: - 'libcxx/docs/**' - 'libc/docs/**' - 'lld/docs/**' + - 'polly/docs/**' jobs: check-docs-build: @@ -67,6 +69,8 @@ jobs: - 'libc/docs/**' lld: - 'lld/docs/**' + polly: + - 'polly/docs/**' - name: Fetch LLVM sources (PR) if: ${{ github.event_name == 'pull_request' }} uses: actions/checkout@v4 @@ -125,4 +129,9 @@ jobs: run: | cmake -B lld-build -GNinja -DCMAKE_BUILD_TYPE=Release -DLLVM_ENABLE_PROJECTS="lld" -DLLVM_ENABLE_SPHINX=ON ./llvm TZ=UTC ninja -C lld-build docs-lld-html + - name: Build Polly docs + if: steps.docs-changed-subprojects.outputs.polly_any_changed == 'true' + run: | + cmake -B polly-build -GNinja -DCMAKE_BUILD_TYPE=Release -DLLVM_ENABLE_PROJECTS="polly" -DLLVM_ENABLE_SPHINX=ON ./llvm + TZ=UTC ninja -C polly-build docs-polly-html docs-polly-man 
This patch enables building the polly docs in CI to catch warnings/build regressions in the documentation.
@boomanaiden154 boomanaiden154 merged commit 5cfa03e into llvm:main Oct 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

3 participants