[PATCH] D44160: [GVN] Detect fully redundant loads when we have more than one available value

[John Brawn via Phabricator via llvm-commits]

john.brawn added a comment.

Alias analysis uses GetUnderlyingObject from ValueTracking to figure out the 'real' value of the phi and that uses SimplifyInstruction, so I think there's where we should do this. SimplifyPHINode already handles the simple case of a single phi node, and extending it to work on a chain of phi nodes (which is what we need here) looks doable though a bit fiddly to get right in all cases.


Repository:
  rL LLVM

https://reviews.llvm.org/D44160