mibintc added a comment.
In D72841#2022354 <https://reviews.llvm.org/D72841#2022354>, @uabelho wrote:
> It seems to be this change in SemaStmt.cpp that makes the FENV-pragma have
> some effect regardless of the warning saying that the pragma doesn't have
> any effect:
>
> index aa0d89ac09c3..f76994a6dab3 100644
> @@ -394,6 +394,11 @@ StmtResult Sema::ActOnCompoundStmt(SourceLocation L,
> SourceLocation R,
> ArrayRef<Stmt *> Elts, bool isStmtExpr)
> {
> const unsigned NumElts = Elts.size();
>
> + // Mark the current function as usng floating point constrained
> intrinsics
> + if (getCurFPFeatures().isFPConstrained())
> + if (FunctionDecl *F = dyn_cast<FunctionDecl>(CurContext))
> + F->setUsesFPIntrin(true);
> +
>
Thank you I will follow up!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D72841/new/
https://reviews.llvm.org/D72841
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
