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

Arthur Eubanks via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 25 13:04:29 PDT 2025


================
@@ -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(
-               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;
+    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);
+
+    llvm::TimerGroup::printAll(*llvm::CreateInfoOutputFile());
+    llvm::TimerGroup::clearAll();
----------------
aeubanks wrote:

ultra nit: I'd move this to the end of NewPMDriver.cpp

https://github.com/llvm/llvm-project/pull/131941


More information about the llvm-commits mailing list