[PATCH] D49425: [MemorySSAUpdater] Update Phi operands after trivial Phi elimination

[George Burgess IV via Phabricator via llvm-commits]

george.burgess.iv accepted this revision.
george.burgess.iv added a comment.
This revision is now accepted and ready to land.

LGTM after one nit is addressed. Thanks again!



================
Comment at: lib/Analysis/MemorySSAUpdater.cpp:90
+      // create a new one.
+      if (Phi && Phi->getNumOperands() != 0) {
+        // FIXME: Figure out whether this is dead code and if so remove it.
----------------
`Phi` should always be non-null now


https://reviews.llvm.org/D49425