rorth wrote:

> > the remaining failures seem all to be XPASSes on AArch64. I guess it's time 
> > to just remove aarch64 from the XFAIL in bindings.sh then
> 
> Sounds great! Two more things from my side:
> 
>     1. Please resolve the conflict so CI can run.

Done now.

>     2. You removed the changed CI trigger path again (clang/tests/bindings... 
> or sth in libclang-python-tests.yml, can't check rn since I'm on mobile). I 
> only noticed after merging yesterday, please add that one back in so the CI 
> id triggered also when the tests in their new location are changed

Right, I accidentally reverted all changes to `libclang-python-tests.yml` 
instead of just the new `run:` statement.  Fixed now.


https://github.com/llvm/llvm-project/pull/145855
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to