nikic wrote:

> @nikic Sorry for the mess :/
> 
> I took the `target=` approach from 
> [here](https://llvm.org/docs/TestingGuide.html#constraining-test-execution). 
> Following the current buildbot status, this also does not seem to crash. Does 
> this mean the rests are still incorrect and are not being executed at all? My 
> local setup has all targets enabled, so it's hard for me to reproduce this 
> locally.

I expect the tests just don't run at all now.

> If this is still incorrect, I'd kindly ask you to point out the correct way 
> of doing this across all tests. I guess it would have `REQUIRES: 
> aarch64-registered-target` for AArch64 and `REQUIRES: 
> riscv64-registered-target` for RISCV?

The RISCV check would be `REQUIRES: riscv-registered-target`.

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

Reply via email to