[Patch] GVN fold conditional-branch on-the-fly

[Shuxin Yang]

ping?

On 9/9/13 10:43 PM, Shuxin Yang wrote:
> Hi,
>
>  These helper routines are used to save time. I don't need to 
> construct dominance frontier for GVN.
> If DF were available, we don't need them at all.
>
>  I can pull all these stuff in another file/class, and only expose 3 
> member functions.
>  - tryFoldCondBranch(br)
>  - isDeadBlock(BB)
>   - deleteBlock()
>
>  How does this sound to you?
>
>   If we judge added complexity by the # of interaction point to
> the existing code, this is quite a small change.  If you want to 
> quantify complex by LOC,
> please explain why your simpler rudimentary implementation has about 
> 2x LOCs of
> existing complicated GVN.
>
>