[PATCH] D87528: Enable '#pragma STDC FENV_ACCESS' in frontend

Blower, Melanie I via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 27 10:37:37 PDT 2020


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 <reviews at reviews.llvm.org>
> Sent: Tuesday, October 27, 2020 11:58 AM
> To: Blower, Melanie I <melanie.blower at intel.com>; sepavloff at gmail.com;
> rjmccall at gmail.com; richard at metafoo.co.uk; aaron.ballman at gmail.com
> Cc: dmajor at mozilla.com; dexonsmith at apple.com; scanon at apple.com;
> kevin.neal at sas.com; cfe-commits at lists.llvm.org; mlekena at skidmore.edu;
> blitzrakete at gmail.com; shenhan at google.com; paul.robinson at 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



More information about the cfe-commits mailing list