[PATCH] D54706: [LoopSimplifyCFG] Remove phis whilst removing CFG edges

[Dave Green via Phabricator via llvm-commits]

dmgreen added inline comments.


================
Comment at: lib/Transforms/Scalar/LoopSimplifyCFG.cpp:248
+          for (PHINode &Phi : Succ->phis())
+            Phi.removeIncomingValue(BB);
+        }
----------------
This can remove empty phi's I believe, which I'm not 100% sure about.

I think it's OK as-is but, Max, this may change with your followon patches. Feel free to commandeer if you think that's easier, otherwise let me know what you think.


https://reviews.llvm.org/D54706