kwk wrote:

> This test was failing on non-x86 systems (or rather, builds where that wasn't 
> the default triple): https://lab.llvm.org/buildbot/#/builders/29/builds/17827 
> https://lab.llvm.org/buildbot/#/builders/201/builds/6934

Thank you for bringing this up. I was out shortly after the commit was 
auto-merged. That wasn't ideal and I hope you accept my apologies.

> So I have added an x86 target to the first command to fix it. The test now 
> passes, but please check this change does not defeat the point of the test.
> 
> If you really need it to check what happens without any target argument, we 
> can probably find a set of `REQUIRES` to only run it in the right places.

Honeslty, I thought clang would take the sysroot (`Inputs/fedora_39_tree`) and 
only make the containing targets available. I hope no more change or revert is 
needed.



https://github.com/llvm/llvm-project/pull/165886
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to