[PATCH] D84609: [MemDepAnalysis] Cut-off threshold reshuffling

[Alina Sbirlea via Phabricator via llvm-commits]

asbirlea added a comment.

In D84609#2180222 <https://reviews.llvm.org/D84609#2180222>, @lebedev.ri wrote:

> I wasn't really asking about NewGVN story, i know it's stagnant somewhat.
> I was only asking, would it be better to instead look into porting
> `GVN::processNonLocalLoad()` to be MemorySSA-driven.

If we're building two analyses (MSSA & MemDepAnalysis) instead of one, I expect we'll see a spike in compile-time. Additionally, MemDepAnalysis is a piece of technical debt that would be nice to replace altogether.
IMO it makes sense to make the transition for the whole pass, or for the pipeline of 3 passes in one go.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84609