[llvm] 325c529 - [NewPM][opt] Assert PassPipeline and Passes don't both contain passes

Arthur Eubanks via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 24 18:00:48 PDT 2020


Author: Arthur Eubanks
Date: 2020-06-24T18:00:22-07:00
New Revision: 325c52956bfd64f4cb156996ca1ccde44609f40b

URL: https://github.com/llvm/llvm-project/commit/325c52956bfd64f4cb156996ca1ccde44609f40b
DIFF: https://github.com/llvm/llvm-project/commit/325c52956bfd64f4cb156996ca1ccde44609f40b.diff

LOG: [NewPM][opt] Assert PassPipeline and Passes don't both contain passes

Reviewers: asbirlea

Subscribers: llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D82512

Added: 
    

Modified: 
    llvm/tools/opt/NewPMDriver.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/tools/opt/NewPMDriver.cpp b/llvm/tools/opt/NewPMDriver.cpp
index cdde8a1f0149..322397438349 100644
--- a/llvm/tools/opt/NewPMDriver.cpp
+++ b/llvm/tools/opt/NewPMDriver.cpp
@@ -333,6 +333,8 @@ bool llvm::runPassPipeline(StringRef Arg0, Module &M, TargetMachine *TM,
     MPM.addPass(NewPMDebugifyPass());
 
   if (!PassPipeline.empty()) {
+    assert(Passes.empty() &&
+           "PassPipeline and Passes should not both contain passes");
     if (auto Err =
             PB.parsePassPipeline(MPM, PassPipeline, VerifyEachPass, DebugPM)) {
       errs() << Arg0 << ": " << toString(std::move(Err)) << "\n";


        


More information about the llvm-commits mailing list