[PATCH] D99987: [NewGVN] Track simplification dependencies for phi-of-ops.

Alina Sbirlea via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 22 15:28:16 PDT 2021


asbirlea accepted this revision.
asbirlea added a comment.
This revision is now accepted and ready to land.

Thank you! Looks good.

A note for the dependent patch, if it will also adds predicate users, perhaps the name `addAdditionalUsers` need not be overloaded (though `addAdditionalAndPredicateUsers` is a bit verbose).
If it makes sense to inline the predicate API into the new one, and remove it, I'd be ok with that.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D99987/new/

https://reviews.llvm.org/D99987



More information about the llvm-commits mailing list