[PATCH] D128583: CodeGen: Remove AliasAnalysis from regalloc

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added a comment.

In D128583#3613323 <https://reviews.llvm.org/D128583#3613323>, @efriedma wrote:

> The general approach here seems fine.  I suspect we end up calling pointsToConstantMemory for most memory operands at some point anyway.
>
> In general pointsToConstantMemory doesn't imply the pointer is dereferenceable. But in practice, the cases where that happens are probably pretty obscure (like out-of-bounds references to constant arrays).

Do you think this is worth fixing in a follow on patch? It seems the API requires two separate queries that mostly do the same thing


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

https://reviews.llvm.org/D128583