vvereschaka wrote:

@MaskRay 
>// UNSUPPORTED: system-windows was a fairly reasonable workaround, after the 
>previously imprecise // REQUIRES: x86-registered-target had been removed.
I'm not quite agree here, sorry. That kind of workarounds disable the tests for 
the cross platform builders on the Windows build hosts, but they could need 
that kind of test even more than the other build hosts on my sight. Sure, this 
test didn't work for Arm/aarch64 builders before and this workaround does not 
change anything for them, but this should not be like that I suppose.

> (It seems that there is an arm/aarch64 window bot that does not enable x86 
> targets.)
Correct. These bots enable only Arm/Aarch64 targets.

> #94276 seems to fix the test properly.
yes, it fixes `clang/test/Driver/sysroot.c` test

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

Reply via email to