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

[Alan Zhao via llvm-commits]

alanzhao1 wrote:

> the fact that we have to duplicate `printAll/clearAll` in multiple places suggests that we should just make `ManagedTimerGlobals` deterministically destruct its timers. can we do that? somehow sort by name before destructing?

This is doable, but I'm not a huge fan of the print metrics on object destruction pattern as it ends up being a painful exercise in object lifetime management rather than something that improves code quality. This is even worse given that this is a global.

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