Actually kludging it by just removing the assert isn't going to work. I'll ping 
Pengfei to see about developing a patch for this problem. 

> -----Original Message-----
> From: dmajor via Phabricator <revi...@reviews.llvm.org>
> Sent: Tuesday, October 27, 2020 11:58 AM
> To: Blower, Melanie I <melanie.blo...@intel.com>; sepavl...@gmail.com;
> rjmcc...@gmail.com; rich...@metafoo.co.uk; aaron.ball...@gmail.com
> Cc: dma...@mozilla.com; dexonsm...@apple.com; sca...@apple.com;
> kevin.n...@sas.com; cfe-commits@lists.llvm.org; mlek...@skidmore.edu;
> blitzrak...@gmail.com; shen...@google.com; paul.robin...@sony.com
> Subject: [PATCH] D87528: Enable '#pragma STDC FENV_ACCESS' in frontend
> 
> dmajor added a comment.
> 
> After this commit our bots have a failure on the wasm target, could you please
> take a look?
> 
> test.c:
> 
>   double a, b;
>   c() {
>   #pragma STDC FENV_ACCESS ON
>     b == a;
>   }
> 
> Run `clang -cc1 -triple wasm32-unknown-wasi -emit-obj test.c` (clang needs to
> be build with `-DLLVM_EXPERIMENTAL_TARGETS_TO_BUILD=WebAssembly`)
> 
> Result:
> 
>   clang: /home/vm/src/llvm-
> project/llvm/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp:3764: bool
> (anonymous namespace)::SelectionDAGLegalize::ExpandNode(llvm::SDNode *):
> Assertion `!IsStrict && "Don't know how to expand for strict nodes."' failed.
> 
> 
> Repository:
>   rG LLVM Github Monorepo
> 
> CHANGES SINCE LAST ACTION
>   https://reviews.llvm.org/D87528/new/
> 
> https://reviews.llvm.org/D87528

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

Reply via email to