Subzero: Improve the use of timers.

src/IceCompiler.cpp

Index: src/IceCompiler.cpp
diff --git a/src/IceCompiler.cpp b/src/IceCompiler.cpp
index 7a0dcbb5fec73461662ca787e1ba22b915123a37..ff77533190b287925cf5891cdd6a7a47e4d2c991 100644
--- a/src/IceCompiler.cpp
+++ b/src/IceCompiler.cpp
@@ -139,8 +139,8 @@ void Compiler::run(const Ice::ClFlagsExtra &ExtraFlags, GlobalContext &Ctx,
     Ctx.dumpTimers();
 
   if (Ctx.getFlags().getTimeEachFunction()) {
-    constexpr bool DumpCumulative = false;
-    Ctx.dumpTimers(GlobalContext::TSK_Funcs, DumpCumulative);
+    constexpr bool NoDumpCumulative = false;

[Karl, 2016/03/11 23:05:50]

What about?

  constexpr bool DumpCumulative = true;
  Ctx.DumpTimers(..., !DumpCumulative);

[Jim Stichnoth, 2016/03/11 23:32:34]

I think we've been converging to the style that I "corrected" this to.  See "git
grep 'bool No'".

+    Ctx.dumpTimers(GlobalContext::TSK_Funcs, NoDumpCumulative);
   }
   constexpr bool FinalStats = true;
   Ctx.dumpStats("_FINAL_", FinalStats);