[Github] Build clang docs in CI if autogenerated files change

[boomanaiden154]

These are unlikely to produce errors in the build, assuming that the tablegen emitter works as expected, but we're still losing some documentation build testing coverage but not building upon changes to these files.

[llvmbot]

@llvm/pr-subscribers-github-workflow

Author: Aiden Grossman (boomanaiden154)

Changes

These are unlikely to produce errors in the build, assuming that the tablegen emitter works as expected, but we're still losing some documentation build testing coverage but not building upon changes to these files.

---

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

1 Files Affected:

• (modified) .github/workflows/docs.yml (+9)

diff --git a/.github/workflows/docs.yml b/.github/workflows/docs.yml
index 6329d777a182031..ae68dfd3a4d12e0 100644
--- a/.github/workflows/docs.yml
+++ b/.github/workflows/docs.yml
@@ -15,6 +15,9 @@ on:
     paths:
       - 'llvm/docs/**'
       - 'clang/docs/**'
+      - 'clang/include/clang/Basic/AttrDocs.td'
+      - 'clang/include/clang/Driver/ClangOptionDocs.td'
+      - 'clang/include/clang/Basic/DiagnosticDocs.td'
       - 'clang-tools-extra/docs/**'
       - 'lldb/docs/**'
       - 'libunwind/docs/**'
@@ -28,6 +31,9 @@ on:
     paths:
       - 'llvm/docs/**'
       - 'clang/docs/**'
+      - 'clang/include/clang/Basic/AttrDocs.td'
+      - 'clang/include/clang/Driver/ClangOptionDocs.td'
+      - 'clang/include/clang/Basic/DiagnosticDocs.td'
       - 'clang-tools-extra/docs/**'
       - 'lldb/docs/**'
       - 'libunwind/docs/**'
@@ -62,6 +68,9 @@ jobs:
               - 'llvm/docs/**'
             clang:
               - 'clang/docs/**'
+              - 'clang/include/clang/Basic/AttrDocs.td'
+              - 'clang/include/clang/Driver/ClangOptionDocs.td'
+              - 'clang/include/clang/Basic/DiagnosticDocs.td'
             clang-tools-extra:
               - 'clang-tools-extra/docs/**'
             lldb:

[boomanaiden154]

Still need to try and find a more scalable solution than copying and pasting the paths into three separate parts of the document. Will look into that in a bit.

[tstellar]

Still need to try and find a more scalable solution than copying and pasting the paths into three separate parts of the document. Will look into that in a bit.

I think you have to abandon the builtin github path filtering if you only want to maintain the list in one place. Creating a separate workflow file for each documentation build would get you from 3 copies of the list to 2, but I'm not sure that's really that much better.