[NewPM] Hide pass manager debug logging behind -debug-pass-manager-verbose

Printing pass manager invocations is fairly verbose and not super
useful.

This allows us to remove DebugLogging from pass managers and PassBuilder
since all logging (aside from analysis managers) goes through
instrumentation now.

This has the downside of never being able to print the top level pass
manager via instrumentation, but that seems like a minor downside.

Reviewed By: ychen

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

1 files changed, 5 insertions, 7 deletions

diff --git a/polly/lib/Support/RegisterPasses.cpp b/polly/lib/Support/RegisterPasses.cpp
index 05cad2b5e52a..1c8703f16521 100644
--- a/polly/lib/Support/RegisterPasses.cpp
+++ b/polly/lib/Support/RegisterPasses.cpp
@@ -734,16 +734,15 @@ static bool isScopPassName(StringRef Name) {
 
 static bool
 parseTopLevelPipeline(ModulePassManager &MPM, PassInstrumentationCallbacks *PIC,
-                      ArrayRef<PassBuilder::PipelineElement> Pipeline,
-                      bool DebugLogging) {
+                      ArrayRef<PassBuilder::PipelineElement> Pipeline) {
   std::vector<PassBuilder::PipelineElement> FullPipeline;
   StringRef FirstName = Pipeline.front().Name;
 
   if (!isScopPassName(FirstName))
     return false;
 
-  FunctionPassManager FPM(DebugLogging);
-  ScopPassManager SPM(DebugLogging);
+  FunctionPassManager FPM;
+  ScopPassManager SPM;
 
   for (auto &Element : Pipeline) {
     auto &Name = Element.Name;
@@ -773,9 +772,8 @@ void registerPollyPasses(PassBuilder &PB) {
       });
   PB.registerParseTopLevelPipelineCallback(
       [PIC](ModulePassManager &MPM,
-            ArrayRef<PassBuilder::PipelineElement> Pipeline,
-            bool DebugLogging) -> bool {
-        return parseTopLevelPipeline(MPM, PIC, Pipeline, DebugLogging);
+            ArrayRef<PassBuilder::PipelineElement> Pipeline) -> bool {
+        return parseTopLevelPipeline(MPM, PIC, Pipeline);
       });
 
   if (PassPosition != POSITION_BEFORE_VECTORIZER)