phosek added a comment.

In D98023#2609328 <https://reviews.llvm.org/D98023#2609328>, @mstorsjo wrote:

> In D98023#2609318 <https://reviews.llvm.org/D98023#2609318>, @phosek wrote:
>
>> We see two test failures that appear to have been introduced by this change:
>
> Hmm - I'm pretty surprised about that, because the change literally added an 
> `&& !TC.getTriple().isOSCygMing()`, which should evaluate to `&& true` when 
> targeting anything else than mingw/cygwin - and the errors you quoted clearly 
> show a linux triple.
>
> I presume you're not actually running the build in a mingw environment, in 
> case that would happen to surprisingly interfere somehow?
>
> Feel free to revert - but I'd appreciate if you could try to figure out why 
> it breaks things, as to me, it should be a clear no-op for linux targets.

The only thing that might be somewhat unusual is that we use 
`CLANG_DEFAULT_RTLIB=compiler-rt`, see 
https://github.com/llvm/llvm-project/blob/fb2cf0dd609238931b0fc53ef0ea2172a99307ae/clang/cmake/caches/Fuchsia-stage2.cmake#L34


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D98023/new/

https://reviews.llvm.org/D98023

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to