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

Sanjoy Das via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 31 01:06:30 PST 2017

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.


More information about the llvm-commits mailing list