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

[Chandler Carruth]

And I'm CC-ing Danny specifically to get the URL to his github branch w/
this in it. =D


On Fri, Aug 10, 2012 at 6:32 PM, Chandler Carruth <chandlerc at google.com>wrote:

> 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/0b348826/attachment.html>