[PATCH] D44564: [BasicAA] Relax restriction on PHI node handling.

[John Brawn via Phabricator via llvm-commits]

john.brawn added a comment.

A bit of an update on this: instead of the circuitous route of having PhiInfo be optionally used by BasicAA then having MemoryDependenceAnalysis use PhiInfo in order to make it available to BasicAA I've gone for making BasicAA use it directly but have PhiInfo use ValueHandles (in a way rather similar to AssumptionCache does) in a way that gets it automatically updated if the function changes. This makes everything much simpler in terms of how it gets used, but it's been taking a while to get the implementation right.


Repository:
  rL LLVM

https://reviews.llvm.org/D44564