[LLVMbugs] BreakCriticalEdges and DominanceFrontier

[Chris Lattner]

> I thought it should be both easy and consistent for BreakCriticalEdges
> to update DominanceFrontier information - I was right.

Yup, very nice!

> Also, I changed BreakCriticalEdges to name the new block that spits the
> edge between blocks bb0 and bb1 "bb0.bb1": this makes it a little easier
> for me to understand the output of my PRE pass.  This is purely a style
> issue, and has no technical merit, so feel free to reject this part of
> the patch.

Sounds good to me, I made it produce names of the form "bb0.bb1_crit_edge"
to be nice and extra verbose.  :)

http://mail.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20021028/001165.html
http://mail.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20021028/001166.html

Thanks for the patches!

-Chris

-- 
http://llvm.cs.uiuc.edu/
http://www.nondot.org/~sabre/Projects/