[llvm-commits] CVS: llvm/lib/Transforms/Utils/BreakCriticalEdges.cpp

Chris Lattner sabre at nondot.org
Tue Oct 3 21:59:12 PDT 2006



Changes in directory llvm/lib/Transforms/Utils:

BreakCriticalEdges.cpp updated: 1.33 -> 1.34
---
Log message:

This case isn't implemented yet.  It seems unlikely to be needed, but if it
ever is, we want to get an assert instead of silent bad codegen.


---
Diffs of the changes:  (+2 -4)

 BreakCriticalEdges.cpp |    6 ++----
 1 files changed, 2 insertions(+), 4 deletions(-)


Index: llvm/lib/Transforms/Utils/BreakCriticalEdges.cpp
diff -u llvm/lib/Transforms/Utils/BreakCriticalEdges.cpp:1.33 llvm/lib/Transforms/Utils/BreakCriticalEdges.cpp:1.34
--- llvm/lib/Transforms/Utils/BreakCriticalEdges.cpp:1.33	Tue Oct  3 02:02:02 2006
+++ llvm/lib/Transforms/Utils/BreakCriticalEdges.cpp	Tue Oct  3 23:58:58 2006
@@ -253,11 +253,9 @@
   if (DominanceFrontier *DF = P->getAnalysisToUpdate<DominanceFrontier>()) {
     // If NewBBDominatesDestBB hasn't been computed yet, do so with DF.
     if (!OtherPreds.empty()) {
-#if 0
       // FIXME: IMPLEMENT THIS!
-      OtherPreds.clear();
-#endif
-      NewBBDominatesDestBB = false;
+      assert(0 && "Requiring domfrontiers but not idom/domtree/domset."
+             " not implemented yet!");
     }
     
     // Since the new block is dominated by its only predecessor TIBB,






More information about the llvm-commits mailing list