diff options
author | Arthur Eubanks <aeubanks@google.com> | 2021-02-10 22:06:49 -0800 |
---|---|---|
committer | Arthur Eubanks <aeubanks@google.com> | 2021-02-10 22:06:49 -0800 |
commit | 8334cdde2e830787029ca819a26b745c47432a64 (patch) | |
tree | 0bd50773a8ce42a58b7821949461e519fdc45170 /llvm/lib/LTO/LTOBackend.cpp | |
parent | bac1f12727835bd8b80ad3db256457ef91eed63b (diff) | |
download | llvm-8334cdde2e830787029ca819a26b745c47432a64.zip llvm-8334cdde2e830787029ca819a26b745c47432a64.tar.gz llvm-8334cdde2e830787029ca819a26b745c47432a64.tar.bz2 |
[NFC] Don't pass redundant arguments
Some parameters were already part of the Config passed in.
Diffstat (limited to 'llvm/lib/LTO/LTOBackend.cpp')
-rw-r--r-- | llvm/lib/LTO/LTOBackend.cpp | 19 |
1 files changed, 8 insertions, 11 deletions
diff --git a/llvm/lib/LTO/LTOBackend.cpp b/llvm/lib/LTO/LTOBackend.cpp index 3839cc1..bf49e5d 100644 --- a/llvm/lib/LTO/LTOBackend.cpp +++ b/llvm/lib/LTO/LTOBackend.cpp @@ -289,17 +289,15 @@ static void runNewPMPasses(const Config &Conf, Module &Mod, TargetMachine *TM, } static void runNewPMCustomPasses(const Config &Conf, Module &Mod, - TargetMachine *TM, std::string PipelineDesc, - std::string AAPipelineDesc, - bool DisableVerify) { + TargetMachine *TM) { PassBuilder PB(Conf.DebugPassManager, TM); AAManager AA; // Parse a custom AA pipeline if asked to. - if (!AAPipelineDesc.empty()) - if (auto Err = PB.parseAAPipeline(AA, AAPipelineDesc)) + if (!Conf.AAPipeline.empty()) + if (auto Err = PB.parseAAPipeline(AA, Conf.AAPipeline)) report_fatal_error("unable to parse AA pipeline description '" + - AAPipelineDesc + "': " + toString(std::move(Err))); + Conf.AAPipeline + "': " + toString(std::move(Err))); RegisterPassPlugins(Conf.PassPlugins, PB); @@ -330,11 +328,11 @@ static void runNewPMCustomPasses(const Config &Conf, Module &Mod, MPM.addPass(VerifierPass()); // Now, add all the passes we've been requested to. - if (auto Err = PB.parsePassPipeline(MPM, PipelineDesc)) + if (auto Err = PB.parsePassPipeline(MPM, Conf.OptPipeline)) report_fatal_error("unable to parse pass pipeline description '" + - PipelineDesc + "': " + toString(std::move(Err))); + Conf.OptPipeline + "': " + toString(std::move(Err))); - if (!DisableVerify) + if (!Conf.DisableVerify) MPM.addPass(VerifierPass()); MPM.run(Mod, MAM); } @@ -395,8 +393,7 @@ bool lto::opt(const Config &Conf, TargetMachine *TM, unsigned Task, Module &Mod, } // FIXME: Plumb the combined index into the new pass manager. if (!Conf.OptPipeline.empty()) - runNewPMCustomPasses(Conf, Mod, TM, Conf.OptPipeline, Conf.AAPipeline, - Conf.DisableVerify); + runNewPMCustomPasses(Conf, Mod, TM); else if (Conf.UseNewPM) runNewPMPasses(Conf, Mod, TM, Conf.OptLevel, IsThinLTO, ExportSummary, ImportSummary); |