diff options
| author | Fangrui Song <i@maskray.me> | 2022-03-26 00:57:06 -0700 |
|---|---|---|
| committer | Fangrui Song <i@maskray.me> | 2022-03-26 00:57:06 -0700 |
| commit | c37accf0a207a09b80c1109eb7a114876c45f959 (patch) | |
| tree | 3d0bd9681d1e71400bc19f5b9747187e752b5345 /clang/lib/Frontend/CompilerInvocation.cpp | |
| parent | da62a5c6610dd2087ce2f527ca84ba43e93d5e30 (diff) | |
| download | llvm-c37accf0a207a09b80c1109eb7a114876c45f959.zip llvm-c37accf0a207a09b80c1109eb7a114876c45f959.tar.gz llvm-c37accf0a207a09b80c1109eb7a114876c45f959.tar.bz2 | |
[Option] Avoid using the default argument for the 3-argument hasFlag. NFC
The default argument true is error-prone: I think many would think the
default is false.
Diffstat (limited to 'clang/lib/Frontend/CompilerInvocation.cpp')
| -rw-r--r-- | clang/lib/Frontend/CompilerInvocation.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/clang/lib/Frontend/CompilerInvocation.cpp b/clang/lib/Frontend/CompilerInvocation.cpp index e18c7ed..be7828b 100644 --- a/clang/lib/Frontend/CompilerInvocation.cpp +++ b/clang/lib/Frontend/CompilerInvocation.cpp @@ -1982,8 +1982,8 @@ bool CompilerInvocation::ParseCodeGenArgs(CodeGenOptions &Opts, ArgList &Args, else if (Args.hasArg(options::OPT_fno_finite_loops)) Opts.FiniteLoops = CodeGenOptions::FiniteLoopsKind::Never; - Opts.EmitIEEENaNCompliantInsts = - Args.hasFlag(options::OPT_mamdgpu_ieee, options::OPT_mno_amdgpu_ieee); + Opts.EmitIEEENaNCompliantInsts = Args.hasFlag( + options::OPT_mamdgpu_ieee, options::OPT_mno_amdgpu_ieee, true); if (!Opts.EmitIEEENaNCompliantInsts && !LangOptsRef.NoHonorNaNs) Diags.Report(diag::err_drv_amdgpu_ieee_without_no_honor_nans); |