[llvm-commits] CVS: llvm/include/llvm/Transforms/Utils/BasicBlockUtils.h

Devang Patel dpatel at apple.com
Thu Jun 7 17:02:30 PDT 2007



Changes in directory llvm/include/llvm/Transforms/Utils:

BasicBlockUtils.h updated: 1.16 -> 1.17
---
Log message:

Do not preserve ETForest.


---
Diffs of the changes:  (+9 -9)

 BasicBlockUtils.h |   18 +++++++++---------
 1 files changed, 9 insertions(+), 9 deletions(-)


Index: llvm/include/llvm/Transforms/Utils/BasicBlockUtils.h
diff -u llvm/include/llvm/Transforms/Utils/BasicBlockUtils.h:1.16 llvm/include/llvm/Transforms/Utils/BasicBlockUtils.h:1.17
--- llvm/include/llvm/Transforms/Utils/BasicBlockUtils.h:1.16	Sat Apr  7 02:17:27 2007
+++ llvm/include/llvm/Transforms/Utils/BasicBlockUtils.h	Thu Jun  7 19:02:08 2007
@@ -61,15 +61,15 @@
                     bool AllowIdenticalEdges = false);
 
 /// SplitCriticalEdge - If this edge is a critical edge, insert a new node to
-/// split the critical edge.  This will update ETForest, ImmediateDominator,
-/// DominatorTree, and DominatorFrontier information if it is available, thus
-/// calling this pass will not invalidate either of them.  This returns true if
-/// the edge was split, false otherwise.  If MergeIdenticalEdges is true (the
-/// default), *all* edges from TI to the specified successor will be merged into
-/// the same critical edge block.  This is most commonly interesting with switch
-/// instructions, which may have many edges to any one destination.  This
-/// ensures that all edges to that dest go to one block instead of each going to
-/// a different block, but isn't the standard definition of a "critical edge".
+/// split the critical edge.  This will update DominatorTree, and DominatorFrontier 
+/// information if it is available, thus calling this pass will not invalidate 
+/// either of them. This returns true if the edge was split, false otherwise.  
+/// If MergeIdenticalEdges is true (the default), *all* edges from TI to the 
+/// specified successor will be merged into the same critical edge block.  
+/// This is most commonly interesting with switch instructions, which may 
+/// have many edges to any one destination.  This ensures that all edges to that 
+/// dest go to one block instead of each going to a different block, but isn't 
+/// the standard definition of a "critical edge".
 ///
 bool SplitCriticalEdge(TerminatorInst *TI, unsigned SuccNum, Pass *P = 0,
                        bool MergeIdenticalEdges = false);






More information about the llvm-commits mailing list