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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to