[PATCH] D36591: [PredicateInfo] Add a helper routine to remove SSA copies from a Function.

[Davide Italiano via Phabricator via llvm-commits]

davide added a comment.

I'm not necessarily against this, and I think having PI garbage collecting the intrinsics is a good thing.
What I'm wondering here if you hit a case  when this actually matters? I and Dan debugged several cases where PI intrinsics weren't reclaimed but those were all legitimate bugs in NewGVN.
In case we decide to use PI for some other pass (LVI is an obvious candidate) then having this logic handled in the utility may make things easier.


https://reviews.llvm.org/D36591