[PATCH] D21335: [TailDuplication] Split up NumInstrDups statistic.

[Geoff Berry via llvm-commits]

gberry created this revision.
gberry added a subscriber: llvm-commits.
Herald added a subscriber: mcrosier.

Split NumInstrDups statistic into separate added/removed counts to avoid
negative stat being printed as unsigned.

http://reviews.llvm.org/D21335

Files:
  lib/CodeGen/TailDuplicator.cpp

Index: lib/CodeGen/TailDuplicator.cpp
===================================================================
--- lib/CodeGen/TailDuplicator.cpp
+++ lib/CodeGen/TailDuplicator.cpp
@@ -33,7 +33,10 @@
 
 STATISTIC(NumTails, "Number of tails duplicated");
 STATISTIC(NumTailDups, "Number of tail duplicated blocks");
-STATISTIC(NumInstrDups, "Additional instructions due to tail duplication");
+STATISTIC(NumTailDupAdded,
+          "Number of instructions added due to tail duplication");
+STATISTIC(NumTailDupRemoved,
+          "Number of instructions removed due to tail duplication");
 STATISTIC(NumDeadBlocks, "Number of dead blocks removed");
 STATISTIC(NumAddedPHIs, "Number of phis added");
 
@@ -145,7 +148,7 @@
 
   // If it is dead, remove it.
   if (isDead) {
-    NumInstrDups -= MBB->size();
+    NumTailDupRemoved += MBB->size();
     removeDeadBlock(MBB);
     ++NumDeadBlocks;
   }
@@ -805,7 +808,7 @@
     // Simplify
     TII->AnalyzeBranch(*PredBB, PredTBB, PredFBB, PredCond, true);
 
-    NumInstrDups += TailBB->size() - 1; // subtract one for removed branch
+    NumTailDupAdded += TailBB->size() - 1; // subtract one for removed branch
 
     // Update the CFG.
     PredBB->removeSuccessor(PredBB->succ_begin());


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D21335.60722.patch
Type: text/x-patch
Size: 1233 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160614/b94e3b2a/attachment.bin>