zahiraam created this revision.
zahiraam added a reviewer: aaron.ballman.
Herald added a project: All.
zahiraam requested review of this revision.
Herald added a project: clang.

Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D135963

Files:
  clang/lib/CodeGen/CGCall.cpp


Index: clang/lib/CodeGen/CGCall.cpp
===================================================================
--- clang/lib/CodeGen/CGCall.cpp
+++ clang/lib/CodeGen/CGCall.cpp
@@ -1863,9 +1863,9 @@
     if (LangOpts.ApproxFunc)
       FuncAttrs.addAttribute("approx-func-fp-math", "true");
     if ((LangOpts.FastMath ||
-         !LangOpts.FastMath && LangOpts.AllowFPReassoc && LangOpts.AllowRecip 
&&
+         (!LangOpts.FastMath && LangOpts.AllowFPReassoc && LangOpts.AllowRecip 
&&
              !LangOpts.FiniteMathOnly && LangOpts.NoSignedZero &&
-             LangOpts.ApproxFunc) &&
+             LangOpts.ApproxFunc)) &&
         LangOpts.getDefaultFPContractMode() != 
LangOptions::FPModeKind::FPM_Off)
       FuncAttrs.addAttribute("unsafe-fp-math", "true");
     if (CodeGenOpts.SoftFloat)


Index: clang/lib/CodeGen/CGCall.cpp
===================================================================
--- clang/lib/CodeGen/CGCall.cpp
+++ clang/lib/CodeGen/CGCall.cpp
@@ -1863,9 +1863,9 @@
     if (LangOpts.ApproxFunc)
       FuncAttrs.addAttribute("approx-func-fp-math", "true");
     if ((LangOpts.FastMath ||
-         !LangOpts.FastMath && LangOpts.AllowFPReassoc && LangOpts.AllowRecip &&
+         (!LangOpts.FastMath && LangOpts.AllowFPReassoc && LangOpts.AllowRecip &&
              !LangOpts.FiniteMathOnly && LangOpts.NoSignedZero &&
-             LangOpts.ApproxFunc) &&
+             LangOpts.ApproxFunc)) &&
         LangOpts.getDefaultFPContractMode() != LangOptions::FPModeKind::FPM_Off)
       FuncAttrs.addAttribute("unsafe-fp-math", "true");
     if (CodeGenOpts.SoftFloat)
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D135963: Fix b... Zahira Ammarguellat via Phabricator via cfe-commits

Reply via email to