[PATCH] D33136: [ValueTracking] Don't let isAddOfNonZero look at adds with a PHI node as input

Daniel Berlin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 14 22:50:02 PDT 2017


dberlin added a comment.

In https://reviews.llvm.org/D33136#773589, @Farhana wrote:

> What is the performance implication of this change-set? Have you done any performance testing? This seems to make the alias analysis very conservative. It is now going to find two non-equal numbers equal very easily.
>
>     


Other than rewrite it all, what are you suggesting he do?


https://reviews.llvm.org/D33136





More information about the llvm-commits mailing list