Subzero: Clean up a few areas.

src/IceTimerTree.cpp

 //===- subzero/src/IceTimerTree.cpp - Pass timer defs ---------------------===//
 //
 //                        The Subzero Code Generator
 //
 // This file is distributed under the University of Illinois Open Source
 // License. See LICENSE.TXT for details.
 //
 //===----------------------------------------------------------------------===//
 //
 // This file defines the TimerTree class, which tracks flat and
@@ skipping 123 matching lines @@
 }
 
 namespace {
 
 typedef std::multimap<double, IceString> DumpMapType;
 
 // Dump the Map items in reverse order of their time contribution.
 void dumpHelper(Ostream &Str, const DumpMapType &Map, double TotalTime) {
   if (!ALLOW_DUMP)
     return;
-  // TODO(stichnot): Use llvm::make_range with LLVM 3.5.
-  for (auto I = Map.rbegin(), E = Map.rend(); I != E; ++I) {
+  for (auto &I : reverse_range(Map)) {
     char buf[80];
-    snprintf(buf, llvm::array_lengthof(buf), "  %10.6f (%4.1f%%): ", I->first,
-             I->first * 100 / TotalTime);
-    Str << buf << I->second << "\n";
+    snprintf(buf, llvm::array_lengthof(buf), "  %10.6f (%4.1f%%): ", I.first,
+             I.first * 100 / TotalTime);
+    Str << buf << I.second << "\n";
   }
 }
 
 // Write a printf() format string into Buf[], in the format "[%5lu] ",
 // where "5" is actually the number of digits in MaxVal.  E.g.,
 //   MaxVal=0     ==> "[%1lu] "
 //   MaxVal=5     ==> "[%1lu] "
 //   MaxVal=9876  ==> "[%4lu] "
 void makePrintfFormatString(char *Buf, size_t BufLen, size_t MaxVal) {
   if (!ALLOW_DUMP)
@@ skipping 59 matching lines @@
   dumpHelper(Str, FlatMap, TotalTime);
   Str << "Number of timer updates: " << StateChangeCount << "\n";
 }
 
 double TimerStack::timestamp() {
   // TODO: Implement in terms of std::chrono for C++11.
   return llvm::TimeRecord::getCurrentTime(false).getWallTime();
 }
 
 } // end of namespace Ice