[PATCH] D44160: [GVN] Detect fully redundant loads when we have more than one available value

[Daniel Berlin via llvm-commits]

FWIW: It's also possible to just fix AA without InstSimplify just by
replacing what it uses with something more powerful.
That would avoid disturbing the other users.

On Fri, Mar 16, 2018 at 7:09 AM, John Brawn via Phabricator <
reviews at reviews.llvm.org> wrote:

> john.brawn added a comment.
>
> I've posted by attempt to fix this in alias anlysis in
> https://reviews.llvm.org/D44564, but the problem is that it's actually
> InstSimplify that's the underlying thing being used to figure out the value
> of the PHI, and that's used all over the place and causes knock-on changes
> that require work to be done elsewhere.
>
>
> Repository:
>   rL LLVM
>
> https://reviews.llvm.org/D44160
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180316/7d06de74/attachment.html>