[llvm] r207187 - blockfreq: Pass the Loop directly into unwrapLoopPackage()

Duncan P. N. Exon Smith dexonsmith at apple.com
Thu Apr 24 21:38:23 PDT 2014


Author: dexonsmith
Date: Thu Apr 24 23:38:23 2014
New Revision: 207187

URL: http://llvm.org/viewvc/llvm-project?rev=207187&view=rev
Log:
blockfreq: Pass the Loop directly into unwrapLoopPackage()

<rdar://problem/14292693>

Modified:
    llvm/trunk/lib/Analysis/BlockFrequencyInfoImpl.cpp

Modified: llvm/trunk/lib/Analysis/BlockFrequencyInfoImpl.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Analysis/BlockFrequencyInfoImpl.cpp?rev=207187&r1=207186&r2=207187&view=diff
==============================================================================
--- llvm/trunk/lib/Analysis/BlockFrequencyInfoImpl.cpp (original)
+++ llvm/trunk/lib/Analysis/BlockFrequencyInfoImpl.cpp Thu Apr 24 23:38:23 2014
@@ -824,10 +824,8 @@ static void scaleBlockData(BlockFrequenc
 /// Visits all the members of a loop, adjusting their BlockData according to
 /// the loop's pseudo-node.
 static void unwrapLoopPackage(BlockFrequencyInfoImplBase &BFI,
-                              const BlockNode &Head) {
-  assert(Head.isValid());
-
-  LoopData &LoopPackage = BFI.getLoopPackage(Head);
+                              LoopData &LoopPackage) {
+  BlockNode Head = LoopPackage.getHeader();
   DEBUG(dbgs() << "unwrap-loop-package: " << BFI.getBlockName(Head)
                << ": mass = " << LoopPackage.Mass
                << ", scale = " << LoopPackage.Scale << "\n");
@@ -851,8 +849,8 @@ void BlockFrequencyInfoImplBase::unwrapL
   for (size_t Index = 0; Index < Working.size(); ++Index)
     Freqs[Index].Floating = Working[Index].Mass.toFloat();
 
-  for (const LoopData &L : Loops)
-    unwrapLoopPackage(*this, L.getHeader());
+  for (LoopData &Loop : Loops)
+    unwrapLoopPackage(*this, Loop);
 }
 
 void BlockFrequencyInfoImplBase::finalizeMetrics() {





More information about the llvm-commits mailing list