[opt][timers] Fix time-passes.ll test failing on reversed iterators

After llvm#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,24 @@
 ; 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,22 +64,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

llvm/tools/opt/optdriver.cpp

@@ -48,6 +48,7 @@
 #include "llvm/Support/SystemUtils.h"
 #include "llvm/Support/TargetSelect.h"
 #include "llvm/Support/TimeProfiler.h"
+#include "llvm/Support/Timer.h"
 #include "llvm/Support/ToolOutputFile.h"
 #include "llvm/Support/YAMLTraits.h"
 #include "llvm/Target/TargetMachine.h"
@@ -722,14 +723,16 @@ extern "C" int optMain(
     // The user has asked to use the new pass manager and provided a pipeline
     // string. Hand off the rest of the functionality to the new code for that
     // layer.
-    return runPassPipeline(
+    bool result =  runPassPipeline(
                argv[0], *M, TM.get(), &TLII, Out.get(), ThinLinkOut.get(),
                RemarksFile.get(), Pipeline, PluginList, PassBuilderCallbacks,
                OK, VK, PreserveAssemblyUseListOrder,
                PreserveBitcodeUseListOrder, EmitSummaryIndex, EmitModuleHash,
-               EnableDebugify, VerifyDebugInfoPreserve, UnifiedLTO)
-               ? 0
-               : 1;
+               EnableDebugify, VerifyDebugInfoPreserve, UnifiedLTO);
+
+    llvm::TimerGroup::printAll(*llvm::CreateInfoOutputFile());
+    llvm::TimerGroup::clearAll();
+    return result ? 0 : 1;
   }
 
   if (OptLevelO0 || OptLevelO1 || OptLevelO2 || OptLevelOs || OptLevelOz ||
@@ -905,5 +908,8 @@ extern "C" int optMain(
   if (ThinLinkOut)
     ThinLinkOut->keep();
 
+  llvm::TimerGroup::printAll(*llvm::CreateInfoOutputFile());
+  llvm::TimerGroup::clearAll();
+
   return 0;
 }