[PATCH] D59315: [AliasAnalysis] Second prototype to cache BasicAA / anyAA state.

[Alina Sbirlea via Phabricator via llvm-commits]

asbirlea marked 2 inline comments as done.
asbirlea added inline comments.


================
Comment at: include/llvm/Analysis/AliasAnalysis.h:774-776
+/// esentially making AA work in "batch mode". The internal state cannot be
+/// cleared, so to go "out-of-batch-mode", the user must either use AAResults,
+/// or create a new BatchAAResults.
----------------
chandlerc wrote:
> FWIW, I could imagine adding a `clear` method in the future that nukes the AAQI to make it easier for passes to reset their batch when needed.
> 
> But I like the current design and adding this only once we see it become awkward to create a distinct wrapper for each batch.
SGTM, thank you for the review!


Repository:
  rL LLVM

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

https://reviews.llvm.org/D59315