Skip to content

Commit

Permalink
Driver: Handle -f{no-}honor-infinities, -f{no-}honor-nans, and
Browse files Browse the repository at this point in the history
-ffinite-math-only.
 - No test case yet, I don't know how to construct a situation where this
   matters.

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@146297 91177308-0d34-0410-b5e6-96231b3b80d8
  • Loading branch information
ddunbar committed Dec 9, 2011
1 parent 2c3c767 commit 001f628
Show file tree
Hide file tree
Showing 4 changed files with 36 additions and 3 deletions.
4 changes: 4 additions & 0 deletions include/clang/Driver/CC1Options.td
Expand Up @@ -166,6 +166,10 @@ def mdebug_pass : Separate<"-mdebug-pass">,
HelpText<"Enable additional debug output">;
def mdisable_fp_elim : Flag<"-mdisable-fp-elim">,
HelpText<"Disable frame pointer elimination optimization">;
def menable_no_infinities : Flag<"-menable-no-infs">,
HelpText<"Allow optimization to assume there are no infinities.">;
def menable_no_nans : Flag<"-menable-no-nans">,
HelpText<"Allow optimization to assume there are no NaNs.">;
def mfloat_abi : Separate<"-mfloat-abi">,
HelpText<"The float ABI to use">;
def mno_global_merge : Flag<"-mno-global-merge">,
Expand Down
11 changes: 10 additions & 1 deletion include/clang/Driver/Options.td
Expand Up @@ -315,7 +315,16 @@ def fexceptions : Flag<"-fexceptions">, Group<f_Group>;
def fextdirs_EQ : Joined<"-fextdirs=">, Group<f_Group>;
def fhosted : Flag<"-fhosted">, Group<f_Group>;
def ffast_math : Flag<"-ffast-math">, Group<clang_ignored_f_Group>;
def ffinite_math_only : Flag<"-ffinite-math-only">, Group<clang_ignored_f_Group>;
def ffinite_math_only : Flag<"-ffinite-math-only">, Group<f_Group>;
def fhonor_nans : Flag<"-fhonor-nans">, Group<f_Group>;
def fno_honor_nans : Flag<"-fno-honor-nans">, Group<f_Group>;
def fhonor_infinities : Flag<"-fhonor-infinities">, Group<f_Group>;
def fno_honor_infinities : Flag<"-fno-honor-infinities">, Group<f_Group>;
// Sic. This option was misspelled originally.
def fhonor_infinites : Flag<"-fhonor-infinites">, Group<f_Group>,
Alias<fhonor_infinities>;
def fno_honor_infinites : Flag<"-fno-honor-infinites">, Group<f_Group>,
Alias<fno_honor_infinities>;

def ffor_scope : Flag<"-ffor-scope">, Group<f_Group>;
def fno_for_scope : Flag<"-fno-for-scope">, Group<f_Group>;
Expand Down
12 changes: 12 additions & 0 deletions lib/Driver/Tools.cpp
Expand Up @@ -1345,6 +1345,18 @@ void Clang::ConstructJob(Compilation &C, const JobAction &JA,
getToolChain().IsStrictAliasingDefault()))
CmdArgs.push_back("-relaxed-aliasing");

// Handle -f{no-}honor-infinities, -f{no-}honor-nans, and -ffinite-math-only.
bool HonorInfinities = Args.hasFlag(options::OPT_fhonor_infinities,
options::OPT_fno_honor_infinities);
bool HonorNaNs = Args.hasFlag(options::OPT_fhonor_nans,
options::OPT_fno_honor_nans);
if (Args.hasArg(options::OPT_ffinite_math_only))
HonorInfinities = HonorNaNs = false;
if (!HonorInfinities)
CmdArgs.push_back("-menable-no-infs");
if (!HonorNaNs)
CmdArgs.push_back("-menable-no-nans");

// Decide whether to use verbose asm. Verbose assembly is the default on
// toolchains which have the integrated assembler on by default.
bool IsVerboseAsmDefault = getToolChain().IsIntegratedAssemblerDefault();
Expand Down
12 changes: 10 additions & 2 deletions lib/Frontend/CompilerInvocation.cpp
Expand Up @@ -183,6 +183,10 @@ static void CodeGenOptsToArgs(const CodeGenOptions &Opts,
Res.push_back("-main-file-name");
Res.push_back(Opts.MainFileName);
}
if (Opts.NoInfsFPMath)
Res.push_back("-menable-no-infinities");
if (Opts.NoNaNsFPMath)
Res.push_back("-menable-no-nans");
// SimplifyLibCalls is only derived.
// TimePasses is only derived.
// UnitAtATime is unused.
Expand Down Expand Up @@ -1083,8 +1087,12 @@ static void ParseCodeGenArgs(CodeGenOptions &Opts, ArgList &Args, InputKind IK,
Opts.HiddenWeakVTables = Args.hasArg(OPT_fhidden_weak_vtables);
Opts.LessPreciseFPMAD = Args.hasArg(OPT_cl_mad_enable);
Opts.LimitFloatPrecision = Args.getLastArgValue(OPT_mlimit_float_precision);
Opts.NoInfsFPMath = Opts.NoNaNsFPMath = Args.hasArg(OPT_cl_finite_math_only)||
Args.hasArg(OPT_cl_fast_relaxed_math);
Opts.NoInfsFPMath = (Args.hasArg(OPT_menable_no_infinities) ||
Args.hasArg(OPT_cl_finite_math_only)||
Args.hasArg(OPT_cl_fast_relaxed_math));
Opts.NoNaNsFPMath = (Args.hasArg(OPT_menable_no_nans) ||
Args.hasArg(OPT_cl_finite_math_only)||
Args.hasArg(OPT_cl_fast_relaxed_math));
Opts.NoZeroInitializedInBSS = Args.hasArg(OPT_mno_zero_initialized_in_bss);
Opts.BackendOptions = Args.getAllArgValues(OPT_backend_option);
Opts.NumRegisterParameters = Args.getLastArgIntValue(OPT_mregparm, 0, Diags);
Expand Down

0 comments on commit 001f628

Please sign in to comment.