[PATCH] D29316: Add predicateinfo intrinsic, analysis pass, and basic NewGVN support

[Sanjoy Das via Phabricator via llvm-commits]

sanjoy added a comment.

In https://reviews.llvm.org/D29316#661376, @sanjoy wrote:

> Generally, (as I've mentioned on IRC) the only concern I had with this was the compile time hit we may have because of the extra dereferences that will now be necessary to go from a use to its "true" operand.


Noticed a potential ambiguity here -- I meant going from a use to the value that use actually uses, skipping all of the intermediate predicate_info calls in the use chain.


https://reviews.llvm.org/D29316