[clang-tools-extra] Add new tool: clang-read-diagnostics (PR #118522)
Jon Roelofs via cfe-commits
cfe-commits at lists.llvm.org
Wed Dec 4 09:44:40 PST 2024
jroelofs wrote:
> Looking at the test failure, it appears that when only building `check-clang-tools`, no `clang` binary was built. So the test driver cannot find `clang`. Maybe this diagnostics binary file should be checked in for the test? That will not catch future errors if the diagnostics binary format changes.
We should stay away from committed binaries, per the `xz` fiasco. A better way to solve this, I think, would be to add a `REQUIRES:` clause for it. See the `available_features.add()` calls in `llvm/utils/lit/lit/llvm/config.py`.
https://github.com/llvm/llvm-project/pull/118522
More information about the cfe-commits
mailing list