[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