diff options
author | Francis Visoiu Mistrih <francisvm@yahoo.com> | 2019-12-18 13:16:09 -0800 |
---|---|---|
committer | Francis Visoiu Mistrih <francisvm@yahoo.com> | 2019-12-18 14:31:41 -0800 |
commit | a8c678cb9a90c4f82505f573e9b9fa23fec93a64 (patch) | |
tree | 4ec777994709d4c057ad81aecdd12788da3392b8 /clang/lib/Driver/ToolChains | |
parent | bce1cce6bf1286541c57690ab1129fbc02c60f93 (diff) | |
download | bcm5719-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.cpp | 27 |
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, |