[llvm] r294086 - [CodeGen]: BlockPlacement: Skip extraneous logging.

Kyle Butt via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 3 18:26:35 PST 2017


Author: iteratee
Date: Fri Feb  3 20:26:34 2017
New Revision: 294086

URL: http://llvm.org/viewvc/llvm-project?rev=294086&view=rev
Log:
[CodeGen]: BlockPlacement: Skip extraneous logging.

Move a check for blocks that are not candidates for tail duplication up before
the logging. Reduces logging noise. No non-logging changes intended.

Modified:
    llvm/trunk/lib/CodeGen/MachineBlockPlacement.cpp

Modified: llvm/trunk/lib/CodeGen/MachineBlockPlacement.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/MachineBlockPlacement.cpp?rev=294086&r1=294085&r2=294086&view=diff
==============================================================================
--- llvm/trunk/lib/CodeGen/MachineBlockPlacement.cpp (original)
+++ llvm/trunk/lib/CodeGen/MachineBlockPlacement.cpp Fri Feb  3 20:26:34 2017
@@ -2215,13 +2215,13 @@ bool MachineBlockPlacement::maybeTailDup
     BlockChain &Chain, BlockFilterSet *BlockFilter,
     MachineFunction::iterator &PrevUnplacedBlockIt,
     bool &DuplicatedToLPred) {
-
   DuplicatedToLPred = false;
+  if (!shouldTailDuplicate(BB))
+    return false;
+
   DEBUG(dbgs() << "Redoing tail duplication for Succ#"
         << BB->getNumber() << "\n");
 
-  if (!shouldTailDuplicate(BB))
-    return false;
   // This has to be a callback because none of it can be done after
   // BB is deleted.
   bool Removed = false;




More information about the llvm-commits mailing list