[llvm-commits] [patch] Use the new edge dominance functions in GVN

Chandler Carruth chandlerc at google.com
Fri Aug 10 18:32:09 PDT 2012


FYI, Danny has been working on a pretty significant improvement / rewrite
of GVN.

If you're seeing problems in GVN, might be worth looking at that. Also,
help getting that into the tree / porting improvements over to it, would
likely be very welcome.


On Fri, Aug 10, 2012 at 9:04 AM, Rafael EspĂ­ndola <
rafael.espindola at gmail.com> wrote:

> The attached patch depends on my previous one fixing the handling of
> multiple edges between two basic blocks.
>
> What this patch does is change GVN to use the new edge dominance
> functions so that it is able to reason about an edge dominating a use.
> With this patch in, fixing pr13307 becomes an issues of having logic
> to decide if is more profitable to replace 'a' with 'b' or 'b' with
> 'a'.
>
> Cheers,
> Rafael
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120810/1e5936c3/attachment.html>


More information about the llvm-commits mailing list