[PATCH] D51801: [MemorySSAUpdater] Avoid creating self-referencing MemoryDefs

[Sebastian Pop via Phabricator via llvm-commits]

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

Looks good to me.



================
Comment at: test/Transforms/GVNHoist/pr38807.ll:32
+;CHECK-NOT:   store i64 undef, i64* %{.*}
+
+bb2:
----------------
labrinea wrote:
> I am not sure why but the gep remains after the hoist, and goes away with dead code elimination. Is that correct @sebpop ?
gvn hoist does not need to implement a dce.


https://reviews.llvm.org/D51801