Automerge: [opt][timers] Fix time-passes.ll test failing on reversed iterators (#131941)

After llvm/llvm-project#131217 was submitted,
time-passes.ll fails because `opt` prints `-time-report` when
`ManagedTimerGlobals` is destroyed. `ManagedTimerGlobals` stores
`TimerGroup`s in an unordered map, so the ordering of the output
`TimerGroup`s depends on the underlying iterator.

To fix this, we do what Clang does and use
`llvm::TimerGroup::printAll(...)`, which *is* deterministic. This is
also what Clang does. This does put move analysis section before the
pass section for `-time-report`, but again, this is also what Clang
currently does.

Author: alanzhao1

llvm/test/Other/time-passes.ll

@@ -14,6 +14,23 @@
 ; RUN: rm -f %t; opt < %s -disable-output -passes='default<O2>' -time-passes -info-output-file=%t
 ; RUN:   cat %t | FileCheck %s --check-prefix=TIME
 ;
+; TIME: Analysis execution timing report
+; TIME: Total Execution Time:
+; TIME: Name
+; TIME-PER-RUN-DAG:      ScalarEvolutionAnalysis
+; TIME-PER-RUN-DAG:      LoopAnalysis
+; TIME-PER-RUN-DAG:      DominatorTreeAnalysis
+; TIME-PER-RUN-DAG:      TargetLibraryAnalysis
+; TIME-PER-PASS-DAG:   ScalarEvolutionAnalysis
+; TIME-PER-PASS-DAG:   LoopAnalysis
+; TIME-PER-PASS-DAG:   DominatorTreeAnalysis
+; TIME-PER-PASS-DAG:   TargetLibraryAnalysis
+; TIME-PER-PASS-NOT:   ScalarEvolutionAnalysis #
+; TIME-PER-PASS-NOT:   LoopAnalysis #
+; TIME-PER-PASS-NOT:   DominatorTreeAnalysis #
+; TIME-PER-PASS-NOT:   TargetLibraryAnalysis #
+; TIME: Total{{$}}
+
 ; TIME: Pass execution timing report
 ; TIME: Total Execution Time:
 ; TIME: Name
@@ -46,23 +63,6 @@
 ; TIME-PER-PASS-NOT:   VerifierPass #
 ; TIME: Total{{$}}
 
-; TIME: Analysis execution timing report
-; TIME: Total Execution Time:
-; TIME: Name
-; TIME-PER-RUN-DAG:      ScalarEvolutionAnalysis
-; TIME-PER-RUN-DAG:      LoopAnalysis
-; TIME-PER-RUN-DAG:      DominatorTreeAnalysis
-; TIME-PER-RUN-DAG:      TargetLibraryAnalysis
-; TIME-PER-PASS-DAG:   ScalarEvolutionAnalysis
-; TIME-PER-PASS-DAG:   LoopAnalysis
-; TIME-PER-PASS-DAG:   DominatorTreeAnalysis
-; TIME-PER-PASS-DAG:   TargetLibraryAnalysis
-; TIME-PER-PASS-NOT:   ScalarEvolutionAnalysis #
-; TIME-PER-PASS-NOT:   LoopAnalysis #
-; TIME-PER-PASS-NOT:   DominatorTreeAnalysis #
-; TIME-PER-PASS-NOT:   TargetLibraryAnalysis #
-; TIME: Total{{$}}
-
 define i32 @foo() {
   %res = add i32 5, 4
   br label %loop1

llvm/tools/opt/NewPMDriver.cpp

@@ -14,6 +14,7 @@
 
 #include "NewPMDriver.h"
 #include "llvm/ADT/SmallVector.h"
+#include "llvm/ADT/Statistic.h"
 #include "llvm/ADT/StringRef.h"
 #include "llvm/Analysis/AliasAnalysis.h"
 #include "llvm/Analysis/CGSCCPassManager.h"
@@ -30,6 +31,7 @@
 #include "llvm/Passes/PassPlugin.h"
 #include "llvm/Passes/StandardInstrumentations.h"
 #include "llvm/Support/ErrorHandling.h"
+#include "llvm/Support/Timer.h"
 #include "llvm/Support/ToolOutputFile.h"
 #include "llvm/Support/VirtualFileSystem.h"
 #include "llvm/Support/raw_ostream.h"
@@ -566,6 +568,9 @@ bool llvm::runPassPipeline(
   if (DebugifyEach && !DebugifyExport.empty())
     exportDebugifyStats(DebugifyExport, Debugify.getDebugifyStatsMap());
 
+  TimerGroup::printAll(*CreateInfoOutputFile());
+  TimerGroup::clearAll();
+
   return true;
 }