[PATCH] D36591: [PredicateInfo] Add a helper routine to remove SSA copies from a Function.
Davide Italiano via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 10 23:26:50 PDT 2017
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.
More information about the llvm-commits