summaryrefslogtreecommitdiffstats
path: root/clang/lib/Driver/ToolChains
diff options
context:
space:
mode:
authorFrancis Visoiu Mistrih <francisvm@yahoo.com>2019-12-18 13:16:09 -0800
committerFrancis Visoiu Mistrih <francisvm@yahoo.com>2019-12-18 14:31:41 -0800
commita8c678cb9a90c4f82505f573e9b9fa23fec93a64 (patch)
tree4ec777994709d4c057ad81aecdd12788da3392b8 /clang/lib/Driver/ToolChains
parentbce1cce6bf1286541c57690ab1129fbc02c60f93 (diff)
downloadbcm5719-llvm-a8c678cb9a90c4f82505f573e9b9fa23fec93a64.tar.gz
bcm5719-llvm-a8c678cb9a90c4f82505f573e9b9fa23fec93a64.zip
[Remarks][Driver][NFC] Make shouldEmitRemarks more available in the Driver
Move the function to Driver.h so that it can be re-used in other places.
Diffstat (limited to 'clang/lib/Driver/ToolChains')
-rw-r--r--clang/lib/Driver/ToolChains/Clang.cpp27
1 files changed, 2 insertions, 25 deletions
diff --git a/clang/lib/Driver/ToolChains/Clang.cpp b/clang/lib/Driver/ToolChains/Clang.cpp
index 5bf0efcf050..9840acafd6e 100644
--- a/clang/lib/Driver/ToolChains/Clang.cpp
+++ b/clang/lib/Driver/ToolChains/Clang.cpp
@@ -1415,29 +1415,6 @@ static bool isNoCommonDefault(const llvm::Triple &Triple) {
}
}
-static bool shouldEmitRemarks(const ArgList &Args) {
- // -fsave-optimization-record enables it.
- if (Args.hasFlag(options::OPT_fsave_optimization_record,
- options::OPT_fno_save_optimization_record, false))
- return true;
-
- // -fsave-optimization-record=<format> enables it as well.
- if (Args.hasFlag(options::OPT_fsave_optimization_record_EQ,
- options::OPT_fno_save_optimization_record, false))
- return true;
-
- // -foptimization-record-file alone enables it too.
- if (Args.hasFlag(options::OPT_foptimization_record_file_EQ,
- options::OPT_fno_save_optimization_record, false))
- return true;
-
- // -foptimization-record-passes alone enables it too.
- if (Args.hasFlag(options::OPT_foptimization_record_passes_EQ,
- options::OPT_fno_save_optimization_record, false))
- return true;
- return false;
-}
-
static bool hasMultipleInvocations(const llvm::Triple &Triple,
const ArgList &Args) {
// Supported only on Darwin where we invoke the compiler multiple times
@@ -3718,7 +3695,7 @@ static void RenderDebugOptions(const ToolChain &TC, const Driver &D,
TC.adjustDebugInfoKind(DebugInfoKind, Args);
// When emitting remarks, we need at least debug lines in the output.
- if (shouldEmitRemarks(Args) &&
+ if (willEmitRemarks(Args) &&
DebugInfoKind <= codegenoptions::DebugDirectivesOnly)
DebugInfoKind = codegenoptions::DebugLineTablesOnly;
@@ -5546,7 +5523,7 @@ void Clang::ConstructJob(Compilation &C, const JobAction &JA,
CmdArgs.push_back("-fapple-pragma-pack");
// Remarks can be enabled with any of the `-f.*optimization-record.*` flags.
- if (shouldEmitRemarks(Args) && checkRemarksOptions(D, Args, Triple))
+ if (willEmitRemarks(Args) && checkRemarksOptions(D, Args, Triple))
renderRemarksOptions(Args, CmdArgs, Triple, Input, JA);
bool RewriteImports = Args.hasFlag(options::OPT_frewrite_imports,
OpenPOWER on IntegriCloud