[PATCH] D66033: [MemorySSA] Rename uses when inserting memory uses.

[Alina Sbirlea via Phabricator via llvm-commits]

asbirlea added a comment.

Note: this is incomplete.

We can always have Phis with LoE coming from unreachable blocks, but when doing an insertDef, we replace all uses of the previous def with self. If that def is LoE, we'll end up replacing the incoming value from those unreachable blocks too. So the verification of "if incoming from unreachable, then it must be LoE is too strict".

With this restriction lifted, there is still a failure from the original PR. I will update once I have the fix for that.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D66033/new/

https://reviews.llvm.org/D66033