[PATCH] D83246: [Attributor] Cache query results for isPotentiallyReachable in AAReachability

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 22 22:06:57 PDT 2020


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

In D83246#2163870 <https://reviews.llvm.org/D83246#2163870>, @okura wrote:

> I have difficulty in moving into AAReachability because a member should be const function. If the map move in AAReachability, the function that cache query results cannot be const. That's why I put the cache in InformationCache.


Good point. Let's go with this :)


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

https://reviews.llvm.org/D83246





More information about the llvm-commits mailing list