[PATCH] D22724: [PM] Significantly refactor the pass pipeline parsing to be easier to reason about and less error prone.

[Sean Silva via llvm-commits]

silvas accepted this revision.
silvas added a reviewer: silvas.
silvas added a comment.
This revision is now accepted and ready to land.

LGTM.


================
Comment at: lib/Passes/PassBuilder.cpp:574
@@ +573,3 @@
+      FPM.addPass(createFunctionToLoopPassAdaptor(std::move(LPM)));
+      return true;
+    }
----------------
Thanks for the clarification. That makes sense to me. Can you make sure that ends up in a comment somewhere (if it isn't already)?

================
Comment at: lib/Passes/PassBuilder.cpp:602
@@ -424,2 +601,3 @@
 
-bool PassBuilder::parseLoopPassName(LoopPassManager &FPM, StringRef Name) {
+bool PassBuilder::parseLoopPass(LoopPassManager &FPM, const PipelineElement &E,
+                                bool VerifyEachPass, bool DebugLogging) {
----------------
`LoopPassManager &FPM` looks a bit funky. Might as well make it `LPM` if changing the line.


https://reviews.llvm.org/D22724