[llvm] [ci] ids-check: build tablegen headers for idt (PR #176415)

[via llvm-commits]

llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-github-workflow

Author: Yaxun (Sam) Liu (yxsamliu)

<details>
<summary>Changes</summary>

This PR updates ids-check to build the CommonTableGen targets after LLVM
configuration, so generated headers like llvm/IR/Attributes.inc exist
before idt scans changed headers. It is needed because the current
workflow only lists those targets and never builds them, causing idt to
fail when a changed header includes a generated .inc file.

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


1 Files Affected:

- (modified) .github/workflows/ids-check.yml (+1-1) 


``````````diff
diff --git a/.github/workflows/ids-check.yml b/.github/workflows/ids-check.yml
index 7dcd9ff56d892..5cbde5cb8176e 100644
--- a/.github/workflows/ids-check.yml
+++ b/.github/workflows/ids-check.yml
@@ -55,7 +55,7 @@ jobs:
                 -D CMAKE_EXPORT_COMPILE_COMMANDS=ON \
                 -G Ninja
           cd ${{ github.workspace }}/llvm-project/build/
-          ninja -t targets all | grep "CommonTableGen: phony$" | grep -v "/" | sed 's/:.*//'
+          ninja -t targets all | grep "CommonTableGen: phony$" | grep -v "/" | sed 's/:.*//' | xargs -r ninja
 
       - name: Configure ids
         run: |

``````````

</details>


https://github.com/llvm/llvm-project/pull/176415