[PATCH] D47339: [GVN,NewGVN] Propagate nonnull if K dominates J.

Daniel Berlin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 28 12:50:34 PDT 2018


dberlin added a comment.

So, just curious if we can get rid of the extra parameters you've added.
If you bootstrap with combineMetadata and patchReplacement asserting that the replacement dominates, does the assertion trigger?
If not, we can get rid of the parameters
If so, i'd love to see an example where so it can be analyzed.


https://reviews.llvm.org/D47339





More information about the llvm-commits mailing list