[PATCH] D35741: Add MemorySSA alternative to AliasSetTracker in LICM.

[Davide Italiano via Phabricator via llvm-commits]

davide added a comment.

Thanks, I'll review this soon.
A quick question. a concern people have about MemorySSA is that you pay upfront for O(1) queries, which may have some impact on compile time.
(e.g. in EarlyCSE). AST is already O(N^2), and in some cases badly so, but I wonder if you have compile time measurements of the impact of your change?


https://reviews.llvm.org/D35741