ZijunZhaoCCK wrote:

> > > There's apparently also wasm32-wasi-preview2 and wasm32-wasi-pthread, 
> > > which I suppose are equally broken by this change.
> > 
> > 
> > Yes, I think so. I think adding these environment types in wasi-libc repo 
> > could help fix those errors.
> 
> If wasm can arbitrary environment types, it seems that we can opt out the 
> check for `isWasm()`. We define new environment types when they affect the 
> compiler behavior. If wasm arbitrary sets the environment, we should not 
> define a type for each one.

Okay, I will add this check.

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

Reply via email to