[PATCH] D16300: Enable LoopLoadElimination by default

[Daniel Berlin via llvm-commits]

If you only handle full redundancy, you'll have to explicitly comment out
the code in processNonLocalLoad.
It does not consider full availability phi insertion to be load PRE.

(GVN.cpp:1761)

On Fri, Jan 22, 2016 at 11:42 AM, Adam Nemet <anemet at apple.com> wrote:

> anemet added a comment.
>
> In http://reviews.llvm.org/D16300#333762, @dberlin wrote:
>
> > Out of curiosity, have you tried the pass with load pre disabled?
>
>
> Thanks!  You're making me curious too, I'll do a run.  Obviously we only
> handle full redundancy and no ld->ld but it should be interesting to see
> how far we are.
>
>
> http://reviews.llvm.org/D16300
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160122/5e30014d/attachment.html>