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

[Arthur Eubanks via Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rG325c52956bfd: [NewPM][opt] Assert PassPipeline and Passes don't both contain passes (authored by aeubanks).

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D82512/new/

https://reviews.llvm.org/D82512

Files:
  llvm/tools/opt/NewPMDriver.cpp


Index: llvm/tools/opt/NewPMDriver.cpp
===================================================================
--- llvm/tools/opt/NewPMDriver.cpp
+++ llvm/tools/opt/NewPMDriver.cpp
@@ -333,6 +333,8 @@
     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";


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D82512.273213.patch
Type: text/x-patch
Size: 538 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200625/e48cc3ec/attachment.bin>