[PATCH] D21777: [MemorySSA] Switch to a different walker

[Daniel Berlin via llvm-commits]

dberlin added a comment.

> 

> 

> - The explicit call caching was removed from the upwards query. If it needs to be there, it can be readded without much effort, but the goal was to keep this patch as simple as I could, while not dropping accuracy.


Sure. I added this because it makes newgvn faster, but i can make a walker
for newgvn that caches more, it doesn't have to be the default walker :)

> - The `ClobberWalker::reset()` function exists because an idea that's been thrown around is a bulk-update MSSA API (or some kind of thing where we'll do N updates back-to-back). If we don


FWIW, as mentioned on a different thread, you can't make bulk update faster
than "destroy everything" without a bunch of work.


http://reviews.llvm.org/D21777