[PATCH] D30871: Add forgetBlock in PredIteratorCache.

Daniel Berlin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Mar 12 09:20:26 PDT 2017


dberlin accepted this revision.
dberlin added a comment.
This revision is now accepted and ready to land.

I'm all in favor of cache invalidation when weird things happen, but I also assume you could just tell it when you are removing/adding edges as well?

IE i would expect this kind of invalidation when we do something like "clone a loop", or "clone a region" and it's difficult to tell what happened.

But otherwise, i presume we could just update it instead of destroy it.

Anyway, probably not worth it here.
Just sayin.


https://reviews.llvm.org/D30871





More information about the llvm-commits mailing list