Skip to content

ci: enforce unused include checks with IWYU - #104

Merged
tetsuo-cpp merged 1 commit into
canonfrom
ci/iwyu-unused-includes
Aug 22, 2026
Merged

ci: enforce unused include checks with IWYU#104
tetsuo-cpp merged 1 commit into
canonfrom
ci/iwyu-unused-includes

Conversation

@tetsuo-cpp

Copy link
Copy Markdown
Owner

Summary

  • build and pin IWYU 0.24 for the LLVM 20 toolchain
  • add a check-iwyu target that fails specifically on unused C++ includes
  • remove existing unused includes and preserve intentional MLIR umbrella includes
  • run the check in CI and install IWYU in orb setup

Verification

  • cmake --build build --target check-iwyu
  • synthetic unused-header check (correctly exits nonzero)
  • cmake --build build --target check-warpforth check-clang-tidy check-format

Comment thread tools/check-iwyu.py Outdated
Comment thread tools/warpforthc/warpforthc.cpp Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
@tetsuo-cpp
tetsuo-cpp force-pushed the ci/iwyu-unused-includes branch from c4f7df1 to f9f5ac1 Compare August 22, 2026 09:13
@tetsuo-cpp
tetsuo-cpp force-pushed the ci/iwyu-unused-includes branch from f9f5ac1 to caad13f Compare August 22, 2026 10:07
@tetsuo-cpp
tetsuo-cpp merged commit 7e5ead9 into canon Aug 22, 2026
1 check passed
@tetsuo-cpp
tetsuo-cpp deleted the ci/iwyu-unused-includes branch August 22, 2026 10:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant