[PATCH] D44564: [InstSimplify] Look at the whole PHI graph when simplifying a PHI node

Daniel Berlin via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 16 13:29:34 PDT 2018


Yeah, that is why on the other thread i suggested it probably makes sense
to just change how AA is doing it rather than all simplification users

On Fri, Mar 16, 2018 at 12:21 PM, Eli Friedman via Phabricator <
reviews at reviews.llvm.org> wrote:

> efriedma added a comment.
>
> My primary concern here is that SimplifyInstruction needs to be "constant"
> time; if you put a linear traversal over the entire function here, you'll
> cause compile-time explosions all over the compiler.
>
>
> Repository:
>   rL LLVM
>
> https://reviews.llvm.org/D44564
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180316/a17e5f76/attachment.html>


More information about the llvm-commits mailing list