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

Joshua Cranmer via cfe-commits cfe-commits at lists.llvm.org
Tue Jul 16 08:35:37 PDT 2024


================
@@ -174,9 +174,10 @@ static void addVisualCDefines(const LangOptions &Opts, MacroBuilder &Builder) {
   // transformation unless the transformation is guaranteed to produce a bitwise
   // identical result."
   const bool any_imprecise_flags =
-      Opts.FastMath || Opts.FiniteMathOnly || Opts.UnsafeFPMath ||
-      Opts.AllowFPReassoc || Opts.NoHonorNaNs || Opts.NoHonorInfs ||
-      Opts.NoSignedZero || Opts.AllowRecip || Opts.ApproxFunc;
+      Opts.FastMath || (Opts.NoHonorInfs && Opts.NoHonorNaNs) ||
----------------
jcranmer-intel wrote:

Given that NoHonorInfs and NoHonorNaNs are both individually checked below, you don't need this entry here.

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


More information about the cfe-commits mailing list