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

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 15 07:36:51 PDT 2024


================
@@ -34,8 +34,8 @@ void LangOptions::resetNonModularOptions() {
   // invocations that cannot be round-tripped to arguments.
   // FIXME: we should derive this automatically from ImpliedBy in tablegen.
   AllowFPReassoc = UnsafeFPMath;
-  NoHonorNaNs = FiniteMathOnly;
-  NoHonorInfs = FiniteMathOnly;
+  NoHonorNaNs = 0;
+  NoHonorInfs = 0;
----------------
AaronBallman wrote:

I think we don't need this code at all because `LangOptions.def` defines both of these with a default value of 0 anyway, so the x macros above should already do this automatically, right?

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


More information about the cfe-commits mailing list