[clang] Remove FiniteMathOnly and use only NoHonorINFs and NoHonorNANs. (PR #97342)

Michael Toguchi via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 12 09:30:57 PDT 2024


================
@@ -3295,7 +3295,17 @@ static void RenderFloatingPointOptions(const ToolChain &TC, const Driver &D,
   }
 
   // Handle __FINITE_MATH_ONLY__ similarly.
-  if (!HonorINFs && !HonorNaNs)
+  bool InfValues, NanValues = true;
+  auto processArg = [&](const auto *Arg) {
+    if (StringRef(Arg->getValue()) == "-menable-no-nans")
+      NanValues = false;
+    if (StringRef(Arg->getValue()) == "-menable-no-infs")
+      InfValues = false;
+  };
+  for (auto *Arg : Args.filtered(options::OPT_Xclang)) {
+    processArg(Arg);
+  }
----------------
mdtoguchi wrote:

```suggestion
  for (auto *Arg : Args.filtered(options::OPT_Xclang))
    processArg(Arg);
```

https://github.com/llvm/llvm-project/pull/97342


More information about the cfe-commits mailing list